• 设为首页
  • 点击收藏
  • 手机版
    手机扫一扫访问
    迪恩网络手机版
  • 关注官方公众号
    微信扫一扫关注
    公众号

[翻译]Feedback on the Go Challenge solutions

原作者: [db:作者] 来自: [db:来源] 收藏 邀请

第一次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

鲜花

握手

雷人

路过

鸡蛋
该文章已有0人参与评论

请发表评论

全部评论

专题导读
上一篇:
Go语言-开发环境搭建(windows)发布时间:2022-07-10
下一篇:
Go操作Redis发布时间:2022-07-10
热门推荐
热门话题
阅读排行榜

扫描微信二维码

查看手机版网站

随时了解更新最新资讯

139-2527-9053

在线客服(服务时间 9:00~18:00)

在线QQ客服
地址:深圳市南山区西丽大学城创智工业园
电邮:jeky_zhao#qq.com
移动电话:139-2527-9053

Powered by 互联科技 X3.4© 2001-2213 极客世界.|Sitemap