Python 编码风格：错误检查和处理

Question

我正在用 python 编写一些软件，并且对 python 的首选编码风格有疑问。 想象一下，您有一个函数，它获取一些原始数据，将其解码为 dict 并打印键值对

def printdata(rawdata):
    data = decode(rawdata)
    for key, value in data.items():
        print(key, value)

这一切都很好，直到decode 开始到处抛出异常并且整个程序崩溃了。因此，我们使用 try/catch 块。但是有几种方法可以做到这一点，我想知道哪种方法更受欢迎。

1. try里面的所有东西

   def printdata(rawdata):
       try:
           data = decode(rawdata)
           for key, value in data.items():
              print(key, value)
       except ValueError:
           print("error")
   

2. 仅decode 内try 和return

   def printdata(rawdata):
       data = None
       try:
           data = decode(rawdata)
       except ValueError:
           print("error")
           return
       for key, value in data.items():
          print(key, value)
   

3. 只有decode 在try 和if 内

   def printdata(rawdata):
       data = None
       try:
           data = decode(rawdata)
       except ValueError:
           print("error")
       if data is not None:
           for key, value in data.items():
              print(key, value)
   

所有这些方法都有一些优点和缺点，我不知道该选择哪一种，以及它是否真的重要。

Answer 1

第一个显然是最简单的，但它有一个问题：如果套件中的其他任何东西都可能引发ValueError，则不清楚你是否抓住了@987654323 @您期望并想要处理，或者可能意味着您的代码中存在错误的意外问题，因此您可能应该让中止并打印回溯。

当你确定这不是问题时，就去做吧。

虽然真的，你几乎肯定应该像这样处理错误：

except ValueError as e:
    print("error: {!r}".format(e))

……或类似的东西。这样，如果您确实收到了不可能的意外ValueError，您将能够从意外消息中判断出来，而不是不知道您因为错误而丢弃了有效数据过去 3 个月的运行。

---

如果这不合适，其他两个想法都工作，但使用else 块通常更惯用。

def printdata(rawdata):
    try:
        data = decode(rawdata)
    except ValueError:
        print("error")
    else:
        for key, value in data.items():
           print(key, value)

---

如果你确实需要做#2（也许你有，比如说，try 语句里面有一堆 try 语句或其他东西......），你不需要顶部的 data = None，不应该拥有它。 应该如果不分配给data，您将无法通过return。所以，如果不可能的事情发生了，你想得到一个异常并看到它，而不是默默地把它当作None。

---

在#3 中，None 实际上是必需的。这是一个问题。在设置变量之前“预先声明”变量，然后检查它们是否已设置的整个想法不仅不惯用，而且还经常掩盖错误——例如，如果None 是来自decode 的有效返回怎么办？

Answer 2

“首选编码风格”是不“处理”错误，除非您可以真正处理它们。这意味着在库级别，您应该几乎没有错误处理 - 只是让错误传播到应用程序级别。在您想要的应用程序级别

1. 一个顶级错误处理程序，将正确记录未处理的错误完整的回溯（logging.exception() 是你的朋友），向用户显示用户友好的错误消息并崩溃。

2. 您实际上可以要求用户进行更正（再试一次，选择另一个文件，无论如何），请执行此操作。

仅打印错误消息 - 没有完整的回溯等 - 只是浪费每个人的时间。