Conversation
객체 스스로 비교연산을 진행할 수 있도록 로직을 추가함.
…서 한번에 세개의 정보(matchNumbers,prizeMoney,등수별 당첨횟수)를 저장하여 계산함.
m-a-king
left a comment
m-a-king
left a comment
There was a problem hiding this comment.
페어프로그래밍을 이번에 처음하다보니 많은 시간이 소요되고 그 시간들이 정말 소중한 시간이였던 것 같습니다. 미션 해결을 하며 파트너에게 감탄하고 협업 능력을 기르고 이 미션에 대한 프로그래밍 기여도를 위해 승부욕이 좀 불타오르는 미션이였던 것 같습니다. 어떤 사람에게 코드에 대한 제 생각을 공유하는 것도 연습할 수 있었고 파트너가 제안한 코드 로직과 형식을 이해하는 시간도 너무 좋았습니다. 힘든 만큼 얻어간건 확실히 많았던 것 같습니다.
너무 부럽습니다. 저도 페어프로그래밍 참 좋아해요~ 제 리뷰도 페어와 함께 공유하고, 혼자 해결하기에 어려운 문제가 있다면 함께 해결하셔도 좋습니다!
또한 원시값을 포장하는 요구사항과 일급컬렉션을 사용하라는 요구사항이 발목을 많이 잡았습니다.
하지만 파트너와 최선을 다해 요구사항을 충족시키며 코드를 만들고자 하였습니다.
저도 요구사항에 자주 반항했던 기억이 나네요 ㅎㅎ
이해할 수 없는 요구사항이 주어진다면 그 이유를 적어주시고, 지키지 않으셔도 좋습니다~
그럼 제가 지키게 해드릴게요! 😄
완성하지 못한 부분
메서드 분리가 필요한 메서드가 많은 것 같습니다.
어떤 부분인가요?
저희가 짠 코드가 모든 원시값을 포장했다고 볼 수 있는지 궁금합니다.
네! 잘 지켜주셨습니다. 그치만 원시값 포장이라는 목표에 진짜 의미가 무엇인지 돌아보면 개선할 수 있는 부분이 많은 것 같아요~
일급 컬렉션을 잘 만들고 활용했는지 궁금합니다.
아니요... 이 부분은 상당히 아쉬웠습니다. 어떻게 일급 컬렉션을 잘 활용할 수 있을지 조금 더 학습이 필요하다고 생각합니다. 제가 진짜 잘 알려드릴게요!
하나의 메서드는 하나의 기능을 가져야하는데 책임을 너무 과하게 많이 가진 메서드가 보이시는 지 궁금합니다.
전체적으로 메서드는 잘게 나눠주신 것 같습니다. 그 과정에서 각 메서드들의 위치가 올바른지에 대해서 심도있는 토론을 해보고 싶어요~
이번에는 파트너와 함께 작성했지만 자바에 아직 미숙한 점이 많고 배우는 과정이다보니 부족한 점이 많을 수 있습니다. 잘못된 점이 있다면 말씀주시고 궁금한 부분에 대한 멋진 답변 부탁드립니다! 리뷰어님!
멋집니다 희창님 화이팅!!
| import java.util.List; | ||
|
|
||
| public class Lotto { | ||
| public static final int LOTTO_SIZE = 6; |
There was a problem hiding this comment.
lotto size 라는 상수 명을 어떻게 하면 더 직관적으로 개선할 수 있을까요?
| private final List<LottoNumber> numbers; | ||
|
|
||
| public Lotto(List<LottoNumber> numbers) { | ||
| validatorLotto(numbers); |
There was a problem hiding this comment.
validator은 자바 메서드 네이밍 컨벤션에 맞지 않는 것 같아요~
|
|
||
| public class Lotto { | ||
| public static final int LOTTO_SIZE = 6; | ||
| private final List<LottoNumber> numbers; |
There was a problem hiding this comment.
List<LottoNumber>은 LottoNumbers로 묶지 않은 이유가 있나요?
| public List<LottoNumber> getLotto() { | ||
| return numbers; | ||
| } |
| public List<LottoNumber> getLotto() { | ||
| return numbers; | ||
| } |
There was a problem hiding this comment.
외부에서 lotto.getLotto()를 호출하고, 로또 넘버를 바꿀 수 있는 상황에 대해서 어떻게 생각하시나요?
|
|
||
| ## 새로운 프로그래밍 요구사항 | ||
|
|
||
| - [ ] 모든 원시 값과 문자열을 포장한다. |
| } | ||
|
|
||
| public void valueAdd(Rank rank) { | ||
| result.put(rank, result.get(rank) + 1); |
| import java.util.LinkedHashMap; | ||
| import java.util.Map; | ||
|
|
||
| public class LottoCalculator { |
| } | ||
|
|
||
| @Override | ||
|
|
|
|
||
| import java.util.List; | ||
|
|
||
| public class Lottos { |
There was a problem hiding this comment.
Lottos 클래스가 더 많은 일을 할 수 있을 것 같아요! 30줄 이상으로 만들어보는 미션을 드리고 싶어요~
2 participants
페어프로그래밍을 이번에 처음하다보니 많은 시간이 소요되고 그 시간들이 정말 소중한 시간이였던 것 같습니다. 미션 해결을 하며 파트너에게 감탄하고 협업 능력을 기르고 이 미션에 대한 프로그래밍 기여도를 위해 승부욕이 좀 불타오르는 미션이였던 것 같습니다. 어떤 사람에게 코드에 대한 제 생각을 공유하는 것도 연습할 수 있었고 파트너가 제안한 코드 로직과 형식을 이해하는 시간도 너무 좋았습니다. 힘든 만큼 얻어간건 확실히 많았던 것 같습니다.
또한 원시값을 포장하는 요구사항과 일급컬렉션을 사용하라는 요구사항이 발목을 많이 잡았습니다.
하지만 파트너와 최선을 다해 요구사항을 충족시키며 코드를 만들고자 하였습니다.
완성하지 못한 부분
메서드 분리가 필요한 메서드가 많은 것 같습니다.
궁금한 부분
저희가 짠 코드가 모든 원시값을 포장했다고 볼 수 있는지 궁금합니다.
일급 컬렉션을 잘 만들고 활용했는지 궁금합니다.
하나의 메서드는 하나의 기능을 가져야하는데 책임을 너무 과하게 많이 가진 메서드가 보이시는 지 궁금합니다.
이번에는 파트너와 함께 작성했지만 자바에 아직 미숙한 점이 많고 배우는 과정이다보니 부족한 점이 많을 수 있습니다. 잘못된 점이 있다면 말씀주시고 궁금한 부분에 대한 멋진 답변 부탁드립니다! 리뷰어님!