V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
推荐关注
Meteor
JSLint - a JavaScript code quality tool
jsFiddle
D3.js
WebStorm
推荐书目
JavaScript 权威指南第 5 版
Closure: The Definitive Guide
KirbySD
V2EX  ›  JavaScript

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

  •  
  •   KirbySD · 2021-11-19 00:22:23 +08:00 · 2582 次点击
    这是一个创建于 1106 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

    我的尝试:

    • 每个方法里直接简单粗暴地 try catch:不知道这个异常究竟是哪里抛出的( error 只有 type 和 message )
    • try catch 里抛出本层信息到 error 里继续抛出:高层的会覆盖底层的
    • 手动实现一个栈:感觉不是最佳实践?
    • try catch 里确认没有底层 error 信息才加入本层信息并抛出:感觉不如实现一个栈?

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

    9 条回复    2021-11-19 12:03:11 +08:00
    dcsuibian
        1
    dcsuibian  
       2021-11-19 00:50:44 +08:00
    如果是我的话,异常只在最顶层处理,中间不做 try...catch...
    KirbySD
        2
    KirbySD  
    OP
       2021-11-19 01:16:41 +08:00 via iPhone
    @dcsuibian 中间不做 try catch 我也想过,但感觉和第一条尝试一样无法捕获出现异常的具体位置…
    yanguango
        3
    yanguango  
       2021-11-19 02:52:09 +08:00
    * 只在最外层 catch ,stacktrace 会有哪里 throw 的信息,不需要自己处理
    * 尽量使用 unchecked exception
    dustinth
        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.
    dustinth
        5
    dustinth  
       2021-11-19 07:54:48 +08:00
    @dustinth 看节点是 JavaScript... 那关于 checked 那个讨论就不用了, 只需要关心在哪一层是否需要包装,是否需要处理了.
    dcsuibian
        6
    dcsuibian  
       2021-11-19 10:10:32 +08:00
    wancoco
        7
    wancoco  
       2021-11-19 10:21:06 +08:00
    加异常处理器集中处理是不是可以解决这个问题,只需要分场景判断是否需要加 try-catch ,对于二方、三方的接口就加上 try-catch ,其他引起的错误直接交给异常处理器去做
    wancoco
        8
    wancoco  
       2021-11-19 10:37:11 +08:00
    @wancoco 搞错辽,以为是 java 呢
    aguesuka
        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
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1131 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 25ms · UTC 22:43 · PVG 06:43 · LAX 14:43 · JFK 17:43
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.