2016福州大学软件工程 团队Git Review

时间:2022-05-05 10:16:15

606Connected组:

git地址https://github.com/606notconnected/Beta.git

文件忽略:使用gitignore较正确

版本控制:只有1次commit,5个分支,可能是最后才提交到库里的,开发的时候并没有很好地进行版本控制;没有任何issue。

readme:没有内容

文件结构:良好,遵循了安卓开发规范

命名:项目命名只有一个Beta不太合理

编程规范:基本遵守一定的规范,注意到代码中硬编码的字符串比较多,另外java中字符串的比较建议都用 equals()!equals()代替==!=

Aruba组:

git地址https://git.coding.net/aa5680737/Cento.git

文件忽略:使用gitignore正确,未见临时文件和.class等二进制文件,apk文件其实应该忽略掉

版本控制:开发期间能够按时有效地commit,且每次commit都有相应日志,维护了master和开发版2个分支,没有issue。

readme:主要包括3部分,即运行环境、更新信息以及关于我们,信息准确有效,还应增加主要功能说明、编译、使用等说明。

文件结构和命名:良好

编程规范:基本遵守一定的规范,if-else语句等能够一直坚持自己团队的风格,很好,存在的问题:

1. 字符串的问题在606Connected组里提到了,不建议硬编码。

2. 请保证分支和循环语句体都被大括号包括,就算只有一行语句也是如此。

3. 另外发现存在空的函数体,这个并没有大的影响,建议增加todo的异常。

CLover组:

git地址https://github.com/liezhengli/put-me-down.git

文件忽略:使用gitignore基本正确

版本控制:共有179次commit,开发期间能够按时有效地commit,且每次commit都有相应日志,共有53个已关闭issue,10个开放issue,其中已关闭issue均已解决。

readme:主要包括简介、编码规范和更新日志,其中更新日志不全,另外还应增加主要功能说明、编译、使用等说明。

文件结构和命名:能够按照readme中的规范进行

编程规范:基本遵守一定的规范,有一些诸如硬编码、缩进、空函数等共有的问题

TAC组:

git地址https://github.com/AiProgram/TAC-Android.git

文件忽略:使用gitignore基本正确,并且gitignore添加了注释,很好,其中后台版本中有疑似测试的空文件,不应该出现

版本控制:共有211次commit,开发期间能够按时有效地commit,有的commit日志只是简单的日期或者一个'.',不建议这样做。只有一次分支,体现不出版本控制的作用,且每次commit都有相应日志,没有issue

readme:主要包括项目介绍、功能介绍和编译安装说明,其中编译说明中的图没有显示出来,后台版的readme比较简略。

编程规范:基本遵守规范,有一些共性的问题

1.请保证分支和循环语句体都被大括号包括,就算只有一行语句也是如此。

2.switch语句最好有default出口。

3.代码风格方面,双目运算符前后最好加上空格。

4.尽量不要出现if(exp){return true;}else{return false;}这样形式的代码,直接return表达式就可以了。

Transcend组:

git地址https://github.com/cafe3165/admin.git

文件忽略:使用gitignore较正确,建议gitignore文件中加上注释,或者按类别中间添加空行,项目备份文件也可以写入ignore文件中

版本控制:共有200余次commit,且附有准确的commit日志;

readme:没有内容

文件结构:良好,遵循了安卓开发规范

命名:项目命名为admin不太合理,建议修改

编程规范:这组代码风格比较好,可以适当多一些注释,另外发现有空的{}语句块,不是大问题,但是最好不要这样。

一不小心就火了组:

git地址https://coding.net/u/YC_chen/p/StudentTutorSelect/git

文件忽略:使用gitignore较正确,都是安卓系统配置大同小异。

版本控制:平均每个组员都能做到几十次commit,且有commit日志,没有issue

readme:没有内容,希望尽快补上

文件结构:良好,遵循了安卓开发规范

编程规范:这组代码量惊人~值得鼓励,不过由此带来的类文件可能比较相对复杂的问题,以后有机会重构可以考虑。另外有关switch分支语句,在case后面一定要保证有break,在这组的代码中发现了有省略break的情况,另外硬编码、语句块的大括号都是老生常谈的问题了。

NO Bug组:

git地址https://git.coding.net/Yuaoi/BuyTogether.git

文件忽略:安卓的gitignore基本都差不多,apk文件应该加入到其中,php没有见到gitignore文件,不过也没有见到需要忽略的文件。

版本控制:开发期间能够正确有效地commit,commit的日志显得稍有些杂乱,有的就是一个改动的文件名,不知道具体做了什么修改,没有issue。

readme:没有内容,希望尽快补上

文件结构:良好,遵循了安卓和php开发规范,开发文档建议组织在一个文件夹中

编程相关:安卓部分的规范前面的组讲了很多了,这组主要看了一下php部分,有一个问题是php函数中的默认参数应该放在最后,发现有的地方会出现正常参数反而在默认参数后的情况,其他都很好。

我说的都队:

git地址https://github.com/Tutordistribution/Tutor-distribution.git

文件忽略:该组与其他组不同的地方在于将.idea文件夹加入到了忽略中,是可取的

版本控制:开发期间能够正确有效地commit,共有541次commit,并且能够准确使用日志,已有119个已关闭issues,关闭的issue均有说明和标签,很好。

readme:没有内容,希望尽快补上

文件结构:很好,其中"原型.md"文件可以合并入doc文件夹中.

编程相关:这组是为数不多的采用B/S的,主要编程语言是php,这组代码量更加惊人,而且由于前后端的特性,采用的语言比较多,js、css、php甚至cpp各种代码都有,这对于pm的组织能力是很大的考验,php方面发现的问题主要有case语句要保证有确定的break或者return出口,还有exit()和die()用得比较多,个人不太建议,总体很好!