SQL更新中的变量

Question

只需建立此更新脚本即可更新数据库中用于Div样式的颜色。 我知道这对SQLInjection开放，但就目前而言，它仅是一个内部站点和一个概念。

  <?php
    $con = mysqli_connect('HOST', 'USER', 'PASSWORD','DATABASE');

    if (mysqli_connect_errno())
      {
      echo "Failed to connect to MySQL: " . mysqli_connect_error();
      }

    $user = $_GET['user'];
    $colour = $_GET['colour'];

    $sql= " UPDATE phoneavail
    SET Colour = '$colour' WHERE Name = '$user'"

    if (!mysqli_query($con,$sql))
      {
      die('Error: ' . mysqli_error($con));
      }

    header('Location: ' . $_SERVER['HTTP_REFERER']);

    mysqli_close($con);
    ?>

问题是它不会正确运行，我尝试了不同的查询，但均未成功。 颜色变量是从URL中提取的； 我通过回显$ colour变量进行了测试。

URL示例： http://localhost/test/php/phone/busy-demo.php?colour=green&user=Jack

表结构非常简单：

列| 姓名| 颜色|

样本数据| 杰克| 红色|

Answer 1

更新查询中缺少分号。

$sql= " UPDATE phoneavail SET Colour = '$colour'";

Answer 2

改成

if (mysqli_connect_errno())
  {
  echo "Failed to connect to MySQL: " . mysqli_connect_error();
  }

$user = $_GET['user'];
$colour = $_GET['colour'];

$sql= " UPDATE phoneavail SET Colour = '$colour' WHERE Name = '$user'"

if (!mysqli_query($con,$sql))
  {
  die('Error: ' . mysqli_error($con));
  }

mysqli_close($con);
header('Location: ' . $_SERVER['HTTP_REFERER']);
?>

并在重定向之前关闭连接。