0

问题：

我正在尝试从名为 users 的数据库中的变量 cash_amount 中减去 0.05，并且我正在通过 ajax 调用此文件，但没有发生任何事情。为了解决这个问题，我在浏览器中打开了文件，我收到了这个错误：

错误：

您的 SQL 语法有错误；检查与您的 MySQL 服务器版本相对应的手册，以在第 1 行的 '' 附近使用正确的语法

编码：

PHP：

<?php
session_start();

$servername = "localhost";
$username = "myUser";
$password = "myPass";
$dbname = "myDBname";
$cash_amount = $_SESSION['cash_amount'];

// Create connection

$userid = $_SESSION['id'];

// You must enter the user's id here. /\

$conn = new mysqli($servername, $username, $password, $dbname);
// Check connection

if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
}

// Fetch the existing value of the cash_amount against that particular user here. You can use the SELECT cash_amount from users where userid = $userid
$_SESSION['cash_amount'] -= 0.05;
$newAmount = $cash_amount - 0.05;

$sql = "UPDATE users SET cash_amount = $newAmount WHERE id = $userid";
$result = $conn->query($sql);

if($result)
{
   echo "5 cents have been subtracted!";
}
else
{
   echo mysqli_error($conn);
   session_start();
   session_unset();
   session_destroy();
}

$conn->close();
?>

Javascript/AJAX：

function countdownEnded() {
    //make serverscreen dissapear
        document.getElementById('serverScreenWrapper').style.display = 'none';
        document.getElementById('serverScreenWrapper').style.opacity = '0';
        document.getElementById("cashOutNumTwo").style.right = '150%';
        document.getElementById("cashOutNumOne").style.right = '150%';
//start Timer
        setInterval(gameTimer.update, 1000);
//make player move again
        socket.emit('4');
        socket.emit('6');
//make game appear
        document.getElementById('gameAreaWrapper').style.opacity = 1;
//play sound
        document.getElementById('spawn_cell').play();
//cut 5 cents from account - php function
        $.ajax({
        type: "POST",
        url: 'http://cashballz.net/game/5game/subtract5.php',
        data: { },
        success: function (data) {
            alert(data);
        }
    });
}

我的数据库：

我的表称为用户，它位于数据库 casball_accounts 内部。

这是格式：

编号 | 名字 | 姓氏 | 电子邮件 | 密码 | 现金金额 | 4 之间 | 哈希 | 积极的

结论：

我对为什么我的 php 代码不起作用感到很困惑，我已经尝试过寻找修复程序，但我找到了“SQL Injection”这个词，但我仍然没有找到错误。我在 JS 方面很先进，但对 PHP 是初学者，所以请多多包涵。谢谢！

4

4 回答 4

1

您可以简化查询并执行

$sql = "UPDATE users SET cash_amount = cash_amount - 0.05 WHERE id = $userid";

但是为了避免 SQL 注入的可能性，我建议也更改代码以使用像这样的参数化和绑定查询

$sql = "UPDATE users SET cash_amount = cash_amount - 0.05 WHERE id = ?";
$stmt = $conn->prepare($sql);
$stmt->bind_param('s', $userid);
$result = $stmt->execute();
于 2018-06-26T23:15:05.610 回答
0

您正在将数据插入到查询中，如果这是您想要的，您可以使用 Mostafa 的答案并在这些值周围加上引号，以防它们格式错误。这也意味着您必须在将它们放入查询之前检查它们是否是合法值。

但是，它们也可能会受到用户输入的影响，因此请使用 PDO 的准备好的语句。

$sql = "UPDATE users SET cash_amount = :newAmount WHERE id = :userid";
$conn = $conn->prepare($sql, [PDO::ATTR_CURSOR => PDO::CURSOR_FWDONLY]);
$conn->execute([':newAmount' => $newAmount, ':userid' => $userid])

...应该解决这个问题。但是确保你拥有你认为你拥有的东西可以防止很多红鲱鱼。

于 2018-06-26T20:52:15.197 回答
0

语法错误抱怨附近有问题''，这意味着在它混淆的地方没有任何内容。即，它在查询结束时变得混乱。

如果$userid为空，您的 SQL 查询将是

UPDATE users SET cash_amount = ... WHERE id =

这显然是错误的语法，因为=需要两个操作数。

您需要确保您$userid的值为非空值。

其他答案中的建议尝试通过使用引号或查询参数来解决该问题，但真正的问题是您在使用它之前没有检查它$userid是否具有值。

于 2018-06-26T21:00:39.150 回答
-1

代替

   $sql = "UPDATE users SET cash_amount = '$newAmount' WHERE id = '$userid'";

反而

$sql = "UPDATE users SET cash_amount = $newAmount WHERE id = $userid";
于 2018-06-26T20:33:23.603 回答