ass和class之间有的没有空两行
def和def之间有的空了一行有的没有空
变量命名还可以
很多数字常量应该定义成常量, 这样好更改, 而且更加语义化, 一目了然
注释很少
参数列表的逗号后面应该有空格
程序就一个文件, 虽然抽象了类, 但是没有分文件出来
main过于长了
##周润来
python格式稍微好一点
也是基本没有注释
能自己处理WebSocket握手应答还是不错的
同样是缺少配置, 应该把一些数字, 字符串常量抽离出来定义成变量, 好修改
mysql.py 注释了过多不必须注释的地方
##李凯杰
数据库ci_blog.sql是copy的吧 时间都没改
把substr函数放到了blog.php的控制器里面, 不合理, 应该放在helper里面
而且这个函数为什么$length参数是字符串??
没有发挥动态语言的特性, 仍然保持着静态语言每次先生命变量的习惯
注释不太多
缩进还可以
用CI自己的模板引擎
虽然说前端不是考察重点, 但是很多文件用到了同样的css, view里面的文件只是简单的粘贴复制, 没有写一个css文件抽离出来, 这样改起来很麻烦
##李昊洋
python爬虫
运行效果, 爬出来的还不错
代码风格上, 比前几个要好, 至少分离了一些常量, 但是忽略了一些正则表达式的分离, 只是常量字符串的形式出现在了代码里, 最好是用变量定义出来比较好
空行的掌握不是很好
##王大为
django框架写的博客
模块化分的可以
数据库用的是sqlite 不太符合我们的要求
django 为他做的事情太多了
有些class和class之间没有两行分隔
为什么根目录有了images文件夹, 在static里面还有一个images文件夹? grappelli下面有images又有img文件夹
##王浩文
用php手写的聊天室
手写的页面上的echo, 代码很多地方没有注意排版
缺少把功能封装成类的思想, 还是单个文件形成接口
也缺少数据过滤, 安全性是个大问题
sql是用phpmyadmin生成的, 其实没有反应sql语句的功底
注释也基本没有
css没有缩进, 看来是不拘小节啊....
##蒋奇志
说明较详细
程序放注释的还可以, 粒度还行
使用的是图形化的界面, 并不是websocket但是是本地socket
多线程管理
client和server的代码都混在一个类里面的了, 不太好
没有意识到用MVC, 数据, 视图和控制分离
##魏俊杰
编码没有使用标准的utf-8 有些是gb2312 有些是iso-8859-2的
函数的命名还可以
没有找到他的sql文件放哪里了
只有一个blog控制器的设计不太好, 应该分开几个
注释不多
竟然把获取数据的操作放在了model里面! MVC理解不够