为了避免这样的混淆,我总是sprintf()
在需要连接字符串的地方使用
改变:
global $wpdb;
if ( isset ( $_POST['id'] ) && ! empty ( $_POST['id'] )) {
$wpdb->query("DELETE " . PRO_TABLE_PREFIX . "tutorial WHERE id='{$_POST['id']}'");
}
至:
global $wpdb;
if ( isset ( $_POST['id'] ) )) {
$wpdb->query(sprintf("DELETE %stutorial WHERE id='%s'", PRO_TABLE_PREFIX, $_POST['id']));
}
有几点需要注意:
1)你很容易受到 SQL 注入的攻击
2)一旦你isset()
确定了 key 是否$_POST['id']
真的不是NULL
,你就不需要检查它是否为空 viaempty()
更新
你真的应该测试$_POST['id']
它是否有效。我建议你实现一个功能,比如,is_id_valid()
function is_id_valid(&$id){ //<-- IMPORTANT, Argument should be a reference
if ( ! isset($id) ){
return false;
}
if ( empty($id) ){
return false;
}
// add this if you expect $id to be a numeric value
// otherwise just ignore - do not add
if ( ! is_numeric($id) ){
return false;
}
//it's also a good to validate the length
if ( strlen($id) > ... && strlen($id) < ... ){
return false;
}
//seems like all tests passed
return true;
}
然后你会使用它,比如
if ( is_id_valid($_POST['id']) !== false ){
....
}
警告:它仍然容易受到 SQL 注入的影响