来 review 代码

[v2eb]

https://i.bmp.ovh/imgs/2022/01/c75f78147c9fcbb0.png

[chihiro2014]

没有新增失败这种可能么？

[v2eb]

@chihiro2014 #1
有补充个注解 @Transactional(rollbackFor = {Exception.class, Error.class})

[SuperManNoPain]

除了事务以外，看着没啥问题，crud 基本操作

[micean]

表单没校验

[Habyss]

1. 参数校验
2. contractForm.getList().stream 非 null 校验

[chihiro2014]

@v2eb 但好像也没有捕获异常。顺带一提，那个 stream 操作如果很多的话，封装成一个 function 如何

[v2eb]

@micean #4
@Habyss #5
表单交给前端校验，List 确实大意了

[chihiro2014]

@v2eb 如果直接艹接口，那你后端不校验，不一样有问题么

[v2eb]

@chihiro2014 #8 已补充 @Validated 了，感谢

[v2eb]

@chihiro2014 #6
https://i.bmp.ovh/imgs/2022/01/1bb44523db997067.png
这里不知道怎么封装为好

[chihiro2014]

@v2eb

```java
public interface WrapperUtils {
BiFunction<ContractMoney, String, ContractMoney> convert = ((contractMoney, id) -> {
contractMoney.setId(id);
return contractMoney;
});
}
```

对应的那部分直接替换成 return WrapperUtils.convert.apply(contractMoney, id);

具体封装，还是看业务逻辑，可做可不做

[chihiro2014]

其实这种没什么意义的转换，写成函数式就好了=。=，很方便

[kekxv]

前端的不叫校验，叫做友好提示🐶