用于将函数作为属性进行比较的 Python 代码检查器

Question

我偶尔会花费大量时间来追踪代码中的脑残……虽然我通常会针对它运行 pylint，但有些东西会忽略 pylint。对我来说最容易忽视的问题是……

# normally, variable is populated from parsed text, so it's not predictable
variable = 'fOoBaR'
if variable.lower == 'foobar':
    #       ^^^^^<------------------ should be .lower()
    do_something()

pylint 和 Python 都没有对此提出异议……是否有一个 python 代码检查工具可以标记这个特定问题？

Answer 1

您建议如何使用代码检查器来验证这一点？这是完全合法的语法。与其检查此类错误，不如养成使用更好模式的习惯。

而不是：这样

variable = 'fOoBaR'
if variable.lower == 'foobar':
    #       ^^^^^<------------------ should be .lower()
    do_something()

做：

variable = 'fOoBaR'
sane_variable = variable.lower()
if sane_variable == 'foobar':
    do_something()

这样，您总是显式地对要比较的值调用 .lower()，而不是依赖就地方法调用和比较，这会导致到你正在经历的陷阱。

Answer 2

@Mike Pennington 我只想首先说我也经常遇到这种情况 -.-

@eyquem 'lower()' 是一个函数。 “lower”是一个函数指针（如果我没记错的话）。 Python 将允许您尝试运行此代码，但不会调用该函数。

我认为这很难捕捉的原因是你并不总是知道你调用方法的变量的类型。例如，假设我有 2 节课。

class Foo()
   def func(self):
      #do stuff
      pass

class Bar()
   self.func = "stuff"

如果您的代码中有一个函数接受参数“baz”，如下所示：

def myfunction(baz):
    print baz.func

def myfunction(baz):
    baz.func()

根据 baz 的类型，其中任何一个都可能有效。
但实际上无法知道 baz 的类型是“Foo”还是“Bar”。

编辑：我的意思是静态分析......

Answer 3

这是 pylint 票证#65910