“天天團購”的壞代碼


 
 介紹
 
也許有人說這個不是壞代碼。——呵呵,壞與好永遠是相對的,對我來說,這些代碼是壞代碼。
 
我最近看了一個源碼,是團購網站的源碼,看到了很多的問題。
 
 
 常量的使用
 

                                     

把常量定義在方法體內,而且不只一個地方。

                                     

                                     

居然還有常量文件,而且也不只一個。

                                     

 

我建議把所有的常量放在一個常量文件,這樣便於我們將來維護方便。

 

一個文件放置多個類
 

                                     

這種代碼更是讓人一頭霧水,為什么這么寫。

 

我的建議是,一個文件放置一個類。這樣我們就能很好的找到我們要找的類,代碼的直觀性更強一些。

 

多個文件擁有相同的類
  

 
                                     

 

建議,每個類名可以跟文件名相關聯。如index.mod.php我們可以命名里面的類名叫IndexModuleObject。

 框架代碼混亂
 
所有的module調用,就是上面提到的,都用一個名字ModuleObject,怪怪的。
 
加載的template文件,寫在logic文件中(Model文件中)。

我的建議是,盡量把整個框架代碼簡單些,能讓程序員立馬找到module和template。

總結
 
整個代碼不太好的地方很多,如果逐個挑出來,我估計我得寫成系列了。——我在想初期是不是沒有做很好的規划呢?
 
我覺得如果項目緊,沒有人專門做php框架,那為什么不使用現在比較成熟的php框架(如thinkphp,zend framework)呢?這樣整個代碼更加規范些,也容易維護。——這種代碼,讓后來的程序員,情何以堪?
 
也許是我見的php框架代碼太相似了,所以看到這種,大驚小怪了些,大家就權當我“年紀大了,喜歡嘮叨吧"。
 
推薦
 
 

 

 


免責聲明!

本站轉載的文章為個人學習借鑒使用,本站對版權不負任何法律責任。如果侵犯了您的隱私權益,請聯系本站郵箱yoyou2525@163.com刪除。



 
粵ICP備18138465號   © 2018-2025 CODEPRJ.COM