service层代码相互调用, 导致spring循环依赖，设计上的优化

管理员创建用户需要发送激活邮件, 而发送激活邮件的时候需要判断发件人是不是合法的用户, 因此设计到一个循环依赖的问题

//UserService
@Service
class UserService{
    @Autowired
    private MailService mailService;

    @Autowired
    private UserMapper userMapper;

    public void createUser(User user){
        //创建用户
        userMapper.save(user);

        //发送邮件
        mailService.send(xxxx);
    }

    //判断用户是否合法
    public boolean isExist(String userId){//.... userMapper.xxxx}
}

//MailService
@Service
class MailService{
    @Autowired
    private UserService userService;

    public void send(Mail mail){
        //判断发件人是否存在
        String currentUserId = SessionContext.getCurrentUser().getUserId();
        if(!userService.isExist(currentUserId)){
            //用户不存在抛异常
        }
            //send mail
    }
}

改良：提出一个辅助的service，或者将

UserService 替换成UserMapper

//UserServiceHelper
@Service
class UserServiceHelper{
    @Autowired
    private UserMapper userMapper;

    //判断用户是否合法
    public boolean isExist(String userId){//.... userMapper.xxxx}

}

@Service
class MailService{

    @Autowired
    private UserServiceHelper userServiceHelper; //这里不同了

    public void send(Mail mail){
        //判断发件人是否存在
        String currentUserId = SessionContext.getCurrentUser().getUserId();
        if(!userServiceHelper.isExist(currentUserId)){
            //用户不存在抛异常
        }
            //send mail
    }
}

有人会说注入 UserMapper 不就行了, 是可以但是每次都调用Mapper的方法较为麻烦, 即使只多一行代码, 但是这种代码是没有语义化的

tip：

发送邮件和判断用户是否存在是两个独立的部分，不应该紧耦合在一起；重写一个单独的service，专门用来表达用户发送邮件，最敏捷的是放到controller里处理

UserService {
  // CRUD
  boolean isExists(String userId);
}

MailService {
  void send(Mail mail);
}

UserExistsCheckMailService {
  UserService userService;
  MailService mailService;
  void send(Mail mail) {
    if (!userService.isExists(...)) {
      throw Exception;
    }
    mailService.send()
  }
}

Tip：service自身调用自身也属于调用的循环，serviceX有两个方法，一个A，一个B，A里调用B，则自己造成循环，解耦办法：将A方法拆解，放到controller里或

将B方法放到不掉用serviceX里的方法的其他sevcie类里