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

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()用得比较多，个人不太建议，总体很好！