用户
 找回密码
 入住 CI 中国社区
搜索
查看: 4182|回复: 6
收起左侧

看了这个cms,感觉代码一点都不规范....

[复制链接]
发表于 2011-9-26 10:24:54 | 显示全部楼层 |阅读模式
本帖最后由 phpcaicai 于 2011-9-26 10:27 编辑

看了这个cms,感觉代码一点都不规范....,作者是不是考虑下规范问题
发表于 2011-9-26 10:29:49 | 显示全部楼层
嗯,接受您的批评,我也是菜鸟一枚
另外,说明下你脑海中的规范。
发表于 2011-9-29 10:54:46 | 显示全部楼层
呵呵 刚开发的还有待改善
发表于 2011-9-30 16:04:05 | 显示全部楼层
呵呵,都是达人们牺牲休息时间写的,多些包容吧
发表于 2011-11-12 02:10:16 | 显示全部楼层
加油!我们与你同在……
发表于 2011-11-20 18:35:36 | 显示全部楼层
刚下载来看了一下,代码是有点不规范,但不算是“一点都不规范”。

简单的规范,可参考一下Wiki上的Indent style,选定其中一个。如CI便是基于Allman style。

http://en.wikipedia.org/wiki/Indent_style

if, while, foreach等是语言的一部份,后面应跟随空格后再加括号。is_array,isset等是函数,括号应紧随而无空格。


initialize时应查看settings目录有否修改权,否则报了“初始化完成!”,但实际上没有成功。


看时在想,settings目录的东西为何不都改在数据库内?settings目录是不是可看成一个cache目录?

从目前看,settings内生成的php scripts,在php语法或函数有所修改的,如何处理?


整体来看,算是不错。野心似乎颇大,布局有点大,小心别弄得太复杂。

评分

参与人数 1威望 +15 收起 理由
jeongee + 15 非常感谢中肯的评论。

查看全部评分

发表于 2011-11-20 19:40:10 | 显示全部楼层
燃雲 发表于 2011-11-20 18:35
刚下载来看了一下,代码是有点不规范,但不算是“一点都不规范”。

简单的规范,可参考一下Wiki上的Indent ...

嗯,代码不规范的问题,我会慢慢改正的
2.0.0版本,settings目录下所有的内容都已经放到数据库中,就是1个cache目录了
如果运行在SAE中,这个文件夹没用,因为写到memcache中了,呵呵
其实没有什么野心哈。
感谢燃云同学的指导。

本版积分规则