zoukankan      html  css  js  c++  java
  • 每日代码

     
    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对象创建出来,然后再对它进行验证。我们应该尽力避免创建非法对象。

  • 相关阅读:
    Jmeter(十四)
    Jmeter(十一)
    Jenkins
    对前端模块化应如何理解
    Vue面试相关
    vue-router使用总结
    Vue全家桶__Vue-router&Vuex
    Vue-cli3开发单文件
    Vue_组件化开发
    Vue_问题梳理
  • 原文地址:https://www.cnblogs.com/idior/p/3158896.html
Copyright © 2011-2022 走看看