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個以上になると、こちらのほうがいいように思います。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2015/03/20 22:35