使用 Spring Framework 时常犯的十大错误

通常，打破 关注点分离 的是将新功能简单地 “倒” 在现有类中。当然，这是一个很好的短期解决方案(对于初学者来说，它需要更少的输入)，但它也不可避免地会在将来成为一个问题，无论是在测试期间、维护期间还是介于两者之间。考虑下下面的控制器，它将从数据库返回 TopTalentData。

@RestController 
public class TopTalentController { 
 private final TopTalentRepository topTalentRepository; 
 @RequestMapping("/toptal/get") 
 public List<TopTalentData> getTopTalent() { 
 return topTalentRepository.findAll() 
 .stream() 
 .map(this::entityToData) 
 .collect(Collectors.toList()); 
 } 
 private TopTalentData entityToData(TopTalentEntity topTalentEntity) { 
 return new TopTalentData(topTalentEntity.getName()); 
 } 
}复制代码 

起初，这段代码似乎没什么特别的问题;它提供了一个从 TopTalentEntity 实例检索出来的 TopTalentData 的 List。然而，仔细观察下，我们可以看到 TopTalentController 实际上在此做了些事情;也就是说，它将请求映射到特定端点，从数据库检索数据，并将从 TopTalentRepository 接收的实体转换为另一种格式。一个“更干净” 的解决方案是将这些关注点分离到他们自己的类中。看起来可能是这个样子的：

@RestController 
@RequestMapping("/toptal") 
@AllArgsConstructor 
public class TopTalentController { 
 private final TopTalentService topTalentService; 
 @RequestMapping("/get") 
 public List<TopTalentData> getTopTalent() { 
 return topTalentService.getTopTalent(); 
 } 
} 
@AllArgsConstructor 
@Service 
public class TopTalentService { 
 private final TopTalentRepository topTalentRepository; 
 private final TopTalentEntityConverter topTalentEntityConverter; 
 public List<TopTalentData> getTopTalent() { 
 return topTalentRepository.findAll() 
 .stream() 
 .map(topTalentEntityConverter::toResponse) 
 .collect(Collectors.toList()); 
 } 
} 
@Component 
public class TopTalentEntityConverter { 
 public TopTalentData toResponse(TopTalentEntity topTalentEntity) { 
 return new TopTalentData(topTalentEntity.getName()); 
 } 
}复制代码 

这种层次结构的另一个优点是，它允许我们通过检查类名来确定将功能驻留在何处。此外，在测试期间，如果需要，我们可以很容易地用模拟实现来替换任何类。

4. 常见错误四：缺乏异常处理或处理不当

（编辑：惠州站长网）

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