Pylint为何对含return分支的if/else结构报no-else-return冗余警告

规则的判定逻辑 #

Pylint 对该问题的判定没有特殊的复杂语义识别，规则逻辑非常直白：

• 只要某段 if 分支内存在无条件的 return 语句，该分支关联的 else 分支就属于语法上的冗余结构
• 当 if 分支的条件满足时，代码执行流会直接通过 return 跳出整个函数，根本不会进入后续的 else 块，这个 else 的语法块没有存在的必要性

代码的等价性对比 #

你编写的原始代码：

def example(inp):
    if not isinstance(inp, list):
        return 'Not list'
    else:
        return 'List'

按照 Pylint 建议去掉 else 后的改写版本，和原代码的运行逻辑完全一致：

def example(inp):
    if not isinstance(inp, list):
        return 'Not list'
    # 代码会自然跳过if块的分支，走到这里执行后续return
    return 'List'

规则的设计目的 #

no-else-return 属于代码风格类建议，不属于代码错误告警：

• 平展的代码结构能减少不必要的嵌套层级，提升整体可读性
• 降低后续维护的误操作风险：保留 else 结构时，后续开发者如果在 if 块和 else 块之间插入逻辑，很容易被 if 块的提前 return 打断执行流，出现非预期的运行结果
• 如果你和团队约定要保留这种对称的 if-else 结构，完全可以在 Pylint 配置中关闭这条规则，它不会对代码的运行正确性造成任何影响

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