erinus / 91tdd Goto Github PK

Good job.

Some suggestions:

1. Although you declare List as readonly, the elements of List still can be modified. You should use structure like ReadonlyCollection.

2. Column as a string argument has some advantages and some constraints, as a first version of delivery, you did it very well. Try to fulfill the other flexibility requirements, like pass a lambda for choosing what columns client want to sum.

建議找機會重寫（或許是上課完）

• 第二個測試案例就把一堆紅燈補上去，然後硬幹了 production code (這個動作是沒意義的)，導致瞬間產生一大堆不好維護的 production code, 連重構都不容易下手，幾乎只能重寫。這跟作業的規範也不符合。

• 最終的 production code 可讀性很差，你可以跟任何一個上課的同學交換一下想法，他們瞭解需求，但他們是否能不經過你解釋，就看懂、甚至維護你的程式碼。需求異動之後，是不是能輕易應付需求？

• 不該 public 的，就不要 public。命名可以再精準一點。

我不馬上給你 sample solution, 因為沒太大意義，只能說作法差異挺大的。