프로그래밍

실무에서 PR 만들면 보통 몇 명이 코드 리뷰하냐?

씨발 진짜

ㅈㄱㄴ

 

11개의 댓글

2019.05.21

케바케아님?

0
@잉텔

지금 인턴으로 들어온 회사는 한 팀당 2명 QA, 3명 프론트엔드, 4명 백엔드인데, 백엔드는 하청업체에서 일하는 사람도 있음.

아무튼 백엔드로 일하고 있는데, 아니 8명이나 코드리뷰하니까, 말이 왔다갔다하는데, 너무 짜증난다

0
2019.05.21

와 나도 그런환경에서 성장하고싶다 코드 피드백 받으면서

0
@잠적자

말이 왔다갔다하면 개짜증남

0
2019.05.21
@일째프로그래밍

그런거 리뷰할때 어떻게 진행됨? 그냥 코드 도메인언어에 맞춰서 객체간의 응집도랑 결합도 평가함? 아니면 단순 코딩컨벤션리뷰야?

0
@잠적자

우리 회사는 Bitbucket쓰는데, 아무튼 PR만들면 리뷰어들이 이름을 넣음. 그러면 그 리뷰어들이 내 코드를 읽는데 아무튼

작게는

- 뭐 메쏘드이름이나 클래스이름이 A보다는 B가 적합하다.

- 메쏘드 C 안에 있는 이 부분은 다른 메쏘드로 뺸 다음 불러라.

- 이 알고리즘 너무 복잡하다 더 간단하게 바꿔바라.

 

크게는

- 이 클래스 너무 무겁다. structure를 바꿔라

 

 

씨바 온 지 3주째인데, 씨바씨바.

0
2019.05.22
@일째프로그래밍

우리도 비슷하게 하는대 같은팀 1-2명만 리뷰어로검

0
2019.05.21

그런거 받아보고싶다. 근데 막연하게 복잡하다 하면 그것도 답없을듯 내부구현이랑 로직이 맞고 최적화가 잘되어있는데 별다른 근거없이 그러면 짜증날듯.

그런거 잘되어있으면 호출부만 신경써도 될텐데

0
2019.05.22

보는사람이 너무 많은거 아니냐

0
@GTX1080Ti

씨바 내 말이

0
2019.05.22

보통 한두명 아닌가

그리고 리뷰내용이 납득안되면 찾아가서 토론해보는게 좋당

0
무분별한 사용은 차단될 수 있습니다.
번호 제목 글쓴이 추천 수 날짜 조회 수
180592 [컴퓨터] 이거 사운드카드 죽은거냐? 1 와신상담 0 1 시간 전 32
180591 [컴퓨터] m.2케이스에 넣어서 sd카드 대신에 사용해도 괜찮아?? 3 코싸멘뚜 0 1 시간 전 47
180590 [잡담] 플스5 아무거나 사면됨?? 4 년째재수강 0 1 시간 전 42
180589 [모바일] 24 울트라 5G 512기가는 재고가 아예없네 마법부오러사무국장 0 2 시간 전 89
180588 [컴퓨터] 아니 바탕화면 창에 ㅈ 같은 이거 도대체 뭐라고 해요? 7 사슴만지게해줘 0 2 시간 전 130
180587 [모바일] 갤럭시 처음 써보는데 문자알림 내용표시 어떻게 해??? 17 bulusuky 1 3 시간 전 61
180586 [프로그래밍] 엥 구글 플러터 유기각 재는거임?? 4 최수연 0 3 시간 전 108
180585 [모바일] 모바일 크롬 탭창 표정 바뀜 ㅎㄷㄷ 1 살에눈나와어른의... 1 4 시간 전 112
180584 [모바일] 폰 충전기 발열이 넘 심한데 바꿔야 하나 2 우히히힣 0 6 시간 전 95
180583 [컴퓨터] i9 13900k 쓰고있는데 질문! 3 래더넉 0 10 시간 전 136
180582 [컴퓨터] 키보드 -> 방향키가 눌릴때 삐걱거림 3 뮤잉운동케겔운동... 0 12 시간 전 62
180581 [모바일] 기기변경할때 유심을 같은거쓰면 정보 그대로 보존됨? 2 마법부오러사무국장 0 12 시간 전 91
180580 [잡담] 결국 레이니 75 구매함 5 이제는끝내자 0 13 시간 전 189
180579 [컴퓨터] 이륙 허가를 요청한다! 15 와플맛 0 14 시간 전 134
180578 [컴퓨터] 찐따 컴하하 컴퓨터 구매 완료 2 69746974 0 14 시간 전 140
180577 [모바일] a34주문했는데 품절이네 ㅅㅂ 울그락푸르락 0 16 시간 전 127
180576 [컴퓨터] 보통 모니터 스펙이 높을수록 발열이 심해짐? 7 오늘하루도빚갚으리오 0 17 시간 전 147
180575 [컴퓨터] 스마트폰용 SD카드 샀는데 이거 잘 인식 되려나? 5 마두라지 0 17 시간 전 112
180574 [모바일] 갤탭s10 존버중이였는데 그냥 s9 살까 6 한림예고 0 18 시간 전 149
180573 [모바일] 오 플립4 one ui 6.1 업뎃 나왔다 대나무표효자손 0 18 시간 전 128