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

[Refact] #250 - 리팩토링을 위한 폴더 구조 변경사항입니다 #251

Merged
merged 3 commits into from
May 3, 2024

Conversation

seongmin221
Copy link
Contributor

🌁 Background

클린 아키텍쳐를 적용하는 단계에서 폴더 구조의 변경이 필요해서 폴더링 구조를 변경했습니다

👩‍💻 Contents

  • 클린 아키텍쳐 적용 전 설계한 프로젝트 폴더링 구조을 토대로 구성하였습니다

  • 지금 돌리면 빌드가 안될텐데 ... lint가 또 발목을 잡네요 하하ㅏ 기존에 최상위에 있던 .swiftlint.yml 파일을 Resource 폴더로 옮겼는데, lint 특성상 본인이 포함되어 있는 폴더 디렉토리와 그 하위 폴더까지 적용되더라구요

e.g. GEON-PPANG-iOS 에 위치해 있으면 파일 전체에 적용 / GEON-PPANG-iOS/Resource에 위치해 있으면 Resource 폴더 내부의 것들만 적용

그래서 이건 제안인데 ... 그냥 lint 안쓰는건 ... 어떨지..?

✅ Testing

📝 Review Note

📣 Related Issue

📬 Reference

Copy link
Contributor

@jeongdung-eo jeongdung-eo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

폴더링하느라 고생하셨슴다:)
coordiantor 관련 폴더를 따로 분리해서 관리하는건 어떨까여?
DI를 적용해서 coordinator를 구현하고, coordinator에서 직접 vc를 생성하지 않고 factory pattern을 적용해서 vc를 생성하기로 했던 것 같은데 coordinator 파일들이 분리되어 있는 것 보다는 coordinator 폴더를 만들어서 그 안에서 coordinator 관련 로직들을 관리하는게 좋지 않을까 생각함다
이렇게 하면 프로젝트의 구조를 더욱 명확하게 파악할 수 있을 것 같습니당

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분은 이야기 못했던 것 같은디 이미지나 컬러 에셋에서도 변경 사항이 있나여?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다 !!!

저도 DIContainer는 일단은 미뤄두려고 했는데, 그냥 이참에 공부하고 적용해보려구요 ㅋㅎㅋ
팩토리패턴이랑 DI랑 다 해봅시다 !!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그리고 이미지랑 컬러 에셋 등은 네이밍 변경을 할 필요는 있어 보여유 ...
그래서 그냥 싹 함 갈아엎을까 하고 있긴 합니다 !
요 PR 머지되면 최우선적으로 제가 해두려고 했슴다 !

@seongmin221 seongmin221 merged commit 47665de into develop May 3, 2024
1 check failed
@seongmin221 seongmin221 changed the title [Refactor] #250 - 리팩토링을 위한 폴더 구조 변경사항입니다 [Refact] #250 - 리팩토링을 위한 폴더 구조 변경사항입니다 May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refact] 리팩토링을 위한 초기 폴더링
2 participants