[일반] 갤주가 말하는 코드리뷰가 어떤 느낌임?

[익명(117.111)]

코드리뷰가 없는 기업에선 성장이 불가능하다는 말을 봤는데 그게 어떤 느낌인지 모르겠음...그게 pr할때 로직에 이상 없는지 체크하는 수준은 아닐 것 같은데

[ㅇㅇㅇ(qwebnm9)]

로직 작동만 체크하는게 아니라 그 코드가 클린한지, 유지보수 하기 좋은 코드인지, 확장하기 좋은 코드인지 등등 여러 관점에서 코드를 살펴보고 토론하는거지

[백갤러 1(221.145)]

면접때 그런 식으로 디테일하게 리뷰를 하느냐고 물어봤는데 여긴 학원이 아니기 때문에 그건 본인이 코드를 짤때 고민을 많이 해야 한다, 그런 고민 없이 짠 코드를 pr 올리면 많이 혼나겠죠? 라는 답변을 받았음... 여긴 이상한 곳인가

[백갤러 1(221.145)]

답변해줘서 너무너무 고마워요

[익명(183.107)]

일 터지는 크기에따라 시니어가 수습해야할 확률이 올라가서 코드리뷰는 빡세게 하는게 맞다

[ㅇㅇㅇ(qwebnm9)]

물론 고민없이 수준낮은 코드를 올리면 혼나긴 하겠지. 하지만 고민을 아무리 해도 사람마다 코드를 보는 시각은 다를 수 밖에 없어서 대화를 많이 할 수 밖에 없는 구조일텐데..

[김대기(waitkim)]

필요하다면 처음부터 다시 작성하는 수준의 코드리뷰를 의미하는건데(당연히 시니어랑 페어프로그래밍) 일정이 있으니까 매번 그 정도 레벨로는 힘듬

[김대기(waitkim)]

중요한건 틈이 날때마다 주니어의 성장을 위한 코드리뷰를 해야한다는거야 팀의 처리 능력은 가장 못하는놈이 결정함

[백갤러 2(221.145)]

후 중소 위주로 최근에 면접 다녀보는데 어렵구만 괜찮아보이는 곳도 갤주가 말한 부분에서 한 두가지씩 충족이 안되니깐...

[백갤러 2(221.145)]

답변 고맙습니다

[김대기(waitkim)]

중소에선 찾기 힘들지..

[백갤러 2(221.145)]

그럼 만약 중소를 가는 걸로 타협하게 된다면 코드리뷰는 포기하는 게 맞는 걸까요

[김대기(waitkim)]

극단적으로 생각할건 없고 최대한 챙긴다는 생각으로 해봐