mysqli_fetch_assoc() 期望参数 1 mysqli_result,在 while 循环中给出字符串

mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, string given in while-loop

提问人: 提问时间:2/4/2020 更新时间:2/4/2020 访问量:156

问:

我目前有这段代码,但它给了我错误mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, string given on line 22

代码是这样的:

1   $servername = "localhost";
2   $user = "root";
3   $pass = "";
4   $db = "mafioso";
5
6   $con = mysqli_connect($servername, $user, $pass, $db);
7
8   $cash_utbetaling[0] = 50000000;
9   $cash_utbetaling[1] = 40000000;
10  $cash_utbetaling[2] = 30000000;
11  $cash_utbetaling[3] = 20000000;
12  $cash_utbetaling[4] = 10000000;
13
14  $kuler_utbetaling[0] = 25;
15  $kuler_utbetaling[1] = 20;
16  $kuler_utbetaling[2] = 15;
17  $kuler_utbetaling[3] = 10;
18  $kuler_utbetaling[4] = 5;
19
20  $i = 0;
21  $result = mysqli_query($con, "SELECT * FROM daily_exp ORDER BY exp DESC LIMIT 5");
22  while($row_best = mysqli_fetch_assoc($result)) {
23    
24    $acc_id = $row_best['acc_id'];
25
26    $sql = "SELECT * FROM accounts WHERE ID='".$acc_id."'";
27    $query = mysqli_query($con, $sql) or die (mysqli_error());
28    $row_top5 = mysqli_fetch_assoc($query);
29    
30    $result = "UPDATE accounts SET money = (money + ".$cash_utbetaling[$i]."), 
      bullets = (bullets + ".$kuler_utbetaling[$i].")  WHERE ID = ".$acc_id."";
31    mysqli_query($con, $result) or die("Bad query: $result");
32
33    $i++;
34  }

我似乎找不到错误,我在另一个文件中运行相同的代码并且没有问题。

php sql mysqli while-loop

评论

0赞 Bhargav Rao 2/5/2020
评论不用于扩展讨论;此对话已移至 Chat

答:

2赞 Cid 2/4/2020 #1

你写得太过分了$result

// This is supposed to be
//  a mysqli_result object ----------v-----v
while($row_best = mysqli_fetch_assoc($result)) 
{
    // some code
    // $result is now a string. Next iteration will raises the warning
    $result = "UPDATE accounts SET ...";
}

因此,您需要为变量提供不同的名称。命名查询不是最佳选择。$result


旁注。

内部查询容易受到 SQL 注入的攻击。应该使用预处理语句而不是连接字符串。

在示例中:

// prepare the query
$query = "SELECT * FROM accounts WHERE ID=?";
if ($stmt = mysqli_prepare($con, $query)) {

    // bind the param
    mysqli_stmt_bind_param($stmt, "s", $acc_id);

    // execute the query
    mysqli_stmt_execute($stmt);

    // get the result. Of course, avoir using the same variable name, again :)
    $result = mysqli_stmt_get_result($stmt);
}

有关使用 mysqli 预准备语句的更多信息,请阅读官方文档@YourCommonSense 编写的教程

评论

0赞 Dharman 2/4/2020
mysqli 上的官方文档。呸!我更喜欢链接到 YCS 的教程,这些教程是在线提供的关于 MySQLi 的最正确和最新的文档。
0赞 Cid 2/4/2020
@Dharman我不认识他们。如果您认为它们真的比官方文档更好,请编辑我的答案并提供链接
1赞 Dharman 2/4/2020
我会把它留在评论中:phpdelusions.net/mysqli/examples
1赞 Dharman 2/4/2020 #2

这里的主要问题是,您调用的 SQL 查询变量的名称与您在 while 循环中使用的结果相同。在第 30 行用字符串覆盖它:

30    $result = "UPDATE accounts SET money = (money + ".$cash_utbetaling[$i]."), 
      bullets = (bullets + ".$kuler_utbetaling[$i].")  WHERE ID = ".$acc_id."";

简单的答案是这是一个错别字,但你犯了更多的错误。

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$con = mysqli_connect("localhost", "root", "", "mafioso");
$con->set_charset('utf8mb4');


$cash_utbetaling[0] = 50000000;
$cash_utbetaling[1] = 40000000;
$cash_utbetaling[2] = 30000000;
$cash_utbetaling[3] = 20000000;
$cash_utbetaling[4] = 10000000;
$kuler_utbetaling[0] = 25;
$kuler_utbetaling[1] = 20;
$kuler_utbetaling[2] = 15;
$kuler_utbetaling[3] = 10;
$kuler_utbetaling[4] = 5;
$i = 0;

$result = mysqli_query($con, "SELECT * FROM daily_exp ORDER BY exp DESC LIMIT 5");
$daily_exp = $result->fetch_all(MYSQLI_ASSOC);

// prepare update
$stmt = $con->prepare('UPDATE accounts SET money = (money + ? ), bullets = (bullets + ? ) WHERE ID=?');

foreach ($daily_exp as $row_best) {
    $stmt->bind_param('sss', $cash_utbetaling[$i], $kuler_utbetaling[$i], $row_best['acc_id']);
    $stmt->execute();
    $i++;
}
  1. 您需要启用错误报告,而不是使用 ,因为缺少参数,这永远不会起作用。or die (mysqli_error())
  2. 不要使用 while 循环来浏览结果。最好一次获取所有记录或使用 循环执行结果。如果你使用我的建议,那么你很可能会完全避免你的错别字。foreach
  3. 您必须使用预准备语句。在您的代码中,值是整数和常量,但我假设您在某个时候会使用变量输入,这意味着您需要使用占位符和参数。
  4. 循环中的 SELECT 查询没有执行任何操作,因此我将其从答案中删除。
  5. 您应该努力避免 N+1 查询问题。如果可能,请仅在一个查询中尝试执行相同的操作。