1
Asens 2019-08-16 16:31:00 +08:00
试试 fluent-validator
|
2
arrow8899 2019-08-16 16:33:25 +08:00
assert 没毛病,符合 FailFast 理念,if 太多,容错性虽然好了,但是出了问题不好查原因
|
3
lihongjie0209 2019-08-16 16:37:04 +08:00
那就用 guava 的 precondition
|
4
chendy 2019-08-16 16:38:24 +08:00
一堆 if-else 不可取,很多库都有封装这种参数校验 /断言的功能,不满足需求的话就自己封装一些
另外一说还有 jsr-303 可用 再另外一说 Assert 是测试用的不是上线用的怕不是对 Assert 有什么误解 |
5
Cbdy 2019-08-16 16:39:46 +08:00 2
我认为用 Assert 是个好习惯
类似的比如 Guava 的 Preconditions,JDK 的 Objects::requireNonNull 看各大框架源代码基本已经是公认的最佳实践了,异常情况抛异常,Happy path 处理逻辑,比 Golang 的 if err != nil 不知道高到哪里去 |
6
Takamine 2019-08-16 16:44:57 +08:00
请问一下用了 Assert 之后抛出响应异常之后是怎么处理的。
|
8
Cbdy 2019-08-16 16:52:20 +08:00
@Takamine 库、组件让调用方处理,应用让用户处理
比如:Assert.hasText(searchConditon, "请输入合法搜索条件!"); |
9
qwerthhusn 2019-08-16 16:52:36 +08:00 2
抛 RuntimeException 确实非常效率低,但是也是建立在有大量异常抛出的情况下,如果这种情况发生了,只能说明这程序有 BUG。。
有些框架使用 fail-fast。几乎都是用的 Assert。这时候他们会用自定义的 Assert,跑出来的异常类重写了 fillInStackTrace 方法直接 return this,这样异常也变得没那么慢了,坏处就是没有了异常堆栈 |
10
grittiness OP @Cbdy 其实现在是要不要和其他同事统一的问题了,现在就显得你很另类的一样,就你一个人用 Assert。
我是不想用一堆 if 的,用 Assert 还有一个理由是在 Service 里面封装了两三层逻辑后,发现流程有问题要返回失败信息给前端的时候,我只能想到用 Exception 的方式来处理,要不然一层一层 return 再判断是否继续,这样的逻辑写起来就感觉很难受。 不过我是新人,入职不到一年,然后同事是好几年的老员工,我感觉也不好对同事反驳什么😂 |
11
warcraft1236 2019-08-16 16:57:54 +08:00
validator 啊
|
12
grittiness OP @Takamine 因为是前后端分离项目,我是直接用 Spring 的全局异常处理方式,实现 HandlerExceptionResolver 以后把异常信息转 JSON 返回前端
|
13
grittiness OP @qwerthhusn 学习到了,我以后去尝试一下这样的异常处理方式,谢谢
|
14
bobuick 2019-08-16 17:04:12 +08:00 2
参数校验稍微多一点的话, 用 assert 其实算不上方便, validator 吧.
另外这么用是没毛病的, failfast 原则. if 一大堆才是有毛病, 写个 web 还讲究到 jvm 的异常性能级别了, 你以为你在写 OS 呢, 做什么层级的工作考虑什么层级的事. crud 就是要讲究可读和可维护性, 这点破性能还不够塞牙缝的, 而且异常既然叫异常就是偶尔才会发生的 |
15
BBCCBB 2019-08-16 17:15:01 +08:00
该用的时候就用, 99.9%的情况不需要考虑 Assert 带来的性能影响.
|
16
luozic 2019-08-16 17:41:55 +08:00 via iPhone
用了 spring 了还在乎这点性能?
|
17
sampeng 2019-08-17 12:05:32 +08:00 via iPhone
lz,不要把工龄当回事。很多人是把 1 年的事重复做 N 年。如果你不想这样。按自己的想法走
|
18
Jrue0011 2019-08-17 16:34:35 +08:00
if/else 和 Assert 其实目的都不一样吧。。。参数在这个场景下是必须的就用 Assert 或者 Objects.requireNonNull(),不是必须的就用 if/else 或者 Optional
|