Help us understand the problem. What is going on with this article?

コードレビューのやり方

More than 5 years have passed since last update.

コードレビュー

コードレビューについてのメモ

コードレビューについて

メリット

悪い兆候のあるコードに対する抑止力になり、「動けばよい」という妥協を封じ込める
教育的な効果
コードを共有することで、一人が死んでも大丈夫的な効果
コミュニケーション的な効果

デメリット

コードレビューに対するコスト

悪い兆候

コピー&ペーストのコード(=>変更に弱くなる)
1つの関数やメソッドが長過ぎるコード(=>見通しが悪く、バグを生みやすい)
テストコードが存在しなかったり、通らないコード(=>保守性が低まる)

レビューの心得

コミットを綺麗にする

git rebase -i HEAD~3 // とかでコミットを綺麗にしてpushするとか

バグはレビュアでもコミッタの責任でもなくチームの責任なので、思ったことをコメントする
違う人がレビュアでも気になったら、横やりでコメントする
ピアレビュー = 同僚などフラットな間柄で互いにレビューしあう

レビューの手法

上記みたいな詳細な説明が見つけられなかったので、以下は

http://masasuzu.github.io/document/book/PeerReview/chapter03.html

みてください。

  • チームレビュー

  • ウォークスルー

  • ペアレビュー

  • パスアラウンド

  • アドホックレビュー

参考URL

書籍

レビューツール

diffコメント型

Rietveld
Gerrit Code Review
Review Board
Barkeep

統合型

Gitlab
RhodeCode
Phabricator
GitHub
Bitbucket
Chromium
Lhaz

yanap
Existing in essence or effect though not in actual fact or form. みかけや形はそのものではないが、本質あるいは効果としてはそのものであること
https://github.com/yanap
showroom
SHOWROOMは、アイドルやアーティストとインターネット上でコミュニケーションが楽しめるライブ動画ストリーミングプラットフォームです。
https://www.showroom-live.com/
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