fugafuga.write

日々のログ



レビュー時のこころがけ

開発時にコードレビューや設計レビューを行う機会がある。

良いレビュー、悪いレビューって何だ?ということを考えたりしたので書く。

悪いレビュー

レビュアー

  • 主観しかない
  • 何かに基いて指摘していない
    • 根拠がゆるふわである
  • 宗教上の理由で、とか言う
  • 感情にあふれすぎている

レビュイー

  • コードの指摘を人格否定と受け取る
  • 指摘されたことを考えなしに修正、実行する

良いレビュー

悪いレビューの反対のことをすればよい

レビュアー

  • 根拠に基づいた指摘をする
  • 事実を見る
  • 冷静に指摘する
  • 相手に寄り添う

レビュイー

  • 指摘を事実として受け取る
    • 良くするためにやってる
    • 実際無力感でつらいこともあるが卑屈にならないようにする
    • 感謝の気持ち
  • 必ず自分で考える
    • 指摘の意味がわからなければ質問する
    • 納得できるまでやる

まとめ

問題 vs 私達

これができればチームは良くなると信じている。

アジャイルな見積りと計画づくり ?価値あるソフトウェアを育てる概念と技法?

アジャイルな見積りと計画づくり ?価値あるソフトウェアを育てる概念と技法?

アジャイルサムライ――達人開発者への道

アジャイルサムライ――達人開発者への道