LIMIT句でのsprintf関数について

Question

LIMIT句についてです。

$page = filter_input(INPUT_GET, 'page');
$start = (int) $page * 5;
$sql = sprintf("SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT %d, 5", $start);
$stmt = $pdo->prepare($sql);
$stmt->execute();

var_dump($stmt);

【結果】
object(PDOStatement)#3 (1) { ["queryString"]=> string(68) "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT 10, 5" }

正常に動いてるのでいいのですが

LIMIT 取り出す位置,取り出す件数
の順で記述していくのに

LIMIT %d, 5", $start
ではなく
**LIMIT %d,$start, 5" **
にすると
LIMIT 取り出す位置,取り出す件数
のはずなのにエラーになってしまいます。

その原因と
LIMIT 取り出す位置,取り出す件数
の順で記述するはずなのに
LIMIT %d, 5", $start
で記述して結果が
DESC LIMIT 10, 5
になっている理由について教えてください。

LIMIT %d, 5", $start
なら
DESC LIMIT 5,10
になるはずだと思うのです。

Answer 1

区切る位置がおかしいですね…

sprintf("... LIMIT %d, 5", $start) 

これは%dの位置に整数キャストして$startを当てはめる使い方をしています．sprintfのマニュアルを読めば

sprintf("... LIMIT %d,$start, 5")

これがおかしいことぐらいすぐに気づくと思うんですが…
(%dが使われていないのでエラーになります)

なお，プレースホルダを使うかsprintfを使うかの選択ですが，これは状況次第というところでしょう．PDOはどうしても生のままだと整数の取り扱いが苦手なので，部分的にsprintfの力を借りるのはよくあることです．ただ，これがコーディング作法として良いかと言われると微妙なところです．

もしプレースホルダで書くなら

$page = max(0, (int)filter_input(INPUT_GET, 'page'));
$start = $page * 5;
$sql = "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT ?,5";
$stmt = $pdo->prepare($sql);
$stmt->bindValue(1, $start, PDO::PARAM_INT);
$stmt->execute();

となります．(念のため負の数は0になおしています)

Comments

[退会済みユーザー]

$page = max(0, (int)filter_input(INPUT_GET, 'page'));

max() を使うのは、新たな気づきです！人のソース読むのは楽しいですね！

[earnest_gay]

回答ありがとうございます。

max()で負の数にならないようにしてますが、$_POST['page']が負の数になることはあるんでしょうか？
--------------------------------------------------------------------------
$page = max(0, (int)filter_input(INPUT_GET, 'page'));
$start = $page * 5;
$sql = "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT ?,5";
$stmt = $pdo->prepare($sql);
$stmt->bindValue(1, $start, PDO::PARAM_INT);
$stmt->execute();

この結果
object(PDOStatement)#2 (1) { ["queryString"]=> string(66) "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT ?,5" }
になっており、うまくバインドできていないように伺えるのですが...

[mpyw]

>> $_POST['page']が負の数になることはあるんでしょうか？

普通にWebブラウザでアクセスしてくる限りはないですが，任意のプログラムからHTTPリクエストを飛ばせることを考慮すると「やろうと思えばできる」といえます．LIMIT/OFFSETが負の値でもエラーにならなければ問題ないのですが，なったような記憶があったので，絶対に何があってもエラーが発生しないように書いている…というところです．

>> うまくバインドできていないように伺えるのですが...

queryStringプロパティからはバインドした結果は見えません．

[earnest_gay]

1つ目については分かりました！

２つ目ですが、queryStringプロパティとは何で、なぜバインドした結果は見えないのでしょうか？

[mpyw]

queryStringプロパティはprepareの第1引数がそのまま格納されたものです．バインドしか結果は内部的に保持されていて，外からは見えないのです．
（PDOクラスおよびPDOStatementクラスの実装はC言語で書かれており，そこで保持している状態変数全てがPHP側に晒されているわけではありません）

Answer 2

こんにちは

sprintfの使い方として形式は

string sprintf ( string $format [, mixed $args [, mixed $... ]] )
string $format = "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT %d, 5"
$args = $start

に相当します。

"SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT 5, 10"

と表示したいなら

sprintf ( "SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT 5,%d",$start);

になるかと思います。

http://www.phpbook.jp/func/string/index7.html

Comments

[earnest_gay]

詳しくありがとうございます！

Answer 3

以下のふたつが同じことであることは、ご理解できますか？

<?php
$start = 10;
$recordCount = 5;
$sql = sprintf("SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT %d, %d", $start, $recordCount);
var_dump($sql);

<?php
$start = 10;
$sql = sprintf("SELECT pref,city FROM user_data WHERE 1 ORDER BY id DESC LIMIT %d, 5", $start);

var_dump($sql);

Comments

[earnest_gay]

今理解できました！
ありがとうございます。

Answer 4

sprintfではなくプリペアドステートメントを使用されたほうが良いです。

---

質問文を見る限りsprintfの使い方を勘違いされています。
PHP: sprintf - Manual
マニュアルときちんと読んでsprintfの動作を理解されることをおすすめします。

Comments

[earnest_gay]

回答ありがとうございます。