SlideShare a Scribd company logo
1 of 19
Download to read offline
Review를 잘 하자!
어떻게 하면 잘 할까? 무조건 더 노력?!
올림포스 @ 쿠팡
지난 review 되집어 보기.
Review를 왜 했었나요?
여러 분들의 생각을 마구마구 이야기 해주세요~
했던 Review는 어떤 느낌인가요?
여러 분들의 생각을 마구마구 이야기 해주세요~
이런 경험이 있나요?
리뷰를 거친 뒤에, 배포를 보냈는데.
리뷰를 했던 부분에서 에러가 발생이 되는 일 발생.
실제 장애로 이어지기까지 한 경우.
리뷰도 했는데, 에러가 발견이 안 되었네요.
다 같이 모여서 그렇게 같이 봤는데.
“조금 더 열심히 리뷰를 해야겠습니다.”
어떻게 하면 Review를 잘 할까요?
여러 분들의 생각을 마구마구 이야기 해주세요~
처음부터 생각해 봅시다.
리뷰를 하는 목적은?
잠재적인 문제를 미리 발견하고자 함.
비용 효과가 높은 문제를 찾아내서, 미리 해결하자.
호미로 막을 것을 가래로 막는다.
100% 완벽한 문서를 만드는 것은 이상이다.
효율적인 문제 도출에 집중을.
이 관점에서 리뷰에서 도출해야 할 문제
는?
● 오탈자 확인
● 메소드 리팩토링 방안
● 데이터베이스 Connection 자원 미반환 지점
● 모호한 클래스 이름
● Exception에 대한, Log 하지 않는 부분들
집중해야 할 문제에 집중할 방법은?
● 문제 유형을 명확히 정의
o 리뷰 관점 축소.
● 해당 유형에 집중한 리뷰 수행
o 집중된 관점 없이 리뷰를 수행할 때
 사소한 문제에 집중을 하는 경우가 많음
o 제한된 관점에서 리뷰를 수행할 때
 중요 문제 추출 건수가 1.4배 증가 효과 (실험 결과)
