解決go lint上顯示的warning，包含縮寫及error命名、一致的receiver命名等

[titaneric]

👏 解決掉的 issue / Resolved Issues

• close #52
• close #53

⛏ 變更內容 / Details of Changes

• 縮寫命名
  • JSON
  • UI
  • ID
• 一致的receiver命名
• Error命名
• increment (E.g., idx++)
• unreachable code
• typo
• PosOf省略bbs名稱

[titaneric]

準備可以review了，但CI好像有go build的問題要幫忙看一下

[minchao]

@titaneric CI 裡的問題是 #37 及 https://github.com/Ptt-official-app/go-bbs/pull/58#issuecomment-789741746 提到的 module declare，注意到 development 分支在剛剛已經修正了。

[PichuChen]

可以再幫我 pull 一次 development 嗎？

[minchao]

簡單在 merge current development 後的版本，透過 golangci-lint 跑 golint，發現幾個漏網之魚，提供參考：

$ golangci-lint run
cache/systemvshm.go:50:2: var `shmId` should be `shmID` (golint)
    shmId, err := Shmget(key, size, flag)
    ^
cache/systemvshm.go:95:2: var `shmId` should be `shmID` (golint)
    shmId, err := Shmget(key, 0, 0)
    ^
cache/systemvshm.go:112:1: receiver name m should be consistent with previous receiver name s for SHM (golint)
func (m *SHM) Close() error {
^
pttbbs/board.go:69:23: method BoardId should be BoardID (golint)
func (b *BoardHeader) BoardId() string            { return b.BoardID() }
                      ^
pttbbs/cache.go:14:2: struct field `posOfUserId` should be `posOfUserID` (golint)
    posOfUserId     int
    ^
pttbbs/cache.go:297:17: method UserId should be UserID (golint)
func (c *Cache) UserId(uid int) string {
                ^

[titaneric]

感謝 @minchao 與 @PichuChen ，目前CI過囉，再麻煩幫忙review

[minchao]

@titaneric 剩下最後一個 receiver 修改就過 lint 囉

% golangci-lint run
cache/systemvshm.go:112:1: receiver name m should be consistent with previous receiver name s for SHM (golint)
func (m *SHM) Close() error {
^