天天看點

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日志;

命名:項目命名為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。

檔案結構:良好,遵循了安卓和php開發規範,開發文檔建議組織在一個檔案夾中

程式設計相關:安卓部分的規範前面的組講了很多了,這組主要看了一下php部分,有一個問題是php函數中的預設參數應該放在最後,發現有的地方會出現正常參數反而在預設參數後的情況,其他都很好。

我說的都隊:

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

檔案忽略:該組與其他組不同的地方在于将.idea檔案夾加入到了忽略中,是可取的

版本控制:開發期間能夠正确有效地commit,共有541次commit,并且能夠準确使用日志,已有119個已關閉issues,關閉的issue均有說明和标簽,很好。

檔案結構:很好,其中"原型.md"檔案可以合并入doc檔案夾中.

程式設計相關:這組是為數不多的采用B/S的,主要程式設計語言是php,這組代碼量更加驚人,而且由于前後端的特性,采用的語言比較多,js、css、php甚至cpp各種代碼都有,這對于pm的組織能力是很大的考驗,php方面發現的問題主要有case語句要保證有确定的break或者return出口,還有exit()和die()用得比較多,個人不太建議,總體很好!