Python中赋值时抛异常是否为不良实践？有无优化方案？

替代方案 #

可以通过以下几种方式减少重复代码，同时保持可读性：

• 封装通用检查函数
  写一个工具函数统一处理“值为空则抛异常”的逻辑，复用性强，调用直观：

  def require_not_empty(value, exc):
      if not value:
          raise exc
      return value
  
  # 使用示例
  some_variable = require_not_empty(some_function_call(), AnException("变量不能为空"))
  

  如果需要严格区分None和其他假值（比如0、空列表），可以把判断条件改成value is None，适配更精准的场景。

• 使用海象运算符（Python 3.8+）
  利用海象运算符将赋值和判断合并到一行，无需额外函数：

  if not (some_variable := some_function_call()):
      raise AnException("变量不能为空")
  

  这种写法简洁，且逻辑清晰，符合Python“显式优于隐式”的原则。

• 给函数加装饰器（针对自定义函数）
  如果是你自己实现的函数，可以给它加装饰器，让函数自身在返回空值时自动抛异常，调用时无需再做检查：

  def ensure_not_empty(exc_type):
      def decorator(func):
          def wrapper(*args, **kwargs):
              result = func(*args, **kwargs)
              if not result:
                  raise exc_type("返回值不能为空")
              return result
          return wrapper
      return decorator
  
  # 给目标函数应用装饰器
  @ensure_not_empty(AnException)
  def some_function_call():
      # 原函数逻辑，可能返回None或空值
      ...
  
  # 调用时直接赋值，无需额外检查
  some_variable = some_function_call()
  

关于or raise_exception写法的分析 #

你提到的这种写法属于不良实践，原因如下：

1. 可读性差
   这种利用or短路特性触发异常的写法非常不直观，大部分Python开发者需要反应一下才能理解逻辑，后续维护成本高。

2. 异常实例被提前创建
   不管some_function_call()是否返回真值，raise_exception(AnException())里的AnException()都会先执行。如果some_function_call()返回有效值，异常实例的创建就是不必要的开销；如果异常的初始化逻辑有副作用（比如写日志、占用资源），还会导致意外行为。

3. 布尔值判断的局限性
   or是基于值的布尔真假性判断，比如函数返回0、空字符串、空列表这些合法的假值时，也会触发异常。如果你的需求只是在返回None时抛异常，这种写法会误判。

4. 违背Python设计原则
   Python推崇“显式优于隐式”，这种技巧性的写法刻意隐藏了异常抛出的逻辑，不符合惯用的代码风格。

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