PHP重构count()调用，解决“参数需为数组或Countable对象”警告

你的第二个方案是完全正确的，符合原代码的预期逻辑，可以直接替换所有对应代码行。

原错误代码的实际运行逻辑 #

你对原代码的理解完全正确，PHP 7.4以前对字符串/Null调用count()的行为是：

• 任意字符串（包括空字符串''）调用count()都会返回1
• null调用count()会返回0
  所以原代码的判断逻辑本质是：只要delete_suff()的返回值不是null，就返回$ret_str，和返回的字符串是否为空没有关系。

方案1的问题 #

直接去掉count()的写法if($ret_str = $this->delete_suff(xxx))会把空字符串''判定为布尔值false，触发不符合预期的行为：
当delete_suff()返回空字符串时，条件判断不通过，不会执行return，和原代码逻辑完全不一致。

方案2的正确性 #

!is_null($ret_str = $this->delete_suff(xxx))的判断逻辑和原代码完全等价：

• 当返回值为非空字符串/空字符串时，is_null()返回false，取反后条件成立，执行return
• 当返回值为null时，is_null()返回true，取反后条件不成立，不执行return
  完全匹配原代码的业务逻辑，同时完全规避了对非可计数类型调用count()的警告。

如果你希望代码可读性更高，也可以把赋值和判断分开写，逻辑完全一致：

$ret_str = $this->delete_suff(REDACTED LIST OF ARGUMENTS);
if (!is_null($ret_str)) {
    return $ret_str;
}

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