乐趣区

关于java:3月30日-代码检视日志

jiezi

if 异样判断尽量应用 Assert

例如：

    if(CollectionUtils.isEmpty(hotContents)) {log.error("社区主题获取 7 *24 小时精选内容获取数据谬误：{}",hotContents);  
     throw new BusinessAPIException("社区主题获取 7 *24 小时精选内容获取数据谬误");  
    }

能够改为 Assert

    Assert.notEmpty(hotContents, "社区主题获取 7 *24 小时精选内容获取数据谬误");

再如：

    if (CommunityConsts.TAB_NEWS_SUCCEED_STATUS.equals(tabNewResponse.getCode())) {// do somethings} else {throw new APICallException(tabNewResponse.getCode(), tabNewResponse.getMsg());  
    }

能够改为：

    Assert.isTrue(TAB_NEWS_SUCCEED_STATUS.equals(tabNewResponse.getCode()),  
     "7×24tab 资讯举荐获取资讯接口 Fail! code: {} - msg: {}",  
     tabNewResponse.getCode(), tabNewResponse.getMsg());

如果 Assert 抛出的异样不满足要求（例如要抛出个别的异样），那至多也包装一个静态方法，例如 BizAssert。这里次要的问题是呈现了大量反复的判断语句，所以最好尽量打消这些反复代码。

异样解决不是常态

异样解决不是常态，尽量不要侵入失常业务逻辑。

例如，对返回值进行校验，之前很多代码会这样写：

    if (validateUtil.checkInvalid(data)) {log.error("接口返回数据谬误：{}",data.getRemList());  
        throw new BusinessAPIException("接口返回数据谬误");  
    }

这里仅仅只是做了个异样转换，没啥养分，齐全能够封装一个新的校验办法并在外面抛出异样。封装完后写成，

    validateUtil.bizCheck(data, "xxx msg");

Domain Model 尽可能封装，不要适度裸露

来看个例子，

@Data  
@AllArgsConstructor  
public class ReviewWealthFocusInfo {  
  
    List<ReviewWealthFocus> reviewWealthFocus;  
  
     public List<ReviewWealthFocus> getReviewWealthFocus() {if (reviewWealthFocus.size() > CommunityListConsts.FIRST_COMMUNITY_LIST) {return reviewWealthFocus.stream()  
                    .limit(3)  
                    .collect(Collectors.toList());  
     }  
        return reviewWealthFocus;  
     }  
}

能够看到，这个类外面，只有一个成员，而且曾经有了 getter 了，但有些同学还是习惯加上 lombok 的注解 @Data。lombok 有时候只是图个不便，其实毁坏了封装性。

能够改成这样，增加一个构造函数，外面还能够加些入参校验。并没有比 lombok 麻烦，IDEA 有快捷键能够疾速增加 constructor, getter, setter, override 等。

public class ReviewWealthFocusInfo {  
  
    private List<ReviewWealthFocus> reviewWealthFocus;  
  
    public ReviewWealthFocusInfo(List<ReviewWealthFocus> reviewWealthFocus) {Validate.notNull(reviewWealthFocus);  
        this.reviewWealthFocus = reviewWealthFocus;  
     }  
      
    public List<ReviewWealthFocus> getReviewWealthFocus() {  
        return reviewWealthFocus.subList(0,  
                  Math.min(FIRST_COMMUNITY_LIST, reviewWealthFocus.size()));  
     }  
}

上述例子还有一个有毒的中央，就是 List 的截取，棘手改了，不细说。

须要反复调用的中央，请应用循环

检视过程中，我还发现有个中央，同一个办法反复调用了两次。getTabNewsIds() 这个办法竟然反复了两次，令人非常费解。

    List<String> tabNewsIds = getTabNewsIds();  
    List<String> otherTabNewsIds = getTabNewsIds();  
    ids = getTotalTabNewsId(tabNewsIds,otherTabNewsIds);

且不说这是出于何种起因要这么写，至多整个循环吧？一个简略的循环就包装起来了 

    private List<String> getEnoughTabNewsIds() {List<String> ids = Lists.newArrayList();  
        while (ids.size() <= NEWS_NUM_THRES) {ids.addAll(getTabNewsIds());  
        }  
        return ids;  
    }
退出移动版