如果它抛出一个异常，那么有一个不返回的函数是不好的做法吗？ [重复]

Question

这个问题在这里已有答案：

• Conventions for exceptions or error codes 22回答

我有一些计算功能，用于驱动硬件设置的软件配置。在某些情况下，用户可能输入无效的配置值。我通过在使用无效值时抛出异常来处理此问题，或者只是在配置有效时返回我的计算值：

def calc_exhale_dur(breath_rate, inh_dur, breath_hold_dur):
    """Calculate exhalation duration.

    Args:
        breath_rate (float): animal breath rate (br/min)
        inh_dur (float): animal inhalation duration (ms)
        breath_hold_duration (float): animal breath hold duration (ms)
    """

    single_breath_dur = calc_breath_dur(breath_rate)
    exhale_dur = single_breath_dur - (inh_dur + breath_hold_dur)

    if (inh_dur + breath_hold_dur + exhale_dur) > single_breath_dur:
        raise error.ConfigurationError
    elif exhale_dur <= 0:
        raise error.ConfigurationError
    else:
        return exhale_dur

以这种方式这样做被认为是不好的做法吗？如果有一个返回值，我是否总是需要一些有效的返回值？我正在努力学习如何最好地编写Pythonic代码，同时仍然满足我的计算方法的需要。

Answer 1

引发异常的目的是在无法找到有效返回值的情况下提供备用退出点。您正在使用与其预期完全相同的异常。

但是，我可能会先检查exhale_dir是否为非正数，这样可以避免执行带有无效值的计算。

if exhale_dur <= 0:
    raise error.ConfigurationError
elif (inh_dur + breath_hold_dur + exhale_dur) > single_breath_dur):
    raise error.ConfigurationError

# You can also omit the else, since the only way to reach this point
# is to *not* have raised an exception. This is a matter of style, though.
return exhale_dur

Answer 2

不会。例外情况要么在一段代码中处理，要么被抛出，并留给调用者决定如何处理它们。如果您返回某些内容，则必须选择第一个选项。你选择了第二个。有一个完美的感觉。这就是抛出异常的全部想法