Informal-Review

데스크 체크(Desk Check) 데스크 체크는 코드를 작성한 개발자가 자신의 “책상에서” 수행하는 자체 검토 활동이다. 이는 마치 작가가 원고의 초안을 검토하는 것과 유사하다. 개발자는 자신이 작성한 코드를 한 줄씩 꼼꼼히 읽어가며 논리적 오류나 잠재적 문제를 찾아낸다. 데스크 체크의 실제 적용 예시: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 // 데스크 체크 과정의 예시 public class PaymentProcessor { public boolean processPayment(double amount, String cardNumber) { // 데스크 체크 포인트 1: 입력값 검증 // - amount가 음수인 경우는 없는지? // - cardNumber가 null이거나 빈 문자열은 아닌지? if (amount <= 0 || cardNumber == null || cardNumber.isEmpty()) { return false; } // 데스크 체크 포인트 2: 카드 번호 형식 검증 // - 숫자로만 구성되어 있는지? // - 길이가 올바른지? if (!validateCardNumber(cardNumber)) { return false; } // 데스크 체크 포인트 3: 결제 처리 로직 // - 예외 처리가 적절한지? // - 트랜잭션 처리가 정확한지? try { return executePayment(amount, cardNumber); } catch (PaymentException e) { logError("Payment failed", e); return false; } } } 데스크 체크 수행 방법 체계적 검토 프로세스 개발자는 다음과 같은 순서로 코드를 검토한다: 코드 구조 검토 논리적 흐름 확인 예외 상황 고려 성능 관련 검토 코드 스타일 확인 체크리스트 활용 효과적인 데스크 체크를 위한 체크리스트 예시: 기본적인 검증 사항들 null 참조 가능성 검사 경계 조건 검사 리소스 관리 확인 보안 관련 검토 문서화 적절성 확인 데스크 체크의 장점과 효과 즉각적인 문제 발견 개발자가 코드를 작성한 직후에 검토함으로써 문제를 빠르게 발견할 수 있다: ...

워크스루(Walkthrough) 워크스루는 마치 박물관 가이드가 관람객들을 안내하듯이, 코드 작성자가 리뷰어들을 코드를 통해 “안내"하는 과정이다. 이 과정에서 코드의 의도, 구현 방식, 그리고 잠재적인 문제점들을 함께 발견하고 논의할 수 있다. 워크스루 세션의 실제 예시: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 // 워크스루 세션 중 코드 설명 예시 public class OrderProcessor { // 발표자: "주문 처리 시스템의 핵심 클래스입니다. // 주문의 유효성 검사부터 결제, 배송 처리까지 담당합니다." private final PaymentService paymentService; private final InventoryService inventoryService; private final ShippingService shippingService; public OrderResult processOrder(Order order) { // 발표자: "먼저 주문의 유효성을 검사합니다. // 여기서 중요한 것은 재고 확인입니다." if (!validateOrder(order)) { throw new InvalidOrderException("Invalid order"); } // 발표자: "재고가 확인되면 결제를 진행합니다. // 결제는 트랜잭션으로 처리됩니다." PaymentResult payment = paymentService.processPayment(order); // 리뷰어 질문: "결제 실패 시 재고는 어떻게 처리되나요?" // 발표자: "좋은 지적입니다. 결제 실패 시 재고를 원복하는 // 로직을 추가해야 할 것 같네요." if (payment.isSuccessful()) { // 발표자: "결제가 성공하면 배송 처리를 시작합니다." return createShippingOrder(order, payment); } return OrderResult.failure("Payment failed"); } } 워크스루의 주요 특징과 장점 상호작용적 학습 참가자들은 실시간으로 질문하고 토론할 수 있다: ...

코드 리뷰 (Code Review) 코드 리뷰는 개발자가 작성한 코드를 다른 개발자들이 검토하고 피드백을 제공하는 과정이다. 이는 마치 작가들이 서로의 글을 읽고 조언을 주고받는 것과 유사하다. 주된 목적은 코드의 품질을 향상시키고 팀 내의 지식 공유를 촉진하는 것이다. 코드 리뷰의 실제 적용 예시: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 // 리뷰가 필요한 코드 예시 public class UserService { public void createUser(String username, String password) { // 데이터베이스에 직접 저장 database.execute( "INSERT INTO users (username, password) VALUES ('" + username + "', '" + password + "')" ); } } // 리뷰어의 피드백 후 개선된 코드 public class UserService { /** * 새로운 사용자를 생성하고 저장합니다. * @param username 사용자 이름 * @param password 비밀번호 * @throws ValidationException 유효하지 않은 입력값 */ public void createUser(String username, String password) { // 입력값 검증 validateInput(username, password); // 비밀번호 해싱 String hashedPassword = passwordHasher.hash(password); // 준비된 구문을 사용하여 SQL 인젝션 방지 userRepository.save(new User(username, hashedPassword)); } } 주요 특징 품질 보증: 코드 리뷰는 버그를 찾아내고, 코드 품질을 향상시키며, 보안 취약점을 발견하는 데 도움을 준다. 여러 눈이 검토하므로 한 사람이 놓칠 수 있는 문제점들을 더 쉽게 발견할 수 있다. 지식 공유: 팀 멤버들이 서로의 코드를 검토하면서 자연스럽게 지식을 공유하고 학습할 수 있다. 주니어 개발자는 시니어의 피드백을 통해 성장할 수 있고, 시니어 개발자도 새로운 관점을 얻을 수 있다. 일관성 유지: 팀의 코딩 표준과 모범 사례를 준수하는지 확인함으로써, 전체 코드베이스의 일관성을 유지할 수 있다. 코드 리뷰 프로세스의 주요 단계 리뷰 준비 개발자는 리뷰를 위해 코드를 준비할 때 다음 사항들을 고려한다: ...

