ブックマーク / crieit.net (1)

  • コードレビュー ありがちな問題への対処例 - Crieit

    コードレビュー、これまでいろんなプロジェクトで経験して、意外と使われていないノウハウがあったり、風習が違ってつらみがあったりしたので、いろいろまとめてみる。 指摘事項について よくある話 - 駄目コードを憎んで人を憎まず。駄目なのはコードであって人格じゃない - 指摘する人は人格攻撃せずにコードのどこが悪いのかを指摘しましょう - 指摘される人も、言われているのはコードの問題であって人間の問題じゃないので、素直な心で受け止めよう この辺はみんな知ってると思うので略。ぼくが思う大事なルール コードレビューで指摘された内容は、対応必須ではない 理由: 対応必須にすると、「これ言ったらリリースできなくなるよね」みたいな忖度が発生してコメントできない人が出現するから。 絶対ダメとは言わないけど、あまりよくはない、みたいな指摘については、そのときは急ぐからリリースするけど、次回から気をつけるとかがあ

    コードレビュー ありがちな問題への対処例 - Crieit
    alt-native
    alt-native 2019/05/27
    前いたプロジェクト レビュー皆無だった。プロトタイプで仕様承認。結合・総合レベルで初めて品質検査。バグが収束しなくて笑ってた。レビュー大事。
  • 1