PHP7.2升级至7.4后字符串校验引发服务器性能超时问题

这个问题的核心是两个问题叠加导致的性能雪崩： #

1. 不同判断写法的底层行为差异 #

原判断if (!$appointment['customer']['name'])存在隐性开销：

• 当$appointment里没有customer键，或是customer下没有name键时，PHP会先抛出E_NOTICE级别的错误，同时自动创建不存在的中间层级数组（比如凭空给当前预约记录加一个空数组类型的customer键），再把最终取不到的name值当作null处理。
• PHP 7.4相对于7.2，默认错误报告规则覆盖了更多E_NOTICE场景，且错误日志写入的开销更高。如果你的生产环境开了错误日志记录，每一条不符合要求的记录都会产生至少2条E_NOTICE日志，大量磁盘IO阻塞直接导致请求超时，数据量越大超时概率越高。

而修改后的!isset($appointment['customer']['name']) || empty($appointment['customer']['name'])从根源上消除了这个开销：
isset()和empty()是PHP的特殊语法结构，访问不存在的数组键时不会抛出任何错误，也不会自动创建中间层级的数组，没有额外的错误日志和数组修改开销。 #

2. 隐藏的unset逻辑错误放大了性能问题 #

你在遍历的时候用了引用&$appointment，本意是要删除外层$appointments数组中不符合条件的元素，正确写法应该是unset($appointments[$k])，但你写的是unset($appointment[$k])。
这行代码的实际作用是删除当前单条预约记录里键为$k的元素（比如遍历到第3条记录时删除这条记录里键为3的值），完全没有业务意义，还会额外抛出一层「数组键不存在」的E_NOTICE，进一步增加了错误日志的产生量。 #

额外优化建议 #

• 尽快修复unset的错误写法，否则不符合条件的元素其实并没有被删除，只是跳过了解密逻辑，仍然会被返回到前端。
• 引用遍历结束后建议加一行unset($appointment)，避免后续代码意外修改最后一个元素的引用。
• 生产环境建议把error_reporting设置为E_ALL & ~E_NOTICE & ~E_WARNING，避免非致命错误占用IO资源。

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