再利用できるPHP関数(データベース用)

質問

こんにちは

PHPの初心者です。

今制作しているウェブサイトで数回同じようなmySQLクエリを使いますが、操作の種類によってクエリを処理できるPHP関数を記述したほうが効率的だと聞きました。たとえばすべてのselectクエリは同じPHP関数で扱う。ですが、テーブルによって引数の数などが変わってきて、どうやってそれを正しく扱うのか迷っています。すべての値をarray項目に格納してwhileで扱うということですか？
最もふさわしい方法がわかる方がいれば教えていただけませんでしょうか？
select, insert, delete, alterを扱いたいと思います。

宜しくお願いします。

回答No.2

hogehoge78 · ベストアンサー

■Insert文の件

ソレで問題ないと思います。MySQLのテーブルを設計した段階で、Primary keyをAUTO_INCREMENTに設定したのであれば、INSERT句に特にそのフィールドを含めなければ勝手に連番でインサートされると思います。
他のフィールドも同様に、defaultを設定しておけば、INSERT句に含めなければ勝手にdefaultの値を挿入すると思います。

■select文の件

こちらもソレで問題ないと思います。ただ、selectの場合は、毎回作成するウェブサイト毎にどの程度変わるのかということと
似たような構文が何回発生するのか、ということを踏まえて考えてみてください。

作成するアプリケーションで、tableの数が２～３ぐらいしかないのであれば、それぞれのテーブル名を含んだ関数を作ったほうが楽できると思います。

サイトに寄ってはとんでもない数のテーブル名が来るとかであればもはやそれは既存のフレームワークを使うような作り方に変えたほうがいいわけですし。

汎用的に作るっていうことは、外からの色々なパラメータを受け取って丁度良く作らなければいけないので、その関数を実行するために様々なオプション的なパラメータ（引数）を渡してやらなければならず、結局面倒なことになるかも知れません。

全体のボリューム感で見て、関数を設計してみてください。
また、どうしても汎用的なものが必要だ、ということであれば、既存のフレームワークやライブラリを検討してみてください。

■エスケープ処理に関して
＞magic_quotes
だめでしょうね。むしろmagic_quotesは、OFFにするのが一般的なのではないでしょうか。

mysqlをどの様に操作しているのか分かりませんが・・・
・mysql関数だった場合
全部の値に対してSQL文に値を収める場合、mysql_real_escape_string関数を使ってエスケープする

・PDOを利用されている場合
プリペアドステートメントを使う

外部からの入力（検索窓など）を受ける場合は特に必須です。

このMySQLを扱うという件に関しては、セキュリティや設定など色々あるので、調べてみてください。

お礼

さまざまな情報を教えていただいて本当にありがとうございます。テーブルが１０個あるので、できるだけ重複を避けるよう関数にまとめたいと思いました。ご提案されたとおり検討してみます。

回答No.1

hogehoge78

どの程度集約するか、というバランス感覚はなかなか難しいと思います。
単純に、同一のクエリが２回以上発生するのであれば、その同一のクエリのものを関数に入れておく、というだけでいいのではないでしょうか。
つまり、ロジック上で、コピペをしないという。

find_foo_data, find_bar_dataのような感じで、SQL文をリターンするだけ、というのでも良いと思います。（まぁそれだったらdefineすればいいだけですが）

テーブルと取得するフィールドは同一で、where句の一部だけ毎回変わる、というのであれば、
<?php
function find_hoge_table_where($where){

return "select a, b, c from hoge where " . $where;

}

//こんなふうに。
$sql = find_hoge_table_where("name='aiueo' and regist=1");

?>

とか。

insert文なんかは、フィールド名に対して、値が決まっていて且つ、複雑なクエリを吐き出さないのであれば、
連想配列を引数で渡してやって動的にsql文を構築して返すだけでいいので簡単ですね。

function insert_query($table, $data){
$sql = "insert into {$table} set ";
$buf = array();
foreach($data as $field=>$value){
$buf[] = "`{$field}`='{$value}'";
}
$sql .= implode(", ", $buf);
return $sql;
}

PDOのprepareを使うとか、値をエスケープするとかする場合はもうちょっと工夫が必要ですが。

deleteは、単一の行を削除するのであれば、引数にテーブル名と、主キーに設定したIDとかを渡してやったらdelete文が返ってくるようにするだけです。

とにかく、もっと汎用的にとか考え出すと大変なので、コレぐらいの簡単なイメージで実装されたらよいのではないかなと思います。

もっときちんと機能的なものをと思われる場合は、ORマッパーなどをご利用されると良いかと思います。
Doctrineとか、ADODBとか、Propelとか。(ORマッパーが組み込まれているフレームワークを使うというのもありうるとは思います。CakePHPとかKohanaとか、Symfonyとか・・・)

お礼

ありがとうございます。とても参考になりました。
もう少し質問させていただきますが

function insert_query($table, $data){
$sql = "insert into {$table} set ";
$buf = array();
foreach($data as $field=>$value){
$buf[] = "`{$field}`='{$value}'";
}
$sql .= implode(", ", $buf);
return $sql;
}

という分でテーブルの第1フィールドがauto increment(データを挿入する必要なく、自動的に値が生成されるタイプ)でも使えますか？フィールド名を明記しない場合、$dataのなかの第一の値を無理矢理そのフィールドに挿入しないのですか？

select文についてなんですが、やはり毎回テーブルとフィールド名などが変わると、このようにarrayでまとめるのは、ふさわしいでしょうか

function select_elements($table, $fieldname){
$sql = "select";
$buf = array();
foreach($fieldname as $value){
$buf[] = $value;
}
$sql .= implode(", ", $buf);
$sql .= "from {$table}" ;
return $sql
}

やはり、エスケープ処理はmagic_quotesだけに頼って安全ではありませんよね？

お手間をおかけしますが、また宜しくお願いします。