- 볼링 게임 점수판 요구사항을 파악한다.
- 요구사항에 대한 구현을 완료한 후 자신의 github 아이디에 해당하는 브랜치에 Pull Request(이하 PR)를 통해 코드 리뷰 요청을 한다.
- 코드 리뷰 피드백에 대한 개선 작업을 하고 다시 PUSH한다.
- 모든 피드백을 완료하면 다음 단계를 도전하고 앞의 과정을 반복한다.
- 질문 데이터를 완전히 삭제하는 것이 아니라 데이터의 상태를 삭제 상태(deleted - boolean type)로 변경한다.
- 로그인 사용자와 질문한 사람이 같은 경우 삭제 가능하다.
- 답변이 없는 경우 삭제가 가능하다.
- 질문자와 답변글의 모든 답변자 같은 경우 삭제가 가능하다.
- 질문을 삭제할 때 답변 또한 삭제해야 하며, 답변의 삭제 또한 삭제 상태(deleted)를 변경한다.
- 질문자와 답변자가 다른 경우 답변을 삭제 할 수 없다.
- 질문과 답변 삭제 이력에 대한 정보를 DeleteHistory를 활용해 남긴다.
- qna.service.QnaService의 deleteQuestion()는 앞의 질문 삭제 기능을 구현한 코드이다. 이 메소드는 단위 테스트하기 어려운 코드와 단위 테스트 가능한 코드가 섞여 있다.
- 단위 테스트하기 어려운 코드와 단위 테스트 가능한 코드를 분리해 단위 테스트 가능한 코드 에 대해 단위 테스트를 구현한다.
- 객체의 상태 데이터를 꺼내지(get)말고 메시지를 보낸다.
- 규칙 8: 일급 콜렉션을 쓴다.
- Question의 List를 일급 콜렉션으로 구현해 본다.
- 규칙 7: 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다.
- 인스턴스 변수의 수를 줄이기 위해 도전한다.
- 테스트하기 쉬운 부분과 테스트하기 어려운 부분을 분리해 테스트 가능한 부분만 단위테스트한다.
- Q2는 사용하지 않는것으로 보입니다. MethodSource를 사용해서 테스트 코드를 작성해보세요 :)
- deleted()는 setter로 보이네요 😯 deleted()에서 DeleteHistory 자체를 리턴해주면 어떨까요? 🤔
- Answers의 테스트 코드도 작성해보면 좋을 것 같아요 ~🙂
- delete()의 반환 값 (DeleteHistories)에 대한 검사도 해주면 좋을 것 같습니다 :)
- 반환된 DeleteHisotries에 특정 값이 포함되었는지 확인하고 싶었는데 외부로 노출하는건 iterator 뿐이고, 테스트만을 위해서 getter를 추가하는게 탐탁치 않아 iterator를 list로 변환후에 테스트를 진행했습니다. 이 부분은 어떤거 같나요?
- 최종 목표는 볼링 점수를 계산하는 프로그램을 구현한다. 1단계 목표는 점수 계산을 제외한 볼링 게임 점수판을 구현하는 것이다.
- 각 프레임이 스트라이크이면 "X", 스페어이면 "9 | /", 미스이면 "8 | 1", 과 같이 출력하도록 구현한다.
- 스트라이크(strike) : 프레임의 첫번째 투구에서 모든 핀(10개)을 쓰러트린 상태
- 스페어(spare) : 프레임의 두번재 투구에서 모든 핀(10개)을 쓰러트린 상태
- 미스(miss) : 프레임의 두번재 투구에서도 모든 핀이 쓰러지지 않은 상태
- 거터(gutter) : 핀을 하나도 쓰러트리지 못한 상태. 거터는 "-"로 표시
- 10 프레임은 스트라이크이거나 스페어이면 한 번을 더 투구할 수 있다.
- 인스턴스 변수를 줄여보세요 🙂 (Frames)
- interface대신 abstract class 로 구현해보면 어떨까요? 🤔
- 반드시 필요한 필드인지 고민해보세요 ! 👀 사용처가 2곳인데 (아래 헤더 출력, 진행 중인 프레임 번호) 다른 방식으로 지원해주면 어떨까요? :) (index)
- 규칙 3: 모든 원시값과 문자열을 포장한다.
- 예를 들면, 점수를 totalScore로 가지고 있지말고, 점수를 구하는 메소드를 지원해주는 방법도 있겠네요 🙂
- 위의 필드를 가지고있고, getPreviousTotalScore(), getPinchings()같은 중복되는 코드가 보이네요. 이 두 필드와 공통 메소드는 Frame이 가지고 있으면 어떨까요?
- 사용하지 않는 값으로 보입니다 ! 👀 (ScoreCalculateTest.java 169 line)
- 처음에 모든 프레임을 초기화시키는 것으로 보이는데요. 이렇게 초기화해주신 의도가 있을까요 ? 👀 볼링을 진행하면서 프레임을 생성해주는 방법은 어떨까요?
- 처음, 마지막, 현재 진행중인 프레임을 포인터처럼 가지고 있군요 🙂 자료구조를 활용해보는 방법도 있을 것 같아요 !
- 1명 이상의 사용자가 사용할 수 있는 볼링게임 점수판을 구현한다.
- 정적 팩토리 메소드에서 List를 만들어 private 생성자에 전달해주면 어떨까요? 그러면 검증된 Player들 일때만 Players객체를 생성할 수 있을 것 같아요 ! (Players.java)
- FrameDto에서 pitchings가 노출되고 있는데요, 도메인이 그대로 노출되어 위험한 것 같아요. 😢 (addPitching 메소드 등 접근 가능) 출력에 필요한 값만 전달해주는 방법이나, 방어적 복사등을 사용해보면 어떨까요?
- 46-48 line이 필요한 분기인지 고민해주세요 ~ 👀 (Frame.java)