来 review 代码

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

V2EX 提问指南

这是一个创建于 1036 天前的主题，其中的信息可能已经有所发展或是发生改变。

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

Review

代码

来

13 条回复 • 2022-01-24 18:15:38 +08:00

chihiro2014

2022-01-24 13:53:44 +08:00

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

v2eb

2022-01-24 13:59:35 +08:00

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

SuperManNoPain

2022-01-24 14:02:04 +08:00

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

micean

2022-01-24 14:11:30 +08:00

表单没校验

Habyss

2022-01-24 14:26:52 +08:00

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

chihiro2014

2022-01-24 14:42:04 +08:00

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

v2eb

2022-01-24 15:12:27 +08:00

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

chihiro2014

2022-01-24 15:24:59 +08:00

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

v2eb

2022-01-24 16:20:46 +08:00

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

v2eb

2022-01-24 16:27:49 +08:00

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

chihiro2014

2022-01-24 17:03:28 +08:00

@v2eb

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

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

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

chihiro2014

2022-01-24 17:04:39 +08:00

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

kekxv

2022-01-24 18:15:38 +08:00 via iPhone

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