每日代码 - 6/27 避免建立非法对象

public class BuyerHelper
{
    private static final UserRepository userRepository;

    public static void createAndCheckBuyer(Cart cart)
    {
        int userId = cart.getUserId();
        User user = userRepository.getUserById(userId);

        Buyer buyer = BuyFactory.createFrom(user);
        BaseCheckResult checkResult = BuyerChecker.checkBuyerPermission(buyer);
        
        if (checkResult.isSuccess())
            cart.setBuyer(buyer);
        else
            throw new InvaildBuyerException();        
    }
}

 

很简单的一段代码，存在如下问题：

1. 责任不清晰。

该方法既包含了验证用户是不是一个合法买家的功能，又实现了根据用户建立买家的功能，而且还隐藏了一个很危险的功能，它还负责将这个建立好的买家设置到cart的属性。而cart是一个外来参数，这意味着后续的代码中颇有可能会依赖这个被附加到cart上的buyer属性上，而这一点实际上是比较隐晦的。

2. 建立非法对象。

本方法实现的功能应该是判断一个用户是不是一个有效的买家，理论上来讲若是一个用户不是有效买家的话，就不该该出现一个买家对象。可是，在这段代码中，确实先无论三七二十一先把Buyer对象建立出来，而后再对它进行验证。咱们应该尽力避免建立非法对象。