PHP和安全表单

时间:2013-09-16 15:33:30

标签: php mysql forms

我正在从PHP&书中做一个练习。 MYSQL简单易行。它涉及一个HTML表单来更新数据库中的行,然后是各种PHP脚本,以检查HTML代码的输入数据并使其成为安全格式。但是,代码并不像本书所说的那样工作。我去了出版商的网站并下载了代码示例,但没有快乐。

而不是下面有行名称的表单,而是我得到表单,然后在“没有提交有效的新名称”下面。然后在下面我要更改的表中当前行的名称。当我尝试输入数据并将其提交到表单中时,它没有任何区别。它显示完全相同的页面。代码如下。

<!DOCTYPE HTML>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Ensuring security
    </title>
</head>
<body>

<form action="secure.php" method="POST">
    <p>New Name : <input type="text" name="name">
        <input type="submit"></p></form>



<?php

require('../connect_db.php');


if (!empty($POST['name']) && !is_numeric($_POST['name'])) {
    $name = $POST['name'];

    $name = mysqli_real_escape_string($dbc, $name);
    $name = strip_tags($name);

    $q = 'UPDATE towels SET name "' . $name . '" WHERE id= 1';
    mysqli_query($dbc, $q);
} else {
    echo 'No valid new name submitted';
}

$q = 'SELECT * FROM towels WHERE id = 1 ';
$r = mysqli_query($dbc, $q);
while ($row = mysqli_fetch_array($r, MYSQLI_NUM)) {
    echo "<p>Name : $row[1] </p>";
}
mysqli_close($dbc);

我很欣赏这方面的任何想法。我花了大约3个小时去了发布商网站,但我仍然在第一个方面。

1 个答案:

答案 0 :(得分:5)

没有超全球数组$POST,因此您必须将$POST['name']更改为$_POST['name']

PHP无法看到该数组,因此它将!empty($POST['name'])评估为false,并且永远不会使用update查询执行代码。

而且,就像@BartFriederichs所说,买更好的书。我不认为你会从现有的东西中学到一些有价值的东西。