Draft
Conversation
Author
|
작성을 하고 보니 Clean swift의 경우 더 분리할 필요가 있어서 (예를 들면 Router의 Routing, Navigation, Passing data..) 추가하도록 하겠습니다 |
Contributor
|
Public 과 Private block으로 나누어지고 있는데, 어떤 기대효과를 위해서 나눈건지 의견을 듣고 싶어요. 저도 나누는 것을 바라고 있는데, 저같은 경우는 외부객체가 사용할 변수와 메서드가 뭔지 한 눈에 빨리 구별하고 싶어서거든요. |
Author
|
@gookhee 제가 생각한 public과 private의 구분은 잘못 구현된 접근제어를 방지하기 위함이었어요. private으로 구현해야 하는데 무의식적으로 public으로 구현하여 외부객체가 사용할 수 있는 경우가 많다고 생각해서(저만..그런가요? ㅎ) block을 구분함으로서 구현할 때 더 명확하게 접근제어를 나눌 수 있지 않을까 하는 기대효과를 얻을 수 있다고 생각합니다 |
Contributor
|
@jhseo 저도 그래요! |
Contributor
|
적용하면 보기 좋겠네요! 또 이전에 있던 클래스나 구조체는 많이 수정해야겠네요 ㅎㅎ |
Contributor
|
논리적으로 잘 나누려면 일단 public, private부터 잘 생각해서 정해야 할 것 같고, 전에 @gookhee 님이 말씀하신 오브젝트 책 빨리 읽어봐야겠네요... 😅 |
|
당연한 이야기이긴 한데, Protocol 구현체를 따로 쪼개는 내용도 들어가면 좋을 것 같습니다! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
논리적으로 block을 나누는 기준에 대해서 정리해보았습니다.
많은 의견 부탁드립니다~