GoでWebアプリ開発時にあるあるだったレビューコメント | フューチャー技術ブログ

The Gopher character is based on the Go mascot designed by Renée French. はじめにTIG DXユニット 1の真野です。 コードレビューについては3,4年ほど前に、コードレビューにおけるレビュアー側のアンチパターン って記事を書いたりもしました。当時はレビュアーの伝え方って大事だよなって話をしてました。いつしかレビュイーからレビュアーに比重が変わることが増えてきました。相互レビューは当たり前にしていますがが、比較的こうしたらもっと良くなるんじゃないかな？と提案される回数より、自分が提案する回数の方が増えてくるタイミングってありますよね？ そういうわけで、最近Goで主にバックエンドのWebAPIや、AWS Lambdaで動くETLアプリ、たまにCLIツールを開発する時に、2回以上同じ指摘したコメントをまとめてます。Go言語

[mano-junki]

書きました

[Keisuke69]

ちゃんと形式知化して素晴らしい。リトライロジック入れるのとかついついおざなりになるのでこれもいいですね

[roshi]

コードと知見ありがたい。コメントに絵文字は斬新だと思った。

[yarumato]

“Goを初めて使うよってメンバーに読んでもらうリンク一覧。Go Tutorialを1日やった後に、環境構築して、次の内容を適時読む。JavaプログラマーのためのGo言語入門、Goを学ぶときにつまずきやすいポイントFAQ、Effective Go”

[kanda_k]

errorラップしないならそのまんまreturnとかめっちゃわかる

[MH4GF]

レビュー内容もめっちゃ最高だし、Go初心者向けのインプット資料のまとめも最高。読んでいこう

[everybodyelse]

Goに限らずリーダブルコード的な話もあってとてもよい。

[mactkg]

よかった。17知らなかった

[fashi]

あとで拡張するだろうと思って分割して書いてるのを1行で済みますよねって直されたらキレそう

[toritori0318]

良い

[hasiduki]

勉強になる！

[tune]

期待値が高かったのかもう一つに感じてしまった。あんまりGo関係ないような

[ryskosn]

読むべき記事へのリンクもあり。

[Ashburton]

goはセキュリティ周りを自分で実装しないといけない。それができないのに採用するのはやめて

[pokuwagata]

もう1回じっくり読もう

[kwhrtsk]

“main関数以外でpanicやos.Exit(1)やlog.Fatalを行うのは原則禁止です。テストがしにくいといったことが理由です。” panicは後の２つと違ってテストでもハンドリングできるので、使うべきところでは使った方が良いと思う。

[yujiorama]

現場感あふれる内容だった

[peketamin]

ちょうどGoをやり直してるとこだったので、めっちゃ参考になりました。感謝

[piro_suke]

“Go言語で幸せになれる10のテクニック - Qiita”