Python：通过引发错误来处理错误。 这是好习惯吗？

Question

我有一个 Python 应用程序，我有一个记录器。

在执行的不同步骤中，应用程序必须读取各种输入文件。 输入文件可以有不同的信息，但都是通过同一个 function 读取的。

我正在查看的特定信息之一是名为computation_id的信息，并且必须存在于其中一个文件中，但可以不存在于所有其他文件中。

我很想知道处理这种情况的正确方法是什么。 目前我正在这样处理它：

def input_reading(filename):
   results = {}
   [...]
   
   try:
     results['computation_id'] = read_computation_id()
   except KeyError:
     pass   
   [...]

   return results

因此，如果正在读取的文件中没有computation_id ID，则代码将继续运行。 但是，在某些时候，我将需要这个计算 ID，因此我需要检查它是否从我希望找到它的文件中正确读取。

我实际上需要这个值远远低于代码。 但是运行代码到此为止（这需要一些时间）然后失败是浪费计算时间。 所以我的想法是尽快检查这个值并通过以下方式处理错误：

def specifc_file_read(filename):
   [...]

   results = input_reading('my_file')
   try:
     results['computation_id']
   except KeyError:
     logger.exception('no computation id provided, aborting')
     raise SystemExit('no computation id provided, aborting')

   [...]

这是好习惯吗？

我有一种感觉，因为我需要编写特殊的代码行来“尽快”检查代码中的错误，以避免浪费计算时间。

由于我在错误处理方面没有太多经验，因此我想知道这是否是一种好习惯，以免养成不良习惯。

Answer 1

这取决于你喜欢什么，imo。 我认为这将更具可读性

if 'computation_id' not in results: raise ...

虽然如果您想在开始一些繁重的数据处理之前检查它是否在任何文件中可用，您可以

for f in get_files():
    if 'computation_id' in f:
        break
else:
    raise SystemExit

因此，如果它没有中断，它将引发SystemExit ，因此如果它至少在一个文件中不可用。