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

♻️ [Refactor] 화면전환 코드 리펙토링 #400

Merged
merged 4 commits into from
Jul 11, 2022

Conversation

hwangJi-dev
Copy link
Member

@hwangJi-dev hwangJi-dev commented Jul 10, 2022

🍎 관련 이슈

closed #395

🍎 변경 사항 및 이유

중복되는 뷰 컨트롤러 인스턴스 생성 코드를 효율적으로 사용하기 위해 Navigator를 만들었습니다.

🍎 PR Point

self.navigator?.instantiateVC(destinationViewControllerType: ReviewWriteVC.self, 
useStoryboard: true, storyboardName: "ReviewWriteSB", naviType: .present, modalPresentationStyle: .fullScreen) 
{ reviewWriteVC in }

💡 화면전환 코드를 사용하고 싶은 VC에서 navigator의 instantiateVC 함수를 호출해주어 사용할 수 있습니다.
📌 클로저를 통해 nextVC의 프로퍼티에 넘겨줄 값이 있다면 후행 클로저를 활용합니다.

📸 ScreenShot

@hwangJi-dev hwangJi-dev added 💡 Refactor 코드 리펙토링 시 사용합니다. ✨🍕 Jieun 🐰✨ 지은이 연 pr, issue에 사용 labels Jul 10, 2022
@hwangJi-dev hwangJi-dev requested a review from dev-jungbin July 10, 2022 15:09
@hwangJi-dev hwangJi-dev self-assigned this Jul 10, 2022
Copy link
Member

@jane1choi jane1choi left a comment

Choose a reason for hiding this comment

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

와 대박,,,리펙토링 수고해써,,,
한가지 궁금한 점은 [weak self]를 써준 곳도 있고 안써준 곳도 있는데
안써준 곳도 메모리 릭이 발생하지 않는건가요??

} else {
if let vc = String(describing: vcType.self).getViewController() {
destinationVC = vc
}
Copy link
Member

Choose a reason for hiding this comment

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

와 여기 제네릭 이용해서 한거 대박이다....

@hwangJi-dev
Copy link
Member Author

와 대박,,,리펙토링 수고해써,,, 한가지 궁금한 점은 [weak self]를 써준 곳도 있고 안써준 곳도 있는데 안써준 곳도 메모리 릭이 발생하지 않는건가요??

일단은 급한 곳만 weak self 써서 메모리 릭 방지했는데 다른 곳들은 앞으로 코드 리펙토링하면서 천천히 바꾸려고 합니다~

@jane1choi
Copy link
Member

와 대박,,,리펙토링 수고해써,,, 한가지 궁금한 점은 [weak self]를 써준 곳도 있고 안써준 곳도 있는데 안써준 곳도 메모리 릭이 발생하지 않는건가요??

일단은 급한 곳만 weak self 써서 메모리 릭 방지했는데 다른 곳들은 앞으로 코드 리펙토링하면서 천천히 바꾸려고 합니다~

좋습니다!!! 고생해써요~~

@hwangJi-dev hwangJi-dev merged commit 0e27800 into develop Jul 11, 2022
@hwangJi-dev hwangJi-dev deleted the refactor/#395-ScreenChange branch July 11, 2022 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨🍕 Jieun 🐰✨ 지은이 연 pr, issue에 사용 💡 Refactor 코드 리펙토링 시 사용합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

♻️ [Refactor] 화면전환 코드 리펙토링
2 participants