代码重构之重构过长的参数列表

举报
i进击的攻城狮 发表于 2022/04/26 23:54:08 2022/04/26
【摘要】 一个函数,往往都需要接收参数,通过参数去完成某些事情,在<<代码整洁之道>>中,作者提出,一个函数的参数应该尽可能的短,最好只有一个,其次是两个,如果出现三个及其以上的参数,那你应该引起注意了。那么如果拿到一段长参数的函数我们应该怎么办?如何去重构呢?

一、过长参数的定义

方法的参数过多,或者存在不必要的参数。

二、过长参数带来的影响

方法不易被理解和使用,方法不稳定,难以维护

三、重构方法

  • 以查询或者计算取代参数:如果参数列表中的某些参数,可以通过现有的其他参数计算得出,那计算的参数就可以省略掉,比如得到一个人的出生日期就能推算出他的年龄。
    如下代码是计算某个演出票的基本信息,票只有符合年龄范围才能购买,学生和儿童票有优惠。Performance对象里是演出的基本信息,里面包含了basicPrice基本票价,所以此方法可以去掉basicPrice参数,改成直接从Performance获取
 /**
     * 获取票据信息
     * 
     * @param name 姓名
     * @param age 年龄
     * @param isChild 是否儿童
     * @param isStudent 是否学生
     * @param ageFloor 年龄上限
     * @param ageCeiling 年龄下限
     * @param performance 演出信息
     * @param basicPrice 基本票价
     * @return 票据信息
     */
    public String getTicketInfo(String name, int age, boolean isChild, boolean isStudent, int ageFloor, int ageCeiling,
        Performance performance, double basicPrice) {
        if ((age < ageFloor || age > ageCeiling)) {
            throw new IllegalArgumentException("age is out of valid range, cannot buy ticket!");
        }

        return getPerformanceInfo(performance)
            + getConsumerInfo(name, age, isStudent, isChild)
            + getPriceInfo(isChild, isStudent, basicPrice);
    }

在这里插入图片描述

  • 保持对像的完整:有时候,一个函数的参数,可能就是一个对象的成员变量,为什么不干脆用这个对象做参数呢?下面的函数中,原本是传入年龄的上限和下线,但其他地方其实有相关的对象,那干脆直接用已经定义好的对象就行了。
    在这里插入图片描述

  • 引入参数对象,函数中的某些参数,可以自己手动封装到一个对象中,如下,参数中有很多自动都是用户相关的信息,所以重构的时候通过新增一个用户类去简化这些参数
    在这里插入图片描述

  • 移动函数到合适的类,将一些参数封装到对象之后,如果某些操作是根据封装的那些字段处理的,不如把这个函数也移动到刚刚封装的对象里边。
    在这里插入图片描述

  • 移除标记参数,某些入参属于标记,根据入参执行两种不同的代码逻辑,可以考虑使用三目运算符简化参数
    在这里插入图片描述

【版权声明】本文为华为云社区用户原创内容,转载时必须标注文章的来源(华为云社区)、文章链接、文章作者等基本信息, 否则作者和本社区有权追究责任。如果您发现本社区中有涉嫌抄袭的内容,欢迎发送邮件进行举报,并提供相关证据,一经查实,本社区将立刻删除涉嫌侵权内容,举报邮箱: cloudbbs@huaweicloud.com
  • 点赞
  • 收藏
  • 关注作者

评论(0

0/1000
抱歉,系统识别当前为高风险访问,暂不支持该操作

全部回复

上滑加载中

设置昵称

在此一键设置昵称,即可参与社区互动!

*长度不超过10个汉字或20个英文字符,设置后3个月内不可修改。

*长度不超过10个汉字或20个英文字符,设置后3个月内不可修改。