[유양우] 연료주입, 블랙잭 - Step1

[YuYangWoo]

안녕하세요 리뷰어님 !
연료주입과 블랙잭 미션 구현하였습니다.
리뷰 잘 부탁드립니다!
감사합니다

[joswlv]

안녕하세요 양우님!
몇 가지 코멘트를 남겼습니다.
확인 부탁드립니다.

[joswlv]

Suggested change

	@Override
	public boolean equals(Object o) {
	if (this == o) {
	return true;
	}
	if (o == null || getClass() != o.getClass()) {
	return false;
	}
	Card card = (Card) o;
	return denomination == card.denomination && suit == card.suit;
	}
	@Override
	public int hashCode() {
	return Objects.hash(denomination, suit);
	}
	public Denomination getDenomination() {
	return denomination;
	}
	public Suit getSuit() {
	return suit;
	}
	public Denomination getDenomination() {
	return denomination;
	}
	public Suit getSuit() {
	return suit;
	}
	@Override
	public boolean equals(Object o) {
	if (this == o) {
	return true;
	}
	if (o == null || getClass() != o.getClass()) {
	return false;
	}
	Card card = (Card) o;
	return denomination == card.denomination && suit == card.suit;
	}
	@Override
	public int hashCode() {
	return Objects.hash(denomination, suit);
	}

코드 컨벤션을 지켜주세요~

[joswlv]

package dir name이 Card로 되어 있습니다. 소문자로 생성 부탁드립니다.

[joswlv]

Suggested change

	super("딜러");
	super(DEALER_NAME);

상수로 변경해보는 것은 어떨까요?

[joswlv]

카드는 고정적인 값이라 매번 생성되는 것보다 static 하게 만들어 보는 것은 어떨까요?

[joswlv]

Suggested change

	private void getCardOrNot(Gamer player) {
	private void drawTo(Gamer player) {

의미를 명확하게 만들어보는 것은 어떨까요?

[joswlv]

Suggested change

	if (dealer.getCardOrNot(dealer)) {
	OutputView.printDealerAddCard();
	}
	drawToDealer()

로직을 분리해보는 것은 어떨까요?

[joswlv]

만약 게임이 계속 진행되어 Cads가 생성되어야 하는 경우 에러가 발생할 것으로 예상됩니다.
카드 뭉치를 정적으로 만드는 것보다 카드를 정적으로 만들어보는 것은 어떨까요?