###### tags: `project`
# 🧮 계산기2
## 일일 스크럼
### 🙌 06/12(월)
- 오늘의 컨디션
- Dasan: (월요일 치고?!)괜찮습니다☺️
- Zion: 좋습니다.
- 특이사항
- Dasan: 없습니다!
- Zion: 공부해야할게 많다.
- 오늘 할 일
- [X] dasan - STEP3 진행하기
- [ ] Zion - Protocol, Extensions, STEP3 리팩토링
- [x] 활동학습 예습
### 🙌 06/13(화)
- 오늘의 컨디션
- Dasan: 눈이 피곤합니다🥹
- Zion: 잠을 많이 자지 못해 피곤합니다.
- 특이사항
- Dasan: 엇 어째서...! 시온의 기력을 제가 가져갔나봐요🤣 저는 기력을 되찾았(?)습니다!
- Zion: 뭔가 무기력함....
- 오늘 할 일
- [x] Zion - STEP3 리팩토링
- [x] Zion - Strong Reference Cycle TIL 작성
- [x] Zion - Protocols docs
- [x] Zion - Extensions docs
- [X] dasan - STEP3 UI 연동(feat. 시온)
- [ ] dasan - STEP3 요구사항에 맞도록 로직 수정 및 추가
### 🙌 06/14(수)
- 오늘의 컨디션
- Dasan: 컨디션 좋습니다! 머리가 안돌아갈뿐...
- Zion: 잠을 많이 자지 못해 피곤합니다.
- 특이사항
- Dasan: 오후 8시에 원비와 멘토링이 있습니다!
- Zion: 졸리다.
- 오늘 할 일
- [X] dasan - STEP3 요구사항에 맞도록 로직 수정 및 추가
- [x] Zion - Generics
- [ ] Zion - Subscripts
- [ ] Zion - Inheritance
### 🙌 06/15(목)
- 오늘의 컨디션
- Dasan: 점점 컨디션이 좋아지는 것 같아요! 방학이 다가와서 그럴까요 껄껄
- Zion: 잠을 잘자서 좋습니다.
- 특이사항
- Dasan: 특별한 사항은 없습니다🙌
- Zion: 특이사항 없습니다!
- 오늘 할 일
- [X] 계산기 코드 Merge
- [X] 합친 코드 리팩토링
- [X] dasan - 활동학습 예습
- [X] zion - 활동학습 예습
- [X] PR 보내기!!!!!!
### 🙌 06/16(금)
- 오늘의 컨디션
- Dasan: 방학 전날이라 힘이 납니다!!
- Zion: 잠을 잘자서 컨디션이 좋스빈다.
- 특이사항
- Dasan: 없습니다
- Zion:
- 오늘 할 일
- [X] 피드백 반영
- [ ] README 작성
- [ ] Zion - Subscripts
- [ ] Zion - Inheritance
- [ ] Dasan - 오토레이아웃 강의 마저 보기(야곰)
- [ ] Zion - App Life Cycle TIL 작성
--------------------------------------------
계산기 II [STEP 2] Zion, Dasan
안녕하세요 건디!(@Gundy93)
기다려주셔서 감사합니다 건디😂
STEP1, STEP2를 함께 하여 PR드립니다.
1학기 마지막 PR인 만큼 건디가 해주시는 말씀 더더욱 새겨듣겠습니다!🙇♀️🙇♂️
---------------------
## STEP 2
### 고민했던 점
#### 많은 예외처리로 떨어지는 가독성
- 계산기에서는 숫자 버튼, 연산자 버튼, "=" 버튼을 눌렀을 때 동작들뿐만 아니라 각 버튼들이 어떤 상황에서 어떻게 입력되었는지에 따라 많은 예외처리 사항들이 생겼습니다. 예외처리 사항들이 늘어나면서 조건문이 늘어났고, 이와 반비례하여 가독성을 떨어진 부분이 아쉬웠습니다.
- 조금이라도 가독성을 높이고, 비대해진 `ViewController`의 역할을 나누고자 역할에 따라 새로운 `Type`들을 만들고 기능에 따른 `method`분리를 해주었습니다.
- 특히 Operand에 대한 Format이 각 상황에 따라 처리해줘야하는 예외들이 많아 코드가 방대해졌습니다. 따라서 `Controller`가 아닌 `Model`에 `OperandFormatter`라는 타입을 만들어, **`Operand`에 대한 Format을 담당**하도록 하였습니다.
#### 코드 병합시 충돌해결
- 같은 `UML`을 기준으로 작성한 코드였지만 생각보다 예외처리 및 동작 로직이 달라 코드 병합시 시간이 많이 들었습니다. 물론 숫자를 입력받아 계산한다는 프로그램의 동작은 같았지만 결과물이 나오기까지의 세부적인 예외처리들 이용하는 메서드들 등 다른 부분들이 상대적으로 많아서 이해하고 진행하는데 애를 먹은 것 같습니다.
- 또한 같은 기능을 하는 `property, method`가 서로 다른 네이밍으로 되어 있어서 그러한 부분들이 가장 헷갈렸고 이러한 부분들은 코드 병합전 서로의 구현한 코드 상세의 UML을 먼저 그려서 비교해보고 진행했더라면 훨씬 수월하게 진행할 수 있었겠다는 생각을 했습니다.
- 같은 기능을 수행하더라고 객체에 대한 서로 다른 의존성을 가지고 있었기 때문에 이러한 부분도 코드 병합시 고민해봐야하는 부분이었다고 생각합니다. 동일한 기능을 수행하는 객체가 서로 다른 의존성을 가지고 있었기 때문에 코드를 병합할 때 해당 의존성을 어떻게 분리할 수 있을지 너무 의존하는 객체에 대해 결합도가 높지는 않았는지 충분히 분리할 수 있을 정도의 의존성인지를 파악하는 것도 코드 병합시 가장 중요한 부분이라는 생각을 했습니다.
### 해결하지 못한 점
### 조언을 얻고 싶은 점
---------------------
README 작성 완료했습니다😄
리뷰어: Gundy🥸
팀원: Dasan🐯, Zion🦁
https://github.com/DasanKim/ios-calculator-app/tree/calculator2-Step2
# 계산기 II
## 🍀 소개
`Dasan`과 `Zion`이 만든 계산기입니다. 사용자로부터 UI 입력을 받아 계산기 동작을 수행합니다.
<br>
## 📖 목차
1. [팀원](#-팀원)
2. [타임라인](#-타임라인)
3. [시각화된 프로젝트 구조](#-시각화된-프로젝트-구조)
4. [실행 화면](#-실행-화면)
5. [트러블 슈팅](#-트러블-슈팅)
6. [참고 링크](#-참고-링크)
7. [팀 회고](#-팀-회고)
<br>
## 👨💻 팀원
| Dasan | Zion |
| :------: | :------: |
|<Img src="https://i.imgur.com/EU67fox.jpg" width="200" height="200"> |<Img src="https://hackmd.io/_uploads/rJqMfSoVn.png" width="200" height="200"> |
|[Github Profile](https://github.com/DasanKim) |[Github Profile](https://github.com/LeeZion94)
<br>
## ⏰ 타임라인
|날짜|내용|
|:--:|--|
|2023.06.12.(월)| - `메모리 구조` 및 `ARC`에 관한 학습 <br> - **Dasan**: STEP3 계산기 코드와 UI 연동(with `Zion`) |
|2023.06.13.(화)| - **Dasan**: STEP3 계산기 `예외처리`(with `Zion`) <br> - **Zion**: STEP3 `리팩토링` | <br> - **Zion**: `Protocols`, `Extensions` 공식문서 학습 |
|2023.06.14.(수)| - **Dasan**: STEP3 계산기 `예외처리`(with `Zion`) <br> - **Zion**: `Generics` 공식문서 학습 |
|2023.06.15.(목)| - `iOS App Life Cycle`에 관한 학습 <br> - Dasan과 Zion 계산기 코드 PR을 통한 **병합** <br> - 병합한 코드 **리팩토링** <br> - `PR` 작성 |
|2023.06.16.(금)| - 피드백 요청사항 반영 및 전체적인 리팩토링 <br> - `README` 작성 |
</br>
## 👀 시각화된 프로젝트 구조
<p>
<img width="" src="https://github.com/DasanKim/ios-calculator-app/blob/calculator2-Step2/Calculate2ClassDiagram.jpg?raw=true">
</p>
<br>
## 💻 실행 화면
|입력을 받은 수 계산화면|소수점 입력을 받은 수 계산화면|
|:--:|:--:|
|<img src="https://hackmd.io/_uploads/ryrYSutvn.gif?" width="400">|<img src="https://hackmd.io/_uploads/H1jZLdFv3.gif" width="400">|
|이전 결과값에 이어서 계산하는 화면|AllClear로 결과값을 전부 지우는 실행화면|
|:--:|:--:|
|<img src="https://hackmd.io/_uploads/BJQ7UdYv3.gif" width="400">|<img src="https://hackmd.io/_uploads/Hy6XUOKwh.gif" width="400">|
</br>
## 🧨 트러블 슈팅
### 1️⃣ **UI를 갱신하는 타이밍 고민** <br>
🔒 **문제점** <br>
계산기의 계산식이 추가될 때마다 ScrollView내부의 StackView에 계산식이 쌓이고 늘어난 Scroll ContentSize에 ScrollToBottom을 해주는 로직을 추가했습니다.
하지만, 계산식이 추가되어 스크롤이 될 때마다 한 개의 계산식 만큼 밀려서 스크롤이 되는 현상이 이슈로 나타나게되었습니다.
🔑 **해결방법** <br>
원인은 UI의 갱신 타이밍이었습니다. ScrollView내부의 StackView에 계산식이 쌓이고 늘어난 ScrollView에 대해 ScrollToBottom이 되어야 했지만 늘어나기 이전의 ContentSize를 기준으로 ScrollToBottom 동작을 하게되어 해당 이슈가 발생했습니다.
따라서, 해당 이슈를 해결하기 위해 UI를 갱신하기 위한 메서드들을 살펴봤습니다.
정리한 내용을 같이 공유드립니다! [UI 갱신 메서드 비교](https://medium.com/@LeeZion94/layoutifneeded-vs-setneedslayout-23a29471582a)
### 2️⃣ **비슷한 기능 및 역할을 하는 객체의 네이밍에 대한 고민**
🔒 **문제점** <br>
소수점, 정수, operand, operator와 같이 비슷한 기능 및 역할을 하지만 조금씩은 다른 특징을 가지는 객체에 대한 네이밍을 정하는데 있어서 많은 어려움이 있었습니다. 따라서 이러한 것들이 쌓이다보니 나중에는 해당 객체가 어떤 역할을 하는지 한번 더 생각해야하는 고민이 늘게 되었습니다.
🔑 **해결방법** <br>
우선 해당 객체의 네이밍의 범위를 해치지 않는 선에서 통합할 수 있는 기능 및 역할들을 통합했습니다. 또한 분명하게 구분지어져야하는 property 및 method에 대해서는 네이밍이 조금은 더 길어지더라도 분명하게 역할이 나누어질 수 있도록 키워드를 추가하여 작성했습니다. Swift Language Guide에서도 언급하고 있듯이 간결함 보다는 명료함에 초점을 맞춘 네이밍을 가져가고자 했습니다.
그 결과, 계산기의 기능 및 동작을 하는데 있어서 역할의 분명함 뿐만 아니라, 역할에 따라 나누다 보니 했던 개체들의 기능분리를 통해 프로그램의 전체적인 가독성의 상승과 유지보수성이 좋아지겠되었습니다.
<br>
## 📚 참고 링크
- [Blog: UI 갱신 타이밍](https://ontheswift.tistory.com/20)
- [Blog: UIScrollView Layout](https://beepeach.tistory.com/613)
- [Blog: Floating Point](https://serynana.tistory.com/entry/swift-%EB%B6%80%EB%8F%99-%EC%86%8C%EC%88%98%EC%A0%90-%EC%98%A4%EB%A5%98%EA%B0%80-%EB%82%98%EB%8A%94-%EC%9D%B4%EC%9C%A0)
- [Blog: Liked-List](https://opentutorials.org/module/1335/8821)
- [Apple Docs: double](https://developer.apple.com/documentation/swift/double)
<br>
## 👥 팀 회고
### 칭찬할 부분
- To. Dasan : Step3에 대한 내용을 부지런히 시간투자를 하셔서 따라와주셨다는 점. 스텝진행을 위해 공부 및 수정해야할 부분을 말씀드리면 다음날 스크럼전까지 꼭 완료해서 와주셨다는 점. 시간약속을 항상 잘지켜주셨다는 점. 무엇이든 항상 배우려고 하셨다는 점이 대단하다고 생각했습니다.
- To. Zion : 시간 약속을 항상 잘 지켜주신 점. 지난 프로젝트에서 Step3를 완료하지 못했었는데, 병합을 먼저 진행하기보다 Step3를 고민하고 구현할 수 있도록 시간을 주신 점. 시간을 주신 것뿐만 아니라 도와주시면서도, 스스로의 공부를 결코 소홀히 하지 않으신 점. Step 진행보다 팀원이 해당 Step에서 얻어가야할 지식들을 얻을 수 있도록 이끌어주신 점. 같이 성장하기를 진심으로 바래주시며 도움과 응원을 아끼지 않아주신 점. 코드에 대한 이해와 코딩 실력이 뛰어나심에도, 겸손하시고 공부를 깊이 하시는 점이 멋지시다고 생각했습니다.
### 개선해야할 부분
-
### 서로에게 하고 싶은 말
- To. Dasan
앞으로의 스텝진행에 있어서도 이번 계산기처럼 매순간이 어려우시겠지만, 이번 계산기 프로젝트처럼 앞으로도 한단계 한단계 훌륭하게 과제를 진행하실 거라고 생각합니다. 이번에 다산이 코드를 구현하고 고민하시는 모습을 보면서 저또한 많이 배웠고 성장할 수 있었습니다. 매번 프로젝트를 진행할 때마다 많은 시간을 들여 노력하시는 걸 알고 있습니다. 그렇게 매 프로젝트마다 노력을 하시는게 쉬운일은 아니겠지만 앞으로도 많은 시간을 들여 노력하신다면 다산이 원하시는 만큼의 성취를 얻으실 것이라 생각합니다. 스스로가 만족하는 지식의 정도는 상대적이라고 생각합니다. 이번 스텝3을 진행하시는 모습을 처음부터 끝까지 보면서 그 짧은 순간에도 다산은 많은 성장을 하셨다고 생각합니다. 저뿐만이 아닌 누구에게나 배울점을 찾고 제가 가볍게 던지는 말들도 메모하시면서 들으셨던 게 기억에 남습니다. 짧지만 같은 팀을 하면서 다산에 대해 더 응원하게 되는 계기가 되었습니다. 앞으로의 개발자의 행보를 항상 응원합니다 다산.
- To. Zion
시온! 이번 일주일은 유독 더 짧게 느껴진 것 같습니다. 그만큼 시온과 함께 열심히 달려왔기 때문이겠죠?☺️
저에게 답을 떠먹여주시지 않고, 생각할 수 있는 기회와 공부할 수 있는 기회를 주셔서 정말 감사했어요!
그래서 더더욱 유익한 공부를 할 수 있었습니다!
시온과 함께하면서 단순히 스텝을 진행하는 것을 넘어 앞으로 어떻게 공부를 해야하는지도 알게되었던 시간이었습니다. 완벽한 저의 본보기..👍
부족했던 저를 이끌어주시는 모습을 보며 시온의 함께 가는 리더십에도 박수를!👍👏 진심으로 감사했어요 시온!!🥹 시온덕분에 보람차고 즐거웠습니다!!