SlideShare uma empresa Scribd logo
1 de 21
Tips for a successful code review
“Introduction to the Personal Software Process – PSP”
                                by Watts S. Humphrey.
how can they achieve a level of code quality that
causes pride among the professionals involved?


The answer: CODE REVIEW.
There are three types of code reviews:




                        pair programming,

                        formal code review,

                        lightweight code review.
benefit of code review
“good idea; bad execution”
Worst code Review
intellectually vain can make code reviews a
                               nightmare of shouting and hurt feelings.




                 But despite all that they are very useful.
Learn to enjoy the code reviews.
how do we make code reviews useful?
Code reviews SHOULD’T be performed to find errors in the code.

Code reviews SHOULD’T be performed to enforce coding standards.
Keep a positive attitude. remember: you are not the code!
Time, Your coworkers are waiting for you.
Code writer (before code review)

          1. Document
          2. At least 2 days before
          3. Code revision number – JIRA open
Code writer (ing)


일단 각오해라.
Reviewer


Ask questions rather than make statements.
Reviewer

Avoid the "Why" questions.
Reviewer


Make sure the discussion stays focused on the code and not the coder.
Standard / Checklist
Moderator
Beware the "Big Brother" effect.

Mais conteúdo relacionado

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

11 best practices_for_peer_code_review
11 best practices_for_peer_code_review11 best practices_for_peer_code_review
11 best practices_for_peer_code_review
SmartBear Software
 

Semelhante a 성공적인코드리뷰 문화 사내 발표 (20)

10 Faulty Behaviors of Code Review - Developer Summit Istanbul 2018
10 Faulty Behaviors of Code Review - Developer Summit Istanbul 201810 Faulty Behaviors of Code Review - Developer Summit Istanbul 2018
10 Faulty Behaviors of Code Review - Developer Summit Istanbul 2018
 
Code review in practice
Code review in practiceCode review in practice
Code review in practice
 
Code review guidelines
Code review guidelinesCode review guidelines
Code review guidelines
 
Software Quality via Unit Testing
Software Quality via Unit TestingSoftware Quality via Unit Testing
Software Quality via Unit Testing
 
Gateway to Agile: XP and BDD
Gateway to Agile: XP and BDD Gateway to Agile: XP and BDD
Gateway to Agile: XP and BDD
 
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pdf
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pdfcode_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pdf
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pdf
 
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pptx
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pptxcode_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pptx
code_review_checklist_6_actions_to_improve_the_quality_of_your_reviews.pptx
 
Peer review
Peer reviewPeer review
Peer review
 
Mob Programming for Continuous Learning
Mob Programming for Continuous LearningMob Programming for Continuous Learning
Mob Programming for Continuous Learning
 
Code quality as a built-in process
Code quality as a built-in processCode quality as a built-in process
Code quality as a built-in process
 
11 best practices_for_peer_code_review
11 best practices_for_peer_code_review11 best practices_for_peer_code_review
11 best practices_for_peer_code_review
 
Let's review it: What designers can learn from (code) review
Let's review it: What designers can learn from (code) reviewLet's review it: What designers can learn from (code) review
Let's review it: What designers can learn from (code) review
 
Code review best practice
Code review best practiceCode review best practice
Code review best practice
 
Code Club - a Fight Club inspired approach to software inspection and review
Code Club - a Fight Club inspired approach to software inspection and reviewCode Club - a Fight Club inspired approach to software inspection and review
Code Club - a Fight Club inspired approach to software inspection and review
 
How to successfully grow a code review culture
How to successfully grow a code review cultureHow to successfully grow a code review culture
How to successfully grow a code review culture
 
Code Review Best Practices
Code Review Best PracticesCode Review Best Practices
Code Review Best Practices
 
Code reviews
Code reviewsCode reviews
Code reviews
 
Are your interns reviewing code? Andrew Lavers, ConFoo Montreal 2020
Are your interns reviewing code? Andrew Lavers, ConFoo Montreal 2020Are your interns reviewing code? Andrew Lavers, ConFoo Montreal 2020
Are your interns reviewing code? Andrew Lavers, ConFoo Montreal 2020
 
Software Defect Prevention via Continuous Inspection
Software Defect Prevention via Continuous InspectionSoftware Defect Prevention via Continuous Inspection
Software Defect Prevention via Continuous Inspection
 
Being Test-Driven: It's not really about testing
Being Test-Driven: It's not really about testingBeing Test-Driven: It's not really about testing
Being Test-Driven: It's not really about testing
 

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