irisshu / gradesystemproject Goto Github PK

代碼

• 第11組的代碼風格相當不錯，在寫法命名上看得出來。 ex: nOfId 的變數名稱，容易瞭解，但也可能讓變數名稱變得太長。
• 風格為了維護相同，在撰寫函數上捨棄掉原本可以利用的 method，這樣的寫法無法斷定是好是壞。Float.valueOf(scanner.next()) 應可直接寫成 scanner.nextFloat() ~
• 由幾個小瑕疵，package name 命名錯誤，小地方非常細心，但是卻將 model 命名成 modle，我想這裏有八成是太忙而忽視。(到底有沒有 pair programming ?)

測試

• @Test(expected = NoSuchCommandExceptions.class) 學到這個挺精簡的寫法。 ~
• 雖然在 UnitTest.java 檔中，巧妙地繞過了上課遇到的 Scanner 問題，但是唯一美中不足的是，全部的測試都是對 class UI 著手，並沒有達到整合測試目的，最基礎要求的成績平均計算也沒有加入 (也許看起來不怎麼必要)。
• test code 調用還沒有確定測試的 method 進行預期比較。 ~
• 沒有測試加權平均，發現有計算公式錯誤，輸入雖然為正整數，但實際應採用比例關係，例如輸入 10，在文件中占有為 0.1，但是代碼處理上忘了這一個部分，排名的測試是測不出來這個錯誤。

曲解

• 對於老師給的文件中，public String promptCommand 的理解也許是我的錯誤，到底是單純的進行命令輸入，還是要進行命令輸入+操作 ? 這個根據函數名稱並沒有定義特別清楚。
• Class header 沒有整理，雖然代碼不錯。

結論

我寫的代碼太糟了

確認新增學生995002903的姓名及成績
姓名  張一二
                 Lab1 90
                 Lab2 90
                 Lab3 90
                 Mid-term   90
                 Final exam 90  (yes/no) yes

操作環境：eclipse
在這項舉動時，發生輸入中文的障礙，導致後來無法輸入數字。

http://www.agilemethod.csie.ncu.edu.tw/

描述客戶的維修任務 (maintenance task). 維修以前開發而現在忘了的Grade System 助教將隨機指定貴組維修別組的軟體 若貴組能依本文件正確迅速完成任務 則該軟體維修性佳 該組原作者加分 反之扣分 最重要的 貴組要遵循本文件的 myAgile 工序

4) A 新增學生 (Add)     
5) D 刪減學生 (Delete)   
6) M 更改分數 (Modify)

將別組代碼增加三項功能

輸入指令 1) G 顯示成績 (Grade) 
2) R 顯示排名 (Rank) 
3) W 更新配分 (Weight) 
4) A 新增學生 (Add)     
5) D 刪減學生 (Delete)   
6) M 更改分數 (Modify)  
7) Q 結束使用  (Quit) 

新的選單操作，將 7) E 修改成 7) Q。

輸入指令如上   E 
輸入ID或 Q (結束使用)？ Q

按照文件開頭的"輸入指令"，我們已經沒有E這個選項，但卻在文件後面出現。