프로그래밍

실무에서 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
무분별한 사용은 차단될 수 있습니다.
번호 제목 글쓴이 추천 수 날짜 조회 수
180376 [컴퓨터] 컴퓨터가 많이 아픈가 본데... 한번만 봐줄래,,?? 꺄꺄룽 0 6 분 전 11
180375 [프로그래밍] 개발자보단 엔지니어가 취업이 잘됨 iillillill 0 49 분 전 77
180374 [모바일] 갤럭시에서 에어팟쓸때 3 뽀3뽀 0 2 시간 전 86
180373 [잡담] 이거 뭐라고 적혀있는거야 6 꺄꺄룽 0 4 시간 전 110
180372 [프로그래밍] 프론트엔드 개발자 연봉 1억 넘는 사람 있어? 8 잠적자 0 4 시간 전 255
180371 [모바일] 배터리 효율 떨어지면 휴대폰도 느려지나? 4 상한가 0 6 시간 전 151
180370 [잡담] 개꿀 당근에서 만원짜리 컴 득템 7 플게이어 2 14 시간 전 500
180369 [컴퓨터] 중고로 키크론 k3 pro? 라는 키보드를 구해왔는데 10 3th3 0 14 시간 전 240
180368 [견적] 이거 에센코어 KLEVV DDR4 PC4-25600 CL22 매우 싸게 나왔는... 11 부터시작하는이세... 0 19 시간 전 192
180367 [잡담] 모니터암 설치하는데 미스테리 4 사나이테스트 0 20 시간 전 326
180366 [컴퓨터] 음향 관련 잘아는 형들 질문... 4 Marlin 0 20 시간 전 139
180365 [잡담] 애드가드 안드로이드 kt-dpi 우회 업데이트 됨 1 어무니 0 21 시간 전 167
180364 [컴퓨터] m2 ssd를 메인보드 기준으로 슬롯 1번에 있는거 2번으로 옮기... 3 오뜨 0 21 시간 전 128
180363 [잡담] 유튜브앱 구간 스킵 방식이 바꼈네?? 6 츄이로 0 22 시간 전 203
180362 [컴퓨터] 이륙 허가좀 18 사촌간부랄빨기 0 22 시간 전 173
180361 [컴퓨터] 완전 컴알못인데 이 레노버 노트북 사도 될까? 15 Fiesta 0 23 시간 전 191
180360 [잡담] 천천히 충전하는 거 폰 배터리에 더 안 좋음? 6 개돼지무한리필 0 23 시간 전 163
180359 [잡담] 메타ai 공개 3 년차html개발자 1 23 시간 전 156
180358 [컴퓨터] p41을 서브 게임저장용은 낭비인가? 8 신우철 0 1 일 전 173
180357 [정보] 해당 모니터암이 여기 모니터2개에 맞을지 모르겠습니다 2 골든베릴 0 1 일 전 129