腾讯工作近十年大佬：不是我打击你！你可能真的不会写Java

我们再来看接口实现：

public class UserInputDTOConvert implements DTOConvert { 
@Override 
public User convert(UserInputDTO userInputDTO) { 
User user = new User(); 
BeanUtils.copyProperties(userInputDTO,user); 
return user; 
} 
} 

我们这样重构后，我们发现现在的代码是如此的简洁，并且那么的规范：

@RequestMapping("/v1/api/user") 
@RestController 
public class UserApi { 
 @Autowired 
 private UserService userService; 
 @PostMapping 
 public User addUser(UserInputDTO userInputDTO){ 
 User user = new UserInputDTOConvert().convert(userInputDTO); 
 return userService.addUser(user); 
 } 
} 

review code

如果你是一个优秀的 Java 程序员，我相信你应该和我一样，已经数次重复 review 过自己的代码很多次了。

我们再看这个保存用户的例子，你将发现，API 中返回值是有些问题的，问题就在于不应该直接返回 User 实体，因为如果这样的话，就暴露了太多实体相关的信息，这样的返回值是不安全的，所以我们更应该返回一个 DTO 对象，我们可称它为 UserOutputDTO：

@PostMapping 
public UserOutputDTO addUser(UserInputDTO userInputDTO){ 
 User user = new UserInputDTOConvert().convert(userInputDTO); 
 User saveUserResult = userService.addUser(user); 
 UserOutputDTO result = new UserOutDTOConvert().convertToUser(saveUserResult); 
 return result; 
} 

这样你的 API 才更健全。

（编辑：西安站长网）

【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容!