● 리뷰에도 효과적인 진행 방법이 있구나.
o 이 방법에 대해서 알아보자!
시간에 떠밀려 종료
장시간 지체되는 리뷰로 인해서, 앞에
서는 집중하다가 뒷 부분에서는 끝내
는 데에만 집중하여, 대충 넘어가는 리
뷰.
리뷰 효율성을 떨어지게 하는 유형들
생각나는 대로 말하기
리뷰를 하러 와서, 보이는 데로 찾아서
이야기를 한다. 오탈자 같은 찾기 쉬운
사소한 이슈들로 리뷰가 가득.
문서 작성자 비난
발견된 문제를 넘어, 작성자에 대한 평
가 및 비난을 하는 경우들. 또한 리뷰
받는 사람은 무조건 자기 방어만 하는
경우.
복수/회피의 장
문제가 아닌, 작성자와 관계로 인한 리
뷰 수위 및 집중도가 달라진 리뷰.
무서운 상사 코드, 얄미운 동료 코드
효과적인 리뷰 절차
우선 작성을 잘 한다.
마무리 전 자가 리뷰는
필수!
리뷰 시나리오 도출 및
공유
집중점을 제시.
리뷰어 문제 도출 시간
시나리오 기준 사전 점검
기준이 없다면, 언제까지 어디까지 해야 할
지 막막.
이것 저것 보다 보면, 큰 문제를 놓치게 된다.
정해진 시나리오당
Timebox
수행을 권장
집중력 좋은 시간 리뷰
개최
체념 방지! 소중한 시간
절약!
효과적인 리뷰 미팅 진행
● 도출해온 문제 사항 모아서
진행
● 문제 발견이 편중되는지 확
인 및 소홀하게 리뷰 되는지
주의
● 논쟁 경계.
● 발견된 문제 연관 문제도 확
인
● 문제 발견에 집중. 수정은 별
도. 문제 별 대응 방침 수립.
● 주제 이탈 방지.
이 안에는 사람이 있
다.
회의 정리
발견 문제 및 대응 방침 공유, 시나리오 업데이트, 수정 결과 확인 및 재발 방지
책 수립
시나리오 관리 = 지식 관리
● 사람은 실수를 반복. 실수들에는 유형이 있는 법.
● 이러한 발생 가능 실수에 대한 점검 방법을 구체적으로
기술한 것이 시나리오.
● 시나리오 Pool을 만들자. (오답 노트 ^^)
o 주의해야 할 관점에 대한 Check List
o 문제 상황들에 대한 노하우 축적 및 발전
o 유사 프로젝트 간에는 유사 시나리오가 효과적
● 시나리오에도 여파에 따라서 우선 순위가 있다.
리뷰 스킬과 기술 스킬은 별개.
Your Attitude Decides Your Altitude.
의식적 불필요 발언 제어 능력
● 베테랑 일수록 말하고 싶은 경험이 많이 겪어, 악의 없이 참견을 하
기 쉽다.
o 전부 쓸데없는 말은 아니다. 그러나 대부분 길어지면서 주제 이
탈.
o 대부분 자신을 드러내고, 인정 받고자 하는 마음의 발현.
o 이런 참견은 회의 시간 지연, 집중력 저하, 중요 문제 놓침.
● 대화를 하라, 게임을 하지 말고.
o 대화라 함은 경청과 생각 그리고 제안 및 인정.
o 이기고 지는 게임이 아니다. 졌다고 내가 못난게 아니다.
o 내가 이기기 위해서, 지속적으로 틀린 점을 찾지 마라.
● 경험이 뛰어 날 수록, 생각은 많이 하고 입은 단속하는 자세가 필요.
o 듣는 이들 입장을 생각하면서, 받아들일 수 있게 이야기 하자.
하기 쉬운 커뮤니케이션 실수들.
문제 발견/해결이 최우선. 나의 뽐
내기 시간 아님.
나를 뽐내기 위한, 지적질에 매료 금지
지적질 경쟁 모드 돌입 금지
장황한 기술 지식 과시. 멀어질 뿐.
의도적 묵인
회의를 빨리 넘어가고자, 문제를 지적
하는 과정을 싫어해서, 의도적 무관심.
감정/가치 평가를 담은 리뷰 목적
벗어난 헐뜯기
내 결론을 강요하며, 틀렸다/맞다라는
가치 판단이 동반된 폭력적 언어 사용
리뷰 받는 이의 정색 및 적반하장
리뷰 받는 입장에서 지속적으로 틀렸
다는 의견 받는 시간.
힘들다고 이성을 잃어서, “늬가 해봐"
라는 식 대응은 금물.
기타 팁.
● 좋은 점에 대해서도 체크 했다가 칭찬해 주도록.
o 너무 지적만 하고 받다 보면, 딱딱해지는 경우 많음.
o 칭찬 사례도 모아두면, 팀 내 좋은 지식 창고.
● 사소한 문제에 얽매이지 말고, 중요한 문제 검출에 집중
o 의식적으로 상세히 읽지 마라.
o 검토하는 시나리오에 집중하고, 세세한 사항은 넘어가자.
● 문제 검출에 집중. 수정 방법(코드 수정)은 따로 고민.
o 효율적인 시간 활용 위한 팁.
o 흐름을 유지하면서 리뷰를 진행할 수 있도록.
o 내부 교육 및 팀원 간 정보 공유는 리뷰 미팅과 별도로
● 민감한 상황에 대한 합의된 신호 사용을 통해, 심각하지 않게.
o 예) 영철님, 파랑새 입니다. = 주제 벗어났습니다.
● 리뷰 통해 문제점 도출되면, 이를 배움의 기회로 생각하고 감사하게 받아들
이자.
o 자기 방어 활동은 성장에서 나를 멀게 한다.
o 자기 방어 활동이 시작되지 않도록, 상대방을 배려한 의사소통.
리뷰에도 유형이.
● Walk Through
o 문서 작성자가 요청해서 실시.
o 문서 작성자가 질문. 능력 향상 및 교육을 위한 용도로도 활용
o 능력 향상과 같은 폭넓은 목적을 위한 리뷰.
● Techinal Review
o 통상적인 리뷰 형태. Technical Leader가 주도하는 리뷰.
o 수정 방향은 Technical Leader가 최종 결정.
● Inspection
o 정해진 규칙에 따라서 코드를 리뷰
o 코드 내 지켜야 할 명확한 규칙에 대한 수행 여부 리뷰
비용 효율성을 위한 사전 문
제 검출 및 수정
이것이 리뷰를 하는 이유이다.
목적/의도를 잊지 않고 접근하자.

