2011-09-28 53 views
-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;     
?> 
+3

對不起,但這絕不是什麼問題。再試一次。 –

+0

你如何衡量'效率'說'我認爲它錯了*低效*' – zerkms

+1

是的,他忘記了問號,但我認爲它不是那麼糟糕。他很可能會問:「有沒有更清晰/更好/更簡潔的方法來編寫這些代碼?」 –

回答

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

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

這種方法沒什麼低效率。儘管如此,你可以用花括號代替塊語法來將它寫得更乾淨(在我看來)。