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

  • gitignore 에는 /Pods 만 넣어주시면 됩니다. Podfile 과 Podfile.lock 은 필요
  • gettable able은 protocol 에서 기능성을 명시해주는 용어입니다. 함수 명으로는 get, post 를 사용해주세요
  • getSearchResult 에서 실패의 경우도 Controller 에게 전달해야 합니다. Completion에 NetworkResult 를 전달해주세요
  • SearchObject 는 ResponseArray 의 형태로 추상화가 가능합니다. 그렇게 변경해주세요
  • gsno 대신 optionalString ?? ""를 사용해주세요
  • userList, tempList 모델 배열을 두개 사용하지 말고 하나로 해결해보세요
  • LoadMore 부분을 추가해보세요
  • Extension 은 기능별로 다른 파일로 구성해주세요. Extensions.swift처럼 불명확한 이름은 좋지 않습니다.
  • dequeueReusableCell 는 force Cast 하지말고 추상화해보기
  • cellForRowAt 에서 out of range 방지하기
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