zero Kim
    • Create new note
    • Create a note from template
      • Sharing URL Link copied
      • /edit
      • View mode
        • Edit mode
        • View mode
        • Book mode
        • Slide mode
        Edit mode View mode Book mode Slide mode
      • Customize slides
      • Note Permission
      • Read
        • Only me
        • Signed-in users
        • Everyone
        Only me Signed-in users Everyone
      • Write
        • Only me
        • Signed-in users
        • Everyone
        Only me Signed-in users Everyone
      • Engagement control Commenting, Suggest edit, Emoji Reply
    • Invite by email
      Invitee

      This note has no invitees

    • Publish Note

      Share your work with the world Congratulations! 🎉 Your note is out in the world Publish Note

      Your note will be visible on your profile and discoverable by anyone.
      Your note is now live.
      This note is visible on your profile and discoverable online.
      Everyone on the web can find and read all notes of this public team.
      See published notes
      Unpublish note
      Please check the box to agree to the Community Guidelines.
      View profile
    • Commenting
      Permission
      Disabled Forbidden Owners Signed-in users Everyone
    • Enable
    • Permission
      • Forbidden
      • Owners
      • Signed-in users
      • Everyone
    • Suggest edit
      Permission
      Disabled Forbidden Owners Signed-in users Everyone
    • Enable
    • Permission
      • Forbidden
      • Owners
      • Signed-in users
    • Emoji Reply
    • Enable
    • Versions and GitHub Sync
    • Note settings
    • Note Insights New
    • Engagement control
    • Make a copy
    • Transfer ownership
    • Delete this note
    • Save as template
    • Insert from template
    • Import from
      • Dropbox
      • Google Drive
      • Gist
      • Clipboard
    • Export to
      • Dropbox
      • Google Drive
      • Gist
    • Download
      • Markdown
      • HTML
      • Raw HTML
Menu Note settings Note Insights Versions and GitHub Sync Sharing URL Create Help
Create Create new note Create a note from template
Menu
Options
Engagement control Make a copy Transfer ownership Delete this note
Import from
Dropbox Google Drive Gist Clipboard
Export to
Dropbox Google Drive Gist
Download
Markdown HTML Raw HTML
Back
Sharing URL Link copied
/edit
View mode
  • Edit mode
  • View mode
  • Book mode
  • Slide mode
Edit mode View mode Book mode Slide mode
Customize slides
Note Permission
Read
Only me
  • Only me
  • Signed-in users
  • Everyone
Only me Signed-in users Everyone
Write
Only me
  • Only me
  • Signed-in users
  • Everyone
