-
[---] IT기업 코드리뷰가 원래 이런건가요?2021.01.04 PM 11:19
그동안 SM만 하다가 코드리뷰가 있는 회사에 처음 왔습니다.
지금까지 4~5번 진행했는데
인수인계도 받지 않은 상태에서 소스코드만 보고 익힌 업무 지식 가지고
구현한 프로그램을 리뷰 받았습니다. 특정 페이지의 사용자 로그를 남기는 프로그램 이었는데요...
하나의 Model을 가지고 각 업무적 행위가 행해지는곳에 코드를 삽입하고 이력을 남기도록 했는데요...
한숨을 푹푹 쉬면서 왜 곳곳에 코드를 이렇게 많이 써놨냐,
6개월이 지났는데 기본적인 로직도 모르느냐 ....매번 1시간 넘게 이러니까
사람들 앞에서 공개망신을 당하는것 같습니다. 지칩니다...
저희 회사 언어는 php를 쓰는데요.
이력정보를 팝업으로 보여주기 위해 백엔드에서 코드값을 이름으로 바꿔주기 위해
함수 array_walk를 이용해 익명함수로 배열 요소를 돌면서
코드->이름으로 바꿔주는 작업을 하고 화면단에 던져줘서 display되게 했는데
왜 화면단에서 한번 Loop 돌릴걸 백엔드에서 한번 돌고 던지느냐 비난조로 지적하고...
맞는말인건 알겠는데 비난하는 어조로 계속 들으니까 점점 일하기가 싫고
코드리뷰가 원래 여러사람 앞에서 공개 비난하는것인가 라는 생각이 듭니다.
답답해서 하소연해봤습니다. ㅠ
- JuliaHart
- 2021/01/04 PM 11:28
갈굼당할수록 실력이 느니까 좋게 생각하세요.
그리고 내꺼 보여주기만 하지 말고
다른사람들 커밋들도 까뒤집어까면서 리뷰 하세요.
당장 쪽팔릴수는 있어도 대충 뭉개고 주먹구구식으로 넘어가는 회사에서보다 몇배는 체계적으로 배울 수 있습니다
개인적인 생각으로 욕먹더라도 내 코드에 관심주고 지적해주는 사람은 스승으로 모셔야한다고 봅니다.
- 밝은
- 2021/01/05 AM 12:09
- E-마토
- 2021/01/04 PM 11:29
글쓴이님이 크게 흉이 될만한 사건이 없었다는 전제하에
비난조로 한다는건 별로 좋은 회사는 아닌거같군요.
- 밝은
- 2021/01/05 AM 12:09
- 철철새
- 2021/01/04 PM 11:31
- 으훗
- 2021/01/04 PM 11:39
막상 부드럽게 이야기하면 1년넘게 조인하나 못거는 개발자도 종종있습니다.
뒤돌면 다 까먹는
- 아르테니
- 2021/01/04 PM 11:47
비난조로 말하는 사람들이 잘못된거니까 너무 신경쓰지 않으셨으면 합니다.
- 밝은
- 2021/01/05 AM 12:08
- 힝~_~
- 2021/01/04 PM 11:48
- 우와으응
- 2021/01/05 AM 12:06
이제 3년차인데 항상 혼자 작업하다보니 뭐가 문젠지 뭘 발전시켜야 하는지... 가끔 귀찮으면 대충 코드 날림으로 쓰게도 되고.. 서글픕니다요 ㅜㅜ
- 밝은
- 2021/01/05 AM 12:08
- 한랭건조
- 2021/01/05 AM 12:10
하나는 저 양반이 원래 까칠한 성격이구
하나는 처음에는 저 양반도 안그랬다가 경력으로 들어온 양반이 자꾸 되도 않는 코드 작성하고 있으니까
짜증나서 그랬을 수도 있구요.
실제 상황이 어떤건지는 모르겠네요.
- 밝은
- 2021/01/05 AM 12:11
- 한랭건조
- 2021/01/05 AM 12:15
저 양반이 좀... 2년밖에 안됐는데.
- 서래마을이반장
- 2021/01/05 AM 01:17
- 마시멜로☆
- 2021/01/05 AM 04:36
user error : Error. B.