[翻譯]Feedback on the Go Challenge solutions

第一次Go Challenge比賽，中國區只有3人參賽。

賽后收到郵件，是一個審閱者的反饋，“Feedback on the Go Challenge solutions”，摘錄如下：

保持簡單粗暴

• 一個語義單元一個文件即可，不要像Java那樣一個文件就一個類型定義
• decoder.go 放解碼相關的內容，而 drum.go 放 Pattern 和 Track 類型，以及 String()
• 不要添加額外的類型，除非需要為其增加方法

Streaming 是個好概念

• 許多代碼基于io.Reader 和 io.Writer，包括都用到的encoding/binary
• 有人直接將文件全部讀到內存中，然后將[]byte封裝到bytes.Reader里，這是不可取的，還不如直接使用打開的file，因為這就是一個 bytes.Reader
• 建議閱讀“Crossing Streams: a Love Letter to io.Reader”

了解標準庫

• 使用 io.LimitedReader 可以封裝已有的io.Reader并設定最大長度來讀取，結合上面提到的直接使用io.Reader更簡單粗暴

Errors are values

• 沒必要不停地做錯誤檢查，這太繁瑣
• 將io.Reader封裝到自定義的Reader中，記錄碰到的最后一個error
  • 解析整個文件，別管那些錯誤
  • 最后檢查下存儲錯誤的結構
• 建議閱讀Rob Pike的“Errors are values”

不要泄露實現細節

• 雖然文件中使用[16]byte來存儲節拍，但是對于ON/OFF類型，用[16]bool就好
• 沒必要為了節省空間使用uint16，因為這會讓API不好搞

寫代碼不是為了通過測試，而是為了解決問題

• Pattern 5在文件末尾有額外的數據，應該通過解碼文件頭獲取文件長度來解析文件，而不是看ID有沒有SPLI
• 通過測試不代表一切OK