php - 使用 PHP 进行表单和 SQL 验证

Question

当用户点击保存时，检查数据是否成功插入记录到表中，并显示相应的信息。如果表单为空，则应显示错误消息并且数据库不应有任何记录。

//SAVE

if (isset($_POST['SAVE'])) {
    $Name = $_POST['name'];
    $City = $_POST['city'];
    $query = "Insert Into Info Values('$Name','$City')";
    $result = mysqli_query($con, $query) or die ("query is failed" . mysqli_error($con));
    $Name = '';
    $City = '';
    if(mysqli_affected_rows($con)>0) {
        echo "record is saved";
    }else {
        echo "record is not saved";
    }

我只能通过 php 进行验证。我不确定我是否做对了。到目前为止，我可以在我的表单上收到“记录已保存”消息，但如果表单为空，我将无法收到后者。它只是在我的数据库中创建一条空记录。

Answer 1

未经测试的代码:

if (!isset($_POST['SAVE'], $_POST['name'], $_POST['city'])) {  // avoid Notices
    echo "Missing required submission data";
} elseif (!strlen(trim($_POST['name']))) {  // validate however you wish
    echo "Name data is not valid";  // explain however you wish
} elseif (!strlen(trim($_POST['city']))) {  // validate however you wish
    echo "City data is not valid";  // explain however you wish
} elseif (!$con = new mysqli("localhost", "root", "", "db")) {  // declare and check for a falsey value
    echo "Connection Failure"; // $con->connect_error <-- never show actual error details to public
} elseif (!$stmt = $con->prepare("INSERT INTO Info VALUES (?,?)")) {
    echo "Error @ prepare"; // $con->error;  // don't show to public
} elseif (!$stmt->bind_param("ss", $_POST['name'], $_POST['city'])) {
    echo "Error @ bind";  // $stmt->error;  // don't show to public
} elseif (!$stmt->execute()) {
    echo "Error @ execute";  // $stmt->error;  // don't show to public
} else {
    echo "Insert Successful"; 
}

提交数据的验证条件确保值不为空且不完全由空白字符组成。如果您希望进一步细化验证要求，只需更新条件即可。

如果只想简单的保证$_POST['name']和$_POST['city']不为空，可以替换前三个条件与

if (empty($_POST['SAVE']) || empty($_POST['name']) || empty($_POST['city'])) {
    echo "Submission data is missing/invalid";
}...

如果您不使用准备好的语句，那么像 Paul O'Malley 这样的名称值将破坏您的查询。更糟糕的是，如果有人想尝试运行一些注入(inject)攻击，您的查询就容易受到攻击。

检查 affected_rows() 是不必要的。如果查询执行没有错误消息，则 INSERT 查询成功。

以上建议都是我敦促您采纳的最佳做法。