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

Review-1 #1

Open
OhKanghoon opened this issue Feb 11, 2019 · 0 comments
Open

Review-1 #1

OhKanghoon opened this issue Feb 11, 2019 · 0 comments

Comments

@OhKanghoon
Copy link
Member

  • GithubSearchViewController+Extension 이름은 적절하지 않은 것 같아요. Extension을 파일로 분리하고 싶다면 각 기능별로 이름이 명확하게 해주세요
  • token 은 네트워킹 부분에서 관리해주세요
  • 네트워킹 레이어를 분리해주세요 (RequestManager / Api / Service / Parsing)
  • dequeueReusableCell force casting 하는 부분을 추상화 해주세요
  • cellForRowAt에서 out of range 발생을 예방해주세요
  • cellForRowAt에서의 구문을 줄여주세요 (cell.configure(user) 형태를 사용하세요)
  • loadmore 가 지속적으로 일어나지 않는 부분 수정해주세요
  • spinner를 footer에 붙이는 건 willDisplay에서 매번 일어나면 안됩니다. initial setting 하게 해주세요
  • appenddisplaySearchUsersItems를 따로 들고 있지 말고 appenddisplaySearchUsersItems 함수 사용시 넘겨주세요 => 전반적인 모델 배열 사용을 줄여주세요
  • willDisplayCell 에서 loadmore를 하게 되면 중복호출될 위험이 있습니다. 해당 부분을 해결해주세요
  • Extension 을 기능별로 다른 파일로 만들고 이름을 명확하게 해주세요 ExtesnionControl 네이밍은 좋지 않습니다.
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

No branches or pull requests

1 participant