PHPでのMySQL

Question

これだとエラーでないのに

$query = "INSERT INTO getTweet(url,name,sentence) VALUES('$url','$screen_name','$text')";

下だとInvalid request (Unexpected EOF)のようなエラーが出てしまいます。

$query = "INSERT INTO getTweet(tweet_id,url,name,sentence) VALUES('$tweet_id','$url','$screen_name','$text')";

どうしてでしょうか??

CREATE TABLE `getTweet` (
  `tweet_id` int(11) DEFAULT NULL,
  `url` varchar(500) DEFAULT NULL,
  `name` varchar(100) DEFAULT NULL,
  `sentence` varchar(255) DEFAULT NULL,
  `upload` datetime DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

Answer 1

'$tweet_id'は、シングルクォートで囲わなくてよいとか？

Comments

[adwadwadw]

すべてシングルクォートで囲わなくしたらできてた方もできなくなりました。

[fromageblanc]

シングルクォートいらないかもってのは、$tweet_idのみです。varcharの項目は取っちゃったらダメです。ちなみにPHPのソース上、$tweet_idの型がstringだったりしませんか？おそらくmysql側は数値型の項目にシングルクォートで囲われてても暗黙で変換してくれたと思いますが、渡ってくる型が文字列型だともしかしたらエラーになるのかもです。

[fromageblanc]

ちなみに、tweet_idは11桁じゃ足りないですよ。

[fromageblanc]

↑あ、twitterのapiで何かしている前提です。違ってたらすいません。

[KaedeKazane]

質問文のようにPDO等を介さず直接SQL文を組み立てているなら、$queryに代入された時点でただの文字列になるので元の変数の型は関係ないように思います。

あとは変数の内容に問題があるとか、SQL文の組み立て自体におかしなところがあるとかでしょうか…

[adwadwadw]

$tweet_idがString型でした。

[adwadwadw]

とりあえずデータベースのtweet_idをvarcharに変えました。

[fromageblanc]

動くのが確認できたら、SQLインジェクション対策してくださいね。pdo使用が無理だとしてもせめて項目をmysql_real_escape_string()に通すとか。

Answer 2

PHPでデータベースに接続するときのまとめ - Qiita

こちらの「初心者がやりがちなミス」から確認してください．そもそも

$query = "INSERT INTO getTweet(url,name,sentence) VALUES('$url','$screen_name','$text')";

こちらのコードにも致命的なSQLインジェクション脆弱性があります．これは「動けばOK」と見過ごしていい範囲を超えているので，必ず正しい書き方に直してください．

Comments

[adwadwadw]

ありがとうございます。PHPはじめて1週間ほどなのでとても参考になります。

Answer 3

データ投入をPDOなどを利用してプレースホルダでやってみてください