밝은 MYPI

밝은
접속 : 3633   Lv. 46

Category

Profile

Counter

  • 오늘 : 4 명
  • 전체 : 83249 명
  • Mypi Ver. 0.3.1 β
[---] IT기업 코드리뷰가 원래 이런건가요? (16) 2021/01/04 PM 11:19


그동안 SM만 하다가 코드리뷰가 있는 회사에 처음 왔습니다. 


 지금까지 4~5번 진행했는데


 인수인계도 받지 않은 상태에서 소스코드만 보고 익힌 업무 지식 가지고


구현한 프로그램을 리뷰 받았습니다. 특정 페이지의 사용자 로그를 남기는 프로그램 이었는데요...


하나의 Model을 가지고 각 업무적 행위가 행해지는곳에 코드를 삽입하고 이력을 남기도록 했는데요...


한숨을 푹푹 쉬면서 왜 곳곳에 코드를 이렇게 많이 써놨냐,


6개월이 지났는데 기본적인 로직도 모르느냐 ....매번 1시간 넘게 이러니까


사람들 앞에서 공개망신을 당하는것 같습니다. 지칩니다...


저희 회사 언어는 php를 쓰는데요.


이력정보를 팝업으로 보여주기 위해 백엔드에서 코드값을 이름으로 바꿔주기 위해


함수 array_walk를 이용해 익명함수로 배열 요소를 돌면서


코드->이름으로 바꿔주는 작업을 하고 화면단에 던져줘서 display되게 했는데


왜 화면단에서 한번 Loop 돌릴걸 백엔드에서 한번 돌고 던지느냐 비난조로 지적하고...


맞는말인건 알겠는데 비난하는 어조로 계속 들으니까 점점 일하기가 싫고


코드리뷰가 원래 여러사람 앞에서 공개 비난하는것인가 라는 생각이 듭니다.


  답답해서 하소연해봤습니다. ㅠ






신고

 

JuliaHart    친구신청

뉘앙스는 다르겠지만 원래 공개적으로 그런거 맞구요.
갈굼당할수록 실력이 느니까 좋게 생각하세요.
그리고 내꺼 보여주기만 하지 말고
다른사람들 커밋들도 까뒤집어까면서 리뷰 하세요.
당장 쪽팔릴수는 있어도 대충 뭉개고 주먹구구식으로 넘어가는 회사에서보다 몇배는 체계적으로 배울 수 있습니다
개인적인 생각으로 욕먹더라도 내 코드에 관심주고 지적해주는 사람은 스승으로 모셔야한다고 봅니다.

밝은    친구신청

ㅠㅠ무슨말하시는지 머리론 이해해도 마음이 그렇지가 않네여

E-마토    친구신청

어디든지 케바케이긴 하지만,
글쓴이님이 크게 흉이 될만한 사건이 없었다는 전제하에
비난조로 한다는건 별로 좋은 회사는 아닌거같군요.

밝은    친구신청

제가 완젼 문제아가 된 기분이에요....

철철새    친구신청

아 다르고 어 다른건데 꼭 밉게 말하는사람들이 있죠

으훗    친구신청

개인적인 취향은 아니지만
막상 부드럽게 이야기하면 1년넘게 조인하나 못거는 개발자도 종종있습니다.
뒤돌면 다 까먹는

아르테니    친구신청

코드리뷰가 비슷비슷하죠..
비난조로 말하는 사람들이 잘못된거니까 너무 신경쓰지 않으셨으면 합니다.

밝은    친구신청

멘탈이 약해서 힘드네여 ㅠ

힝~_~    친구신청

"원래" 그런건 아니예요

우와으응    친구신청

이런말하면 참 죄송한데.. 전 진짜 옆에서 누가 욕해도 리뷰 좀 해줄 사람이 있으면 좋겠습니다 ㅜㅜ
이제 3년차인데 항상 혼자 작업하다보니 뭐가 문젠지 뭘 발전시켜야 하는지... 가끔 귀찮으면 대충 코드 날림으로 쓰게도 되고.. 서글픕니다요 ㅜㅜ

밝은    친구신청

무슨말하시는지 공감합니다.ㅠㅠ 저도 그런거 없다가 처음와봐서..

한랭건조    친구신청

음... 2가지 경우가 있을 것 같은데

하나는 저 양반이 원래 까칠한 성격이구

하나는 처음에는 저 양반도 안그랬다가 경력으로 들어온 양반이 자꾸 되도 않는 코드 작성하고 있으니까

짜증나서 그랬을 수도 있구요.

실제 상황이 어떤건지는 모르겠네요.

밝은    친구신청

경력단절이 있어서 작년에 재취업한 후 실질경력은 2년이 좀 넘습니다. ㅠ

한랭건조    친구신청

2년 정도면 저정도 그냥 좋게 말할것 같은데

저 양반이 좀... 2년밖에 안됐는데.

서래마을이반장    친구신청

하루이틀이면 좋게 말하겠는데 반년 일년 넘어가도록 코드 리뷰 때마다 동일한게 보이면 바라보는 입장에서 인내심 한계가 오긴해요 ㅎㅎ

마시멜로☆    친구신청

회사는 정글이죠
X