zoukankan      html  css  js  c++  java
  • 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()用得比较多,个人不太建议,总体很好!

  • 相关阅读:
    c++构造函数析构函数调用顺序
    c++隐藏实例
    c++子类和父类成员函数重名
    C++虚函数·
    c/c++字符数组和字符串大揭秘
    python 基础回顾 一
    python java scala 单例模式
    推荐一款好用并且免费的markdown软件 Typora
    java 的垃圾回收机制 【转】
    python的垃圾回收机制【转】
  • 原文地址:https://www.cnblogs.com/deerCode/p/6232602.html
Copyright © 2011-2022 走看看