계산기 [Step3] is, june #32

vanism2091 · 2023-01-27T09:13:44Z

🧮 계산기 프로젝트

안녕하세요 코리! @corykim0829
늦은 pr 받아주셔서 감사해요 :)
설날 덕분에 고민하고 구현할 시간이 부족해서 내심 아쉬웠는데요. 어떠한 리뷰든 좋으니 마구 마구 부탁드려요!!
한 주 동안 고생 많으셨고 좋은 주말 보내세요😊

Step3: 요구 사항 및 구현 사항

요구 사항 정리

계산기

Step3

사용자의 터치 이벤트를 수신하여 실제로 연산을 수행할 수 있도록 구현합니다
숫자를 입력하고 계산하는 기능을 구현합니다
스택뷰와 레이블을 활용하여 계산 내역을 표시해줍니다
계산내역이 길어지면 위아래로 스크롤 할 수 있습니다
계산내역이 상단 공간을 넘어 이어지는 경우, 사용자에게 제대로 보일 수 있도록 새로 추가될 때 최하단으로 자동 스크롤 할 수 있도록 합니다

🧹 고민한 부분

☄️ throw Error, ResultType

do {
    let result = try pairs.reduce(operands.values[0]) { (partialResult, pair) in
        guard (0.0, Operator.divide) == pair else { throw FormulaError.dividedByZero }
        let (operand, `operator`) = pair
        return `operator`.calculate(lhs: partialResult, rhs: operand)
    }
    return .success(result)
} catch {
    return .failure(.dividedByZero)
}

guard false == pairs.contains(where: { pair in (0.0, Operator.divide) == pair }) else {
    return .failure(.dividedByZero)
}

let result = pairs.reduce(operands.values[0]) { (partialResult, pair) in
    let (operand, `operator`) = pair
    return `operator`.calculate(lhs: partialResult, rhs: operand)
}

처음 구현은 제안해주셨던대로 reduce 안에서 오류를 throw 했어요.
(0, .divide)가 있다면 굳이 reduce를 하지 않아도 될 것 같아서 밖으로 뺐어요!
그 후 Step3에서 Result Type을 사용해보고싶어서 도입했어요.
이를 바탕으로 제안해주신 방향으로 수정해보았는데, 제가 느끼기에는 수정 전 코드가 가독성이 좋은 것 같아요.
그런데 제가 보기 편한 것이 가독성이 좋은 게 아니기 때문에, 코리의 의견이 어떤지 궁금해요.
❓ 여전히 where절은 복잡하지만, 기존 함수가 error를 throw하지 않고 result를 반환하는 지금도 위의 코드가 더 가독성이 좋을까요?
❓ 코리의 제안을 수용하면서(where절을 복잡하지 않게 수정하고) result type을 사용하는 더 가독성 좋은 코드가 있으면 제안해주세요!

😴 숫자에 쉼표, NumberFormatter

숫자에 쉼표를 붙이기위해 NumberFormatter를 이용했어요.
NumberFormatter로 형변환을 하니 overflow 문제가 생기더라고요.
- 아마 String -> NSNumber -> String 로 변환되는 과정에서,
- NSNumber에서 overflow문제가 일어나는 것 같아요.
overflow를 해결할 방법을 생각해내지 못해서 요구 사항인 20자리를 해결하지 못했어요.
현재는 계산 결과에만 쉼표를 붙여줍니다.
❓ overflow.. 어떻게 해결할 수 있을까요?🥹

그 밖의 궁금증

❓ 전반적으로 이렇게 했으면 더 이해하기/보기 좋았을텐데..! 싶은 부분이 있으면 편하게 알려주세요:)
❓ 코리가 생각하는 좋은 개발자란? 그리고 좋은 개발자가 되기 위해 어떤 노력을 하고 있나요?

- 숫자 최대길이 20자

- Int.max 범위 넘어가는 숫자에 대해서 동작 미구현

- "00" 버튼을 최대길이 - 1에서 눌렀을 때, "0" 하나만 추가

- scrollView 최하단의 하단으로 가야함

- ViewController -> CalculatorViewController - final 키워드 추가

- extension으로 구현

- UILabel 변수명 변경 operationLabel -> operatorLabel currentNumberLabel -> entryNumberLabel - 매직 넘버 20을 maxDigitLength로 변경 - isNumberTyped를 삭제 기존에는 0을 누른 후 operator를 눌렀을 때 history에 추가하기 위해 해당 변수를 사용했으나, 숫자가 0만 있을 떄에는 history에 추가하지 않도록 작동 방식 변경

- history stack 비우기 제외 구현

- formulaString 생성 후 추적. 이를 parser에 넘겨서 result를 받아올 예정 - clear 관련 리팩토링 - 메서드 내 number 변수 네이밍 refactoring (currentNumber -> number)

…ber, displayOperator 구현 - 기존은 displayNumber가 0이 아닐 때 숫자를 추가했다. - 계산 결과에서 숫자를 누르면 추가가 아니라 새로 입력받을 수 있도록 isNumberInTyping 변수를 활용해 구현했다.

- reduce 내 current prefix 제거

- entryNumberLabel.text를 displayNumber 로 변경 - buttonName -> buttonTitle로 변수 네이밍 변경

corykim0829

두분 모두 수고 많으셨습니다 👏👏👏

☄️ throw Error, ResultType

result 사용 잘하셨네요!
가독성도 있지만 저는 순회를 두번할 필요가 없을 것 같아서, reduce 안에서 처리하면 좋을 것 같다고 생각한 거였습니다!

reduce와 Result를 모두 사용하면 throw를 사용하는게 과해보이는건 있네요!
차이점은 다 알고계신게 중요하다 생각합니다 :)

😴 숫자에 쉼표, NumberFormatter

이 부분은 위에를 정리하면 조금 해결될 수도 있다고 생각이 드는데요. NumberFormatter는 마지막에 Label이 표시할 때에만 처리하는 방향으로 처리해보면 좋을 것 같아요.

그 밖의 궁금중

displayNumber를 get set을 잘 활용하셔서 처리한 부분은 매우 좋았습니다! 👍

단, displayNumber 데이터를 get으로 가져올 때 entryNumberLabel에서 가져오는건 어색합니다.

컨트롤러에서 뷰에게 데이터를 넘겨주기만 하고, 뷰는 해당 데이터를 표출해주는 역할만 하는게 깔끔한 로직이라고 생각합니다.

그래서 최대한 데이터와 뷰의 역할은 분리시켜주는게 좋습니다.

지금은 서로 얽혀있는 것 같아서, 코드를 읽기 위해서 위 아래를 자주 왔다갔다 해야하는 느낌이 들고, 나중에 수정하기가 힘들어보여요!

너무 어려운 질문을 해주셨는데... 개인적으로는 그냥 코드를 쓰는게 아니고 이 코드의 이유를 알고 사용하는게 좋은 개발자라고 생각합니다. 어떻게 작동되는지 알고 사용하는게 중요하다고 생각합니다 :)

corykim0829 · 2023-01-28T08:16:10Z