Question

我是一个初学者, 试图获得对 php 的处理。我遇到了一个我似乎无法解决的语法错误。我会给你们看看下面的代码和一些我尝试过的修补方法。如果有人有另一个好主意的话。谢谢 :

$subject_set = mysql_query("SELECT * FROM subjects", $connection);
if(!$subject_set){
  die("Database query failed: " . mysql_error());
}

while($subject = mysql_fetch_array($subject_set)) {
  echo "<li> {$subject[ menu_name ]} </li>";
}

$page_set = mysql_query("SELECT * FROM pages WHERE id_subjects = {$subject["id"]}", $connection);
if(!$page_set){
  die("Database query failed: " . mysql_error());
}

echo "<ul class= pages >";
while($page = mysql_fetch_array($page_set)) {
  echo "<li> {$page[ menu_name ]} </li>";
}
echo "</ul>";

I get: 数据库查询失败 : 您的 SQL 语法有错误 : 您在 SQL 语法中有一个错误; 请检查与您的 MySQL 服务器版本相对应的手册, 以便右语法在 " 第1行 " 附近使用

我知道问题在于 {$subject["id"]} 因为我把"Where id_subjects = 1"放回了内容,没有错误。

{$subject[ id ]}
{$subject["id"]}

但也有同样的错误...

Answer 1

试试

$page_set = mysql_query("SELECT * FROM pages WHERE id_subjects =  ".$subject["id"]." ", $connection);
                if(!$page_set){
                    die("Database query failed: " . mysql_error());
                }

BTW. 您应该真正远离 Mysql 函数。他们正在被贬低, 转移到 PDO 或 Mysqli, 这些功能也非常安全( 您现在很容易被 sql 注入) 。

Answer 2

如果你读回你的文章,你可以清楚地看到这里出了什么问题。

"SELECT * FROM pages WHERE id_subjects = {$subject["id"]}"

正如你所看到的,“id”与其余部分没有连接。这是因为“你关闭了字符串”。

简单地修正此功能

"SELECT * FROM pages WHERE id_subjects = " . $subject["id"]

或者如果您真的想要将变量放入字符串中,您可以为密钥使用一个引用的字符串:

"SELECT * FROM pages WHERE id_subjects = {$subject[ id ]}"

我个人是第一种解决办法的粉丝,但这只是我的看法。

Answer 3

当循环结束循环时, 它将会用尽所有结果。 $subject[ id] 将不会仅仅因为 $subject 不再有任何条目而有任何信息。

我猜你想先列出所有主题, 然后列出每个主题下的所有页面。

使用 MySQL 并不漂亮, 但这里是您想要做的。 (如 Bono 所说使用 PDO 或 Mysqli, 但这里使用 psuedocode 中的解决方案, 与 MySQL 有效 ) 。

loop through first query
    print subject name
    select pages using subject id
    loop through pages under that subject id
       print page names

Answer 4

在引用的字符串中,您不需要任何引号,只要使用

"SELECT * FROM pages WHERE id_subjects = {$subject[id]}"

友情链接