根据数据库中保存的IP地址删除特定对象的投票访问权限

Question

我曾尝试就此问题发表几篇文章，但已决定收集最后一篇文章中的所有内容，希望以某种方式解决它。

我正在建立一个站点，用户可以在该站点上对数据库中的问题进行投票。 没有登录，因此，为确保每个人只能对每个问题投票一次，我将使用他们的IP和问题的ID。

首先，获取ID和IP地址并存储两者，确保它们是整数：

if(isset($_GET['id']))
          {

           //Get IP address

           //Test if it is a shared client
           if (!empty($_SERVER['HTTP_CLIENT_IP'])){
            $ip=$_SERVER['HTTP_CLIENT_IP'];

           //Is it a proxy address
           }elseif (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])){
            $ip=$_SERVER['HTTP_X_FORWARDED_FOR'];
           }else{
            $ip=$_SERVER['REMOTE_ADDR'];
           }

           //Save id and IP address as variables
           $id = $_GET['id'];
           $ip_long = ip2long($ip);

然后，使用两个变量检查​​用户是否已经投票。 我希望在这里出现问题。 我得到：

注意：试图获取非对象的属性

从第116行开始，即： $row_cnt = $result->num_rows 。

此外， var_dump ($result)返回bool(false)而var_dump ($row_cnt)返回Null 。 在查询中的两个变量$ ip_long和$ id周围加上引号可以解决在localhost而不是在我的服务器上的问题。

变量附近带有引号的本地var_dump($result)返回以下内容：

object（mysqli_result）＃2（5）{[“” current_field“] => int（0）[” field_count“] => int（1）[” lengths“] => NULL [” num_rows“] => int（1 ）[“ type”] => int（0）}

我想在特定问题的QuestionVotes中添加1，然后删除针对特定IP地址对该问题进行投票的选项。

//Save id and IP address as variables
           $id = $_GET['id'];
           $ip_long = ip2long($ip);

           ///Check to see if user already voted
           $stmt = $conn->prepare("SELECT * FROM User_Votes where UserID = ? and QuestionID = ?");
           mysqli_stmt_bind_param($stmt, 'ss', $ip_long, $id);
           $stmt->execute();
           $result = $stmt->get_result();
            if($result->num_rows){
                //The user has already voted
                echo "Already voted";
            }else{
                //Add IP Address and ID to the User_Votes table
                $stmt = $conn->prepare("INSERT INTO User_Votes (UserID, QuestionID) VALUES (?, ?)");
                mysqli_stmt_bind_param($stmt, 'ss', $ip_long, $id);
                $stmt->execute();
                $stmt = $conn->prepare("UPDATE Question SET QuestionVotes = QuestionVotes + 1 where QuestionID = ?");
                mysqli_stmt_bind_param($stmt, 's', $id);
                $stmt->execute();
            }

       }

最后，这是我用来构建包含数据库问题信息的html框，添加显示当前投票的投票按钮并在URL后面附加用作QuestionID的代码的代码：

// Build 4 question boxes from database Question table, including voting button
      $stmt = $conn->prepare("SELECT * FROM question ORDER BY QuestionVotes DESC LIMIT 4");
      $stmt->execute();

      $result = $stmt->get_result();
      if ($result->num_rows > 0) {
           // output data of each row
           while($row = $result->fetch_assoc()) {
               //$row["QuestionID"] to add id to url
               echo "<div class=\"col-md-3\"><h2>". $row["QuestionHeader"]. "</h2><p>". $row["QuestionText"]. "</p><p><a href=\"index.php?id=". $row["QuestionID"]. "\" class=\"btn btn-success\"> " . $row["QuestionVotes"] . "</a></p></div>";

           }
      }
      else
      {
        echo "0 results";
      }

我的表格如下：

问题： QuestionID（int11）（pk），QuestionHeader（varchar（20）），QuestionText（text），QuestionVotes（int（5））
用户投票： UserID（unsigned，int（39）），QuestionID（int（11））

Answer 1

我想指出几件事。 首先 ，您的错误：

我从第116行得到一个“通知：试图获取非对象的属性”，它是：$ row_cnt = $ result-> num_rows;。

当您使用不带结果的选择查询调用mysqli->query() ，返回的对象不是对象，而是false 。

其次 ，只需使用*代替COUNT(*) 。

因此，为了保持逻辑，您应该执行以下操作：

//Check to see if user already voted
$result = $conn->query("SELECT * FROM User_Votes where UserID = '$ip_long' and QuestionID = '$id'");

if ($result === false) { 
    //Add IP Address and ID to the User_Votes table
    $result = $conn->query("INSERT INTO `User_Votes` (`UserID`, `QuestionID`) VALUES ('$ip_long', '$id')");
}elseif($result && $result->num_rows) { 
    //The user has already voted
    echo "Already voted";
}

编辑：

//Check to see if user already voted
$result = $conn->query("SELECT * FROM User_Votes where UserID = '$ip_long' and QuestionID = '$id'");

if($result->num_rows){
    //The user has already voted
    echo "Already voted";
}else{
    //Add IP Address and ID to the User_Votes table
    $result = $conn->query("INSERT INTO User_Votes (UserID, QuestionID) VALUES ('$ip_long', '$id')");
}

重新编辑：

您必须在$stmt->store_result()之后调用$stmt->store_result() $stmt->execute() 。 而且这里的$stmt->get_result()是不必要的，因为您没有使用选定的数据。

来自文档的部分评论：

如果您不使用mysqli_stmt_store_result（），并且在执行准备好的语句后立即调用此函数，则该函数通常将返回0，因为它无法知道结果集中有多少行，因为结果集未保存在内存中然而。

因此，您的代码应如下所示：

if(isset($_GET['id']) && !empty($_GET['id'])){
    $id = $_GET['id'];
    $ip_long = ip2long($ip);

    //Check to see if user already voted
    $stmt = $conn->prepare("SELECT * FROM User_Votes where UserID = ? and QuestionID = ?");
    $stmt->bind_param('ss', $ip_long, $id);
    $stmt->execute();
    $stmt->store_result();
    if($stmt->num_rows){
        //The user has already voted
        echo "Already voted";
    }else{
        //Add IP Address and ID to the User_Votes table
        $stmt = $conn->prepare("INSERT INTO User_Votes (UserID, QuestionID) VALUES (?, ?)");
        $stmt->bind_param('ss', $ip_long, $id);
        $stmt->execute();
        $stmt = $conn->prepare("UPDATE Question SET QuestionVotes = QuestionVotes + 1 where QuestionID = ?");
        $stmt->bind_param('s', $id);
        $stmt->execute();
    }
}

旁注：请不要混合mysqli的过程式和面向对象样式。

Answer 2

您应该检查表的名称。

您可以在查询User_Votes之一中使用它，而在另一个user_votes中使用。 如果开发服务器由Windows支持，并且不区分大小写，那么它可能会在您的开发服务器上运行，但是区分大小写的对Linux的支持可能对您的生产服务器有效。

检查此问题以获取更多信息： MySQL中的表名是否区分大小写？

还要注意，从上面的代码来看，您的应用程序看起来对SQL注入是不安全的。 您应该将变量强制转换为int，或者将其转换为期望值。

Answer 3

您的insert语句使用单引号将变量括起来。 这些应该是双引号，因此PHP会将变量解释为值而不是文字字符串。

这似乎是正在发生的事情的根本原因。 在拉动它们之前，您是否能够验证所有内容均已正确写入数据库表？ 然后，验证您的select语句正确提取了数据，数据采用了什么形式？

jcaran的评论是正确的……您需要考虑对所获取变量的一些验证。