코드리뷰28 [Review] 로또 3단계 - 코드 리뷰 반영 (2022/02/27) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 리뷰는 Dion께서 해주셨다! 1. 일급 컬렉션의 상속 사용? 우선 난 일급 컬렉션에 상속을 사용하면 안된다는 생각은 하였지만, 좀 구체적인 근거가 없는것 같아 팀원과 함께 질문하였다. 저희 팀은 1급 객체로 LottoTicket을 만들었습니다. 여기서 궁금한 점은, WinningNumbers 객체를 만들면서 1급 객체인 LottoTicket을 상속하는 방법이 떠올랐습니다.왜냐하면 기존의 1급 객체 에서의 코드를 재사용 하면 중복을 최소화 할 수 있기 때문이었습니다. 하지만 1급 객체는 독립적인 단위로 사용하는 것이다 보니, 상속을 하여 재사용 한다는 것 자체가 확실하지 않았습니다. 이에 대한 리뷰어님의 의견.. CODE SQUAD/FeedBack 정리 2022. 2. 27. [Review] 로또 3단계 - 수동구매 기능 추가 (2022/02/25) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번주는 호눅스가 리뷰를 해주셨다! 1. 일급 컬렉션의 데이터 출력하기 1급 컬렉션의 데이터는 어디서 출력해야 할까? 이에 대한 의문으로 부터 시작한 질문이였다. 호눅스 께서는 선택의 문제라 알려주셨다. 따라서 우리팀은 1급 객체의 데이터 불변성을 보장하기 위해 필드의 list인 lottoNumbers를 반환하는 getter를 만들지 않기로 하였다. 대신 LottoTicket에게 showLottoNumbers() 메시지를 수신할 수 있도록 만들었다. 해당 Class의 코드는 다음과 같다. public class LottoTicket { private static final int NORMAL_TICKET_SIZE .. CODE SQUAD/FeedBack 정리 2022. 2. 25. [Review] 사다리 구현 5단계 - 실행결과 출력 (2022/02/22) 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 지난번에 이어서, 리뷰는 Dion이 해주셨다!! 감사합니다 Dion!! 1. Scanner의 위치 ? 원래 내가 구현한 코드는 다음과 같았다. public class InputView { public static final String INPUT_PERSON_GUIDANCE_MESSAGE = "참여할 사람 이름을 입력하세요. (이름은 쉼표(,)로 구분하세요)"; public static final String INPUT_LADDER_SIZE_GUIDANCE_MESSAGE = "최대 사다리 높이는 몇 개인가요?"; public static final String INPUT_NAME_LENGTH_ERROR_MESSAG.. CODE SQUAD/FeedBack 정리 2022. 2. 23. [Review] 사다리 구현 4단계 - 리팩토링 2 (2022/02/20) 코드리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 이번 리뷰는 Dion이 해주셨다!! 감사합니다 Dion!! 1. Depth 2 지키기 모든 부분에서 depth2를 지켰다고 생각했는데, 인식하지 못한 부분이 있었다... 기존의 코드는 다음과 같았다. public int[] makeStartPositions() { int totalPositions = ladderSize.getTotalPositions(); int countOfLine = ladderSize.getCountOfLine(DEFAULT_PERCENT); int[] startPositions = new int[countOfLine]; for (int i = 0; i < startPositions.lengt.. CODE SQUAD/FeedBack 정리 2022. 2. 20. 이전 1 2 3 다음
