よくわかるPHPの教科書から、

質問

「Twitter風ひとこと掲示板」の作成でのエラー。

［OK］
・データーベース作成
・会員登録画面から登録可能
・データベースを見ると登録されていること確認

http://higelatta.bob.buttobi.net/nise_twitter/index.php

http://higelatta.bob.buttobi.net/nise_twitter/login.php

［NG］
・ログイン画面でログイン後、TOPのindex.php画面に飛んだとき、エラーが発生

■エラー内容
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '-5, 5' at line 1

以上です。
SQL文にエラーがありますと、バージョンがどーのこーのといっているようですが、
具体的な解決方法がわかりません。
以下に、TOPのindex.phpのソースを少々添付いたします。

<?php
session_start();
require('dbconnect.php');

if (isset($_SESSION['id']) && $_SESSION['time'] + 3600 > time()) {
// ログインしている
$_SESSION['time'] = time();

$sql = sprintf('SELECT * FROM members WHERE id=%d',
mysql_real_escape_string($_SESSION['id'])
);
$record = mysql_query($sql) or die(mysql_query());
$member = mysql_fetch_assoc($record);
} else {
// ログインしていない
header('Location: login.php');
}

// 投稿を記録する
if (!empty($_POST)) {
if ($_POST['message'] != '') {
$sql = sprintf('INSERT INTO posts SET member_id=%d, message="%s", reply_post_id=%d, created=NOW()',
mysql_real_escape_string($member['id']),
mysql_real_escape_string($_POST['message']),
mysql_real_escape_string($_POST['reply_post_id'])
);
mysql_query($sql) or die(mysql_error());

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

// 投稿を取得する
$page = $_REQUEST['page'];
if ($page == '') {
$page = 1;
}
$page = max($page, 1);

以上です。
長々と申し訳ありません。
おそらく、
if (isset($_SESSION['id']) && $_SESSION['time'] + 3600 > time()) {
付近に問題があるのかなと思うのですが、
詳しい方ご教授願います。

回答No.2

g_liar · ベストアンサー

動かないからと言って内容を理解せずに安易に削除するのは間違いです。

１回投稿してからプログラムを元に戻したら動くようになった理由は、全体像が見えないから憶測でしかありませんが…。

きっと、未使用の状態（＝レコード数が０件）が考慮されてないのでしょう。
投稿が１件でもあれば問題なく動くプログラムなのでは？
「レコード数が０件の場合はどうなるか？」を頭においてプログラムを見直してみてください。

ちなみに、書籍などに載っているサンプルプログラムは、経験上かなりの確率でバグを含んでいます。
勉強し始めの頃は書籍に頼って、サンプルプログラムの内容を理解せずに動かしてみることは僕もやりがちですが（＾＾；おおざっぱにでも理解をしてから動かしてみた方が結局のところ近道な気がします。

お礼

貴重なご意見ありがとうございます。

>>動かないからと言って内容を理解せずに安易に削除するのは間違いです。

まさにその通りでした。

現に別のサーバー上で同じステップを踏んだら、
ログインはできますが、DBへ書き込まれた情報が画面へ反映されなくなってしまいました。

もう一度きちんと理解しなおそうと思います。

回答No.1

g_liar

よくわかるPHPの教科書と言うことはPHPのお勉強中ですかね。
答えを書いてしまうと勉強になりませんので解決方法だけ。

エラーメッセージはよく読みましょう。
You have an error in your SQL syntax;…つまり、SQLの構文エラーです。
操作内容から言って、エラーになっているのは示されているindex.phpのもっと後ろの方だと思いますが、示されている部分でもSQLの構文エラーがあります。

構文エラーを見つける手っ取り早い方法はどんなSQLを実行しているかを調ること。
SQLの実行前に以下を挿入して実行しているSQLを表示してみましょう。
print "$sql<br>\n";

表示されたSQLを手動で実行してみて同じエラーになるものが問題のSQLです。

お礼

途中経過報告です。

// 最終ページを取得する
$sql = 'SELECT COUNT(*) AS cnt FROM posts';
$recordSet = mysql_query($sql);
$table = mysql_fetch_assoc($recordSet);
$maxPage = ceil($table['cnt'] / 5);
$page = min($page, $maxPage);

$start = ($page - 1) * 5;

$sql = sprintf('SELECT m.name, m.picture, p.* FROM members m, posts p WHERE m.id=p.member_id ORDER BY p.created DESC LIMIT %s, 5',
$start
);
print "$sql<br>\n";
$posts = mysql_query($sql) or die(mysql_error());

上記の様に、
print "$sql<br>\n";
を入れてみたところ、
SELECT m.name, m.picture, p.* FROM members m, posts p WHERE m.id=p.member_id ORDER BY p.created DESC LIMIT -5, 5
と表示されました。
それを手動で実行してみたところ、
元々のエラーと同じものをはきました。

上記構文の
$start = ($page - 1) * 5;
の「５」の値を1などに変更すると
エラー文が
created DESC LIMIT -5, 5
から
created DESC LIMIT -1, 5
に変わります。

従って、
上記構文から
$start = ($page - 1) * 5;
$sql = sprintf('SELECT m.name, m.picture, p.* FROM members m, posts p WHERE m.id=p.member_id ORDER BY p.created DESC LIMIT %s, 5',
$start
);
を削除してみました。

すると無事、ログインできるようになりました＾＾

ただ、
画面に文字を入力しても投稿できませんでした。
何も画面がかわりませんでした。

やはり、上記部分を削除してしまうのではなくて、
上限に合うような数値を入力するべきなのでしょうか？

ん？
いやいや、

先ほど削除したものを再び戻して、結局現状維持でとりあえずサーバーにUPしてみたのですが、
先ほど、削除した状態で投稿したけど、何も変化のなかった画面に
投稿した痕跡が表示されていました。

エラーが発生した当初の状態に戻したはずなのに、
今度はきちんとログインでき、
更には投稿も普通にできました。。。

なぜなのでしょうか。

先ほど、削除した部分は、最初のログインの時は不要で、
ログイン成功した後は、
再び戻せばいいものなのでしょうか？

長々と申し訳ありません。