Only me Signed-in users Everyone
Engagement control Commenting, Suggest edit, Emoji Reply
  • Invite by email
    Invitee

    This note has no invitees

  • Publish Note

    Share your work with the world Congratulations! 🎉 Your note is out in the world Publish Note

    Your note will be visible on your profile and discoverable by anyone.
    Your note is now live.
    This note is visible on your profile and discoverable online.
    Everyone on the web can find and read all notes of this public team.
    See published notes
    Unpublish note
    Please check the box to agree to the Community Guidelines.
    View profile
    Engagement control
    Commenting
    Permission
    Disabled Forbidden Owners Signed-in users Everyone
    Enable
    Permission
    • Forbidden
    • Owners
    • Signed-in users
    • Everyone
    Suggest edit
    Permission
    Disabled Forbidden Owners Signed-in users Everyone
    Enable
    Permission
    • Forbidden
    • Owners
    • Signed-in users
    Emoji Reply
    Enable
    Import from Dropbox Google Drive Gist Clipboard
       Owned this note    Owned this note      
    Published Linked with GitHub
    • Any changes
      Be notified of any changes
    • Mention me
      Be notified of mention me
    • Unsubscribe
    ###### 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 시온! 이번 일주일은 유독 더 짧게 느껴진 것 같습니다. 그만큼 시온과 함께 열심히 달려왔기 때문이겠죠?☺️ 저에게 답을 떠먹여주시지 않고, 생각할 수 있는 기회와 공부할 수 있는 기회를 주셔서 정말 감사했어요! 그래서 더더욱 유익한 공부를 할 수 있었습니다! 시온과 함께하면서 단순히 스텝을 진행하는 것을 넘어 앞으로 어떻게 공부를 해야하는지도 알게되었던 시간이었습니다. 완벽한 저의 본보기..👍 부족했던 저를 이끌어주시는 모습을 보며 시온의 함께 가는 리더십에도 박수를!👍👏 진심으로 감사했어요 시온!!🥹 시온덕분에 보람차고 즐거웠습니다!!

    Import from clipboard

    Paste your markdown or webpage here...

    Advanced permission required

    Your current role can only read. Ask the system administrator to acquire write and comment permission.

    This team is disabled

    Sorry, this team is disabled. You can't edit this note.

    This note is locked

    Sorry, only owner can edit this note.

    Reach the limit

    Sorry, you've reached the max length this note can be.
    Please reduce the content or divide it to more notes, thank you!

    Import from Gist

    Import from Snippet

    or

    Export to Snippet

    Are you sure?

    Do you really want to delete this note?
    All users will lose their connection.

    Create a note from template

    Create a note from template

    Oops...
    This template has been removed or transferred.
    Upgrade
    All
    • All
    • Team
    No template.

    Create a template

    Upgrade

    Delete template

    Do you really want to delete this template?
    Turn this template into a regular note and keep its content, versions, and comments.

    This page need refresh

    You have an incompatible client version.
    Refresh to update.
    New version available!
    See releases notes here
    Refresh to enjoy new features.
    Your user state has changed.
    Refresh to load new user state.

    Sign in

    Forgot password

    or

    By clicking below, you agree to our terms of service.

    Sign in via Facebook Sign in via Twitter Sign in via GitHub Sign in via Dropbox Sign in with Wallet
    Wallet ( )
    Connect another wallet

    New to HackMD? Sign up

    Help

    • English
    • 中文
    • Français
    • Deutsch
    • 日本語
    • Español
    • Català
    • Ελληνικά
    • Português
    • italiano
    • Türkçe
    • Русский
    • Nederlands
    • hrvatski jezik
    • język polski
    • Українська
    • हिन्दी
    • svenska
    • Esperanto
    • dansk

    Documents

    Help & Tutorial

    How to use Book mode

    Slide Example

    API Docs

    Edit in VSCode

    Install browser extension

    Contacts

    Feedback

    Discord

    Send us email

    Resources

    Releases

    Pricing

    Blog

    Policy

    Terms

    Privacy

    Cheatsheet

    Syntax Example Reference
    # Header Header 基本排版
    - Unordered List
    • Unordered List
    1. Ordered List
    1. Ordered List
    - [ ] Todo List
    • Todo List
    > Blockquote
    Blockquote
    **Bold font** Bold font
    *Italics font* Italics font
    ~~Strikethrough~~ Strikethrough
    19^th^ 19th
    H~2~O H2O
    ++Inserted text++ Inserted text
    ==Marked text== Marked text
    [link text](https:// "title") Link
    ![image alt](https:// "title") Image
    `Code` Code 在筆記中貼入程式碼
    ```javascript
    var i = 0;
    ```
    var i = 0;
    :smile: :smile: Emoji list
    {%youtube youtube_id %} Externals
    $L^aT_eX$ LaTeX
    :::info
    This is a alert area.
    :::

    This is a alert area.

    Versions and GitHub Sync
    Get Full History Access

    • Edit version name
    • Delete

    revision author avatar     named on  

    More Less

    Note content is identical to the latest version.
    Compare
      Choose a version
      No search result
      Version not found
    Sign in to link this note to GitHub
    Learn more
    This note is not linked with GitHub
     

    Feedback

    Submission failed, please try again

    Thanks for your support.

    On a scale of 0-10, how likely is it that you would recommend HackMD to your friends, family or business associates?

    Please give us some advice and help us improve HackMD.

     

    Thanks for your feedback

    Remove version name

    Do you want to remove this version name and description?

    Transfer ownership

    Transfer to
      Warning: is a public team. If you transfer note to this team, everyone on the web can find and read this note.

        Link with GitHub

        Please authorize HackMD on GitHub
        • Please sign in to GitHub and install the HackMD app on your GitHub repo.
        • HackMD links with GitHub through a GitHub App. You can choose which repo to install our App.
        Learn more  Sign in to GitHub

        Push the note to GitHub Push to GitHub Pull a file from GitHub

          Authorize again
         

        Choose which file to push to

        Select repo
        Refresh Authorize more repos
        Select branch
        Select file
        Select branch
        Choose version(s) to push
        • Save a new version and push
        • Choose from existing versions
        Include title and tags
        Available push count

        Pull from GitHub

         
        File from GitHub
        File from HackMD

        GitHub Link Settings

        File linked

        Linked by
        File path
        Last synced branch
        Available push count

        Danger Zone

        Unlink
        You will no longer receive notification when GitHub file changes after unlink.

        Syncing

        Push failed

        Push successfully