UserServiceImpl中updateInformation方法不明确

来源：6-7 更新用户个人信息功能开发

小果酱家的小饼干

2017-12-04

public ServerResponse<User> updateInformation(User user){
    //这里用户名是不能被更改的
    //这里的email是需要校验的 为了是不让email在不同的用户中重复出现
    int resultCount = userMapper.checkEmailByUserId(user.getEmail(),user.getId());
    if(resultCount > 0){
        return ServerResponse.createByErrorMessage("亲，不好意思，该邮箱已经被注册了，请换一个试试呢");
    }
    User updateUser = new User();
    updateUser.setId(user.getId());
    updateUser.setPhone(user.getPhone());
    updateUser.setAnswer(user.getAnswer());
    updateUser.setQuestion(user.getQuestion());
    updateUser.setEmail(user.getEmail());
    int updateCount = userMapper.updateByPrimaryKeySelective(updateUser);
    if(updateCount > 0){
        return ServerResponse.createBySuccess("更新个人信息成功",updateUser);
    }
    return ServerResponse.createByErrorMessage("更新个人信息失败");
}

老师我这里不明确的地方有两个：

第一个：校验email 的时候使用了 userMapper的checkEmailByUserId方法，能不能使用之前写的checkValid
方法

第二个：userMapper.updateByPrimaryKeySelective(updateUser); 在更新操作的时候是我们自己建立的一个对象，而不是参数传进来的user参数，是否可以不用建立这个updateUser对象，而在返回这个对象之前，将对象中的密码置空 return ServerResponse.createBySuccess("更新个人信息成功",user); 老师，我不知道这样对不对。

写回答

1回答