- ベストアンサー
2つのプルダウンメニュから受けた値に応じIF分岐するのに効率良い方法
２つのドロップダウンメニューから選択された値に応じて、設定項目の引数を変更したいです。 下の例では、それをするのに15回IF分岐しちゃってますが、もっといい方法はないでしょうか。 PHP4+JpGraphでグラフ（散布図）を描かせているのですが、 目盛りの間隔（細かさを）まばら、少、多、自動の4段階で切り替えてます。 <html> <body> Y軸の目盛間隔 <select name="axis_x"> <option value="0" selected>自動</option> <option value="1">まばら</option> <option value="2">少</option> <option value="3">多</option> </select> Y軸目盛間隔 <select name="axis_y"> <option value="0" selected>自動</option> <option value="1">まばら</option> <option value="2">少</option> <option value="3">多</option> </select> </body> </html> if ($axis_y =='0' and $axis_x == '1'){ $graph->SetTickDensity(TICKD_NORMAL,TICKD_VERYSPARSE); } elseif ($axis_y =='0' and $axis_x == '2'){ $graph->SetTickDensity(TICKD_NORMAL,TICKD_SPARSE); ： ■12セットのelseif文省略←ここが非効率！ ： } elseif ($axis_y =='3' and $axis_x == '3'){ $graph->SetTickDensity(TICKD_DENSE,TICKD_DENSE); }
- みんなの回答 （5）
- 専門家の回答
質問者が選んだベストアンサー
その他の回答 (4)
- yambejp
- ベストアンサー率51% (3827/7415)
普通この手の処理は配列でやりませんか？ （こまかく検証していないので、書式は 間違っているかもしれませんが） <?php $tickd_x[0]="TICKD_NORMAL"; $tickd_x[1]="TICKD_VERYSPARSE"; $tickd_x[2]="TICKD_SPARSE"; $tickd_x[3]="TICKD_DENSE"; $tickd_y[0] ="TICKD_NORMAL"; $tickd_y[1] ="TICKD_VERYSPARSE"; $tickd_y[2] ="TICKD_SPARSE"; $tickd_y[3] ="TICKD_DENSE"; $graph->SetTickDensity($tickd_y[$axis_y ],$tickd_x[$axis_x]); ?>
お礼
yambejpさん、レスありがとうございました。 配列といっても、＃２さんとは記述が異なる アプローチですね。これまた、シンプルで 頭で理解はできるのですが まったく発想にありませんでした。 それと$tickd_y[$axis_y ]というのは連想配列という 記述になるんでしょうかね。 ともかく、ありがとうございました。
- aqucent
- ベストアンサー率39% (78/200)
視認性重視でswitch文にする手はありますね。 switch($axis_x){ case '0': $tickd_x = "TICKD_NORMAL"; break; case '1': $tickd_x = "TICKD_VERYSPARSE"; break; case '2': $tickd_x = "TICKD_SPARSE"; break; case '3': $tickd_x = "TICKD_DENSE"; break; default: echo '<p>エラー: axis_x の値が不正です</p>'; } switch($axis_x) の値は $_POST['axis_x'] or $_GET['axis_x'] にしても良さそうですが、その辺はお好みで。
お礼
aqucentさん、レスありがとうございました。 例文までご提示いただきありがとうございました。 たしかにSWITCHというのも、可能ではあるのですね。 SWITCH ...BREAK構文の存在は知ってましたが 実際に導入したことはなかったです。 確かに、可視性はSWITCH文の方が良いですね、 大変参考になりました。 ちなみに、ご示唆されているように、$axis_xは $_POST['axis_x'] でないと動かないと思われます。 本番環境ではGlobalがOFFですので。。。
- めとろいと（@naktak）
- ベストアンサー率36% (785/2139)
TICKD_VERYSPARSEとかっていう文字が何なのか分かりませんが、 <option>タグのvalueに直接それが入っていれば判定すら要りませんね。 TICKD_VERYSPARSEが定義名であった場合でも、<option>タグに定数値を入れておけば 同様に判定は不要です。
お礼
naktakさん、レスありがとうございました。 これは、超コロンブスの卵でした。 ある意味、衝撃的です。 まったくおっしゃるとおりですね。 ちなみに、TICKD_VERYSPARSEは、JpGraphモジュール が出力するグラフ設定で メモリの細かさが「非常にまばら」のような 設定値です。
お礼
papillon68さん、アドバイスありがとうございました。 配列というのはこんな使い方もできるのですね。 いろいろ構文はあるもののなかなか使い倒せなくて 技術アップできていなかったところです。 例外処理の方法までご提示いただき、感謝です。