가위바위보게임 [STEP 1] caron #11
Conversation
| 기획의 변경에 대해서 최대한 열린 코드로 작성해봅니다 | ||
| */ | ||
|
|
||
| func test_승패판결_주먹_가위_보가아니면_에러() { |
There was a problem hiding this comment.
테스트명과 내용을 보면 어떤 테스트를 진행하는 지 알기 어려워보입니다.
진행할 테스트에 대한 이름으로 수정하거나, 내용을 개선해볼 수 있을까요?
| func test_승패판결() { | ||
| //given | ||
| let input: [String] = ["🖐️", "✊"] //✊ | ||
| var count: [Int] = [0,0] //테스트를 위해 임의로 만듦. |
There was a problem hiding this comment.
count 라는 프로퍼티명과 [0, 0] 배열이 어떤 의미를 나타내는 지 알기 어려워보입니다.
조금 더 구체적으로 표현해볼 수 있을까요? 새로운 타입으로 나타내보는 것은 어떨까요?
|
|
||
| func test_삼세판_진행() { | ||
| //given | ||
| var input: [Int] = [2,1] |
There was a problem hiding this comment.
input: [Int] = [2,1] 배열이 어떤 의미를 나타내는 지 알기 어려워보입니다.
조금 더 구체적으로 표현해볼 수 있을까요?
| XCTAssertEqual(result, [true, false]) | ||
| } | ||
|
|
||
| func test_삼세한_르푸() { |
| func test_삼세판후_대결결과() { | ||
| var input: [Bool] = [true, false] | ||
|
|
||
| } |
| case ["🖐️", "✊"] : return win | ||
| case ["🖐️", "✌️"] : return lose | ||
| case ["✊", "✌️"] : return win | ||
| case ["✊", "🖐️"] : return lose | ||
| case ["✌️", "🖐️"] : return win | ||
| case ["✌️", "✊"] : return lose |
There was a problem hiding this comment.
패를 이모지로 나타내신 이유가 있을까요?
새로 정의하신 Hand 타입을 활용해보는 것은 어떨까요?
| } | ||
|
|
||
|
|
||
| func rpsModify(of rps: [String]) throws -> Bool { |
There was a problem hiding this comment.
rpsModify 메서드명을 보고 어떤 역할을 수행하는 지 모호해 보입니다.
더 이해하기 쉬운 이름으로 변경할 수 있을까요?
| var countting = self.count | ||
|
|
||
| while !besttwooutOfthree(of: 3, in: countting)[0] && !besttwooutOfthree(of: 3, in: countting)[1] { | ||
| print(self.count) |
There was a problem hiding this comment.
어떤 용도로 사용하는 print문일까요? 불필요하다면 제거하는 것이 어떨까요?
| return fightToResult(of: besttwooutOfthree(of: 3, in: countting)) | ||
| } | ||
|
|
||
| //손인지 확인 |
There was a problem hiding this comment.
꼭 필요한 주석일까요? 메서드명을 명확하게 작성해서 의도를 잘 나타내보면 좋을 것 같습니다!
| } | ||
|
|
||
| //손인지 확인 | ||
| private func rpsHandsCheck(of rps: [String]) -> Bool { |
There was a problem hiding this comment.
API 디자인 가이드에 따라 메서드명을 동사형으로 시작해보는 것은 어떨까요?
다른 메서드도 확인해보면 좋을 것 같습니다!
질문사항
참고:
제가,, 다 하고 보니까 뭔가 처음에 가위바위보내는 부분을 랜덤하게 해야했던거 같은데 그부분을 못했고.. ㅠㅠ
3회 처리하는곳도 따로 빼야 했던거같은데..
기능은 미완이지만 아키텍처부분이나 TDD등등 쪽으로 리뷰 부탁드립니다. ㅠㅠ
죄송해요..다음 step2하면서 추가적으로 해보겠습니다!! ㅠㅠ