php - このPHPコードをリファクタリングする方法は？

Question

これが私のコードです：

$aClass = '';
$bClass = '';
$cClass = '';

if ($_GET['data'] == 'a')
{
    $aClass = ' active';
}
else if ($_GET['data'] == 'b')
{
    $bClass = ' active';
}
else if ($_GET['data'] == 'c')
{
    $cClass = ' active';
}

if ($aClass == '' && $bClass == '' && $cClass == '')
{
    $aClass = ' active';
}

何か案は？これは厄介で冗長なようです。

Answer 1

switch-caseコンストラクトを使用できます

switch($_GET['data']) {
    case "a": //redundant it is also the default case
        $aClass = ' active';
        break;
    case "b":
        $bClass = ' active';
        break;
    case "c":
        $cClass = ' active';
        break;
    default:
        $aClass = ' active';
}

Answer 2

たった4行で;）

if ($_GET['data'] == 'b' || $_GET['data'] == 'c')
  ${$_GET['data'] . 'Class'} = ' active';
else
  $aClass = ' active';

Answer 3

cssクラスを設定しているようです。もしそうなら、あなたは試すことができます

if ($_GET['data'] == 'b')
{
    $bClass = ' active';
}
else if ($_GET['data'] == 'c')
{
    $cClass = ' active';
}
else
{
    $aClass = ' active';
}

Answer 4

switch ($_GET['data']) {
case 'a': $aClass = ' active';
break;
//ETC....
case default: 
$aClass = ' active';
break;
}