코드 리뷰 실무 - 버그를 잡고 팀을 키우는 읽기

리뷰의 목적 세 가지, 리뷰로 잘 잡히는 것과 안 잡히는 것, 테스트·정적 검사와 역할 나누기, 완성 코드는 결함 목록을 발견 수단(리뷰·테스트·정적 검사)별로 집계해 표로 출력한다

PR 설명→테스트→공개 인터페이스→핵심 로직→경계 조건 순서, 한 번에 다 보지 않기, 큰 diff 대응, 완성 코드는 변경 파일 목록을 읽는 순서대로 정렬하고 우선 확인 항목을 출력한다

반복 경계, 빈 컬렉션, null, 정수 나눗셈과 반올림, 날짜 경계, 완성 코드는 연체료 계산 변경의 전/후를 경계 입력 표로 돌려 달라지는 지점을 찾는다

예외 삼키기, 너무 넓은 catch, 오류 메시지의 정보량, 자원 해제(try-with-resources), 실패 시 상태가 반쯤 바뀌는 문제, 완성 코드는 대출 처리 중 예외가 났을 때 재고 상태가 일관된지 전/후 버전을 비교한다

너무 많은 일을 하는 클래스, 같은 규칙의 중복, 구체 클래스에 묶인 의존, 이름이 말하지 않는 의도, 리뷰에서 지적할 범위와 다음 PR 로 넘길 범위, 완성 코드는 중복된 요금 규칙을 하나로 모은 전/후의 결과가 같은지 확인한다

공유 가변 상태, 확인 후 실행 경쟁, 동기화 범위, 불변 객체, 동시 컬렉션, 완성 코드는 여러 스레드가 같은 책을 대출하는 상황을 고정된 순서로 흉내 내 결함 버전과 고친 버전의 결과를 출력한다

입력 검증, SQL·경로 주입, 권한 확인 위치, 비밀값과 개인정보 로그, 직렬화, 완성 코드는 요청 처리 코드 조각을 체크리스트로 검사한 결과를 표로 출력한다

반복 안의 조회(N+1 형태), 불필요한 객체 생성, 자료구조 선택, 측정으로 확인하는 습관, 완성 코드는 리스트 검색과 맵 검색의 연산 횟수를 세어 입력 크기별로 비교한다

테스트 이름이 말하는 요구, 기대값을 구현에서 베끼지 않기, 경계·실패 사례, 하나의 테스트가 하나를 검증, 깨지기 쉬운 테스트, 완성 코드는 약한 테스트 묶음에 일부러 버그를 넣어 잡히지 않는 것을 보여 주고 보강한다

질문형·제안형 의견, 필수와 선택 구분 표시, 근거(재현 입력·문서) 붙이기, 사람 대신 코드를 말하기, 칭찬, 완성 코드는 리뷰 의견 데이터에서 근거 없는 지적·모호한 지적을 찾아 표시한다

작은 PR, 설명 틀(목적·변경·검증·위험), 스스로 먼저 리뷰, 의견에 답하는 법, 리뷰 반영 커밋 나누기, 완성 코드는 PR 정보로 리뷰 준비 점수를 계산해 빠진 항목을 알려 준다

정적 검사·AI 리뷰 의견을 1차 거름망으로, 사람은 요구·설계·위험 판단, 오탐 다루기, 종합 실습으로 반납 기능 PR 하나를 처음부터 끝까지 리뷰한다. 완성 코드는 자동 검사 결과와 사람 체크리스트를 합쳐 최종 리뷰 보고서를 출력한다