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 코드내지켜야할명확한규칙에대한수행여부리뷰

비용효율성을위한사전문제검출및수정

이것이리뷰를하는이유이다.

목적/의도를잊지않고접근하자.