为何类父级判断函数不应使用全局$$Obj语句?兼析PHP手册争议评论
为啥这条PHP建议会被狂踩?差评完全合理!
咱们来拆解下这条被喷的建议到底错在哪——它绝对是个糟糕的编程实践,差评一点都不冤枉:
1. 强行依赖全局变量,彻底破坏函数封装性
原评论把函数改成依赖全局作用域里的变量,这完全违背了函数设计的基本原则。本来class_parentage应该接收一个对象实例作为参数,结果现在$obj变成了全局变量的名字,函数只能处理全局范围内的对象,没法用在局部作用域、类方法里的对象,复用性直接降到零。而且全局变量本身就是代码维护的噩梦,越多全局变量,越容易出现变量冲突、难以追踪的bug。
2. $$obj这种变量变量写法,既晦涩又危险
$$obj是PHP里的“变量变量”,意思是先把$obj的值当成变量名,再去取这个变量的值。这种写法可读性极差,别人看代码第一眼根本搞不清你要干嘛,后续维护成本飙升。更要命的是,如果$obj的值不可控(比如来自用户输入、外部接口),很容易意外覆盖或访问到敏感的全局变量,埋下严重的安全隐患。
3. 完全偏离原函数的设计意图,把简单问题复杂化
原函数的核心需求是判断一个对象是否是某个类的子类实例,正确的实现方式本来非常简单:直接接收对象实例,调用is_subclass_of($obj, $class)就够了。评论里的修改完全是画蛇添足——如果原函数出现“Undefined variable”错误,那应该是调用时没正确传递参数,而不是去拉全局变量来“补锅”。
正确的打开方式应该是这样:
function class_parentage($obj, $class) { // 先做个类型校验,避免传入非对象的情况 if (!is_object($obj)) { echo "Le premier paramètre doit être un objet\n"; return; } if (is_subclass_of($obj, $class)) { echo "L'objet appartient à la classe " . get_class($obj) . ", une sous-classe de $class\n"; } else { echo "L'objet n'appartient pas à une sous-classe de $class\n"; } }
直接传对象实例,不依赖全局,代码清晰、安全、复用性强,这才是合理的实现。
总的来说,这条建议不仅没解决本质问题,还引入了一堆坏编程习惯,被大量差评完全是情理之中的事。
内容的提问来源于stack exchange,提问作者Sam Dean
相关产品推荐
相关产品推荐

