mysqli和php代码未能将字段正确插入数据库

Question

我有以下php代码，试图

a）要求用户输入学校密码b）查找数据库表“教师”，然后将新值（输入文本框）覆盖在数据库中学校密码的旧值上。

目前的问题是，它确实用新输入的pin向表添加了一行，但我希望它覆盖该特定教师的现有值（基本上覆盖现有值）。

php处理程序代码如下：

if (isset($_POST["addpin1"]) && !empty($_POST['schoolpin1']) && $_SESSION['teacher']) {
    $school_pin         = mysqli_real_escape_string($con,$_POST['schoolpin1']);
    $email              = $_SESSION['email'];
    $username           = $_SESSION['username'];
    $pass               = $_SESSION['pass'];
    $check_duplicate = mysqli_query($con,"SELECT school_pin FROM teachers WHERE school_pin='$school_pin' ");
    if(@mysqli_num_rows($check_duplicate) == 0){
        $query = mysqli_query($con,"INSERT INTO teachers (school_pin,email,username,pass) 
            VALUES('$school_pin','$email','$username','$pass')");
        if($query){
            header("Location: teachers.php");   
        }
    }
}

HTML表单代码：

 <form action="" method="post">
    <input type="text" name="schoolpin1" id="schoolpin1">
    <input type="submit" class="btn btn-success" name="addpin1" value="CHANGE PIN">

 </form>

数据库结构（tbl:教师）字段如下所示

id  
email  
school_pin  
username  
pass

发生这种情况的页面上有一名教师登录，因此他们的用户名显示在屏幕上。

更新：根据关于开始会话的答案之一。这已经完成了

<?php
require_once("scripts/connect_db.php");

session_start();
if(!isset($_SESSION['teacher'])) header('Location: teacher_login.php');;
$result="";
if (isset($_POST["logout"])) {


    session_destroy();
   header('Location:index.php');

}
$err = "";

Answer 1

这就是你需要的那种结构。如果存在当前记录，请使用新信息更新它。否则请插入。根据您想要更新的内容进行相应调整。也作为进一步的一点。请不要将用户输入直接放入数据库。通过将参数绑定到查询来使用准备好的语句。

 if(@mysqli_num_rows($check_duplicate) == 0){
    $query = mysqli_query($con,"INSERT INTO teachers (school_pin,email,username,pass) 
                VALUES('$school_pin','$email','$username','$pass')");
    if($query){
        header("Location: teachers.php");
    }
}else{

    $query = mysqli_query($con,"
          UPDATE teachers 
           SET email = '$email'  ,username = 
             '$username',pass = '$pass'
          WHERE school_pin='$school_pin'
     ");
    if($query){
        header("Location: teachers.php");
    }

}

Answer 2

不太清楚你在问什么——你的主键是什么？

您说“查找数据库表'teachers'，然后将新值（输入文本框）覆盖到数据库中学校pin的旧值”因此我猜您想更新当前教师的学校pin值？主要的逻辑问题是：

>

你永远不会做更新！

第一点：你有

$check_duplicate = mysqli_query($con,"SELECT school_pin FROM teachers WHERE school_pin='$school_pin' ");

我想你想要

SELECT school_pin FROM teachers WHERE username=` not `SELECT school_pin FROM teachers WHERE school_pin=

第二，如果执行更新，则需要在您的上设置一个其他块。或者（例如，如果username是主键），您可以在重复键更新上使用插入...，然后让mysql处理该行是否存在的检查。看https://dev.mysql.com/doc/refman/5.7/en/insert-on-duplicate.html

请查看如何进行参数化查询，并使用它们代替这里的字符串。从如何防止PHP中的SQL注入开始？

Answer 3

如果您正在使用会话变量，您需要首先启动会话。只需在脚本的开头添加session_start（）。