为什么此get函数会出现致命错误?

时间:2019-05-03 23:28:19

标签: php mysqli

我正在尝试根据3个要求选择帖子表中的所有行。

  1. 帖子表中的用户标识=用户表中的用户标识
  2. 帖子表中的主机ID =用户表中的用户ID
  3. 帖子表中的评论ID =零

我想选择所有符合那些要求的行。 这是错误...

Fatal error: Uncaught Error: Call to a member function fetch_assoc() on bool in C:\xampp\htdocs\loginsystem\includes\posts.inc.php:108 Stack trace: #0 C:\xampp\htdocs\loginsystem\home.php(38): getUploads(Object(mysqli)) #1 {main} thrown in C:\xampp\htdocs\loginsystem\includes\posts.inc.php on line 108

这是我的代码...

function getUploads($conn) {

        $userName = $_GET["user"];
        $sqluserid = "SELECT userid FROM users WHERE userName = $userName";
        $userid = mysqli_query($conn, $sqluserid);
        $sqlusercontent = "SELECT * FROM posts WHERE hostid = $userid AND userid = $userid AND commentid = 0";
        $usercontent = mysqli_query($conn, $sqlusercontent);
        $postid = 1;

    // This is the actual upload content
        while ($row = $usercontent->fetch_assoc()) {
            echo "<div class='postbox'><p>";
            echo $row['title']."<br>";
            echo $row['date']."<br>";
            echo "<div><img src='posts/".$userid."/".$postid.".*></div>";
            echo $row['description']."<br>";
            echo "</p>";
            echo "</div>";
            $postid++;
        }    
    }

2 个答案:

答案 0 :(得分:0)

您没有调用$userid->fetch_assoc()来获取第一个查询的结果。您需要执行此操作,然后使用$row['userid']将用户ID移出该行。

在第一个查询中,您还需要在用户名前后加上引号。最好使用预处理语句和$stmt->bind_param()来防止SQL注入。

但是首先不需要两个查询,您可以将两个表连接起来。

SELECT p.*
FROM posts AS p
JOIN users AS u ON p.userid = u.userid AND p.hostid = u.userid
WHERE u.username = '$userName' AND p.commentid = 0

您应该检查查询的结果,否则将在第一个查询中收到语法错误的通知。

$result = mysqli_query($conn, $sql) or die($conn->error);

答案 1 :(得分:0)

1)尝试添加

//This will prevent if you dont have any data
if ($usercontent = $conn->query($query)) {
    while ($row = $usercontent->fetch_assoc()) {
      //the rest of your code here
    }
/* free result set */
$result->free();
} else {
 echo "No Data";
}

2)您可以通过这种方式传递$_GET['user']作为参数,从而分隔如何传递用户名。

function getUploads($conn, $username) {

}

3)您可以使用INNER JOIN来创建一个查询

您的代码结尾应该是这样

function db () {
    static $conn;
    if ($conn===NULL){ 
        $conn = mysqli_connect ("localhost", "root", "", "database");
    }
    return $conn;
}

function getUploads($userName)
{
    $conn = db();
    $sqlusercontent = "SELECT p.*
    FROM posts AS p
    JOIN users AS u ON p.userid = u.userid AND p.hostid = u.userid
    WHERE u.username = '$userName' AND p.commentid = 0";

   // not suere why hostid is equal to userid ?
    $postid = 1; //$postid is not suppose to come from the db ?

    // This is the actual upload content
    if ($usercontent = mysqli_query($conn, $sqlusercontent)) {
        while ($row = $usercontent->fetch_assoc()) {
            echo "<div class='postbox'><p>";
            echo $row['title'] . "<br>";
            echo $row['date'] . "<br>";
            echo "<div><img src='posts/" . $row['userid'] . "/" . $row['postid']  . "'></div>";
            echo $row['description'] . "<br>";
            echo "</p>";
            echo "</div>";
            $postid++; //$postid is not suppose to come from the db ?

        }
        /* free result set */
        $usercontent->free();
    } else {
        echo "No Data";
    }
}
    $userName = $_GET["user"];
    getUploads($username);