川山甲

追求内心的非常平静!瞬间清空所有的杂念,达到物我两忘!

  博客园  :: 首页  ::  :: 联系 :: 订阅 订阅  :: 管理
  105 随笔 :: 0 文章 :: 1591 评论 :: 82万 阅读
 
 介绍
 
也许有人说这个不是坏代码。——呵呵,坏与好永远是相对的,对我来说,这些代码是坏代码。
 
我最近看了一个源码,是团购网站的源码,看到了很多的问题。
 
 
 常量的使用
 

                                     

把常量定义在方法体内,而且不只一个地方。

                                     

                                     

居然还有常量文件,而且也不只一个。

                                     

 

我建议把所有的常量放在一个常量文件,这样便于我们将来维护方便。

 

一个文件放置多个类
 

                                     

这种代码更是让人一头雾水,为什么这么写。

 

我的建议是,一个文件放置一个类。这样我们就能很好的找到我们要找的类,代码的直观性更强一些。

 

多个文件拥有相同的类
  

 
                                     

 

建议,每个类名可以跟文件名相关联。如index.mod.php我们可以命名里面的类名叫IndexModuleObject。

 框架代码混乱
 
所有的module调用,就是上面提到的,都用一个名字ModuleObject,怪怪的。
 
加载的template文件,写在logic文件中(Model文件中)。

我的建议是,尽量把整个框架代码简单些,能让程序员立马找到module和template。

总结
 
整个代码不太好的地方很多,如果逐个挑出来,我估计我得写成系列了。——我在想初期是不是没有做很好的规划呢?
 
我觉得如果项目紧,没有人专门做php框架,那为什么不使用现在比较成熟的php框架(如thinkphp,zend framework)呢?这样整个代码更加规范些,也容易维护。——这种代码,让后来的程序员,情何以堪?
 
也许是我见的php框架代码太相似了,所以看到这种,大惊小怪了些,大家就权当我“年纪大了,喜欢唠叨吧"。
 
推荐
 
 

 

 
posted on   川山甲  阅读(4187)  评论(22编辑  收藏  举报
编辑推荐:
· [.NET]调用本地 Deepseek 模型
· 一个费力不讨好的项目,让我损失了近一半的绩效!
· .NET Core 托管堆内存泄露/CPU异常的常见思路
· PostgreSQL 和 SQL Server 在统计信息维护中的关键差异
· C++代码改造为UTF-8编码问题的总结
阅读排行:
· CSnakes vs Python.NET:高效嵌入与灵活互通的跨语言方案对比
· 【.NET】调用本地 Deepseek 模型
· Plotly.NET 一个为 .NET 打造的强大开源交互式图表库
· 上周热点回顾(2.17-2.23)
· 如何使用 Uni-app 实现视频聊天(源码,支持安卓、iOS)
点击右上角即可分享
微信分享提示