PHP7迁移后count()函数报错求助：如何修复IP处理代码？

这个坑我之前帮朋友排查过好几次——PHP 7.0开始对count()函数的参数做了严格的类型校验：只有数组或者实现了Countable接口的对象才能作为参数，而在PHP 5.x里，给整数、布尔值这类非Countable类型传count()，会悄悄返回1（非空值）或0（空值），不会报错。你代码里的问题正好踩中了这个变化。

先分析你代码里的问题根源 #

ip2long($ip)的返回值是：

• 成功转换IP时：返回一个整数（可能是正整数，也可能是负整数，取决于IPv4地址的范围）
• 转换失败（比如IP格式无效）时：返回false

你原来的代码里用count($ip)去判断一个整数/false，这在PHP 7+里直接触发count(): must be an array or object that implements Countable警告，而且这个判断本身逻辑就不对——你根本不需要用count()来检查IP转换是否成功。

另外，你后面的preg_match('/^\d+$/', $ip)也是多余的：因为ip2long()成功返回的肯定是整数，转成字符串后必然是纯数字，这个正则完全没必要。

修正后的代码 #

把原来的错误判断逻辑替换成直接检查ip2long()的返回值即可：

// 原IP字符串，比如："192.0.34.166"
$ipStr = $ip;
$ipNum = ip2long($ipStr);

// 判断IP转换是否失败，或者转换结果是否为0（对应0.0.0.0，如果你需要把这个也当成无效IP的话）
if ($ipNum === false || $ipNum == 0) {
    $ip = "0000000000";
} else {
    // 如果需要把整数转回字符串形式的数字
    $ip = (string)$ipNum;
}

额外说明 #

• 如果你的业务允许0.0.0.0作为有效IP，那只需要保留$ipNum === false的判断即可，去掉$ipNum == 0
• 为什么不用empty($ipNum)？因为empty(0)会返回true，如果你不想把0.0.0.0当成无效IP，就别用empty()，直接用严格的=== false判断转换失败的情况

这样修改后，既解决了PHP 7的警告问题，也让你的IP有效性判断逻辑更准确。

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