代码质量记录

service对外提供下面方法会保证 一定能获得ActStepDesc对象，这样外部使用时就不用判断null了。

public ActStepDesc selectByBPK(Integer activityId, Integer tplStepId){
        ActStepDesc actStepDesc= actStepDescMapper.selectByBPK(activityId, tplStepId);
        if (null==actStepDesc||null == actStepDesc.getId()) {
            actStepDesc = new ActStepDesc();
            actStepDesc.setActivityId(activityId);
            actStepDesc.setTplStepId(tplStepId);
            actStepDesc.setProjectId(ProjectContextUtils.getProjectContext().getProjectId());
            actStepDesc.setId(saveOrUpdate(actStepDesc));
        }
        return actStepDesc;
    }

下面是同一类内部调用上面方法的方法，直接使用了底层查询，导致有时会出现null的情况，

也应该直接使用上面的方法，就是说类内部也要使用类本身的方法。

public Integer selectUploadCount(@InvokeParam(value="activityId",required=true) String activityId,@InvokeParam(value="tplStepId",required=true) String tplStepId){
        ActStepDesc actStepDesc= actStepDescMapper.selectByBPK(Integer.valueOf(activityId), Integer.valueOf(tplStepId));

//改成下面 ActStepDesc actStepDesc= selectByBPK(Integer.valueOf(activityId), //Integer.valueOf(tplStepId));
        return resourceService.selectUserCount(Const.RESOURCE_ORIGIN.ACT_STEP, actStepDesc.getId());
    }