---
# System prepended metadata

title: '계산기II [Step2] Mangdi, Jeremy'

---

# 계산기II [Step2] Mangdi, Jeremy

@리뷰어태그

안녕하세요 hyunable! step1 머지되고나서 머지않아 step2 PR 보냅니다 ㅎㅎ
리팩토링을 저희가 진행할수있는부분을 최대한 해보았는데 부족한점이 보인다면 마구마구 코멘트 달아주시면 감사하겠습니다 ㅎㅎ

## 🤔 코드를 작성할 때 고민되었던 점
* 함께 리팩토링을 진행할때 눈에 띄는 메서드들을 타겟으로 진행하려고했습니다. 
대표적으로 showResult, addZero, addOperator 메서드들이 있었습니다.

* showResult메서드를 리팩토링할때 addStackView메서드를 호출하는부분에서 addOperator랑 겹치는 코드가 발견되어 addStackView메서드를 리팩토링해주었는데요
결과적으로 showResult메서드 자체의 리팩토링은 진전이 없었습니다.
이부분에 대해 리팩토링할부분이 보이신다면 알려주시면 감사합니다..!

* addZero랑 addOperator메서드는 조건이 동일하게 2가지씩 붙어있어서 if문으로 &&연산자를 붙여가며 작성했던 코드를 switch문으로 덜 지저분하게? 리팩토링해주었습니다. 이전 step1PR에 코멘트 달아주신것을 참고해서 코드를 간결하게 고쳐보았는데 hyunable은 이점에 대해 추가로 조언해주실부분이 혹시 있으실까 여쭤봅니다!

## 🆘 조언을 얻고 싶은 부분 등 상세하게 작성
* C라는 메서드에 A와 B메서드를 호출한다 가정했을때 A,B,C메서드의 순서를 어떻게 지정해줄까 고민이 있었습니다. hyunable의 의견이 궁금합니다!
1. C에서 호출할메서드들을 미리 위에서 정의해주어서 A,B,C순서로 간다.
2. C에서 사용하는메서드들을 알려주기위해서 C,A,B순서로 간다.