update user information在username处个人认为存在逻辑问题

Question

首先是红线处，其次虽然controller讲username传到service，但是service的updateUser并没有使用

Answer 1

suwei666 2017-10-31 21:07:12

对的，本来上面的那个我一直不理解，因为就算user设置了，穿回来的updateuser里面也是没有username的，所以得在放在session之前设置上，就是下面的红线

Answer 2

Geely 2017-09-01 23:50:08

你好，同学，这个username我们是不允许更改的。 这里的逻辑的确是有冗余的。直接用下面的红线处就可以啦。username用户名无法更改，这个是后来定的逻辑。不过在这之前，写的时候的确没有注意。sorry~~

Answer 3

Geely 2017-06-20 00:24:51

hi 同学，是在第一个红线处的下一行使用了set的username

• 心平灬气和 #1

  geely老师，第一个红线处的在下面一行的方法里面没有涉及到username,username是不能被更新的，更新其他字段是根据userId，这一行应该是没必要的吧

• qq_白色恋人_1 #2

  Geely老师，我也认为这处有疑问，user.setUsername(currentUser.getUsername())这里user被set了一个username，但是在iUserService.updateInformation(user)这里user的username并没有被更新到dao层，我看了updateInformation方法的实现，确实没有。
  
  更新session里面的username用了第二个红线处的语句，所以我也觉得是不是第一个红线处的语句是多余的？

• Geely 回复 qq_白色恋人_1 #3

  你好，同学，这个username我们是不允许更改的。
  这里的逻辑的确是有冗余的。直接用下面的红线处就可以啦。username用户名无法更改，这个是后来定的逻辑。不过在这之前，写的时候的确没有注意。sorry~~