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