Qiita Teams that are logged in
You are not logged in to any team

Log in to Qiita Team
Community
OrganizationAdvent CalendarQiitadon (β)
Service
Qiita JobsQiita ZineQiita Blog
32
Help us understand the problem. What is going on with this article?
@shingohry

「コードレビュー」について調べたのでまとめる

More than 5 years have passed since last update.

レビューする目的を明確にする

  • 開発のどの段階の、どういう文脈でのレビューなのか
    • 開発初期とリリース前では、どこにどの程度着目すべきかは違ってくる
    • プロジェクトにとって何が重要なのか
    • コストと効果を意識する
  • 分類
    • リファクタリング
    • パターンやライブラリ
    • バグの指摘
    • ロジックの整合性、仕様通りかどうか
    • パフォーマンス
    • 互換性、拡張性、メンテナンス性
    • 一貫性
    • 読みやすさ、言語やSDKのルール

人間にしか出来ないことに集中する

  • 機械にまかせられるようなチェックは機械にまかせる
    • インデントの間違い
    • スペースや改行の挿入位置
  • 静的解析
  • コーディング規約:無駄に悩むことを減らす

内容によっては他の手法が適している場合もある

  • ペアプロ

レビューは改善のために行うもの

  • 品質を向上させる
  • 共通の認識を共有する
  • メンバーのスキル向上
    • 人に読まれることを意識するようになる

参考記事

32
Help us understand the problem. What is going on with this article?
Why not register and get more from Qiita?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away

Comments

No comments
Sign up for free and join this conversation.
Sign Up
If you already have a Qiita account Login
32
Help us understand the problem. What is going on with this article?