PHP CodeSniffer报错：闭包内使用self::$types的原因及解决办法

嘿，这个问题我之前也碰到过，咱们先搞清楚为啥CodeSniffer会挑这个刺，再看怎么解决～

为什么会触发这个报错？ #

首先得明确：这不是PHP原生的语法错误（你说代码能正常运行就印证了这点），而是PHP CodeSniffer的代码规范检查规则在生效。

大部分情况下，这条报错来自于对代码可读性和上下文清晰度的要求：闭包是一个独立的匿名函数，直接在闭包内部使用self::$types，可能会让后续阅读代码的人混淆闭包的依赖关系——尤其是当这个闭包被传递到其他类或函数中时，self指向的类很容易被误解。另外，早期PHP版本对闭包访问外部静态成员的处理有一些细节差异，部分规范为了兼顾兼容性和代码整洁性，就把这种写法列为了“不推荐”。

解决办法（几种可选方案） #

根据项目规范和PHP版本，你可以选下面任意一种方案：

方案1：把静态属性提前赋值给变量，通过use引入闭包 #

这是最稳妥、最符合大多数规范要求的写法：

// 先把静态属性赋值给外部变量
$allowedTypes = self::$types;
return array_filter($methods, function ($method) use ($allowedTypes) {
    return in_array(strtolower($method['type']), $allowedTypes);
});

通过use关键字明确告诉闭包要使用外部的$allowedTypes变量，让代码的依赖关系一目了然，CodeSniffer自然就不会报错了。

方案2：用static::$types替代self::$types（PHP≥5.4适用） #

如果你的PHP版本在5.4及以上，可以试试把self换成static：

return array_filter($methods, function ($method) {
    return in_array(strtolower($method['type']), static::$types);
});

static是延迟静态绑定，和self的区别是：self始终指向当前定义这个闭包的类，而static会指向实际调用的类（如果有继承场景的话）。不少CodeSniffer规则会允许这种写法，因为它比self更明确地表示静态属性的来源。

方案3：修改CodeSniffer规则（不推荐） #

如果你的项目团队确实需要保留self::$types的写法，可以找到对应的规则（比如Squiz家族里的相关规则），在项目的phpcs.xml配置文件里禁用这条检查。不过除非有特殊业务理由，否则不建议这么做——毕竟代码规范的存在是为了保持团队代码风格的一致性。

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