Dao层已为CRUD方法添加日志和自定义异常，Service层还需重复记录吗？

核心结论 #

不需要在Service层添加和Dao层完全相同的日志逻辑，如果Service层仅做简单的Dao调用透传，保持现有代码即可；如果Service层包含额外业务逻辑，可补充和业务场景对应的差异化日志，不需要和Dao层日志重复。

具体判断依据 #

1. 重复日志的弊端 #

• 增加不必要的存储成本，排查问题时重复的日志信息也会干扰定位效率
• 违反单一职责原则：数据访问层的操作日志归Dao层维护即可，后续如果要调整日志级别、输出格式，只需要修改Dao层一处，避免多层维护的一致性问题
• Dao层抛出的自定义异常本身已经携带完整的报错信息，异常栈也会明确标注报错的层级和位置，不需要Service层重复打印相同的报错内容。

2. Service层可补充的差异化日志场景 #

只有当Service层存在以下逻辑时，才需要新增日志，注意不要和Dao层已有的日志内容重复：

• 包含业务校验逻辑：比如校验教师关联的课程是否存在、校验教师年龄是否符合入职要求等，可打印业务校验相关的日志
• 包含多Dao的事务操作：比如新增教师同时要绑定授课关系、同步其他系统数据等，可打印业务流程的执行节点日志，方便区分是业务流程问题还是单Dao的数据操作问题
• 需要对Dao层异常做二次包装：比如把Dao层的UniversityDaoException包装成上层业务异常返回给控制层，此时可以补充业务场景的错误描述，不需要重复打印Dao层已经输出的异常内容。

举个符合要求的Service层代码示例：

@Override
@Transactional
public void addTeacher(Teacher teacher, List<Integer> lectureIdList) {
    logger.debug("启动新增教师业务流程，教师ID：{}，待关联授课ID集合：{}", teacher.getID(), lectureIdList);
    // 业务校验：校验关联授课是否存在
    if (lectureDao.countByIds(lectureIdList) != lectureIdList.size()) {
        String errMsg = "新增教师失败：待关联的授课ID不存在";
        logger.warn(errMsg);
        throw new UniversityServiceException(errMsg);
    }
    // 调用Dao层操作
    teacherDao.create(teacher);
    teachersLectureDao.batchBind(teacher.getID(), lectureIdList);
    logger.info("新增教师业务流程执行完成，教师ID：{}", teacher.getID());
}

你当前场景的处理方案 #

你目前的addTeacher方法仅做了Dao层create方法的透传调用，没有任何额外业务逻辑，完全不需要新增任何日志代码，保持现有实现即可。

内容的提问来源于stack exchange，提问作者Spam Garbage