如何更有效地重写这个php

Question

我需要显示手动/自动变速箱车辆信息，我能想到的最好的就是下面的代码。我认为这是错误的并且效率低下，但我不能完全指出它。

<?php
if ($obj->AutoTrans == 'S'):
  echo "Automatic";
  if ($obj->ManualTrans == 'O'):
    echo " (Manual Optional)";
  endif;                   
elseif($obj->ManualTrans == 'S'):
  echo "Manual";
  if ($obj->AutoTrans == 'O'):
    echo " (Automatic Optional)";
  endif;     
endif;                  
?>

Answer 1

if ($obj->AutoTrans == 'S')
    echo "Automatic".($this->ManualTrans=='O'?' (Manual Optional)':'');

if ($obj->ManualTrans == 'S')
    echo "Manual".($this->AutoTrans=='O'?' (Automatic Optional)':'');

Answer 2

这种方法没有什么效率低下的地方。不过，您可以通过使用花括号而不是使用块语法来将其编写得更清晰（在我看来）。