如何更有效地重写这个php

-4

我需要显示手动/自动传输车辆信息，最好的我可以拿出下面的代码。我认为它错了，效率低下，但我无法完全放下手脚。如何更有效地重写这个php

<?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;     
?>

来源

2011-09-28 Dan Gayle

对不起，但这绝不是什么问题。再试一次。 –

你如何衡量'效率'说'我认为它错了*低效*' – zerkms

是的，他忘记了问号，但我认为它不是那么糟糕。他很可能会问：“有没有更清晰/更好/更简洁的方法来编写这些代码？” –

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

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

来源

2011-09-28 23:07:00 romaninsh

这种方法没什么低效率。尽管如此，你可以用花括号代替块语法来将它写得更干净（在我看来）。

来源

2011-09-28 23:04:39 FtDRbwLXw6

如何更有效地重写这个php

回答

相关问题