21
Tips for a successful code review

성공적인코드리뷰 문화 사내 발표

Embed Size (px)

Citation preview

Page 1: 성공적인코드리뷰 문화 사내 발표

Tips for a successful code review

Page 2: 성공적인코드리뷰 문화 사내 발표

“Introduction to the Personal Software Process – PSP” by  Watts S. Humphrey.

Page 3: 성공적인코드리뷰 문화 사내 발표

how can they achieve a level of code quality that causes pride among the professionals involved?

The answer: CODE REVIEW.

Page 4: 성공적인코드리뷰 문화 사내 발표

There are three types of code reviews:

pair programming,

formal code review,

lightweight code review.

Page 5: 성공적인코드리뷰 문화 사내 발표

 benefit of code review

Page 6: 성공적인코드리뷰 문화 사내 발표

“good idea; bad execu-tion”

Page 7: 성공적인코드리뷰 문화 사내 발표

Worst code Review

Page 8: 성공적인코드리뷰 문화 사내 발표

intellectually vain can make code reviews a

nightmare of shouting and hurt feelings. 

But despite all that they are very useful.

Page 9: 성공적인코드리뷰 문화 사내 발표

Learn to enjoy the code re-views.

Page 10: 성공적인코드리뷰 문화 사내 발표

how do we make code reviews use-ful?

Page 11: 성공적인코드리뷰 문화 사내 발표

Code reviews SHOULD’T be performed to find errors in the code.

Code reviews SHOULD’T be performed to enforce coding standards.

Page 12: 성공적인코드리뷰 문화 사내 발표

Keep a positive attitude. remember: you are not the code!

Page 13: 성공적인코드리뷰 문화 사내 발표

Time, Your coworkers are waiting for you.

Page 14: 성공적인코드리뷰 문화 사내 발표

Code writer (before code review) 

1. Document 2. At least 2 days before3. Code revision number – JIRA open

Page 15: 성공적인코드리뷰 문화 사내 발표

Code writer (ing)

일단 각오해라 .

Page 16: 성공적인코드리뷰 문화 사내 발표

Reviewer 

Ask questions rather than make statements.

Page 17: 성공적인코드리뷰 문화 사내 발표

Reviewer 

Avoid the "Why" questions.

Page 18: 성공적인코드리뷰 문화 사내 발표

Reviewer

Make sure the discussion stays focused on the code and not the coder.

Page 19: 성공적인코드리뷰 문화 사내 발표

Standard / Checklist

Page 20: 성공적인코드리뷰 문화 사내 발표

Moderator

Page 21: 성공적인코드리뷰 문화 사내 발표

Beware the "Big Brother" effect.