Q&A
フォローありがとうございます。いい質問と思いますが。特に初心者で周りに聞く人が居なければ。評価させて頂きます。
###前提・実現したいこと
WordPressの固定ページにPHPでデータベースにアクセスしデータを出力したいと思っています。
固定ページにPHPを書き込むことはショートコードを使用することでクリアしています。
表示されたエラーメッセージについて調べてみると、SQL文の書き方にミスがあるらしいのですが、いくら修正をしてみても解決しません。どこを修正すれば良いのでしょうか？回答よろしくお願い致します。
###発生している問題・エラーメッセージ
接続成功 クエリーが失敗しました。 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 '"wp_options" WHERE option_id >= 5' at line 1
###該当のソースコード
try{ $my_Con = mysql_connect("localhost","[user]","[pass]"); if($my_Con == false){ echo '接続失敗<br/>'; }else{ echo '接続成功<br/>'; } $query = 'SELECT option_name FROM "wp_options" WHERE option_id >= 5'; $result = mysql_query($query, $my_Con); if(!$result){ die('クエリーが失敗しました。<br/>'.mysql_error()); }
###試したこと
(1)修正箇所
SELECT option_name FROM wp_options WHERE option_id >= 5
エラーメッセージ
接続成功 クエリーが失敗しました。 No database selected
(2)修正箇所
SELECT "option_name" FROM "wp_options" WHERE "option_id" >= 5
エラーメッセージ
接続成功 クエリーが失敗しました。 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 '"wp_options" WHERE "option_id" >= 5' at line 1
回答5件
0
ベストアンサー
普通ダブルクォートで囲むと文字リテラルとして扱われるので、データベースではエラーになりますから、構文的には「(1)修正箇所」が一番正しいと思います。
で、「データベースが選ばれてないよ！」ってエラーが帰ってきてますから、単純にアクティブにするデータベースを選択してあげればいいのでは？
PHP
1$my_Con = mysql_connect("localhost","[user]","[pass]");
の後に、
PHP
1mysql_select_db("your_db_name", $my_Con);
これでそれ以降のmysql_query()は全て選択されたアクティブなデータベース上で実行されることになります。（your_db_nameはあなたの利用しているデータベース名に変更してくださいね）
【補足】
コメントにもありましたが「普通ダブルクォートで囲むと文字リテラルとして扱われる」と書いたのは私のミスです。
多くのRDBMSでは普通にダブルクォート使ってましたね（恥）
むしろ文字リテラルとして扱われるのはシングルクォートの場合でしたね・・・
MySQLではバッククォートなんかも使いますから、ほんとMySQLはちょっと異端だと思います。
ちょっと質問の内容とずれますが、「-」（ハイフォン）を表名や列名に使う場合、ダブルクォートで囲まないと、SQL文では減算と解釈されますから、ダブルクォートで囲むことで、記号なども名前に使用することもできます。
お勧めはしませんけどね(￣▽￣;)
投稿2017/09/03 08:51
編集2017/09/04 06:54
総合スコア83
この回答が「低評価」になってますが、ちょっと厳しすぎる気がします。間違いに気づいたら、コメントで知らせるのが良いと思います。
なんか間違ってるんですかね？mysql_ 関数はよくわからん。
> 普通ダブルクォートで囲むと文字リテラルとして扱われるので
が、間違っていて、下げたのは、勿論、僕です。理由は、他で “沢山” 説明しています。
MySQLだけがデータベースじゃないし、MySQLのSQLだけがSQLじゃないです。
補足しますが、寧ろ、MySQL こそが、異端です。その辺のところを教えなくて、何が回答ですか？
そうです、そうやって、自分の間違いを素直に認める事こそが、技術者たる証です。
ですから、評価を戻します。
0
データを取るだけなら以下で良いかと…
$query = $wpdb->get_results( 'SELECT option_name FROM wp_options WHERE option_id >= 5');
投稿2017/09/03 09:23
退会済みユーザー
総合スコア0
0
せっかくなので、私も回答を書きます。
まず、大前提
WordPress では テーブル操作用の関数が用意されているので、それを使うのが正しいです。
素の PHP で対応することは、将来のバージョンアップ対応やメンテナンスで、非常にイレギュラーな扱いをしなければならなくなるので、極力避けるべきです。
今回の回答の中では、ay03 さんが WordPress 内で使うべき関数を使用した具体的な方法を提示してくれているので、参考にすると良いです。
また、今回使用している、mysql_connect は、PHP 5.5.0 より非推奨となっている関数なので、素の PHP で書くにしても、現在は避けるべき関数です。質問には PHP のバージョン明記はありませんが、WordPress を使用しているサーバであれば、PDO や MySQLi が使用できると思いますので、そちらを使用してください。
syntax error に関して
やっと、今回の質問内容への回答となりますが、syntax error を出しているのは、テーブル名を「ダブルクオート」で囲ったことが原因です。
「(1)修正箇所」で実はすでに切り分けが出来ているのですが、MySQL の SQL 文として、テーブル名を「ダブルクオート」で囲むことは許されていません。
（厳密には設定されたモードにもよるので、時間のある時に以下のマニュアルを参考にしてください。
9.2 スキーマオブジェクト名 - MySQL マニュアル
こちらのページには、テーブル名を引用符で囲む場合と囲まない場合の違いも記述が有るので合わせて確認しておくと良いです。）
今回のケースでは、テーブル名を引用符「バッククォート」で囲むか、引用符を無くすかすることで、syntax error を無くすことが出来ます。
もう一つの error
本来であれば、「(1)修正箇所」で切り分けが出来ているのですが、この中でも error が出ているため、混乱してしまったようですね。惜しいｗ
urdapple さんが回答しているように、メッセージを素直に読むと、「データベースが選ばれてないよ！」ってエラーなので、それに対応することで、対処は完了します。
ようするに
今回のケースでは、wpdb を使用し、SQL 文の構文を修正するのが、WordPress を使用する上で最も適切な対応になります。
投稿2017/09/05 01:06
退会済みユーザー
総合スコア0
後付けでそんな事力説されても説得力ないです。アホらしいので低評価はしませんが、後で自分で見直して恥かしい投稿とかは、自分を貶めるだけなのでやめた方が良いと思います。
なんで lucker さんに噛みつかれたんだろ＾＾；
ベストアンサーが変わったことで、「WordPressであれば内部の関数を使用する」ってことが希薄になったことと、「mysql_connect を使用した内容がベストアンサーになっている」って流れがあまり良いと感じなかったため、新たに回答を作成しました。
間違っている箇所があれば、普通に指摘していただけると、修正対応しますよ＾＾
> なんで lucker さんに噛みつかれたんだろ＾＾；
ええぇぇぇ… あなた、僕の回答にどれだけ、「貴方の言うところの噛み付き」してたんですか？
本当に訳分からん人だな。
>間違っている箇所があれば、普通に指摘していただけると、修正対応しますよ＾＾
修正してほしのは貴方の回答じゃなくて、貴方の良心です。
0
もー… 回答者全員、何いってんだか意味がわからないんですが、
You have an error in your SQL syntax:
って書いてあるんだから、シンタックスエラーに決まってんじゃないですか。
SQL
1SELECT option_name FROM "wp_options" WHERE option_id >= 5
ですけど、phpmyadmin のタグついてるってことは、DB は MySQL なんでしょ？
何て呼ぶのか忘れましたけど、MySQL のテーブル名およびフィールド名を区切る記号はダブルクオートじゃなくてバッククオートです。これ。
SQL
1`
ダブルクオートは PostgreSQL。
本当に全員、もっと真面目にやってください。
投稿2017/09/03 10:05
退会済みユーザー
総合スコア0
ほら、あっちにベストアンサーついてるし… このサイト、本当にどうなってんですか？ 一体…
あー、わかった、“識別子” だ。って、んなこたぁ、この際どうでもいいや。
その切り分け、(1)修正箇所 で終わってますよ。
それでもエラー出てるね、ていうのが urdapple さんの回答で、それを前提にみんな回答しています。
WordPress に用意された DB 用の関数使用するってことを BA に選ぶのも普通かと。
これは失礼しましたって言いたいところですが、
だって、質問者、最後の最後まで syntax error 出してるじゃないですか。
その事、誰か指摘しましたか？
で、(1)修正箇所の結果が
No database selected
でしたっけ？
それが原因ならなんで、最初に
> 接続成功
> クエリーが失敗しました。
> 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 '"wp_options" WHERE option_id >= 5' at line 1
ってメッセージが出現しうるんですか？
修正箇所って質問にあるから誤解したんですかね？
修正箇所っていうより、質問前に質問者が実施した切り分けと結果を記述しています。
その記述を前提に、urdapple さんが syntax error の原因を指摘しています。
Tomak さんも正解の構文に関しては(（1）修正箇所が正解です)サラッと触れています。
ay03 さんの回答は、その前提で WordPress で実装するならどうするかって回答ですね。
「普通ダブルクォートで囲むと文字リテラルとして扱われるので」←この認識は間違いですよね？ SQL ですよ？ PHP的にはシングルクオート内に書かれてるんですよ？
「質問のコード（1）修正箇所が正解です」←予約語と同じ識別子の場合どうするんですか？
「ay03 さんの回答は、その前提で WordPress で実装するならどうするかって回答ですね。」←つまり、質問者技術向上云々より、対症療法だって言いたいわけですね。
あなたはいろいろと、何いってんだかわからないです。
> 「普通ダブルクォートで囲むと文字リテラルとして扱われるので」←この認識は間違いですよね？ SQL ですよ？ PHP的にはシングルクオート内に書かれてるんですよ？
SQL文の中で、ダブルクォートで囲むと、文字リテラルとして扱われる認識です。普通と書いたのは、「ANSI_QUOTES モード」を意識した回答かと。
> ←予約語と同じ識別子の場合どうするんですか？
今回、予約語ではないです。
> ←つまり、質問者技術向上云々より、対症療法だって言いたいわけですね。
むしろ、WordPress を使用する上での技術向上だと思いますよ。
あのね、あなたのその場しのぎの言い訳を聞きたいわけじゃないです。
本当にそれ、自信を持ってあなたの矜持を元に言ってるんですか？ 「今回、予約語ではないです。」とか。
だとしたら、あなたの発する言葉は聞く価値があるとは思えない。
私も論点をずらされてもちょっとって感じです。
> SQL文の中で、ダブルクォートで囲むと、文字リテラルとして扱われる認識です。普通と書いたのは、「ANSI_QUOTES モード」を意識した回答かと。
こちらの認識が間違っているという指摘が元々の内容だと思いますが、どのようにお考えでしょうか？
どのようにお考えでしょうかもなにも「間違いですよね？」と考えていると言ったとおりです。論点もずらしていないです。
間違いでないなら、論拠を示してください。
> 「普通ダブルクォートで囲むと文字リテラルとして扱われるので」
----------------------------------------------------------------
MySQL uses ' or " to quote values (i.e. WHERE name = "John"). This is not the ANSI standard for databases. PostgreSQL uses only single quotes for this (i.e. WHERE name = 'John'). Double quotes are used to quote system identifiers; field names, table names, etc. (i.e. WHERE "last name" = 'Smith').
http://wiki.postgresql.org/wiki/Things_to_find_out_about_when_moving_from_MySQL_to_PostgreSQL
----------------------------------------------------------------
> こちらの認識が間違っているという指摘が元々の内容だと思いますが
どうしてそれに限定したと思ったんですか？
あなたの言っていることは意味がわからないです。
MySQL の識別子は、以下の通り表現されます。
https://dev.mysql.com/doc/refman/5.6/ja/identifiers.html
> 識別子は引用符で囲むことも囲まないこともあります。（中略）識別子引用符文字は逆引用符 (「`」) です。（サンプル略）ANSI_QUOTES SQL モードが有効になっている場合、二重引用符内で識別子を引用符で囲むことも許可されています。
つまり、識別子を囲む引用符は「なし」「バッククォート」「ダブルクオート」である必要がありますが、「ダブルクオート」を使用する場合は、ANSI_QUOTES SQL モードであることと条件がつくため、普通は「なし」「バッククォート」が使われます。
同じページに以下の記述があります。
> ANSI_QUOTES モードでは、サーバーは二重引用符で囲まれた文字列を識別子として解釈します。この結果、このモードが有効になっているときには、文字列リテラルは単一引用符で囲む必要があります。二重引用符で囲むことはできません。
直接的な表現ではありませんが、ANSI_QUOTES モードでない場合、二重引用符で囲まれた文字列は文字リテラルとして扱われると読めます。
だれが、「MySQL」と限定してあなたの支持する主張を発言したのですか？
そんなこと、言わなきゃ解りませんか？「普通」って言葉の意味、知っていますか？
言ったじゃないですか、「あなたのその場しのぎの言い訳を聞きたいわけじゃないです。」って。
人はエスパーじゃないから、書いてある文章のとおりにしか文章を読まないです。
>「普通ダブルクォートで囲むと文字リテラルとして扱われるので」←この認識は間違いですよね？ SQL ですよ？ PHP的にはシングルクオート内に書かれてるんですよ？
>> こちらの認識が間違っているという指摘が元々の内容だと思いますが
> どうしてそれに限定したと思ったんですか？
これに対する返答、どうしたんですか？
mysql_connect 使って接続は成功しているので MySQL 使ってると判断できます。
lucker さんは別の DB である可能性も考えていたんですね。
普通は mysql_connect 使って接続が成功すれば DB は MySQL です。
なので、他の方の回答は、MySQL を DB として使用することを前提として記述されています。
DB が MySQL だったとして、
> SQL文の中で、ダブルクォートで囲むと、文字リテラルとして扱われる認識です。普通と書いたのは、「ANSI_QUOTES モード」を意識した回答かと。
ここ以外の箇所で指摘された箇所があるということですがどこでしょうか？
文脈からこちらしか指摘箇所があるようには思えませんでした。
lucker様
横から失礼します。
> で、(1)修正箇所の結果が
> No database selected
> でしたっけ？
>
> それが原因ならなんで、最初に
>
> > 接続成功
> > クエリーが失敗しました。
> > 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 '"wp_options" WHERE option_id >= 5' at line 1
>
> ってメッセージが出現しうるんですか？
について、私の手元の環境（MySQL5.7）でデフォルトのデータベースを選択しない状態で質問文のような
「識別子を "" で囲った SQL文」
を実行すると、
You have an error in your SQL syntax:
と表示されました。
mysql> \s
--------------
mysql Ver 14.14 Distrib 5.7.18, for Linux (x86_64) using EditLine wrapper
Connection id: 4
Current database:
Current user: root@localhost
SSL: Not in use
Current pager: stdout
Using outfile: ''
Using delimiter: ;
Server version: 5.7.18 MySQL Community Server (GPL)
Protocol version: 10
Connection: Localhost via UNIX socket
Server characterset: utf8
Db characterset: utf8
Client characterset: utf8
Conn. characterset: utf8
UNIX socket: /var/lib/mysql/mysql.sock
Uptime: 2 min 27 sec
Threads: 1 Questions: 8 Slow queries: 0 Opens: 105 Flush tables: 1 Open tables: 98 Queries per second avg: 0.054
--------------
mysql> SELECT * FROM "no_such_table";
ERROR 1064 (42000): 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 '"no_such_table"' at line 1
どうやら MySQL は、
「SQL文を実行するデータベースが指定されていない」
ことより先に
「構文エラー」
でエラーを発する仕組みになっているようです。
> lucker さんは別の DB である可能性も考えていたんですね。
「普通」という言葉のみを用いて SQL を語る時は、SQLにおける普通と考えるのが適切です。
MySQLしか使った事のない人の事なんかしらないです。
> KiyoshiMotoki さん
ありがとうございます。そういうレスポンスが「意味のある情報」です。
僕は最近 PHP も DB もとんと離れていて、あらたにコードを書いてテストするのが億劫でしたのでしませんでしたが、とりあえず、syntax error だと言っているのにダブルクオートについて誰も何一つ指摘しない事に懐疑を抱いただけです。
いろいろと狂っているんですよ、ここの回答は。
あなたには感謝します。
長かったですね。やった理解できました。
MySQL を DB として使用していることを前提に皆が回答しているってことが、理解できなかったことが原因ですね。
ちなみに、lucker さんの回答の以下は不適切です。
> 何て呼ぶのか忘れましたけど、MySQL のテーブル名およびフィールド名を区切る記号はダブルクオートじゃなくてバッククオートです。これ。
マニュアルに有るように、「なし」「バッククォート」が適切で、「ANSI_QUOTES モード」の場合は、ダブルクオート（つまり今回の件）でも問題ないです。
だーから、「なし」は予約後に対応できないし、モードはどの環境でも一緒じゃないって事が分からん人は、何言っても説得力ないです。
> （つまり今回の件）でも問題ないです。
だーから、だったら、どこに syntax error があったのか明言してください。
SQL のエラーは大抵、その場所そのものじゃなくて、問題箇所以降を指し示す事が多いので、エラーをみっけるのが大変なんです。そのくらい知ってるでしょう？ 仕事しているのなら。
言ってみなさいよ、どこが syntax error だったのか。
> te2ji さん
言ったでしょう？ 「あなたのその場しのぎの言い訳を聞きたいわけじゃないです。」って。
> だったら、どこに syntax error があったのか明言してください。
MySQL に使用する SQL でテーブル名を「ダブルクオート」で囲ってしまったことが原因です。
ごめんなさいね、さすがにイライラしてきたので言いますが、
だったらなんであんたはさっきからグダグダと言い訳ばっかしてんの？
貴方の中では、貴方のしていることは筋が通っているの？
lucker さんが何を指摘したかったのか理解したかっただけです。
urdapple さんの評価が下がった理由が知りたかったってのもあります。
マイナス評価の理由書かせるのってけっこう大変なんですよ。
結局、皆が MySQL を前提に話を進めている中、一人だけ勘違いしちゃっただけのようですね。
本当にそう思っているなら、それは価値があったという事でしょう。
技術者ってね、結局、自分を焼かないと成長しないんですよ。
貴方が勘違いしたといっているのが、この期に及んで僕だと言っているなら、
貴方は救いようがないですけどね。
ですから、彼は、明らかに、マイナスです。
内容を精査して良い答えと判断しました。評価させて頂きます。
アカウントのログ見ると、プラスの勢力とマイナスの勢力の攻防が繰り返されていて、結構楽しかったですよ。
正直、ここの評価がなにか自分の得になる事はなさそうなのでどうでもいいんですけど、少なくとも、「お前の言っていることは分かるが、お前のことが気に入らないから絶対に賛同しない」みたいな人は、技術者たる資格はないよなぁとは、思っています。
同感。
仕様を理解する前に評価って感じですね。
私はエンジニアでも畑違いのケミカル系ですが、経験から役立つノウハウのつもりで回答していますが、マイナス評価する方が得るものは何かが心配です。答えなくても良い回答で、しかも的を射ていればなおさら。貴殿の指摘は的確と理解してます。これだけは言いたい。
エンジニアの精神に畑なんてないと思います。エンジニアに必要なのは民法第1条第2項に基づく、職務に対する信義だけです。誠実であること以上に、エンジニアに必要な要素は無いと考えます。
0
PDOを使ったほうがいいと思いますが、そのままmysql_xxx()を使うのであれば下記のようにすればよいです。
データーベースにSELECT文で結果を受け取るには下記のようにします。
PDOなどでは下記の１．2．を同時に行うことができたりします。
- MySQLサーバーに接続（質問のコードでOK）
- データーベース選択（質問のコードにありません）
- SQLクエリ発行（質問のコード（1）修正箇所が正解です）
- SQLクエリの結果を受け取る
php
1try{ 2 $my_Con = mysql_connect("localhost","[user]","[pass]"); 3 4 if($my_Con == false){ 5 echo '接続失敗<br/>'; 6 }else{ 7 echo '接続成功<br/>'; 8 } 9 10 // データーベース選択 11 mysql_select_db('データーベース名', $my_Con); 12 13 $query = 'SELECT option_name FROM wp_options WHERE option_id >= 5'; 14 $result = mysql_query($query, $my_Con); 15 16 if(!$result){ 17 die('クエリーが失敗しました。<br/>'.mysql_error()); 18 }
投稿2017/09/03 09:17
総合スコア1652
WordPress なら、そもそも DB 操作の準備が出来ている気がします。
なんか関数用意されていそうな。。。
wpdb関数がありますね。
質問の内容とはちょっと違うので書きませんでしたが、CMS上で直接DB上にアクセスが必要なら、インジェクション対策的にもmysql_connectよりもPDOよりも、そのCMSに付随の関数を利用した方がいいでしょうね。
そーですね。wpdbのことも触れるべきでした。
あなたの回答
tips
プレビュー
