php 查询循环不起作用

Question

我有这个代码:

public function updateOrder($num, $ufood, $uquan) {
    $response = array();    

    mysql_query("SET NAMES 'utf8'");

    foreach ($ufood as $index => $f) {
        $result = mysql_query("SELECT food, quantity, uquantity FROM table1 WHERE food ='".$f."'") or die(mysql_error());  
        $no_of_rows = mysql_num_rows($result);

        $response['number rows'] = $no_of_rows;

        if ($no_of_rows>0) {
            while ($row = mysqli_fetch_array($result)); {
                if (!$row['uquantity']) {
                    $w = "INSERT INTO table1(uquantity) VALUES ('$uquan[$index]')";

                    mysql_query($w);
                    $e = (int)$row['quantity'];
                    $q = (int)$uquan[$index];
                    $sum = $e+$q;
                    $s = (string)$sum;
                    $d = "UPDATE table1 SET quantity = '$s' WHERE food = ".$row['$food']." ";
                    mysql_query($d);

                } else if($row['uquantity']) {
                    $c = (int)$row['uquantity'];
                    $q = (int)$uquan[$index];
                    $sumq = $c+$q;
                    $sq = (string)$sumq;
                    $d = "UPDATE table1 SET uquantity = '$sq' WHERE food = ".$row['$food']." ";
                }
            }
        } else {
            $string ="INSERT INTO table1(food,uquantity) VALUES ('".$f."','".$uquan[$index]."')";
            $z = mysql_query($string);      
        }
    }
}

好吧，我无法完成这项工作，而且我正在尝试各种方法，但仍然无法正常工作。所以我有一些问题:

• 这种foreach和while的结构是否有效？

• 虽然 $result 查询从数据库返回一些行，当我尝试使用 $row['quantity'] 作为值时，我获取 null。

在这段代码中，我从 Android 应用程序接收了一些数据，并且我尝试“查看”是否已经有我的 db_table(table1) 类型食物的条目。如果有条目，我希望数据库将发送的 android 的数量条目、数据与我的数据库中的条目相加，并更新该字段。基本上就是这样。但正如我所说，当我尝试使用来自数据库的数据时，我得到了空值。

如果有人可以给我一些提示，因为我真的卡住了..

Answer 1

你的代码有很多问题。我将此答案标记为 Community Wiki，并邀请其他人编辑和添加他们找到的内容。

您也可以考虑发布到 https://codereview.stackexchange.com/相反，当你有很多错误时，直到你有一个更具体的问题。

错误的变量插值

这一行不会做你想做的事:

$w = "INSERT INTO table1(uquantity) VALUES ('$uquan[$index]')";

这不是完全有效的 PHP 语法。您可以连接表达式:

$w = "INSERT INTO table1(uquantity) VALUES ('".$uquan[$index]."')";

或者你可以在花括号中嵌入表达式:

$w = "INSERT INTO table1(uquantity) VALUES ('{$uquan[$index]}')";

或者您可以使用查询参数占位符:

$w = "INSERT INTO table1(uquantity) VALUES (?)";
$stmt = mysqli_prepare($w) or die(mysqli_error());
$uqi = $uquan[$index];
mysqli_stmt_bind_param($stmt, "i", $uqi);
mysqli_stmt_execute($stmt);

混合 MySQL API

您不能将 mysql_query() 与 mysqli_fetch_array() 混合使用。 PHP 为 MySQL 提供了不止一种 API，您不能混合使用它们。您应该标准化使用 mysqli API，因为现在不推荐使用旧的 mysql API。

分号打败while循环

while 语句后的分号使循环成为空操作，当它终止时，$row 不包含任何内容。

while ($row = mysqli_fetch_array($result)); {

应该是:

while ($row = mysqli_fetch_array($result)) {

不恰本地使用变量

用单引号变量引用 $row 键可能不是你的意思，有多种方式:

$d = "UPDATE table1 SET quantity = '$s' WHERE food = ".$row['$food']." ";

您之前的 SELECT 查询的选择列表中的列名称是“food”，而不是“$food”。

此外，即使您打算使用变量名称 $food 作为键，将它放在单引号中也不会使用变量的值，它会是文字字符串 '$food'。

无法引用字符串文字？

此外，您使用带引号的文字与 SELECT 查询中的 food 列进行比较，这让我认为它可能是一个字符串。

所以更新应该是这样的:

$d = "UPDATE table1 SET quantity = '$s' WHERE food = '".$row['food']."' ";

或者:

$d = "UPDATE table1 SET quantity = '$s' WHERE food = " . intval($row['food']);

或者最好使用参数和准备好的查询，这样你就不需要担心引号或类型:

$d = "UPDATE table1 SET quantity = ? WHERE food = ?";
. . .

未能检查错误

每个查询都可能会失败，这可能是因为您有语法错误(例如，字符串没有引号)，或者因为表中没有您引用的名称的列，或者权限问题等。

始终 在运行 SQL 查询时检查查询函数的返回状态。如果出现错误，该函数返回 false，如果发生这种情况，您必须检查错误消息。

mysqli_query($mysqli, $d) or trigger_error(mysqli_error($mysqli), E_USER_ERROR);

执行更新失败

您的第二次更新将 SQL 查询字符串分配给变量 $d，但随后根本不执行该更新查询!