方法多少个参数才算多!
小弟有这样一个方法,按页大小和页索引查询产品,如下。
- C# code
/// <summary> /// 获取产品 /// </summary> /// <param name="manufacturerID">厂商ID,为null时不做查询条件。</param> /// <param name="categoryID">类别ID,为null时不做查询条件。</param> /// <param name="typeID">类型ID,为null时不做查询条件。</param> /// <param name="name">产品名称,模糊匹配。</param> /// <param name="description">描述,模糊匹配。</param> /// <param name="pageSize">页大小</param> /// <param name="pageIndex">页索引</param> /// <returns></returns> public DataTable GetProduct(int? manufacturerID, int? categoryID, int? typeID, string name, string description, int pageSize, int pageIndex) { ...... }
可以看到小弟这个方法参数比较多,有7个,这时有人就说了:“这么多参数,应该封装成一个对象传递!”要写成下面这个样子。
- C# code
public class QueryProduct { private int? manufacturerID; /// <summary> /// 厂商ID,为null时不做查询条件。 /// </summary> public int? ManufacturerID { get { return manufacturerID; } set { manufacturerID = value; } } private int? categoryID; /// <summary> /// 类别ID,为null时不做查询条件。 /// </summary> public int? CategoryID { get { return categoryID; } set { categoryID = value; } } private int? typeID; /// <summary> /// 类型ID,为null时不做查询条件。 /// </summary> public int? TypeID { get { return typeID; } set { typeID = value; } } private string name; /// <summary> /// 产品名称,模糊匹配。 /// </summary> public string Name { get { return name; } set { name = value; } } private string description; /// <summary> /// 描述,模糊匹配。 /// </summary> public string Description { get { return description; } set { description = value; } } private int pageSize; /// <summary> /// 页大小 /// </summary> public int PageSize { get { return pageSize; } set { pageSize = value; } } private int pageIndex; /// <summary> /// 页索引 /// </summary> public int PageIndex { get { return pageIndex; } set { pageIndex = value; } } }
- C# code
/// <summary> /// 获取产品 /// </summary> /// <param name="query">查询条件</param> /// <returns></returns> public DataTable GetProduct(QueryProduct query) { ...... }
小弟心里就想了 ,这才几个参数,连10个都不到,封装成一个对象有必要吗?
下面是各自对问题的看法
应该封装成对象:
1.不封装成对象,代码看起来很乱。
2.可维护性强,以后这个方法如果需要加查询参数,只需要在QueryProduct类中增加一个属性即可,方法不用改。
3.方法的参数越少越好,能少一个,绝不多一个。
4.方法的参数超过3个就要封装成对象。
应该直接传递参数:
1.直接将查询条件写在方法参数上,可读性强,便于后来人维护。
2.封装成对象,项目中就需要增加一个对象,对象越多,越不利于维护,10个以内的参数,如果没有特殊原因,不需要封装成对象。
3.这里的参数每一个都有业务意义,并不是要持久化到数据库的实体属性,在数量不多的时候,不应该封装成对象。
下面是小弟对“应该封装成对象”一派3个理由的回复
1.不封装成对象,代码看起来很乱。
一点都不乱,封装成对象反倒增加了阅读成本。
2.可维护性强,以后这个方法如果需要加查询参数,只需要在QueryProduct类中增加一个属性即可,方法不用改。
可维护性绝对没有增强,反倒因为项目中多了一堆查询条件对象,增加了阅读成本,降低了维护性。当增加查询条件时,改了查询对象的定义,方法的参数就变了,方法也100%的变了,只是方法的代码没改,但方法的定义已经改了,这完全没有优势。都是改代码,都是改方法的参数,都改变了方法的定义。
3.方法的参数越少越好,能少一个,绝不多一个。
这里没有一个参数是多余的,少任何一个参数都不能满足需求,方法的参数越少越好是毋庸置疑的,但绝不是用这种坑爹的方式减少参数。
4.方法的参数超过3个就要封装成对象。
是否要将参数封装成对象,不能只看参数的数量,还要看它的业务意义,作为数据载体的实体类,即使只有两个属性,也要用对象传递。但像这种,不需要持久化,而且每个参数都有各自的业务意义,没有特殊原因,就应该写在方法的参数列表里。
关于这个问题,小弟希望看看大家是怎么对待的,希望大家能在回复时除了表述自己的观点,同时告知在项目中会采取哪种方式。
[解决办法]
[解决办法]
在博客园已经回复过
除非这个对象本身来源自是model层或业务层的类,否则我个人是很不赞同使用查询对象,因为这个对象基本没什么复用性,也只会在这个查询的方法中做为参数出现
为了一个方法的参数就创建一个类,是否是设计过度呢?