可能是一个新手问题,但我是自学成才的,我正在尝试编辑一些超出我的舒适区域的代码,并且没有被困住。请帮助!
是一个从MySQL数据库中选择数据并返回数组的函数。它使用了几个其他的"核心"功能来帮助数据库连接。
原始脚本如下所示:
class Core{
protected $db, $result;
private $rows;
public function __construct() {
$this->db = new mysqli('localhost', 'root', 'password', 'db');
}
public function query($sql){
$this->result = $this->db->query($sql);
}
public function rows(){
for($x = 1; $x <= $this->db->affected_rows; $x++){
$this->rows[] = $this->result->fetch_assoc();
}
return $this->rows;
}
}
class Chat extends Core{
public function fetchMessages(){
$this->query("
SELECT `chat`.`message`,
`users`.`username`,
`users`.`user_id`
FROM `chat`
JOIN `users`
ON `chat`.`user_id` = `users`.`user_id`
ORDER BY `chat`.`timestamp`
DESC
");
return $this->rows();
}
}
问题是,在聊天类的fetchMessages()函数中有一个SELECT子句,该子句将"Chat"表与"users"表连接以获取用户名。如果出于某种原因(删除、禁止、退出等),用户ID在用户表中不存在,则SELECT子句不返回结果。
如果用户不存在,为了使它仍然返回消息,我认为我需要将JOIN分隔为2个SELECT子句:
-
首先,SELECT
然后,SELECT
username
FROMusers
WHEREuser_id
=$user_id
,如果没有找到行返回"Guest"
message
, user_id
FROM chat
ORDER BY timestamp
DESC;(我已经得到了伪代码或逻辑在我的头脑中,我只是不能编码它!)
我的问题是,因为我使用$this->符号,我不知道如何在函数中包含第二个实例。我想做的是这样的:
public function fetchMessages(){
$this->query("
SELECT `message`, `user_id` FROM `chat` ORDER BY `chat`.`timestamp` DESC
");
$rows = $this->rows();
foreach ($rows as $row) {
$uid = $row['user_id'];
$this[2]->query("
SELECT `username` FROM `users` WHERE `user_id` = `$uid`;
");
$user = $this[2]->rows();
if ( $user['username'] == "" ) {
$username = "Guest";
} else {
$username = $user['username'];
}
$return_array[] = array($row['message'],$username);
}
return $return_array;
}
谁能理解我想做什么,并重新编写我的伪代码,使它不使用两个'$this->',实际上工作?
我真的很感激任何帮助…您只需将JOIN users
更改为LEFT JOIN users
。
您当前的代码执行INNER JOIN
,它仅在两个表上都有需要连接的行时才生成结果行。LEFT JOIN
将为左侧表中存在的所有行生成结果,当右侧表中不存在相应的行时,用NULL
替换右侧表中的值。
在您的示例中,这意味着您将获得所有消息的返回行,即使其中一些消息没有相应的用户。
参见SQL连接的可视化解释