🔀 :: (#515) 공통 팝업을 Interface로 뺍니다.

[yongbeomkwak]

💡 배경 및 개요

텍스트 팝업이 많은 곡에서 사용 중인데 Feature에 직접 접근해서 VC를 만듭니다.

Resolves: #515

📃 작업내용

• TextPopUpVC를 사용 중인 곳을 팩토리를 통해 Feature가 아닌 interface로 접근하게 바꿨습니다.

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!
일단 교체만 시켜 놓은 상태, 다른 화면들 구조를 리팩하면서 분명히 수정이 필요함
또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

• 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
• 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
• 작업한 코드가 정상적으로 동작하나요?
• Merge 대상 브랜치가 올바른가요?
• PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

• 교체만 해놨습니다.

[github-actions]

✅ Assign 자동 지정을 성공했어요!

@yongbeomkwak

[youn9k]

팩토리패턴을 써본적이 없어서 기존 코드랑 비교했을 때 어떤 장점이 있나요?

[yongbeomkwak]

다양한 퓨처에서 쓰면 퓨처를 직접 접근하지 않고 interface에 접근하게 해서 의존도를 떨어트리는데요
이 때 VC에 직접 접근하는게 아니라 interface에서 프로토콜에 접근해서 생성하게 하는거에여