より良く
短くするのがいいとかそういうわけではないけど
元のコード
$BASIC = 1;
$ADVANCE = 2;
$MASTER = 3;
$PROFESSIONAL = 4;
switch ($rank) {
case $BASIC:
{
$add_id_arr = [2,20,22];
break;
}
case $ADVANCE:
{
$add_id_arr = [4,105];
break;
}
case $MASTER:
{
$add_id_arr = [8];
break;
}
case $PROFESSIONAL:
{
$add_id_arr = [10];
break;
}
}
修正後
$rank_to_ids = [
$BASIC => [2, 20, 22],
$ADVANCE => [4, 105],
$MASTER => [8],
$PROFESSIONAL => [10],
];
$add_id_arr = $rank_to_ids[$rank] ?? [];
switchして変数の値を条件によって修正する程度で、その条件が固定値なのであれば、
今回のように$rankごとに配列マッピングして参照するほうが効率的