图文并茂的解释“天天团购”坏代码

也许有人说这个不是坏代码。——呵呵，坏与好永远是相对的，对我来说，这些代码是坏代码。

我最近看了一个源码，是团购网站的源码，看到了很多的问题。

源码地址：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