改善参数过多的方法

参数过多的方法，该如何改善？

问题

按照《Clean Code》中的标准，方法参数超过3个就已经是“过多”了。
参数多带来的问题：

• 首先是调用的地方难以理解，特别是多个参数是同一类型的时候。不仅调用时要小心对应每个参数的位置。在方法参数数目变动时，更有造成bug的隐患。
• 过多的参数也体现出方法责任过多，可能缺乏内聚的问题。

这个问题可以用简单的重构入手进行改进。

示例代码

下面示例的重构代码来源于
https://github.com/xpmatteo/birthday-greetings-kata

public void sendGreetings(String fileName, XDate xDate, String smtpHost, int smtpPort) throws IOException, ParseException, AddressException, MessagingException {
        ...
        if (employee.isBirthday(xDate)) {
            String recipient = employee.getEmail();
            String body = "Happy Birthday, dear %NAME%".replace("%NAME%", employee.getFirstName());
            String subject = "Happy Birthday!";
            sendMessage(smtpHost, smtpPort, "sender@here.com", subject, body, recipient);
        }
    }
}

private void sendMessage(String smtpHost, int smtpPort, String sender, String subject, String body, String recipient) throws AddressException, MessagingException {
    // Create a mail session
    java.util.Properties props = new java.util.Properties();
    props.put("mail.smtp.host", smtpHost);
    props.put("mail.smtp.port", "" + smtpPort);
    Session session = Session.getInstance(props, null);

    // Construct the message
    Message msg = new MimeMessage(session);
    msg.setFrom(new InternetAddress(sender));
    msg.setRecipient(Message.RecipientType.TO, new InternetAddress(recipient));
    msg.setSubject(subject);
    msg.setText(body);

    // Send the message
    Transport.send(msg);
}

参数对象 (Parameter Object)

使用抽取参数对象的方式，可以把多个有关联的参数组成一个对象来传递。
往往在采用了这个重构后，会浮现出原本代码中缺失的业务概念。
这个重构也是消除“基本类型迷恋”坏味道的利器。
过程演示如下：

Parameter Object

用Builder代替构造器

从上面的演示可以发现，多个参数从调用方法的地方转移到了产生参数对象的地方。
采取用Builder代替构造器的重构方法，可以使代码的意图更为明确。

Builder

方法对象 (Method Object)

另一种思路是把使用过多参数的方法抽取成一个对象。
采用这种方法，参数会成为对象的属性。
演示如下：

Method Object & Builder

同样，也结合使用了Builder。

结论

• 参数对象可以把多个原始类型的参数合并为一个有业务意义的对象。
• 方法对象可以把较为复杂的操作独立为单独的对象。原本的参数变为对象的状态。
• Builder模式可以有效改善构造器参数参数过多带来的问题。

当然，以上仅仅是进行改进的开始。实际中往往会发现并非所有的参数都应该属于同一对象。也可能被重构的方法本身需要拆分成不同的部分。需要结合其它方法进行更深入的重构。