質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PDO

PDO(PHP Data Objects)はPHPのデータベース抽象化レイヤーです。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

1回答

1795閲覧

PHP&MySQL(PDO)について、効率の良い書き方

miya

総合スコア81

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PDO

PDO(PHP Data Objects)はPHPのデータベース抽象化レイヤーです。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

3クリップ

投稿2015/03/20 13:50

編集2015/03/20 13:51

lang

1function hoge( $dbh, $a, $b, $c ) { 2 ... 3 $sth->bindParam( ':a', $a, PDO:PARAM_STR ); 4 $sth->bindParam( ':b', $b, PDO:PARAM_INT ); 5 $sth->bindParam( ':c', $c, PDO:PARAM_STR ); 6 ... 7}

上記をまとめようと思い、以下の2通りで迷っているのですが、どちらがいいと思いますか?

compact でまとめる

lang

1function hoge( $dbh, $a, $b, $c ) { 2 ... 3 foreach ( compact( 'a', 'b', 'c', func_get_args() ) as $key => $value ) ) { 4 switch ( $key ) { 5 case 'b' : $type = PDO::PARAM_INT; break; // is_int などで振り分けようかとも迷っています。 6 default : $type = PDO::PARAM_STR; break; 7 } 8 $sth->bindParam( ":{$key}", $value, $type ); 9 } 10 ... 11}

外部で連想配列でまとめる

lang

1function hoge( $dbh, $args ) { 2 ... 3 foreach ( $args as $key => $value ) ) { 4 // 今のところ、INT と STR があれば十分。BOOL も格納先は INT だし、NULL は基本使わない 5 $sth->bindParam( ":{$key}", $value, is_int( $value ) ? PDO::PARAM_INT : PDO::PARAM_STR ); 6 } 7 ... 8} 9 10hoge( array( 11 'a' => 'a' 12 ,'b' => 10 13 ,'c' => 'c' 14) );

compact でまとめる方法は、bindPraram のわずらわしさが消えて、カラムの変数名とtype のみに集中すればよくなり(is_int などで判別すればそれすらなくなる)、2番目の外部で連想配列でまとめる方法は、加えて、引数の順を気にしなくてよくなりますが、呼び出しがやや複雑になるのと、データベースのカラム名を関数の呼び出し側で気にしなくてはいけないわずらわしさが生じるので、どれも一長一短との感じが致します。
compact が一番スマートかな?引数が3つとかならいいですが、5個以上になると、こちらのほうがいいように思います。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

私なら、引数の数が多いときはbindParamは使わず、executeに配列渡しにします。

投稿2015/03/20 22:28

chokojori

総合スコア971

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

miya

2015/03/20 22:35

かつて execute の解説読んだ時に配列を渡せるのは便利だと思ったんですが、型の指定ができなかったため、使用を控えていました。 変数の型管理をしっかりしていればそのほうがいいかもしれないですね。is_int を使うくらいならなおさら。 ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問