如何为指定用户批量增加100访问量？现有MySQL更新代码异常

问题根源 #

你的代码犯了两个关键错误：

• UPDATE语句无WHERE条件：每次循环执行UPDATE时，都会修改表中所有记录，而非当前遍历的那条用户数据
• 循环覆盖变量导致最终统一赋值：每次循环会把当前用户的visiter赋值给变量$visiter，最后一次循环后，这个变量会变成最后一条符合条件的数据的访问量，所以所有记录最终都被更新为这个值加100

解决方案1：修复循环逻辑（不推荐，效率低） #

如果一定要保留循环写法，必须在UPDATE语句中加入唯一标识的WHERE条件（假设表中有id作为用户唯一主键），同时建议用预处理语句避免SQL注入：

$views = 100; 
$res_query = "SELECT id, visiter FROM inventory_test WHERE status='0' AND role='0'";
$result = mysqli_query($conn, $res_query);

// 预处理UPDATE语句提升安全性和效率
$updateStmt = mysqli_prepare($conn, "UPDATE inventory_test SET visiter = ? WHERE id = ?");
mysqli_stmt_bind_param($updateStmt, "ii", $newVisiter, $userId);

foreach($result as $res){
    $userId = $res['id'];
    $newVisiter = $res['visiter'] + $views;
    mysqli_stmt_execute($updateStmt);
}

// 关闭预处理语句
mysqli_stmt_close($updateStmt);
$msg="Views has been successfully updated.";

解决方案2：单条UPDATE语句完成（推荐，高效安全） #

完全不需要查询再循环，直接让数据库自己处理每条符合条件记录的数值累加，这是最优写法：

$views = 100; 
// 直接用SQL的字段自增逻辑，一次操作完成所有符合条件的记录更新
$updateQuery = "UPDATE inventory_test SET visiter = visiter + ? WHERE status='0' AND role='0'";
$updateStmt = mysqli_prepare($conn, $updateQuery);
mysqli_stmt_bind_param($updateStmt, "i", $views);
mysqli_stmt_execute($updateStmt);

$msg="Views has been successfully updated.";

这种写法不仅避免了循环带来的错误，还大幅提升了执行效率，同时用预处理语句杜绝了SQL注入风险。

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