2012-08-02 16 views
0

這裏是我的代碼混合PHP和HTML。 (它顯示4X4表)。如何縮小這種混雜的情況?

如何使此代碼獲得更多的可讀性?

如果你改變HTML縮進,U應該改變整個HTML。 (此代碼是整個來源的一部分。)

你對我有什麼想法。請告訴我。助教。

<!-- GALLERY BEGIN --> 
<tr> 
    <td style="padding-top:5px;"><table width="100%" border="0" cellpadding="2" cellspacing="0"> 
    <tr> 
     <?php 
     $ii = 0; 
     $aRows = getArticle('board_table', $DB_CONNECT, 8); 
     foreach ($aRows as $iidx => $aRow) : 
      $ii++; 
      $U = getUpfiles('upload_table',$aRow[uid],''); 
      $pic = getImage($U); 
     ?> 
     <td width="75"><table width="60" border="0" cellspacing="0" cellpadding="0"> 
     <tr> 
      <td><a href="<?=$aRow[link]?>"><img src="<?=$pic?>"/></a></td> 
      </tr> 
     <tr> 
      <td align="center"><?=getStrCut($aRow[subject], 8, '..')?></td> 
      </tr> 
     </table></td> 
      <?php if($ii%4==0): ?> 
     </tr> 
    </table></td> 
    </tr> 
<tr> 
    <td><table width="100%" border="0" cellpadding="2" cellspacing="0"> 
    <tr>  
      <?php endif; ?> 
     <?php endforeach; ?> 
     </tr> 
    </table></td> 
    </tr> 
<!-- END BEGIN --> 
+0

我會首先使用大括號({})來代替BASIC類語法的循環。這將使讀取和縮進(IMO)變得更容易。此外,我會建議代碼塊之間的額外間距。 – Ynhockey 2012-08-02 07:46:37

回答

0

我會建議使用某種模板引擎。像smarty。這將允許你保持你的設計數據更加獨立。

0

甚至不關心HTML縮進,它至少不重要。無論是你的用戶,還是你的瀏覽器和你的調試,你都應該使用類似Chrome的開發者工具或者良好的'Firebug - 這些工具會爲你縮進HTML,不管實際的代碼是什麼樣子。

縮進你最好閱讀代碼的方式,而不是你認爲最適合該程序的東西。至於那,你應該真的避免混合HTML和PHP。正如bkwint所說的,使用SMARTY或TWIG等模板引擎。雖然,在他們的HTML中,仍然存在一些奇怪的「中間」代碼,但它看起來會更清晰,更易於理解。

至於您當前的代碼:無論您是在PHP代碼塊還是在HTML部分中,都要使用標準縮進。這意味着,打開一個循環時,縮進,if或標籤。這至少應該以某種方式清理那些桌面驅動的瘋狂行爲。