捕获异常后抛出无关联新异常：属不良处理还是有合理原因？

我维护的一款应用在执行某流程时失败，但日志里没有足够有效信息。查看日志相关代码后，发现了这样的结构：

public void MyFunction(MyObj param)
{
    try 
    {
        Do(param);
        Something(param);
        Useful(param);
    }
    catch (Exception)
    {
        throw new Exception("Something has gone wrong");
    }
}

这段代码的catch块捕获了包含真实错误原因的Exception，却抛出未关联原异常的新Exception，导致问题根源丢失，无法排查。请问这种做法仅属不良异常处理，还是存在合理实现理由？

结论：绝大多数情况属于不良异常处理，仅极端场景有牵强理由 #

核心问题：完全破坏异常排查链路 #

• 直接丢弃了原异常的堆栈跟踪、错误类型、详细信息，彻底切断了定位问题的关键线索——你根本没法知道是Do、Something还是Useful哪个环节出了问题，也搞不清是参数非法、资源访问失败还是其他具体原因。
• 抛出的通用错误信息“Something has gone wrong”毫无实际价值，既帮不到开发排查，也给用户提供不了有效反馈。

极端场景下的牵强“合理”情况 #

• 敏感信息泄露风险：如果原异常包含数据库密码、用户隐私等敏感数据，且日志系统会全量输出异常信息，同时又无法通过过滤敏感字段等方式处理，可能有人会选择抛出无关联的通用异常。但这属于非常极端的情况，正确做法应该是清理原异常的敏感内容后，以内部异常的形式保留核心排查信息，而非完全丢弃。
• 认知不足导致的错误：部分开发者对异常处理的理解有误，误以为“捕获异常后要抛友好错误”，但不知道可以通过throw new Exception("提示信息", ex)将原异常作为内部异常传递，这种属于技术误区，而非合理设计。

正确的处理方式 #

如果需要统一异常提示，务必保留原异常链路：

catch (Exception ex)
{
    // 可在此处添加日志记录，留存原异常的完整信息
    throw new Exception("执行流程时发生错误", ex);
}

若不需要包装，直接重新抛出原异常即可：

catch (Exception ex)
{
    // 记录原异常日志
    throw;
}

内容的提问来源于stack exchange，提问作者Luke