[ TIL ] 저 칭찬받았어요..!! ( 숫자야구 게임 구현에 대한 피드백 ) / 24년 3월 16일

 

출처 : 핀터레스트 / 최고심

 

안녕하세요! 오늘은 어제 제출했던 숫자 게임에 대한 피드백을 가져왔습니다. ( 사실 한번 날려먹은 상태라 멘탈이 너덜너덜이에욤..ㅠ 그래도 뭐 어떡해 다시 작성해야죠..ㅠ )

 

 

저.. 칭찬받았어요!!!ㅎㅎㅎㅎㅎ

사실 어제 확장에 대한 개념을 공부하던 중 확장의 데이터 구조에 대해서 이해가 안되는 부분이 있더라구요. 그래서 주변에 도움을 요청했는데, 알고보니 제가 확장의 데이터 구조를 아예 처음부터 이해를 잘 못한거더라구요.. (사실 아직 해결 못함.. 아직도 이해 못 하는중.. ) 그래서 아주 낙담하며 우울해하고 있었는데 이런 단비같은 칭찬이....🌸

 

 

특히 커밋 내역 잘 나누어서 남겨주셔서 흐름 따라가기 매우 용이했습니다.
...
매우 잘하셨습니다!
...
커밋 내역은 감동이에요 ㅜ

 

 

제가 더 감동입니다.. 튜텨님..🩵

 

 

 

 

 

이제 피드백을 받았으니 하나하나 뜯어봐야죠!

첫 번째, " RecordManager은 아주 잘 작성하셨습니다! "

 

TableView 에 관한 앱 강의를 들으면서 테이블 뷰를 따라해볼때, TableViewManager 를 썼던 기억이 나더라구요. 그래서 게임 기록을 보관하고 있는 RecordManager 가 따로 구현이 되어있으면 좋겠다고 생각해 게임 기록을 보관하는 배열과 게임 기록을 업데이트하거나 반환할 수 있는 함수를 RecordManager 클래스로 따로 구현하였습니다.

 

class RecordManager {
    
    // 유저들의 게임 기록을 기록할 배열
    private var recordArray: [Int] = []
    
    // 유저들의 게임 정보를 저장할 함수
    func setRecordArray(_ count: Int) {
        recordArray.append(count)
    }
    
    // 유저들의 게임 정보를 출력하는 함수
    func getRecordArray() {
        if recordArray == [] {
            print("게임 기록이 존재하지 않습니다.")
        }
        recordArray.enumerated().forEach{ (i, count) in
            print("\(i+1)번째 게임 : 시도 횟수 \(count)번")
        }
    }
}

 

이 부분에서 게임 기록을 보관하고 있는 recordArray 를 private 로 구현해 클래스 외부에서 접근을 하지 못하도록 하고, 게임 정보를 저장하는 set 함수와 게임 정보를 출력하는 get 함수로 접근을 하도록 하였어요. 이 부분에서 칭찬을 받은 듯 합니다ㅎ!

 

 

 

두 번째, "User와 Number도 비슷한 구조로 만들 수 있을 거 같아요. "

 

User와 Number 클래스를 따로 구현하였지만, 내부적으로 선언된 변수에는 클래스 외부에서 접근을 하도록 구현을 하였어요. 이 부분을  피드백에 따라 private로 바꾸어 주면서 필요한 함수를 추가해봅시다.

 

UserClass

class User {
    
    // 정답까지의 횟수
    var count: Int = 0
    
    // 유저가 제시한 숫자
    var userNum: Int?
    
    // 유저에게 입력값을 받는 함수
    func getUserInput() {
        let input = readLine()
        
        // 문자열이 입력되지 않았을 경우 에러처리
        guard let input = input else { return getUserInput() }
        
        // 문자열이 3자리가 아닐 경우 에러처리
        if input.count != 3 {
            print("0부터 9까지를 자릿수로 가지는 3자리 숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열의 첫번째가 0이 포함되어 있는 경우 에러처리
        if input.first == "0" {
            print("세자리 숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열이 숫자가 아닐 경우 에러 처리
        guard let input = Int(input) else {
            print("숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열이 중복이 된 경우 에러처리
        // 백의 자리, 십의 자리, 일의 자리
        let hundred = input / 100
        let tenDigit = ( input % 100 ) / 10
        let one = input % 10
        
        if hundred == tenDigit || tenDigit == one || hundred == one {
            print("숫자가 중복되었습니다.")
            return getUserInput()
        }
        
        self.userNum = input
    }
    
}

 

정답을 제시한 횟수를 담는 count 와 유저에게서 입력받은 숫자 정보를 담을 userNum 을 클래스 외부에서 접근하지 못하도록 private 로 구현하였어요. 또한 count 의 횟수를 올려줄 set 함수와 count 를 불러올 get 함수를 따로 구현해주고, userNum 을 받아오는 get 함수를 추가로 구현하였습니다. 클래스 외부에서 수정되지 못하니 더더욱 안전하게 데이터를 저장할 수 있더라구요.

 

UserClass (수정)

class User {
    
    // 정답까지의 횟수
    private var count: Int = 0
    
    // 유저가 제시한 숫자
    private var userNum: Int?
    
    // 유저에게 입력값을 받는 함수
    private func getUserInput() {
        let input = readLine()
        
        // 문자열이 입력되지 않았을 경우 에러처리
        guard let input = input else { return getUserInput() }
        
        // 문자열이 3자리가 아닐 경우 에러처리
        if input.count != 3 {
            print("0부터 9까지를 자릿수로 가지는 3자리 숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열의 첫번째가 0이 포함되어 있는 경우 에러처리
        if input.first == "0" {
            print("세자리 숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열이 숫자가 아닐 경우 에러 처리
        guard let input = Int(input) else {
            print("숫자를 입력해주세요.")
            return getUserInput()
        }
        
        // 문자열이 중복이 된 경우 에러처리
        // 백의 자리, 십의 자리, 일의 자리
        let hundred = input / 100
        let tenDigit = ( input % 100 ) / 10
        let one = input % 10
        
        if hundred == tenDigit || tenDigit == one || hundred == one {
            print("숫자가 중복되었습니다.")
            return getUserInput()
        }
        
        self.userNum = input
    }
    
    func getUserNum() -> Int? {
        getUserInput()
        return userNum
    }
    
    func setUserCount() {
        count += 1
    }

    func getUserCount() -> Int {
        return count
    }
    
}

 

이제 Number도 변경해봅시다.

 

NumberClass

class Number {
    
    var num: Int?
    
    func getRandomNum() {
        var num = [Int]()
        
        while num.count < 3 {
            let a = Int.random(in: 0...9)
            if num == [] && a != 0 {
                num.append(a)
            }else if num != [] && !num.contains(a) {
                num.append(a)
            }
        }
        
        self.num = num[0] * 100 + num[1] * 10 + num[2] * 1
    }
}

 

위의 코드에서 게임의 정답을 저장할 num 을 private 로 수정해주면서 해당 num 을 출력할 수 있는 getRandomNum 함수까지 구현을 해주었어요.

 

NumberClass(수정)

class Number {
    
    private var num: Int?
    
    private func makeRandomNum() {
        var num = [Int]()
        
        while num.count < 3 {
            let a = Int.random(in: 0...9)
            if num.isEmpty && a != 0 {
                num.append(a)
            }else if !num.contains(a) {
                num.append(a)
            }
        }
        
        self.num = num[0] * 100 + num[1] * 10 + num[2] * 1
    }
    
    func getRandomNum() -> Int? {
        makeRandomNum()
        return num
    }
}

 

 

 

세 번째, 추가적인 코드 구현의 피드백

 

"빈 배열인지 체크할 때는 someArray.isEmpty를 활용해보세요."

 

저는 코딩테스트를 풀 때에도 항상 빈 배열을 체크하는 문자에서는

if someArray == [] {
	// 내부 코드
}

이렇게 구현을 하였었는데, 배열에 isEmpty 라는 함수가 있었는진 몰랐네요!ㅎ 이제 isEmpty 를 적극적으로 활용해야할 것 같습니다.

 

 

"String(user).map{ Int(String($0))! }은 String(user).compactMap{ Int(String($0))} 과 같습니다."

 

일단 이 부분은 제가 map 과 compactMap 의 차이에 대한 지식이 부족했던 거 같아요. 이 문장을 보자마자 오늘 해야할 일은 map , compactMap , flatMap 을 공부하고 그 차이를 포스팅해야겠다고 생각이 들었습니다. 바로 공부해서 포스팅하여 글 링크 첨부해둘게요! ㅎ

 

🔗 map, compactMap, flatMap의 차이

 

 

 

네 번째, "enum을 활용해서 choice부분을 리팩토링해보세요."

 

같은 종류의 한정적인 케이스는 enum으로 구현하여 그 이외의 케이스가 들어오는 것을 막을 수 있다고 저번 과제때도 말씀해주셨는데, 고새 까먹고 그냥 switch로만 구현한 바보 몽충이 나.. 이건 저의 불찰이므로 반성함니댜..ㅎ

 

기존의 MainPlay에서 choice에 사용자가 선택한 값을 받아와 switch 문을 통해 각각의 상태에 따른 코드를 짰는데요. choiceNum 함수에서 1, 2, 3과는 다른 케이스가 들어왔을 때의 조건을 일일히 구현해야했어요.

 

MainPlay 클래스

class MainPlay {
    
    init() {
        main()
    }
    
    let recordManager = RecordManager()
    
    // MARK: - 숫자를 선택하는 choiceNum() 함수
    private func choiceNum() -> Int {
        print("---------------------------------------")
        print("1. 게임 시작하기   2. 게임 기록보기  3. 종료하기 ")
        print("=======================================")
        
        let choice = readLine()
        
        guard let choice = choice else {
            return choiceNum()
        }
        guard let choice = Int(choice) else {
            print("숫자를 입력해주세요.")
            return choiceNum()
        }
        
        if 1 > choice || choice > 3 {
            print("번호를 잘못 입력하셨습니다. 다시 입력해주세요. ")
            return choiceNum()
        }else {
            return choice
        }
    }
    
    // MARK: - 메인 함수
    func main() {
        print("=======================================")
        print("환영합니다!! 원하시는 번호를 입력해주세요.")
        
        // 번호를 입력하는 함수
        let choice = choiceNum()
        
        switch choice {
        case 1 :
            let startGame = GamePlay()
            let gameCount = startGame.start()
            recordManager.setRecordArray(gameCount)
            return main()
        case 2 :
            // 기록보기
            recordManager.getRecordArray()
            return main()
        case 3 :
            // 종료하기
            print("[ 숫자 야구 게임을 종료합니다 ]")
            break
        default :
            return main()
        }
    }
}

 

이거를 아래의 코드로 수정했습니다!

 

MainPlay 클래스

class MainPlay {
    
    init() {
        main()
    }
    
    let recordManager = RecordManager()
    
    enum Choice: String {
        case start = "1"
        case record = "2"
        case exit = "3"
    }
    
    // MARK: - 숫자를 선택하는 choiceNum() 함수
    private func choiceNum() -> Choice {
        
        let choice = readLine()
        
        guard let choice = choice else { return choiceNum() }
        guard let choice = Choice(rawValue: choice) else {
            print("잘못 입력하였습니다. 다시 입력해주세요.")
            return choiceNum()
        }
        return choice
    }
    
    // MARK: - 메인 함수
    func main() {
        print("=======================================")
        print("환영합니다!! 원하시는 번호를 입력해주세요.\n")
        print("1. 게임 시작하기   2. 게임 기록보기  3. 종료하기 ")
        print("=======================================")
        
        switch choiceNum() {
        case .start :
            let startGame = GamePlay()
            let gameCount = startGame.start()
            recordManager.setRecordArray(gameCount)
            return main()
        case .record :
            // 기록보기
            recordManager.getRecordArray()
            return main()
        case .exit :
            // 종료하기
            print("[ 숫자 야구 게임을 종료합니다 ]")
            break
        }
    }
}

 

사용자에게 문자열을 받아오는 과정에서 1, 2, 3을 제외한 문자열이면 다 다시 입력을 받아와 코드가 아주 깔끔하게 간결해줬어요! 또한 문자열을 굳이 숫자로 바꾸지 않아도 되어 코드가 한 줄 더 줄었습니다.ㅎㅎ

 

추가적으로 튜텨님께서 failable init 에 대해서 생각해보라고 하셨는데, 열심히 찾아보니 failable init 은 실패할 가능성이 있는 생성자이더라구요! 원래 클래스, 구조체, 열거형의 생성자는 무조건 성공하도록 구현이 되어있어요. 하지만 failable init 은 성공하면 초기값을 부여하고, 실패하면 nil 을 반환합니다. 그래서 옵셔널같다는 생각이 들었어요.

다시 위의 코드를 살펴보면 Choice( rawValue: String ) 부분이 열거형의 failable init 이여서 guard let 을 통한 unwrapping 과정을 거치고 있어요. 만약 unwrapping 되지 않는다면 즉, 사용자가 입력한 문자가 "1", "2", "3"을 제외한 문자열이라면 다시 chioceNum 코드가 실행이 되는 거죠!

 

 

---

 

이렇게 피드백에 대한 반영을 해보았는데요?!

오늘도 성장한 나 아주 멋쥐자나?!ㅎㅎㅎㅎㅎㅎㅎ

기분 좋습니다요!😊