Попытка упростить мои операторы if

Question

Следующий запрос mysqli всегда будет вытаскивать один результат. Однако, посмотрев на мой код, я чувствую, что должен быть лучший способ упростить раздел html выпадающего списка. Как вы можете видеть, у меня есть куча операторов if и массива. Я сделал это так, чтобы установить selected в html в качестве текущего раскрывающегося списка. Может ли кто-нибудь помочь мне сделать это проще?

$query = mysqli_query($mysqli, "SELECT * From referrals WHERE id = '".$edit."';"); 

    while($row = mysqli_fetch_array($query)) 

    { 
     $editstatus = $row['status']; 
    } 

      if($editstatus == "N") 
       { 
       $estatus = "N/A"; 
       } 

      if($editstatus == "I") 
       { 
       $estatus = "Installation Comp"; 
       } 

      if($editstatus == "SI") 
       { 
       $estatus = "Site Inspection"; 
       } 

      if($editstatus == "S") 
       { 
       $estatus = "Sold"; 
       } 

      if($editstatus == "C") 
       { 
       $estatus = "Cancelled"; 
       } 

      if($editstatus == "P") 
       { 
       $estatus = "Press/Follow-Up"; 
       } 

      if($editstatus == "W") 
       { 
       $estatus = "Being Installed"; 
       } 

        $bstatus[] = "N/A"; 
        $bstatus[] = "Installation Comp"; 
        $bstatus[] = "Site Inspection"; 
        $bstatus[] = "Sold"; 
        $bstatus[] = "Cancelled"; 
        $bstatus[] = "Press/Follow-Up"; 
        $bstatus[] = "Being Installed"; 
?> 

    <div class="status"><label for="edit_status">Edit Status</label> 
    <select id="edit_status" name="edit_status"> 

<?php 

           foreach($bstatus as $cstatus) { 

             if($cstatus == "N/A") 
              { 
              $dstatus = "N"; 
              } 

             if($cstatus == "Installation Comp") 
              { 
              $dstatus = "I"; 
              } 

             if($cstatus == "SI") 
              { 
              $dstatus = "Site Inspection"; 
              } 

             if($cstatus == "Sold") 
              { 
              $dstatus = "S"; 
              } 

             if($cstatus == "Cancelled") 
              { 
              $dstatus = "C"; 
              } 

             if($cstatus == "Press/Follow-Up") 
              { 
              $dstatus = "P"; 
              } 

             if($cstatus == "Being Installed") 
              { 
              $dstatus = "W"; 
              } 

?> 

    <option <?php if($cstatus == $estatus) { echo "selected=\"selected\""; } ?> value="<?php echo $dstatus; ?>"><?php echo $cstatus ?></option> 

<?php 
                  } 
?> 

    </select>  
    </div> 

Как я уже говорил, посмотрев на этот код, я знаю, что должен быть лучший способ сделать это, я просто не знаю, как. Любая помощь будет принята с благодарностью.

Answer 1

отличный кандидат для switch

$editstatus = $row['status']; 
switch ($editstatus) 
{ 
    case 'N': 
     $estatus = 'N/A'; 
     break; 
    case 'I': 
     $estatus = 'Installation Comp'; 
     break; 
    // Fill in the rest here ... 
    default: 
     // None of my case statements were matched 
     break; 
} 

Answer 2

Почему бы не создать ассоциативный массив переменных editstatuses для estateses?

http://php.net/manual/en/language.types.array.php

$statuses = array(
     "N" => "N/A", 
     "I" => "Installation Comp", 
     ... 
    ) 

Answer 3

Используйте один из указанных ниже способов

Метода 1

switch ($editstatus) { 
    case "N": 
     $dstatus = "N"; 
     break; 
    case "I": 
     $estatus = "Installation Comp" 
     break; 
//rest of your code goes here 
} 

http://php.net/manual/en/control-structures.switch.php

Метод 2

Ассоциативный массив будет вполне подходящим слишком

$array = array(
    "N" => "N", 
    "I" => "Installation Comp", 
    .. 
    .. 
); 

http://php.net/manual/en/language.types.array.php