Skip to content

Comments

셀 재사용 문제#3

Open
mohok wants to merge 3 commits intoyk1028:masterfrom
mohok:jeonghwa
Open

셀 재사용 문제#3
mohok wants to merge 3 commits intoyk1028:masterfrom
mohok:jeonghwa

Conversation

@mohok
Copy link

@mohok mohok commented Jan 20, 2019

여기에 코멘트를 달 수 있었네요.ㅠㅠ 커밋 메세지를 확인해주세요.

  1. 함수 이름은 lowerCamelCase를 따릅니다. 무의식적으로 몇개 대분자로 쓰신것 같아요.
  2. 공통 코드 분리는 겹치는 코드가 많아보여서 해봤어요. 사실 저렇게 하면 안되는거 같은데 좋은 방법을 알게되면 다시 풀리퀘스트 하겠습니다.
  3. 화면을 위아래로 빠르게 왔다갔다할때 셀이 재사용되면서 인덱스에 맞게 들어가지 않는 문제가 보입니다(빠르게 위아래로 왔다갔다해보세요). prepareForReuse 메소드를 사용해 재사용되기 전에 초기화 시켜주는 게 쉬운 문제 해결법 같습니다!

@mohok mohok changed the title Jeonghwa 셀 재사용 문제 Jan 20, 2019
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.

1 participant