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

[Fix] #209 - 데이터를 계속 append하여 셀이 증식하는 문제 해결 #218

Merged
merged 2 commits into from
Jan 29, 2022

Conversation

L-j-h-c
Copy link
Member

@L-j-h-c L-j-h-c commented Jan 28, 2022

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 데이터를 append 하기 전에 removeAll()을 해줘서 값이 계속 늘어나는 문제 해결

📌 참고 사항

  • API 통신을 하기 전에 List에 있는 정보들을 모두 삭제해주는 방식으로 문제를 해결했습니다.
  • 생각보다 간단하게 해결할 수 있는 문제였네염...
  • 추가적인 방법으로 append를 할 때 List에 이미 값이 들어있는지 확인하고 없는 경우에만 append를 해줄 수 있을 것 같아요. 어떤 방법이 더 메모리 효율적일지는 모르겠서요! help...
  • 다시 생각해보니까 바로 위 방법으로 하면 viewWillappear에서 API통신하는 의미가 없어지네요.. 머쓱..

📮 관련 이슈

데이터를 계속 append하여 셀이 증식하는 문제 해결
@L-j-h-c L-j-h-c added 🧚‍♀️Devil『쥬노』 뀨의 의지 공식후계자입니다 아몰랑 P1 / Priority High labels Jan 28, 2022
@L-j-h-c L-j-h-c self-assigned this Jan 28, 2022
Copy link
Member

@yangsubinn yangsubinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • API 통신을 하기 전에 List에 있는 정보들을 모두 삭제해주는 방식으로 문제를 해결했습니다.
  • 생각보다 간단하게 해결할 수 있는 문제였네염...
  • 추가적인 방법으로 append를 할 때 List에 이미 값이 들어있는지 확인하고 없는 경우에만 append를 해줄 수 있을 것 같아요. 어떤 방법이 더 메모리 효율적일지는 모르겠서요! help...
  • 다시 생각해보니까 바로 위 방법으로 하면 viewWillappear에서 API통신하는 의미가 없어지네요.. 머쓱..

고생하셨슴다⚡️
움 저도 이 문제에 대해서 removeAll을 해주는 방법으로 생각하구 있었어여!
append할 때 list에 이미 값이 들어있는지 확인하고 없는 경우에만 append 해주는 경우에는
이미 값은 있지만 추가적으로 새로운 값이 들어오는 경우 그것만 따로 분리해서 append 해주는 것이 번거로울 것 같아서! removeAll 도 괜찮은 방법이 되지 않으까.. 생각합니다 메모뤼...에 어떤 방식이 더 효율적일지는...😢😵‍💫

Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍LZTM~

@hyun99999
Copy link
Member

우리가 구현한 무한스크롤 원리대로면 계속 append 를 해주어야하고 수빈이 말대로 분기처리가 꽤 까다로워보이네요! 대신우리는 생명주기를 사용해서 충분히 의미있도록 짠것 같습니당!
고생했어여~ 두분다 고민해보느라 더 좋은 코드도 있을거에요! 앞으로 더 많이 공유해보자구여

@L-j-h-c L-j-h-c merged commit 85ba0ec into TeamSparker:develop Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧚‍♀️Devil『쥬노』 뀨의 의지 공식후계자입니다 아몰랑 P1 / Priority High
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 보관함 리스트 viewwillappear append 관련 문제 해결하기
3 participants