SQL文で昇順にデータが取得出来ません

Question

【質問内容】
PHPの関数の中でSQLを使用してデータを取得する際に、
変数「rank」に入っている整数値を参照し、「昇順」にデータを取り出したいのですがエラーが出てうまくいきません。
初心者で大変申し訳ありませんが、何卒よろしくお願い致します。

---

【備考】
・「ORDER BY rank ASC」 ⇒「ORDER BY rank」としたところ順不同ではありますがデータは取得出来ました。
(ORDER BY rank ASCとするとエラーが出てしまいデータを取得することが出来ません)

・データベースは「MariaDB」を使用しています。

---

【エラー】
エラー：SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'AND ( w.obj_key like ?)' at line 12

---

public function get_waitobj($search_key){
    $sql = <<<EOS
SELECT
       w.rank       as rank,
       w.obj_key    as obj_key,
       w.dotdata    as dotdata

FROM
       waitobj w

ORDER BY rank ASC

EOS;
       if($search_key != ""){
           $sql .= " AND ( w.obj_key  like :obj_key) ";
       }

       try {
           $stmh = $this->pdo->prepare($sql);
           if($search_key != ""){
               $search_key = '%' . $search_key . '%'; 
               $stmh->bindValue(':obj_key',  $search_key, PDO::PARAM_STR );
           }
           $stmh->execute();
           
           // 検索結果を多次元配列で受け取る
           $i=0;
           $data = [];
           while ($row = $stmh->fetch(PDO::FETCH_ASSOC)){
               foreach( $row as $key => $value){
                       $data[$i][$key] = $value;
               }
               $i++;
           }
       } catch (PDOException $Exception) {
           print "エラー：" . $Exception->getMessage();
       }
       return $data;
   }

Answer 1

public function get_waitobj($search_key)
{
	$sql = "SELECT ";
	$sql .= "w.rank as rank, ";
	$sql .= "w.obj_key as obj_key, ";
	$sql .= "w.dotdata as dotdata ";
	$sql .= "FROM ";
	$sql .= "waitobj w ";
	$sql .= "WHERE 1 ";
	if ($search_key != "") {
		$sql .= " AND ( w.obj_key like :obj_key) ";
	}
	$sql .= "ORDER BY rank ASC";

	try {
		$stmh = $this->pdo->prepare($sql);
		if ($search_key != "") {
			$search_key = '%' . $search_key . '%';
			$stmh->bindValue(':obj_key', $search_key, PDO::PARAM_STR);
		}
		$stmh->execute();

		// 検索結果を多次元配列で受け取る
		return $stmh->fetchAll(PDO::FETCH_ASSOC);
	} catch (PDOException $Exception) {
		print "エラー：" . $Exception->getMessage();
	}
}

Comments

[t33]

無事昇順にソート出来ました！ありがとうございます。
エラーが出た理由としてEOSで記載していたのがダメだったのでしょうか。。

[退会済みユーザー]

直接の原因はそこじゃない。
SQLそのものが理解されていないようです。
SELECT ... FROM ... WHERE ... ORDER この順番で書かれていない。
提示のものには WHERE というキーワードすら書かれていない。

[t33]

WHERE句がないのが原因だったのですね。。
お教え頂きありがとうございます。

[退会済みユーザー]

動くコード見せてるんだから、自分のコードと見比べてください。他にも問題の部分はありますよ。

[t33]

AND ( w.obj_key like :obj_key)を記載するタイミングでしょうか。。
（間違っていたらごめんなさい）

[退会済みユーザー]

http://www.dbonline.jp/mysql/select/index1.html

[KiyoshiMotoki]

横から失礼します。

> 提示のものには WHERE というキーワードすら書かれていない。

について、誤解を招きそうなので補足させていただくと、
WHERE キーワードは記載しなければいけない、【というわけではありません】。

テーブルに格納されている全てのレコードを取得したい場合、

　SELECT ... FROM [テーブル名] ORDER BY ...

のように WHERE句を付けなくても良いですし、
Kosuke_Shibuya様の回答のように

　SELECT ... FROM [テーブル名] WHERE 1 ORDER BY ...

と、全てのレコードに対して true となる(※)ような WHERE句を付けても良いのです。

※ MariaDB の場合、数値の 0 が false、それ以外の値が true とみなされます。
　https://mariadb.com/kb/en/mariadb/boolean/
　> A value of zero is considered false. Non-zero values are considered true:

実際、Kosuke_Shibuya様が示された URL に記載されている SELECT文の書式を見ても、

　[WHERE where_condition]

と、WHERE句が "[ ]" で囲われています。

"[ ]" で囲われている部分は、
「付けても付けなくても良い」
ことを表しています。

[退会済みユーザー]

> KiyoshiMotokiさん
補足ありがとうございます。" AND ( w.obj_key like :obj_key) ";という条件式があるのに、WHEREがないというのが正確な表現ですね。

[KiyoshiMotoki]

Kosuke_Shibuya様

レスありがとうございます。

> " AND ( w.obj_key like :obj_key) ";という条件式があるのに、WHEREがないというのが正確な表現ですね。

まさに、仰る通りですね。

[t33]

＞KiyoshiMotokiさん、詳細なご説明ありがとうございます。
大変勉強になりました。
＞Kosuke_Shibuyaさん
コードのご提示、及び参考URLありがとうございます。
精進致します。

Answer 2

具体的にどんなエラーが発生しているのでしょうか。

Comments

[t33]

エラー：SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'AND ( w.obj_key like ?)' at line 12
というエラーが出ています。質問箇所にも追記致しました。
よろしくお願い致します。

[turbgraphics200]

WHERE句より先にORDER BY句が記述されているからなのではないでしょうか？

[t33]

初心者であまり分かっていないので恐縮なのですが、WHERE句は今回記載しておりません。
(ないといけないものなのでしょうか。。)
ちなみにですが、「ORDER BY rank」としたところ順番がばらばら（昇順ではない）ですがデータは取得することが出来ました。

[turbgraphics200]

ないといけません。
コードを見た感じだと
SELECT ~~ FROM xx ORDER BY rank ASC AND (w.obj_key like ?) AND (w.obj_key like ?)
というSELECT分を実行しようとしていますよね。
ただしくは
SELECT ~~ FROM xx WHERE (w.obj_key like ?) AND (w.obj_key like ?) ORDER BY rank ASC
のようなSELECT文にしなければなりません。

[t33]

WHEREはないとダメなのですね。詳細なご説明ありがとうございます。

Answer 3

ORDER BYのデフォルトはASCなので通常記述しません。エラーになるDBってあるんですね。
同じSQLでもDBによって方言が多いです。質問毎にお使いのDBが何か明記された方が良いです。

Comments

[t33]

失礼致しました。「MariaDB」を使用している旨記載致しました。
ORDER BY rankと記述したところデータは取得出来るのですが、順不同（昇順にならない）で困っています。。

[Orlofsky]

他の人も書かれているように条件はORDER BY句の前にWHERE句がなければならないのに、ORDER BY句の次にWHERE句がないのに条件がかいてあるせいですね。

[t33]

SQLで分からないことが多かったので勉強になりました。
ご返信ありがとうございました。

[Orlofsky]

いちおう、正規マニュアルです。今回の件でマニュアルの見方も進歩したでしょう。
https://dev.mysql.com/doc/refman/5.6/ja/select.html

[t33]

>Orlofskyさん
正規マニュアルのリンクありがとうございます。
精進致します。