检查路径有效性的函数失败时应抛异常还是返回false？

核心问题点： #

• 违背单一职责原则：同一个函数同时承载了「返回检查结果」和「抛出异常中断流程」两种完全不同的行为，增加了调用方的理解成本和出错概率，一旦参数传错会直接导致非预期的程序崩溃或逻辑失效。
• 混淆了正常业务判断和异常的适用场景：路径不存在是CheckPathValid的预期内检查结果，不属于意料之外的系统错误，用参数控制是否抛出异常，相当于把正常业务逻辑和异常处理混为一谈。而且你现有示例中捕获通用Exception的写法还会吞掉其他非路径不存在的异常（比如参数非法、权限不足、系统IO错误等），后续排查问题时很难定位根因。
• 现有调用示例存在逻辑漏洞：遍历场景下你调用了CheckPathValid但完全不处理返回值，相当于检查逻辑没有起到任何作用。

优化方案 #

第一步：调整函数设计，保持职责单一 #

移除throwExceptionIfOffline参数，函数仅负责返回路径是否有效的布尔值，仅在遇到入参非法等真正的预期外错误时才抛出异常：

public bool CheckPathValid(string fullFilePath, int maxTimeMillisecond)

第二步：调用方根据场景自行控制流程 #

如果需要路径不存在就中断的场景，由调用方自行判断返回值后处理即可：

try {
    if (!CheckPathValid(@"\\somepath", 1000))
    {
        Log("func1 failed due to path not existing.");
        return; // 或者抛出自定义异常，完全由上层决定
    }
    // 后续需要跳过的代码
} catch (Exception ex) {
    Log($"func1 failed with unexpected error: {ex.Message}");
}

遍历路径的场景直接判断返回值处理即可：

try {
    foreach (var path in somepathlist) {
       if (CheckPathValid(path, 1000))
       {
           // 处理有效路径的逻辑
       }
       // 无效路径自动进入下一轮循环
    }
} catch (Exception ex) {
    Log($"func1 failed with unexpected error: {ex.Message}");
}

如果频繁用到「路径不存在就抛异常」的场景，可以单独提供一个重载方法，比用参数控制行为清晰得多：

public void CheckPathValidOrThrow(string fullFilePath, int maxTimeMillisecond)
{
    if (!CheckPathValid(fullFilePath, maxTimeMillisecond))
    {
        throw new IOException($"路径 {fullFilePath} 不可访问");
    }
}

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