多层嵌套异常捕获/抛出的最佳实践?

2021-11-19 00:22:23 +08:00
 KirbySD

需求:多个方法一层一层嵌套,希望在各个方法出现异常时打断不必要的函数执行,进行异常处理并抛出给用户

现在的实现:
有调用链为 a() -> b() -> c() -> d() 的方法
每层如果出现异常,将会往 class ErrorHandler 添加异常,并往外抛出
抛出到 a() 时会使用调用 ErrorHandler 获取已记录的异常( d 抛出的记录为 d ,以此类推)并进行处理

我的尝试:

想请教:这种设计的最佳实践是怎么样的? 是否尽量把调用拍扁更好?(但拍扁感觉无法有效复用)

2289 次点击
所在节点    JavaScript
9 条回复
dcsuibian
2021-11-19 00:50:44 +08:00
如果是我的话,异常只在最顶层处理,中间不做 try...catch...
KirbySD
2021-11-19 01:16:41 +08:00
@dcsuibian 中间不做 try catch 我也想过,但感觉和第一条尝试一样无法捕获出现异常的具体位置…
yanguango
2021-11-19 02:52:09 +08:00
* 只在最外层 catch ,stacktrace 会有哪里 throw 的信息,不需要自己处理
* 尽量使用 unchecked exception
dustinth
2021-11-19 07:51:34 +08:00
异常有两个维度: 是否可纠错, 是否需感知.

针对不同的层面(比如提供接口给上游或者外部), 这两个维度又是相对的, 比如 IO 错误, 如果让最终用户重试的话可以直接抛出 unchecked 错误, 如果让中间层重试的话则需要抛出 checked 错误; 中间层对于自己不能处理的错误是不需要感知的, 可以抛出 unchecked Exception, 然后包装给外部方可处理的情况下, 又可以把 unchecked 转为 checked Exception.

当然为了方便可以统一用 unchecked, 感兴趣的中间层可以 catch 特定的 exception 做处理.

至于 Exception 是否需要包装, 取决于系统怎么抽象自己的错误, 比如 IOException 和 NetworkException 或者有些 RuntimeException 对外部来说都是 InternalException.
dustinth
2021-11-19 07:54:48 +08:00
@dustinth 看节点是 JavaScript... 那关于 checked 那个讨论就不用了, 只需要关心在哪一层是否需要包装,是否需要处理了.
dcsuibian
2021-11-19 10:10:32 +08:00
wancoco
2021-11-19 10:21:06 +08:00
加异常处理器集中处理是不是可以解决这个问题,只需要分场景判断是否需要加 try-catch ,对于二方、三方的接口就加上 try-catch ,其他引起的错误直接交给异常处理器去做
wancoco
2021-11-19 10:37:11 +08:00
@wancoco 搞错辽,以为是 java 呢
aguesuka
2021-11-19 12:03:11 +08:00
js 的 Error 对象是有异常栈的, 也许你 ErrorHandler 没有继承 Error?
https://developer.mozilla.org/en-US/docs/web/javascript/reference/global_objects/error

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/816419

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX