가위바위보 게임 [STEP 1] jaySong #13
Conversation
| gameView = GameView(delegate: self) | ||
| guard let gameView = gameView else { return } | ||
| view = gameView |
There was a problem hiding this comment.
뷰가 로드되는 단계에서 조기 종료가 발생할 수 있는 구문을 작성하기 보다는 안전하게 처리할 수 있는 구문을 사용해보는 것도 괜찮을 것 같습니다!
let gameView = GameView(delegate: self)
self.gameView = gameView
view = gameViewThere was a problem hiding this comment.
지역변수를 활용해서 옵셔널언래핑 없이 코드구현이 가능해졌네요, 좋은 것 같습니다
| } | ||
|
|
||
| private func play(userCard: Card) { |
There was a problem hiding this comment.
게임 시작과 진행, 종료까지 게임에 대한 로직이 ViewController에서 수행하고 있는 것으로 보입니다. 뷰와는 별개로 게임 수행을 위한 새로운 타입을 만들어 보는 것은 어떨까요? 역할과 책임을 나눠볼 수 있을 것 같습니다!
There was a problem hiding this comment.
예를 들면, 게임 로직을 다루는 'GameController' 타입을 만들고, ViewController를 GameController와 GameView를 연결해주는 역할로 변경하는 식일까요?
There was a problem hiding this comment.
게임 로직을 다루는 GameController 타입을 만들어서 ViewController(GameView)의 책임을 줄일 수 있는 것도 좋은 방법일 것 같습니다!
|
|
||
| private func judgement() { | ||
| guard let userCard = user.selectedCard, let computerCard = computer.selectedCard else { return } | ||
| let cal = userCard.rawValue - computerCard.rawValue |
There was a problem hiding this comment.
cal 이름이 표현하는 의미가 모호한 것 같습니다. 조금 더 구체적인 이름을 사용해볼 수 있을까요?
There was a problem hiding this comment.
음.. 가위바위보 결과를 계산하는 의미로, calculateResult 는 어떨까요
| guard let userCard = user.selectedCard, let computerCard = computer.selectedCard else { return } | ||
| let cal = userCard.rawValue - computerCard.rawValue | ||
|
|
||
| if cal == 0 { |
There was a problem hiding this comment.
코드를 작성하고 이후에 if cal == 0 구문을 보았을 때, 그 의미를 파악하기 어려울 것 같습니다. 동일한 값을 나타낸다는 표현을 사용해보는 것이 어떨까요? 의미를 보다 명확하게 이해할 수 있을 것 같습니다!
아래의 if문도 확인해보면 좋을 것 같습니다.
There was a problem hiding this comment.
0, 1, 2 같은 숫자들을 위한 enum타입을 만들어서 same, win, lose 등으로 case를 나누면 이해가 좀 더 용이할 것 같네요!
| } | ||
| } | ||
|
|
||
| private func showResultAlert(_ isUserFinalWinner: Bool) { |
| static let paper: String = "🖐️" | ||
| static let rock: String = "✊" | ||
| static let scissor: String = "✌️" | ||
| protocol GameViewDelegate: AnyObject { |
| @@ -114,10 +176,11 @@ class GameView: UIView { | |||
| ]) | |||
| } | |||
|
|
|||
| init() { | |||
| init(delegate: GameViewDelegate) { | |||
| computer.select(card: .rock) | ||
|
|
||
| //when | ||
| let cal = user.selectedCard!.rawValue - computer.selectedCard!.rawValue |
There was a problem hiding this comment.
테스트 코드 내에서 비즈니스 로직에 대한 코드가 작성된 것 같습니다.
비즈니스 로직을 담고 있는 구현부 코드를 테스트하는 형태로 변경해보는 것은 어떨까요? 테스트 대상과 어떤 테스트를 진행하는 지 명확하게 파악할 수 있을 것 같습니다!
There was a problem hiding this comment.
구현부 코드를 테스트하는 형태란 예를들면 어떤 식인가요? user와 computer가 카드 선택하는 부분은 제외하고 승패 판정하는 부분만 테스트하도록 하는 것을 말씀하시는 건가요?
There was a problem hiding this comment.
given 단계에서 카드를 선택하는 것은 그대로 작성해도 괜찮을 것 같습니다. when 단계에서 게임의 진행과 결과를 판단하는 로직이 테스트 코드 내에 작성되어 있어서 코멘트를 작성해보았습니다. 어떤 코드를 테스트해보고 싶으셨는지 궁금했습니다!
There was a problem hiding this comment.
결과 판단 코드를 테스트해보고 싶었습니다, 이제보니 메서드를 불러오면 되는것을, 내부 코드를 넣어버렸네요..!
| let cal = user.selectedCard!.rawValue - computer.selectedCard!.rawValue | ||
|
|
||
| if cal == 0 { | ||
| user.draw() |
There was a problem hiding this comment.
묵찌빠로 바꾸면서 draw() 함수가 삭제되었는데, 그 이후에는 단위테스트를 실행하지 않아서 발견을 못했네요..!
| //then | ||
| XCTAssertNil(user.selectedCard) | ||
| XCTAssertEqual(user.winCount, 0) | ||
| XCTAssertEqual(user.drawCount, 0) |
There was a problem hiding this comment.
묵찌빠로 바꾸면서 draw() 함수가 삭제되었는데, 그 이후에는 단위테스트를 실행하지 않아서 발견을 못했네요..
User와 Computer가 카드선택은 동일하게 있는데, 나머지 승패 등은 유저만 관리하면 될 것 같아서 Player 프로토콜을 만들었습니다.
승패를 판단하는 기능을 구현할 때 if문이 많이 들어갔는데, else를 피하려고 early pattern을 적용해 봤습니다. 다른 방법은 뭐가 있을까요?
UI와 연동을 안해서, user카드도 랜덤으로 선택하도록 일단 해놨습니다.
@zdodev