Talk:Go Fish/Go: Difference between revisions
Content added Content deleted
(→Not idiomatic: new section) |
|||
Line 13: | Line 13: | ||
—[[User:dchapes|dchapes]] ([[User talk:dchapes|talk]] | [[Special:Contributions/dchapes|contribs]]) 18:45, 22 February 2015 (UTC) |
—[[User:dchapes|dchapes]] ([[User talk:dchapes|talk]] | [[Special:Contributions/dchapes|contribs]]) 18:45, 22 February 2015 (UTC) |
||
It's been refactored on two separate occasions since this was posted. How do you feel about it now? |
|||
—[[User:Joshua.Snider|Joshua.Snider]] ([[User talk:Joshua.Snider|talk]]) 05:21, 12 September 2015 (UTC) |
Latest revision as of 05:21, 12 September 2015
Not idiomatic
This is not an idiomatic or good example of Go code.
See Effective Go and the Go Authors code review comments page to start.
For example:
- Identifier names should not have underscores and no leading underscores. Names like
kFoo
andtBar
are also non-idiomatic. - Function/method comments are usually in
//
rather than/* */
. They shouldn't be wrapped so short and they should be written for reasonable godoc parsing (e.g. start with a summary sentence that starts with the name of the function/method/type). - Receiver names should not be
self
,this
, orme
- …
The overall structure and quality of this example is also sub-par.
—dchapes (talk | contribs) 18:45, 22 February 2015 (UTC)
It's been refactored on two separate occasions since this was posted. How do you feel about it now?
—Joshua.Snider (talk) 05:21, 12 September 2015 (UTC)