这样做是值得的,尽管性能并不总是会提高,但通知通常是坏代码或坏代码的迹象。
在您引用的示例中,实际上存在一个错误和潜在的安全问题:
$geolocation = esc_sql($_COOKIE[\'geolocation\']);
<代码假定存在cookie,并且
geolocation
cookie存在,它使用
esc_sql
不应该在这里使用,
esc_sql
几乎从未被WP开发人员使用过,因为它主要在
WPDB
类地理位置不是SQL语句
esc_sql
是一个转义函数,而不是清除函数
对于未定义的时间没有默认值,更好的方法是:$geolocation = \'\';
if ( !empty( $_COOKIE[\'geolocation\'] ) ) {
$geolocation = wp_strip_all_tags( $_COOKIE[\'geolocation\'] );
}
在这里,我不确定默认值应该是什么,或者该值的格式,但您可以将其替换掉wp_strip_all_tags
对其进行消毒的等效物。您还需要一个验证步骤来确保它实际上是一个地理位置。通知和警告可能由很多事情引起,其中很多可能对性能没有影响,但它们可能是bug、安全问题或糟糕的代码质量的迹象!如果你买了一个产品,它开始向你发送PHP通知,那么这是一个坏兆头。
PHP警告和通知就像洗碗机里设计脱落的杯子,没有正确拧上的门把手,商店里有污渍的衣服,或者开始发出奇怪和不祥噪音的汽车。仅仅因为它不能让你的网站更快,并不意味着它们不应该被修复。