介紹
也許有人說這個不是壞代碼。——呵呵,壞與好永遠是相對的,對我來說,這些代碼是壞代碼。
我最近看了一個源碼,是團購網站的源碼,看到了很多的問題。
常量的使用
把常量定義在方法體內,而且不只一個地方。
居然還有常量文件,而且也不只一個。
我建議把所有的常量放在一個常量文件,這樣便於我們將來維護方便。
一個文件放置多個類
這種代碼更是讓人一頭霧水,為什么這么寫。
我的建議是,一個文件放置一個類。這樣我們就能很好的找到我們要找的類,代碼的直觀性更強一些。
多個文件擁有相同的類

建議,每個類名可以跟文件名相關聯。如index.mod.php我們可以命名里面的類名叫IndexModuleObject。
框架代碼混亂
所有的module調用,就是上面提到的,都用一個名字ModuleObject,怪怪的。
加載的template文件,寫在logic文件中(Model文件中)。
我的建議是,盡量把整個框架代碼簡單些,能讓程序員立馬找到module和template。
總結
整個代碼不太好的地方很多,如果逐個挑出來,我估計我得寫成系列了。——我在想初期是不是沒有做很好的規划呢?
我覺得如果項目緊,沒有人專門做php框架,那為什么不使用現在比較成熟的php框架(如thinkphp,zend framework)呢?這樣整個代碼更加規范些,也容易維護。——這種代碼,讓后來的程序員,情何以堪?
也許是我見的php框架代碼太相似了,所以看到這種,大驚小怪了些,大家就權當我“年紀大了,喜歡嘮叨吧"。
推薦
