PHPLint的建议对我们的社区有好处吗？

Question

我最近发现了一个名为PHPLint的静态PHP代码分析器。它能做的事情之一就是像这样看代码：

/**
 * @return string
 */
function getText() {
  return 5;
}

抱怨一个5，不是一个字符串。在我看来，这可能会减少bug，甚至我们代码中的安全问题。一个好的方面是，我看到它可以给出的是，对于函数的返回值，只需要检查(=== null)。如果它不是null，那么您就得到了所期望的数据。

从表面上看，这似乎是对我们的代码库的一个很好的补充，核心和控制一样。但也许我看不出这有什么问题？

我开始应用适当的文档块来拥有我自己的模块，而我自己遇到的一个问题是，由于core经常使用包含混合数据的数组，所以PHPLint能够进行的检查是有限制的，这会引起很多警告。

编辑:一个可以防止的问题的具体例子是本通知来自php 5.4。这段代码总是失败的，只是在PHP 5.4隐式转换之前没有人注意到这一点。在这种情况下，bug是相对无害的，但是更微妙的错误和安全问题可能会以这种方式蔓延。

Answer 1

从理论上讲，这是有好处的。然而，做这件事有一些问题，因为你已经部分地发现了自己。

• 可能会有上百万个错误的警告，很难找到真正的问题，就像简单地阅读代码时一样。
• Drupal有很多定制的编码标准。例如，除了@return $class之外，我们目前不在docblocks中使用类型声明。

我不知道PHPLint有多可配置，也许有几个检查可以在我们的代码上运行。

对于编码样式问题，使用http://drupal.org/project/drupalcs可能更有意义。