也許有人說這個不是壞代碼。——呵呵，壞與好永遠是相對的，對我來說，這些代碼是壞代碼。

我最近看了一個源碼，是團購網站的源碼，看到了很多的問題。

源碼地址：http://www.tttuangou.net/download.html

常量的使用

                                     [[76950]]

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

                                     [[76950]]

                                     [[76950]]

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

                                     [[76950]]

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

#p#

一個文件放置多個類

                                     [[76950]]

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

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

多個文件擁有相同的類

< style="text-align: center">                                     [[76950]]

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

#p#

框架代碼混亂 

所有的module調用，就是上面提到的，都用一個名字ModuleObject，怪怪的。

加載的template文件，寫在logic文件中（Model文件中）。

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

總結

整個代碼不太好的地方很多，如果逐個挑出來，我估計我得寫成系列了。——我在想初期是不是沒有做很好的規劃呢？

我覺得如果項目緊，沒有人專門做php框架，那為什么不使用現在比較成熟的php框架(如thinkphp,zend framework)呢？這樣整個代碼更加規范些，也容易維護。——這種代碼，讓后來的程序員，情何以堪?

也許是我見的php框架代碼太相似了，所以看到這種，大驚小怪了些，大家就權當我“年紀大了，喜歡嘮叨吧"

原文鏈接：http://www.cnblogs.com/baochuan/archive/2012/05/22/2511093.html