Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[iOS] 전반적인 리팩토링 #355

Closed
wants to merge 6 commits into from

Conversation

SwiftyJunnos
Copy link
Member

@SwiftyJunnos SwiftyJunnos commented Jan 29, 2024

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • MSPersistentStorage 리팩토링

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

1️⃣ KeychainStorage

MSKeychainStorage의 로직이 PersistentStorage와 동일한데도 불구하고 MSPersistentStorage 프로토콜을 따르지 않고 있었습니다.
따라서 해당 프로토콜을 채택하여 사용하도록 수정했습니다.

또한 MSKeychainStorage 대신 FileManagerStorageCoreDataStorage와 동일하게 MS prefix를 제거한 KeychainStorage로 네이밍을 수정하였습니다.

2️⃣ RecordingJourneyStorage

  1. RecordingJourneyStorageMSPersistentStorage를 채택하지 않고 사용된다는 점
  2. 저장소의 역할 보다는 비즈니스 로직을 수행한다는 점
    의 사유로

RecordingJourneyStorage의 네이밍을 RecordingJourneyService로 변경해주었습니다.

📸 스크린샷

@SwiftyJunnos SwiftyJunnos added the ♻️ 리팩토링 코드 동작 방식 변경, 리팩토링 label Jan 29, 2024
@SwiftyJunnos SwiftyJunnos self-assigned this Jan 29, 2024
@SwiftyJunnos SwiftyJunnos deleted the iOS/story/Refactorings#354 branch April 17, 2024 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩토링 코드 동작 방식 변경, 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant