php的if判断出现的问题思考

问题

工作中有一个“大神”写的代码,其中出现了这么一幕,不看答案,你知道结果是什么吗?

<?php

function testIf($if){
    
    if($if == "on" || $if == 1){
       return true;
    }
    
    return false;
}

$if = 0;
var_dump(testIf($if));//运行结果:true

为什么执行结果是 true???$if这个变量既不是["on"],也不是[1],怎么就能够进这个判断里面去呢???

测试

在判断进行到 [if($if == "on"...] 的时候,他们其实是这样执行的

 if($if == (int)"on" || $if == 1)

因为php是弱类型语言,所以在比较一个整型和一个字符串的时候,执行时候会帮你强转,
字符串["on"]被强转后就是[0],也就等于[$if]了,所以bug就来了。。。我们的本意就是要判断他们不相等从而跳出去的,所以要达到目的,应该使用全等号。代码应该改为:

function testIf($if){
    
    if($if === "on" || $if === 1){
       return true;
    }
    
    return false;
}

建议

因为php是弱类型语言,带来方便的同时也容易出现各种各样预料之外的问题,所以我们在编码上应该避免这种类似的问题.$\color{#FF0000}{判断或传值的时候,尽量让相同类型的数据在一起,不同类型不要放在一起}$

posted @ 2021-02-03 14:38  续断  阅读(211)  评论(0编辑  收藏  举报