推荐关注
› Meteor
› JSLint - a JavaScript code quality tool
› jsFiddle
› D3.js
› WebStorm
推荐书目
› JavaScript 权威指南第 5 版
› Closure: The Definitive Guide
› Closure: The Definitive Guide
这是一个创建于 1093 天前的主题，其中的信息可能已经有所发展或是发生改变。
需求：多个方法一层一层嵌套，希望在各个方法出现异常时打断不必要的函数执行，进行异常处理并抛出给用户
现在的实现：
有调用链为 a() -> b() -> c() -> d() 的方法
每层如果出现异常，将会往 class ErrorHandler 添加异常，并往外抛出
抛出到 a() 时会使用调用 ErrorHandler 获取已记录的异常（ d 抛出的记录为 d ，以此类推）并进行处理
我的尝试：
- 每个方法里直接简单粗暴地 try catch：不知道这个异常究竟是哪里抛出的（ error 只有 type 和 message ）
- try catch 里抛出本层信息到 error 里继续抛出：高层的会覆盖底层的
- 手动实现一个栈：感觉不是最佳实践？
- try catch 里确认没有底层 error 信息才加入本层信息并抛出：感觉不如实现一个栈？
想请教：这种设计的最佳实践是怎么样的？ 是否尽量把调用拍扁更好？（但拍扁感觉无法有效复用）
 |
1
dcsuibian 2021-11-19 00:50:44 +08:00
如果是我的话，异常只在最顶层处理，中间不做 try...catch...
|
 |
2
KirbySD OP @dcsuibian 中间不做 try catch 我也想过，但感觉和第一条尝试一样无法捕获出现异常的具体位置…
|
 |
3
yanguango 2021-11-19 02:52:09 +08:00
* 只在最外层 catch ，stacktrace 会有哪里 throw 的信息，不需要自己处理
* 尽量使用 unchecked exception |
 |
4
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. |
|
5
dustinth 2021-11-19 07:54:48 +08:00
@dustinth 看节点是 JavaScript... 那关于 checked 那个讨论就不用了, 只需要关心在哪一层是否需要包装,是否需要处理了.
|
|
6
dcsuibian 2021-11-19 10:10:32 +08:00
|
 |
7
wancoco 2021-11-19 10:21:06 +08:00
加异常处理器集中处理是不是可以解决这个问题，只需要分场景判断是否需要加 try-catch ，对于二方、三方的接口就加上 try-catch ，其他引起的错误直接交给异常处理器去做
|
 |
9
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 |
Select Language