在 php 7.4.3 环境中使用 `strpos()` 判断子串存在时,因未正确处理返回值(如 `0` 被误判为 `false`)导致逻辑失效,需严格使用 `!== false` 进行比较。
在 PHP 开发中,strpos() 是兼容 PHP 5.0+ 的经典函数,常用于替代 PHP 8.0+ 新增的 str_contains()。但一个极易被忽视的关键点是:strpos() 在目标子串位于字符串开头时返回 0,而 0 == false 为 true(弱比较)——这会导致本应匹配成功的判断被错误跳过。
你的代码中:
echo (stristr($result, 1) !== false) ? '...delete...' : '...update...';
虽已使用 !== false(✅ 正确),但存在两个潜在风险:
✅ 正确做法如下:
① 显式初始化并构建可安全匹配的结构
$result = []; // 使用数组存储 ID,避免字符串歧义
$stmt = $mysql->prepare("SELECT PRIVACY_ID FROM privacy_user WHERE USER_ID = ? AND VALID_TILL IS NULL");
$stmt->execute([$_SESSION['USER_ID']]);
while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
$result[] = (int)$row['PRIVACY_ID'];
}
// 转为索引数组便于快速查找(O(1))
$consentedIds = array_flip($result); // 如 [1=>0, 3=>1, 5=>2]② 安全检查(推荐)
③ 若坚持用字符串 + strpos(),必须确保精确匹配
// 将 ID 用分隔符包裹(如逗号),避免 1 匹配到 10/11
$result = ',' . implode(',', $result) . ',';
// 检查是否包含 ",1,"(注意前后逗号)
if (strpos($result, ',1,') !== false) { /* 已同意 */ }⚠️ 额外重要提醒:
'USER_ID'] 到 SQL 查询("WHERE USER_ID = '" . $_SESSION['USER_ID'] . "'"),这是严重安全隐患!务必改用预处理语句绑定参数(如上例所示);function str_contains_safe(string $haystack, string $needle): bool {
return strpos($haystack, $needle) !== false;
}综上,问题根源不在 strpos() 本身失效,而在于数据结构设计不合理 + 类型比较不严谨 + 安全实践缺失。优先采用数组键值映射方案,既高效、安全、可读性强,又彻底规避字符串解析歧义。