日期:2014-05-20  浏览次数:20992 次

方法多少个参数才算多!
小弟有这样一个方法,按页大小和页索引查询产品,如下。
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%的变了,只是方法的代码没改,但方法的定义已经改了,这完全没有优势。都是改代码,都是改方法的参数,都改变了方法的定义。