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()用得比較多，個人不太建議，總體很好！