패스 어라운드(Pass Around) 패스 어라운드는 마치 책을 여러 사람이 돌려가며 읽는 것처럼, 코드를 여러 개발자들이 순차적으로 검토하는 방식이다. 각 리뷰어는 자신의 전문 분야나 관점에서 코드를 검토하고 피드백을 제공한다. 예를 들어, 한 개발자는 성능 관점에서, 다른 개발자는 보안 관점에서 같은 코드를 검토할 수 있다. 실제 패스 어라운드 프로세스의 예시: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 // 첫 번째 리뷰어 (성능 전문가)의 검토 public class DataProcessor { public List<Result> processData(List<Data> dataList) { // 성능 관련 코멘트: // "대용량 데이터 처리 시 메모리 문제가 발생할 수 있습니다. // 스트림을 사용하여 처리하는 것이 좋겠습니다." return dataList.stream() .filter(Data::isValid) .map(this::transform) .collect(Collectors.toList()); } } // 두 번째 리뷰어 (보안 전문가)의 검토 후 수정된 버전 public class DataProcessor { public List<Result> processData(List<Data> dataList) { // 보안 관련 코멘트: // "입력 데이터의 유효성 검증이 필요합니다. // 또한 처리 과정에서의 로깅이 필요합니다." if (dataList == null) { throw new IllegalArgumentException("Data list cannot be null"); } logger.info("Starting data processing for {} items", dataList.size()); return dataList.stream() .filter(this::validateData) .map(this::transform) .collect(Collectors.toList()); } } // 세 번째 리뷰어 (테스트 전문가)의 검토 후 추가된 테스트 코드 @Test public class DataProcessorTest { // 테스트 관련 코멘트: // "경계 조건과 예외 상황에 대한 테스트가 필요합니다." @Test void testProcessDataWithNullInput() { assertThrows(IllegalArgumentException.class, () -> processor.processData(null)); } @Test void testProcessDataWithEmptyList() { assertTrue(processor.processData(Collections.emptyList()).isEmpty()); } } 프로세스 코드 작성자가 리뷰 대상 코드를 공유 참여자들이 개별적으로 코드 검토 각자의 의견을 메일이나 시스템에 기록 코드 작성자가 피드백을 수집하고 필요한 수정 진행 패스 어라운드의 장점과 효과 다양한 관점에서의 검토 여러 전문가의 시각으로 코드를 검토할 수 있다: ...

페어 프로그래밍(Pair Programming) 페어 프로그래밍에서는 두 명의 개발자가 서로 다른 역할을 맡는다. ‘드라이버(Driver)‘는 실제로 코드를 작성하는 사람이고, ‘네비게이터(Navigator)‘는 코드를 검토하고 방향을 제시하는 사람이다. 이 두 역할은 주기적으로 교대한다. 실제 페어 프로그래밍 예시: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 // 페어 프로그래밍 세션 예시 // Driver가 코드를 작성하고, Navigator가 실시간으로 리뷰와 제안을 합니다 // Navigator: "사용자 등록 기능을 구현해볼까요? 먼저 입력값 검증부터 시작하는게 좋겠어요." // Driver: "네, 동의합니다. 사용자 이름과 이메일 유효성을 체크하는 메서드부터 작성할게요." public class UserRegistration { public boolean registerUser(String username, String email) { // Navigator: "null 체크도 필요할 것 같네요." // Driver: "네, 좋은 지적입니다. 추가하겠습니다." if (username == null || email == null) { return false; } // Navigator: "이메일 형식 검증도 필요할 것 같아요." // Driver: "이메일 정규식을 사용해서 검증하면 좋겠네요." if (!validateEmailFormat(email)) { return false; } // Navigator: "사용자 이름 길이 제한도 있어야 할 것 같아요." // Driver: "네, 최소 3자, 최대 20자로 제한하겠습니다." if (username.length() < 3 || username.length() > 20) { return false; } // 실제 등록 로직 구현… return saveUser(username, email); } } 페어 프로그래밍의 주요 이점 실시간 코드 리뷰 두 명이 함께 작업하면서 즉각적인 피드백이 가능하다: ...