More Related Content

Similar to Review 어떻게 할까 20140820

[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기
[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기 [NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기
[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기 Jungsoo Lee
 
브레인스토밍 아이디어발상법
브레인스토밍 아이디어발상법브레인스토밍 아이디어발상법
브레인스토밍 아이디어발상법seekly
 
ApprenticeshipPatterns/Chapter5
ApprenticeshipPatterns/Chapter5ApprenticeshipPatterns/Chapter5
ApprenticeshipPatterns/Chapter5Suan Lee
 
SWDeveloprStory201601
SWDeveloprStory201601SWDeveloprStory201601
SWDeveloprStory201601Suho Kwon
 
How to startup 02- 5factors
How to startup 02- 5factorsHow to startup 02- 5factors
How to startup 02- 5factors종익 주
 
[2012 11 12]애자일 회고
[2012 11 12]애자일 회고[2012 11 12]애자일 회고
[2012 11 12]애자일 회고Jong Pil Won
 
직장인경쟁력강화방법
직장인경쟁력강화방법직장인경쟁력강화방법
직장인경쟁력강화방법인태 박
 
00 갈등 Disc
00 갈등 Disc00 갈등 Disc
00 갈등 Discparkds2
 
리뷰의 기술 소개
리뷰의 기술 소개리뷰의 기술 소개
리뷰의 기술 소개beom kyun choi
 
0. review. 린과 애자일 개발
0. review. 린과 애자일 개발0. review. 린과 애자일 개발
0. review. 린과 애자일 개발Unyong (Sheldon) Choi
 
적을만들지않는대화법
적을만들지않는대화법적을만들지않는대화법
적을만들지않는대화법Ji Young Kim
 
00 직장인 경쟁력(Shrinked)
00 직장인 경쟁력(Shrinked)00 직장인 경쟁력(Shrinked)
00 직장인 경쟁력(Shrinked)parkds2
 
조직개발_아이디어 연결 및 결합하기_다이애나크롤
조직개발_아이디어 연결 및 결합하기_다이애나크롤조직개발_아이디어 연결 및 결합하기_다이애나크롤
조직개발_아이디어 연결 및 결합하기_다이애나크롤od_KHU
 
이상현 - 노력이 배신하는 이유
이상현 - 노력이 배신하는 이유이상현 - 노력이 배신하는 이유
이상현 - 노력이 배신하는 이유상현 이
 
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조Freewill Inc.
 
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018devCAT Studio, NEXON
 
Interviewbox 140227
Interviewbox 140227Interviewbox 140227
Interviewbox 140227pxdstory
 
스마일게이트 서버개발캠프 - 5vengers
스마일게이트 서버개발캠프 - 5vengers 스마일게이트 서버개발캠프 - 5vengers
스마일게이트 서버개발캠프 - 5vengers ServerDevCamp
 

Similar to Review 어떻게 할까 20140820 (20)

[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기
[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기 [NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기
[NDC 16] 당신은 사랑 받기 위해 태어난 사람: 3년차 게임 디자이너의 자존감 찾기
 
브레인스토밍 아이디어발상법
브레인스토밍 아이디어발상법브레인스토밍 아이디어발상법
브레인스토밍 아이디어발상법
 
Pair programming how_to_20140930-1
Pair programming how_to_20140930-1Pair programming how_to_20140930-1
Pair programming how_to_20140930-1
 
ApprenticeshipPatterns/Chapter5
ApprenticeshipPatterns/Chapter5ApprenticeshipPatterns/Chapter5
ApprenticeshipPatterns/Chapter5
 
SWDeveloprStory201601
SWDeveloprStory201601SWDeveloprStory201601
SWDeveloprStory201601
 
How to startup 02- 5factors
How to startup 02- 5factorsHow to startup 02- 5factors
How to startup 02- 5factors
 
[2012 11 12]애자일 회고
[2012 11 12]애자일 회고[2012 11 12]애자일 회고
[2012 11 12]애자일 회고
 
직장인경쟁력강화방법
직장인경쟁력강화방법직장인경쟁력강화방법
직장인경쟁력강화방법
 
00 갈등 Disc
00 갈등 Disc00 갈등 Disc
00 갈등 Disc
 
리뷰의 기술 소개
리뷰의 기술 소개리뷰의 기술 소개
리뷰의 기술 소개
 
애자일프랙티스
애자일프랙티스애자일프랙티스
애자일프랙티스
 
0. review. 린과 애자일 개발
0. review. 린과 애자일 개발0. review. 린과 애자일 개발
0. review. 린과 애자일 개발
 
적을만들지않는대화법
적을만들지않는대화법적을만들지않는대화법
적을만들지않는대화법
 
00 직장인 경쟁력(Shrinked)
00 직장인 경쟁력(Shrinked)00 직장인 경쟁력(Shrinked)
00 직장인 경쟁력(Shrinked)
 
조직개발_아이디어 연결 및 결합하기_다이애나크롤
조직개발_아이디어 연결 및 결합하기_다이애나크롤조직개발_아이디어 연결 및 결합하기_다이애나크롤
조직개발_아이디어 연결 및 결합하기_다이애나크롤
 
이상현 - 노력이 배신하는 이유
이상현 - 노력이 배신하는 이유이상현 - 노력이 배신하는 이유
이상현 - 노력이 배신하는 이유
 
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조
[2023-1학기 아산 유스프러너 앙트십 프로젝트] 성일고등학교 4조
 
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018
홍성우, 게임 프로그래머는 어떻게 가르치나요?, NDC2018
 
Interviewbox 140227
Interviewbox 140227Interviewbox 140227
Interviewbox 140227
 
스마일게이트 서버개발캠프 - 5vengers
스마일게이트 서버개발캠프 - 5vengers 스마일게이트 서버개발캠프 - 5vengers
스마일게이트 서버개발캠프 - 5vengers
 

More from Unyong (Sheldon) Choi

만화 SlamDunk로 보는 scrum
만화 SlamDunk로 보는 scrum만화 SlamDunk로 보는 scrum
만화 SlamDunk로 보는 scrumUnyong (Sheldon) Choi
 
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.Unyong (Sheldon) Choi
 
Concept Design For AchieveIt Service. (generated at 2012/06)
Concept Design For AchieveIt Service. (generated at 2012/06)Concept Design For AchieveIt Service. (generated at 2012/06)
Concept Design For AchieveIt Service. (generated at 2012/06)Unyong (Sheldon) Choi
 
팀웍에 대한 새로운 견해. (출처: 원아웃)
팀웍에 대한 새로운 견해. (출처: 원아웃)팀웍에 대한 새로운 견해. (출처: 원아웃)
팀웍에 대한 새로운 견해. (출처: 원아웃)Unyong (Sheldon) Choi
 

More from Unyong (Sheldon) Choi (8)

만화 SlamDunk로 보는 scrum
만화 SlamDunk로 보는 scrum만화 SlamDunk로 보는 scrum
만화 SlamDunk로 보는 scrum
 
Team work 2014
Team work 2014Team work 2014
Team work 2014
 
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.
2013년 말에 들었던 IoT 관련 세미나에서 인상적이던 부분들.
 
Jobs comment about obama
Jobs comment about obamaJobs comment about obama
Jobs comment about obama
 
Concept Design For AchieveIt Service. (generated at 2012/06)
Concept Design For AchieveIt Service. (generated at 2012/06)Concept Design For AchieveIt Service. (generated at 2012/06)
Concept Design For AchieveIt Service. (generated at 2012/06)
 
The Art Of Readable Code.
The Art Of Readable Code.The Art Of Readable Code.
The Art Of Readable Code.
 
팀웍에 대한 새로운 견해. (출처: 원아웃)
팀웍에 대한 새로운 견해. (출처: 원아웃)팀웍에 대한 새로운 견해. (출처: 원아웃)
팀웍에 대한 새로운 견해. (출처: 원아웃)
 
Seminar agile samurai
Seminar agile samuraiSeminar agile samurai
Seminar agile samurai
 

Review 어떻게 할까 20140820

  • 1. Review를 잘 하자! 어떻게 하면 잘 할까? 무조건 더 노력?! 올림포스 @ 쿠팡
  • 3. Review를 왜 했었나요? 여러 분들의 생각을 마구마구 이야기 해주세요~
  • 4. 했던 Review는 어떤 느낌인가요? 여러 분들의 생각을 마구마구 이야기 해주세요~
  • 5. 이런 경험이 있나요? 리뷰를 거친 뒤에, 배포를 보냈는데. 리뷰를 했던 부분에서 에러가 발생이 되는 일 발생. 실제 장애로 이어지기까지 한 경우. 리뷰도 했는데, 에러가 발견이 안 되었네요. 다 같이 모여서 그렇게 같이 봤는데. “조금 더 열심히 리뷰를 해야겠습니다.”
  • 6. 어떻게 하면 Review를 잘 할까요? 여러 분들의 생각을 마구마구 이야기 해주세요~
  • 8. 리뷰를 하는 목적은? 잠재적인 문제를 미리 발견하고자 함. 비용 효과가 높은 문제를 찾아내서, 미리 해결하자. 호미로 막을 것을 가래로 막는다. 100% 완벽한 문서를 만드는 것은 이상이다. 효율적인 문제 도출에 집중을.
  • 9. 이 관점에서 리뷰에서 도출해야 할 문제 는? ● 오탈자 확인 ● 메소드 리팩토링 방안 ● 데이터베이스 Connection 자원 미반환 지점 ● 모호한 클래스 이름 ● Exception에 대한, Log 하지 않는 부분들
  • 10. 집중해야 할 문제에 집중할 방법은? ● 문제 유형을 명확히 정의 o 리뷰 관점 축소. ● 해당 유형에 집중한 리뷰 수행 o 집중된 관점 없이 리뷰를 수행할 때  사소한 문제에 집중을 하는 경우가 많음 o 제한된 관점에서 리뷰를 수행할 때  중요 문제 추출 건수가 1.4배 증가 효과 (실험 결과) ● 리뷰에도 효과적인 진행 방법이 있구나. o 이 방법에 대해서 알아보자!
  • 11. 시간에 떠밀려 종료 장시간 지체되는 리뷰로 인해서, 앞에 서는 집중하다가 뒷 부분에서는 끝내 는 데에만 집중하여, 대충 넘어가는 리 뷰. 리뷰 효율성을 떨어지게 하는 유형들 생각나는 대로 말하기 리뷰를 하러 와서, 보이는 데로 찾아서 이야기를 한다. 오탈자 같은 찾기 쉬운 사소한 이슈들로 리뷰가 가득. 문서 작성자 비난 발견된 문제를 넘어, 작성자에 대한 평 가 및 비난을 하는 경우들. 또한 리뷰 받는 사람은 무조건 자기 방어만 하는 경우. 복수/회피의 장 문제가 아닌, 작성자와 관계로 인한 리 뷰 수위 및 집중도가 달라진 리뷰. 무서운 상사 코드, 얄미운 동료 코드
  • 12. 효과적인 리뷰 절차 우선 작성을 잘 한다. 마무리 전 자가 리뷰는 필수! 리뷰 시나리오 도출 및 공유 집중점을 제시. 리뷰어 문제 도출 시간 시나리오 기준 사전 점검 기준이 없다면, 언제까지 어디까지 해야 할 지 막막. 이것 저것 보다 보면, 큰 문제를 놓치게 된다. 정해진 시나리오당 Timebox 수행을 권장 집중력 좋은 시간 리뷰 개최 체념 방지! 소중한 시간 절약! 효과적인 리뷰 미팅 진행 ● 도출해온 문제 사항 모아서 진행 ● 문제 발견이 편중되는지 확 인 및 소홀하게 리뷰 되는지 주의 ● 논쟁 경계. ● 발견된 문제 연관 문제도 확 인 ● 문제 발견에 집중. 수정은 별 도. 문제 별 대응 방침 수립. ● 주제 이탈 방지. 이 안에는 사람이 있 다. 회의 정리 발견 문제 및 대응 방침 공유, 시나리오 업데이트, 수정 결과 확인 및 재발 방지 책 수립
  • 13. 시나리오 관리 = 지식 관리 ● 사람은 실수를 반복. 실수들에는 유형이 있는 법. ● 이러한 발생 가능 실수에 대한 점검 방법을 구체적으로 기술한 것이 시나리오. ● 시나리오 Pool을 만들자. (오답 노트 ^^) o 주의해야 할 관점에 대한 Check List o 문제 상황들에 대한 노하우 축적 및 발전 o 유사 프로젝트 간에는 유사 시나리오가 효과적 ● 시나리오에도 여파에 따라서 우선 순위가 있다.
  • 14. 리뷰 스킬과 기술 스킬은 별개. Your Attitude Decides Your Altitude.
  • 15. 의식적 불필요 발언 제어 능력 ● 베테랑 일수록 말하고 싶은 경험이 많이 겪어, 악의 없이 참견을 하 기 쉽다. o 전부 쓸데없는 말은 아니다. 그러나 대부분 길어지면서 주제 이 탈. o 대부분 자신을 드러내고, 인정 받고자 하는 마음의 발현. o 이런 참견은 회의 시간 지연, 집중력 저하, 중요 문제 놓침. ● 대화를 하라, 게임을 하지 말고. o 대화라 함은 경청과 생각 그리고 제안 및 인정. o 이기고 지는 게임이 아니다. 졌다고 내가 못난게 아니다. o 내가 이기기 위해서, 지속적으로 틀린 점을 찾지 마라. ● 경험이 뛰어 날 수록, 생각은 많이 하고 입은 단속하는 자세가 필요. o 듣는 이들 입장을 생각하면서, 받아들일 수 있게 이야기 하자.
  • 16. 하기 쉬운 커뮤니케이션 실수들. 문제 발견/해결이 최우선. 나의 뽐 내기 시간 아님. 나를 뽐내기 위한, 지적질에 매료 금지 지적질 경쟁 모드 돌입 금지 장황한 기술 지식 과시. 멀어질 뿐. 의도적 묵인 회의를 빨리 넘어가고자, 문제를 지적 하는 과정을 싫어해서, 의도적 무관심. 감정/가치 평가를 담은 리뷰 목적 벗어난 헐뜯기 내 결론을 강요하며, 틀렸다/맞다라는 가치 판단이 동반된 폭력적 언어 사용 리뷰 받는 이의 정색 및 적반하장 리뷰 받는 입장에서 지속적으로 틀렸 다는 의견 받는 시간. 힘들다고 이성을 잃어서, “늬가 해봐" 라는 식 대응은 금물.
  • 17. 기타 팁. ● 좋은 점에 대해서도 체크 했다가 칭찬해 주도록. o 너무 지적만 하고 받다 보면, 딱딱해지는 경우 많음. o 칭찬 사례도 모아두면, 팀 내 좋은 지식 창고. ● 사소한 문제에 얽매이지 말고, 중요한 문제 검출에 집중 o 의식적으로 상세히 읽지 마라. o 검토하는 시나리오에 집중하고, 세세한 사항은 넘어가자. ● 문제 검출에 집중. 수정 방법(코드 수정)은 따로 고민. o 효율적인 시간 활용 위한 팁. o 흐름을 유지하면서 리뷰를 진행할 수 있도록. o 내부 교육 및 팀원 간 정보 공유는 리뷰 미팅과 별도로 ● 민감한 상황에 대한 합의된 신호 사용을 통해, 심각하지 않게. o 예) 영철님, 파랑새 입니다. = 주제 벗어났습니다. ● 리뷰 통해 문제점 도출되면, 이를 배움의 기회로 생각하고 감사하게 받아들 이자. o 자기 방어 활동은 성장에서 나를 멀게 한다. o 자기 방어 활동이 시작되지 않도록, 상대방을 배려한 의사소통.
  • 18. 리뷰에도 유형이. ● Walk Through o 문서 작성자가 요청해서 실시. o 문서 작성자가 질문. 능력 향상 및 교육을 위한 용도로도 활용 o 능력 향상과 같은 폭넓은 목적을 위한 리뷰. ● Techinal Review o 통상적인 리뷰 형태. Technical Leader가 주도하는 리뷰. o 수정 방향은 Technical Leader가 최종 결정. ● Inspection o 정해진 규칙에 따라서 코드를 리뷰 o 코드 내 지켜야 할 명확한 규칙에 대한 수행 여부 리뷰
  • 19. 비용 효율성을 위한 사전 문 제 검출 및 수정 이것이 리뷰를 하는 이유이다. 목적/의도를 잊지 않고 접근하자.

Editor's Notes

  1. 이제 부터는 사람에 대한 이야기입니다. 아무리 기술 스킬이 좋아도 리뷰 스킬이 좋지 않은 경우가 많습니다. 이러한 관점에서 순수하게 리뷰 스킬 측면에서 나를 바라봅시다.
  2. 가장 대표적인 증상 중 하나인 그리고 조직상에서 여파가 큰 베테랑이 가지는 주의해야 할 점. 이 점이 아무래도 리뷰라는 상황에서는 기술적 우위라는 측면에서 상대적으로 베테랑들이 입이 쎄게 작용하기 때문에. 우리는 이 문제에 정말 집중해서 우리 자신을 돌아봐야 한다. 내가 이겨야 하는 게임이 아니에요. 우리 팀이 잘 되기 위해서, 좋은 결론을 찾아가는 과정이니깐요. 마음을 열고, 서로 받아들일 수 있도록 상대방 입장을 생각하면서 공감하면서 이야기를 하는게 좋아요. 이 외에 많이 있는 실수에 대해서는 다음 장에서 살펴봅니다.
  3. 기타 다른 커뮤니케이션 실수들에 대해서 하나씩 생각해보기로 해요. 이러한 접근을 의식적으로 알고 있으면, 내가 하는 행동들이 그러한지를 체크할 수 있어요. 이러한 측면에서 주의해야할 anti-pattern들을 다시 한 번 보는 시간이에요. 이 외에 추가적으로 리뷰를 효과적으로 하는 팁을 다음 장에서 보겠습니다.
  4. 작은 팁들이지만, 참으로 유요한 것들이에요. 특히 가운데 수정 방법은 따로 고민한 다는 것이 참으로 중요합니다. 리뷰 결과 수정을 하자고 하면, 수정에 대한 시간을 따로 잡아서 진행하면 진행이 효율적이에요.
  5. 본 장은 마지막 장입니다. 마지막으로 참고로 리뷰 유형에 대한 소개. 마지막으로, 공식 용어인 3가지 유형을 공유를 하면, 서로와 의사소통을 명료하게 해 줄 수 있어요. “Walk Through” 미팅을 하기로 해요~ 이는 제가 원하는 부분이 있으니 검토해 주셨으면 좋겠어요라고 이야기 하는 것을 함축하고 있게 되죠.
  6. 마무리 차원에서 다시 정리합니다.