2つのプルダウンメニュから受けた値に応じIF分岐するのに効率良い方法

Question

２つのドロップダウンメニューから選択された値に応じて、設定項目の引数を変更したいです。

下の例では、それをするのに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);
}

papillon68 · Accepted Answer

効率がいいか分かりませんが、配列はどうでしょうか。
種類が増えたら配列に追加すれば対応できます。
if文を追加する必要がないので保守性がアップするかもしれません。

$tickd_x = array("TICKD_NORMAL", "TICKD_VERYSPARSE", "TICKD_SPARSE", "TICKD_DENSE");
$tickd_y = array("TICKD_NORMAL", "TICKD_VERYSPARSE", "TICKD_SPARSE", "TICKD_DENSE");

// 範囲外の値ならグラフを生成させないための処理
if((isset($tickd_x[$axis_x])) && (isset($tickd_y[$axis_y]))) {
  $graph->SetTickDensity($tickd_y[$axis_y],$tickd_x[$axis_x]);
}

litton102 · Answer

<?php

//すみません、元質問者です。
//投稿後にすごいことを思いつきました。
//↓↓↓これで良かったですね（寒）。
//でも、まだ効率良い方法あったら是非教えてください。
if ($axis_x = '0') {
　　$tickd_x = "TICKD_NORMAL";
}elseif ($axis_x = '1') {
　　$tickd_x = "TICKD_VERYSPARSE";
}elseif ($axis_x = '2') {
　　$tickd_x = "TICKD_SPARSE";
}elseif ($axis_x = '3') {
　　$tickd_x = "TICKD_DENSE";
}

if ($axis_y = '0') {
　　$tickd_y = "TICKD_NORMAL";
}elseif ($axis_y = '1') {
　　$tickd_y = "TICKD_VERYSPARSE";
}elseif ($axis_y = '2') {
　　$tickd_y = "TICKD_SPARSE";
}elseif ($axis_y = '3') {
　　$tickd_y = "TICKD_DENSE";
}

$graph->SetTickDensity($tickd_y,$tickd_x);
?>

naktak · Answer

TICKD_VERYSPARSEとかっていう文字が何なのか分かりませんが、

aqucent · Answer

視認性重視で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 '

エラー: axis_x の値が不正です

'; } switch($axis_x) の値は $_POST['axis_x'] or $_GET['axis_x'] にしても良さそうですが、その辺はお好みで。

yambejp · Answer

普通この手の処理は配列でやりませんか？
（こまかく検証していないので、書式は
間違っているかもしれませんが）

<?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]);
?>

2つのプルダウンメニュから受けた値に応じIF分岐するのに効率良い方法

<?php

効率がいいか分かりませんが、配列はどうでしょうか。

TICKD_VERYSPARSEとかっていう文字が何なのか分かりませんが、

視認性重視でswitch文にする手はありますね。

普通この手の処理は配列でやりませんか？

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング