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

Grouping extension 정리 #3

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Grouping extension 정리 #3

wants to merge 3 commits into from

Conversation

jhseo
Copy link

@jhseo jhseo commented Apr 24, 2020

논리적으로 block을 나누는 기준에 대해서 정리해보았습니다.

많은 의견 부탁드립니다~

@jhseo
Copy link
Author

jhseo commented Apr 27, 2020

작성을 하고 보니 Clean swift의 경우 더 분리할 필요가 있어서 (예를 들면 Router의 Routing, Navigation, Passing data..) 추가하도록 하겠습니다

@gookhee
Copy link
Contributor

gookhee commented Apr 27, 2020

Public 과 Private block으로 나누어지고 있는데, 어떤 기대효과를 위해서 나눈건지 의견을 듣고 싶어요. 저도 나누는 것을 바라고 있는데, 저같은 경우는 외부객체가 사용할 변수와 메서드가 뭔지 한 눈에 빨리 구별하고 싶어서거든요.

@jhseo
Copy link
Author

jhseo commented Apr 27, 2020

@gookhee 제가 생각한 public과 private의 구분은 잘못 구현된 접근제어를 방지하기 위함이었어요. private으로 구현해야 하는데 무의식적으로 public으로 구현하여 외부객체가 사용할 수 있는 경우가 많다고 생각해서(저만..그런가요? ㅎ) block을 구분함으로서 구현할 때 더 명확하게 접근제어를 나눌 수 있지 않을까 하는 기대효과를 얻을 수 있다고 생각합니다

@gookhee
Copy link
Contributor

gookhee commented Apr 27, 2020

@jhseo 저도 그래요!

@agiletalk
Copy link
Contributor

적용하면 보기 좋겠네요! 또 이전에 있던 클래스나 구조체는 많이 수정해야겠네요 ㅎㅎ

@agiletalk
Copy link
Contributor

agiletalk commented Apr 28, 2020

논리적으로 잘 나누려면 일단 public, private부터 잘 생각해서 정해야 할 것 같고, 전에 @gookhee 님이 말씀하신 오브젝트 책 빨리 읽어봐야겠네요... 😅

@deholic
Copy link

deholic commented Mar 26, 2021

당연한 이야기이긴 한데, Protocol 구현체를 따로 쪼개는 내용도 들어가면 좋을 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants