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 13, 2019 · 0 comments
Open

Review-1 #1

OhKanghoon opened this issue Feb 13, 2019 · 0 comments

Comments

@OhKanghoon
Copy link
Member

  • ResponseBody, items 의 이름은 부적절합니다. 추상화 대상이 아닌 Model 이기 때문에 User 라는 이름을 사용해주세요 ( 배열 단위를 추상화한 ResponseArray 는 적절합니다)
  • NetworkLayer 를 (API / Manager(Request) / Parsing)으로 분리해주세요 지금은 SearchService 안에 모든 것이 있습니다. (AlamofireObjectMapper 를 사용하셔서 파싱은 따로 분리 안해도 될 것 같습니다)
  • Podfile, Podfile.lock 은 gitignore 처리하면 안됩니다.
  • ViewViewController 를 분리해주세요. 지금처럼 폴더링 하려면 Module 이라는 이름이 적절합니다.
  • willDisplay Cell에서 loadmore 를 처리하면 여러번 호출될 가능성이 있습니다. 이 부분을 처리해주세요
  • willDisplay Cell에서 spinner를 매번 붙여줄 필요가 없습니다. 초기 설정에서 처리해주세요
  • cellforrowat에서 네트워킹 콜을 하게 되면 스크롤 할 때마다 지속적으로 네트워킹 처리를 하게 됩니다. 분리해주세요
  • != nil 비교 후 force cast 하지 말고 if let 이나 guard let 을 사용해주세요
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