MySQLの似たような命令を ループさせて行数を減らすやり方について

下記のような、MySQLの命令が5つあり、数字の部分だけ連続して1～5あるので、行数を減らしたいので、forでループすることにして短縮したいと思います。

<li>
<SELECT name="cat1" >
<OPTION value="0" >--カテゴリを選択してください--</OPTION>
<?php foreach ($cats1 as $cat1) { ?>
<OPTION value="<?=$cat1['cat_id']?>" <?php if( $cat1['cat_id'] == $row['cat1'] ){ echo "selected" ;}?> > <?=$cat1['name']?></OPTION>
<?php } ?>
</SELECT>
</li>

<li>
<SELECT name="cat2" >
<OPTION value="0" >--カテゴリを選択してください--</OPTION>
<?php foreach ($cats2 as $cat2) { ?>
<OPTION value="<?=$cat2['cat_id']?>" <?php if( $cat2['cat_id'] == $row['cat2'] ){ echo "selected" ;}?> > <?=$cat2['name']?></OPTION>
<?php } ?>
</SELECT>
</li>

　　　　　　　　　　　　　　・・・・あと3回　続く

上記の文を 下記のように可変変数で変えたのですが、うまくうごきませんでした。可変変数のつかいかたが悪いのかもしれません。原因がわかりましたら教えてください。よろしくお願いします。

<?php for ($i = 1; $i <= 5; $i++){
$a = 'cats'. $i;
$b = 'cat'. $i; ?>

<li>
<SELECT name="<?=$b ?>" >
<OPTION value="0" >--カテゴリを選択してください--</OPTION>
<?php foreach ($$a as $$b) { ?>
<OPTION value="<?=$$b['cat_id']?>" <?php if( $$b['cat_id'] == $row[$b] ){ echo "selected" ;}?> > <?=$$b['name']?></OPTION>
<?php } ?>
</SELECT>

</li>
<?php } ?>

No.1 ベストアンサー 回答者： kmee 回答日時： 2016/05/01 11:31

> うまくうごきません

どう「うまくうごきません」なのかを正確に把握して、その現象が起る原因を予測することが、修正への近道です。

> MySQLの命令が5つあり

この部分はPHPのプログラムであり、MySQLの命令はいっさい含まれていません。



<?php foreach ($cats1 as $cat1) { ?>

の $cat1 は、おそらくこのループの中だけで使う変数でしょう。
それならば、5つ書くにしても

<?php foreach ($cats1 as $cat) { ?>
<?php foreach ($cats2 as $cat) { ?>
...と同じ名前にすれば

<OPTION value="<?=$cat['cat_id']?>" <?php if( $cat['cat_id'] == $row['cat1'] ){ echo "selected" ;}?> > <?=$cat['name']?></OPTION>
と、( $row['cat1'] を除いて )同じ書き方にできます。
'cat1'という文字列を変数にしてしまえば

$cat_id = 'cat1' ;
<SELECT name=$cat_id >
<?php foreach ($cats1 as $cat) { ?>
<OPTION value="<?=$cat['cat_id']?>" <?php if( $cat['cat_id'] == $row[$cat_id] ){ echo "selected" ;}?> > <?=$cat['name']?></OPTION

$cat_id = 'cat2' ;
<SELECT name=<?=$cat_id?> >
<?php foreach ($cats2 as $cat) { ?>
<OPTION value="<?=$cat['cat_id']?>" <?php if( $cat['cat_id'] == $row[$cat_id] ){ echo "selected" ;}?> > <?=$cat['name']?></OPTION
....
とループの中も SELECT name= の部分もまったく同じになります。

残りは $cats1,$cats2 .. だけです。
可変変数を使うにしても、この一点だけ済みます。

$a='cats1' ;
$cats = $$a ;
$cat_id = 'cat1' ;
<SELECT name=$cat_id >
<?php foreach ($cats as $cat) { ?>
<OPTION value="<?=$cat['cat_id']?>" <?php if( $cat['cat_id'] == $row[$cat_id] ){ echo "selected" ;}?> > <?=$cat['name']?></OPTION



また、この一連の内容を、 関数にしてしまえば、可変変数を使う必要もなくなり、
option_list('cat1',$cats1,$row);
option_list('cat2',$cats2,$row);
option_list('cat3',$cats3,$row);
option_list('cat4',$cats4,$row);
option_list('cat5',$cats5,$row);
等と、ベタに書いてもそれなりに見易いものになります。

この回答へのお礼

ありがとうございます。質問のタイトルと全文が今おかしいことに気付きました。SQLから引っ張り出していたので、当初SQLの構文に間違えがあるのかと、質問を書いていましたが、文章を書いているときに、SQLの構文には問題がないことに気づき、途中で変更して投稿してしまったため、変な文章になりすいませんでした。それにもかかわらず回答して感謝します。効率の良い組み立て方をするのが苦手で、どうしても長々の構文になってしまいます。回答者さまのように、関数の例をみて、こういう場面で使うととてもよいことが分かりました。教えてくれて感謝します。

お礼日時：2016/05/01 17:21