このPHPソースをすっきりさせる為のいい案無いでしょうか？

Question

PHPでSQLを組み立てて実行するプログラムを書いています。

ただ、この手のプログラム、書くのはそれほど苦ではないのですが、綺麗にと言いますか、コンパクトに無駄がなくまとめるのが苦手なのです。

例えば、このプログラム、

$sql = "SELECT * FROM rk_list";
$jyoken = 0;

// タイプ
if ($typeno > 0){
    if($jyoken == 0){
		$sql = $sql . " where ";
		$jyoken = 1;
	}else{
		$sql = $sql . " and ";
	}
     $sql = $sql . "typeno = " . $typeno;
}

// 価格1
if ($price1 > 0){
    if($jyoken == 0){
		$sql = $sql . " where ";
		$jyoken = 1;
	}else{
		$sql = $sql . " and ";
	}
     $sql = $sql . "price >= " . $price1;
}

// 価格1
if ($price2 > 0){
    if($jyoken == 0){
		$sql = $sql . " where ";
		$jyoken = 1;
	}else{
		$sql = $sql . " and ";
	}
     $sql = $sql . "price <= " . $price2;
}

// 年
if ($year > 0){
    if($jyoken == 0){
		$sql = $sql . " where ";
		$jyoken = 1;
	}else{
		$sql = $sql . " and ";
	}
	if($year == "2016
	    $sql = $sql . "year <= " . $year;
	}else{
		$sql = $sql . "year = " . $year;
	}
}

$sql = $sql . ";";

つまり、フォームで入力された値を元にWhere句を組み立てているのですが、未入力もありなので、何も入力されなければ、

SELECT * FROM rk_list;

で終わりなのですが、条件が入力された場合は、最初の条件の場合は前に"Where"を付け、2個め以降は、"and"でつないでいくので、項目ごとに最初かどうかの判定を入れており、大変冗長性のある、長いプログラムになってしまっています。

こういうの、なんとか条件判定分を減らし、でも、あまりにトリッキーではなく、見やすい、いい方法は無いかと思いまして。。。

一人で考えているのも煮詰まってしまっていますので、誰か一緒に考えて頂けないかと。。。

よろしくお願い致します。

Answer 1

コード書いてる間に回答出てた…。せっかくなので貼っておきます。
maisumakun さんの2番目の手にあたります。

<?php

$sql    = 'SELECT * FROM rk_list';

$typeno = 3;
$price1 = 1000;
$price2 = 3000;
$year   = 2016;

// 条件配列を作る
$input = array();

if ( $typeno > 0 ) {
	$input[] = array(
		'key'     => 'typeno',
		'value'   => $typeno,
		'compare' => '=',
	);
}

if ( $price1 > 0 ) {
	$input[] = array(
		'key'     => 'price',
		'value'   => $price1,
		'compare' => '>=',
	);
}

if ( $price2 > 0 ) {
	$input[] = array(
		'key'     => 'price',
		'value'   => $price2,
		'compare' => '<=',
	);
}

if ( $year > 0 ) {
	$input[] = array(
		'key'     => 'year',
		'value'   => $year,
		'compare' => 2016 === $year ? '<=' : '=',
	);
}

// where 句を作る
$where = implode(
	' and ', array_map(
		function ( $query ) {
			// key = value のような形に加工
			return implode( ' ', array( $query['key'], $query['compare'], $query['value'] ) );
		}, $input
	)
);

// 空じゃない場合のみ where を先頭につける
$where = empty( $where ) ? '' : ' where ' . $where;

// 合体！
$sql = $sql . $where . ';';

echo $sql;
// SELECT * FROM rk_list where typeno = 3 and price >= 1000 and price <= 3000 and year <= 2016;

Comments

[退会済みユーザー]

おお～。なるほど！
これぞプロのソースですね！
ありがとうございました！

[退会済みユーザー]

このコードSQLインジェクションの脆弱性ありますよ。

[yhg]

主旨と違うのでSQLインジェクションについては考慮してません。
return implode( ' ', array( $query['key'], $query['compare'], '?') );
とでもしておけば簡単に対応できると思います。

[退会済みユーザー]

なるほど。奥が深いですね。

Answer 2

よくある手としては2つあります。

• 最初にWHERE 1 = 1のようなダミーの条件を入れて、残りはすべて機械的にANDでつなぐ
• 条件を配列に入れておいて、ANDでimplodeする

Comments

[退会済みユーザー]

あ、WHERE 1 = 1は以前、パッケージソフトのコードで見かけて、何なんだろうと思っていましたが、そういう事でしたか。。
ありがとうございました！

Answer 3

SQL の組み立ては、組立自体よりもデータのバインドの方に気を使ったほうが良いです。

回答はコピペですが参考まで。
参考コードは多分未完です。型を使用したバインドが完成形だと思います。

条件検索におけるプリペアドステートメントの使用
escape_wildcard()の実装は、本を買って確認してくださいｗ

answerer ockeghem [12月18日 17:19]

検索条件が動的に変わる場合、プレースホルダ（プリペアードステートメント）をどう使って良いかはよくある質問だと思います。「体系的に学ぶ 安全なWebアプリケーションの作り方」という本のP135にサンプルが出ています。以下に該当する部分を引用します。

    // 基本となるSQL
    $sql = 'SELECT id, title, author, publisher, date, price FROM books';
    if ($title !== '') {　// 検索条件titleの追加（LIKE）
      $conditions[] = "title LIKE ? ESCAPE '#'";
      $ph_type[] = 'text';
      $ph_value[] = escape_wildcard($title);
    }
    if ($price !== '') {　// 検索条件priceの追加（大小比較）
      $conditions[] = "price <= ?";
      $ph_type[] = 'integer';
      $ph_value[] = $price;
    }
    if (count($conditions) > 0) { // WHERE句がある場合
      $sql .= ' WHERE ' . implode(' AND ', $conditions);
    }
    $stmt = $mdb2->prepare($sql, $ph_type); // SQL文の準備
    $rs = $stmt->execute($ph_value); // バインド・問い合わせ実行

ご覧のように、WHERE句の断片を配列に追加していき、同時に型と値も別の配列に追加しているだけです。

面倒なように見えるかもしれませんが、単純な繰り返しなので、難しくはないと思います。

Comments

[退会済みユーザー]

なるほど。ありがとうございます！
勉強になりました。