享受代码,享受人生

SOA is an integration solution. SOA is message oriented first.
The Key character of SOA is loosely coupled. SOA is enriched
by creating composite apps.
posts - 213, comments - 2314, trackbacks - 162, articles - 45
  博客园 :: 首页 :: 新随笔 :: 联系 :: 订阅 订阅 :: 管理

公告

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

Posted on 2013-06-27 15:38 idior 阅读(...) 评论(...) 编辑 收藏
 
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对象创建出来,然后再对它进行验证。我们应该尽力避免创建非法对象。