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

Horizontal Pager 적용에 관해... #143

Open
6 of 7 tasks
likppi10 opened this issue Oct 25, 2023 · 1 comment
Open
6 of 7 tasks

Horizontal Pager 적용에 관해... #143

likppi10 opened this issue Oct 25, 2023 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@likppi10
Copy link
Collaborator

likppi10 commented Oct 25, 2023

유지 되어야하는 기존의 기능

  • 앱 시작 시, 최근 반다라트 띄워주기
  • 셀 수정 시, 변경사항 띄워주기
  • 반다라트 추가 시, 그 반다라트 띄워주기
  • 반다라트 삭제 시, 첫번째 반다라트 띄워주기
  • 완료 페이지 갔다 올 시, 그 표 보여주기
  • 스와이프 시, 해당 반다라트가 마지막 반다라트로 인지하기

현재 구현

현재 시나리오

  • 추가된 변수
    • bandalartIdx : 최근 반다라트를 찾아가기 위한 반다라트 리스트의 idx
    • recentBandalartKey : 앱 시작 시, 최근 반다라트를 찾아갈 때는 스와이프 하는 과정에서 지나가는 표들은 최근 반다라트로 지정하지 않기 위한 비교 용도로 사용함
      • 이를 사용하지 않으면 최근 반다라트로 이동하다가 지나가던 반다라트를 최근 반다라트로 지정해버리기 때문에 포커스가 왔다갔다 함
  1. recentBandalartKey에 먼저 최근 반다라트 키값을 넣는다.
  2. 앱이 시작되면 getList를 자동으로 불러온다.
  3. getList로 비교할 목록까지 존재하게 되면 HomeScreen 224 줄에 있는 LaunchedEffect로 최근 반다라트와 일치하는 idx를 찾고 bandalartIdx에 저장한다.
  4. HomeScreen 202 줄에 있는 LaunchedEffect가 작동하여 Pager를 넘긴다.
  5. HomeScreen 210 줄에 있는 LaunchedEffect가 페이지가 바뀌었기 때문에 작동한다.
  6. 잘 찾아간다..

문제점

  1. 반다라트 삭제 시 첫번째 반다라트가 아닌, bandalartIdx에 해당하는 반다라트를 띄운다. 근데 표는 첫번째 반다라트임 (잘 불러오긴한단 뜻)
  2. 반다라트 추가 시 추가한 반다라트가 아닌, bandalartIdx에 해당하는 반다라트를 띄운다. 근데 표는 추가한 반다라트임 (잘 불러오긴한단 뜻)
  3. 반다라트 완료 후 복귀하면, bandalartIdx의 인디케이터와 recentBandalartKey의 표를 띄운다. (완료한 표가 아닌)
  4. 로그를 여기저기 찍어보면 확인하겠지만, 코드가 이쁘지 않아서 쓸데 없이 도는 루프가 존재하는 듯 하다.

현재 유지되는 기능

  • 앱 시작 시, 최근 반다라트 띄워주기
  • 셀 수정 시, 변경사항 띄워주기
  • 반다라트 추가 시, 그 반다라트 띄워주기
  • 반다라트 삭제 시, 첫번째 반다라트 띄워주기
  • 완료 페이지 갔다 올 시, 그 표 보여주기
  • 스와이프 시, 해당 반다라트가 마지막 반다라트로 인지하기
  • 첫 실행 시, 표 삭제 다이얼로그 부재
@likppi10 likppi10 added the documentation Improvements or additions to documentation label Oct 25, 2023
@likppi10 likppi10 self-assigned this Oct 25, 2023
@easyhooon
Copy link
Collaborator

로그 찍을 때 Timber 로 찍으면 tag 가 알아서 해당 클래스로 지정되서 편해~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants