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

dahyee step1 #3

Open
wants to merge 4 commits into
base: kimdahyee
Choose a base branch
from
Open

dahyee step1 #3

wants to merge 4 commits into from

Conversation

kimdahyee
Copy link

@kimdahyee kimdahyee commented Nov 4, 2021

Description

  • 구현 내용
  1. 검색 창에 user id를 입력하고 search 버튼 클릭시 검색 결과가 아래 recyclerview에 출력
  2. recyclerview item 클릭 시 해당 user의 상세 정보 보기 화면으로 이동
  3. 상세보기 화면에 있는 user의 github 주소 클릭시 해당 브라우저로 이동 가능
  • 어려웠던 점
  1. 데이터바인딩, 라이브데이터, 엠비비엠을 한 번에 처음 구현을 해보려니 정신이 혼미했음
  2. (뷰는 뷰모델과 모델을 모르고 뷰모델은 뷰를 참조 안해) 이걸 계속 생각하면서 구현하려 노력은 많이 햇음
  3. user의 검색 결과로 보여지는 recyclerview에 clickListener를 적용하느라 너무 머리가 아팠음
  4. method를 view와 하나하나 연결해주는 방법으로 구현을 하려 하였으나 잘 작동이 되지 않아
  5. object를 직접 넘겨주는 방법으로 시도를 해서 성공을 하긴 햇다고 생각하나 내가 한 방식이 오브젝트를 직접 넘겨주는 방식이 맞는건지 확신은 없음 ....
  6. 유저 검색 화면을 위해서 user의 profile imageurl를 가져올때는 null이 안들어왔는데 상세보기 화면에서는 null이 자꾸 들어와서 당황하였으나 잘 해결함
  7. 이미지를 로딩하기 위해서 기능별 BindingAdapter를 구현해줬는데 이거 굳이 두개 만들어야 되는건가 한번에 UserBindingAdapter로 묶어서 처리하는건 안되나 궁금함
  8. 근데 adapter는 mvvm 중에 view인거임 ??????????? 리스너 구현할 때 여기서 해줘야 되는게 맞나, (뷰는 뷰모델과 모델을 모르고 뷰모델은 뷰를 참조 안해) 이걸 어기는건 아닐까하고 고민을 아주 많이 함 근데도 잘 모르겠음
  • 느낀점
  1. 남들이었으면 더 빨리 잘 했을텐데 역시 난 느리고 멍청하구나를 또 한 번 깨달았음
  2. 그래도 새로운걸 배우고 적용해볼 수 있어서 재밌었고
  3. 0.5걸음정도 성장한거 같아서 뿌듯함
  4. 근데 중간엔 진짜 노트북 부셔버리고 싶었음
  5. 아무튼 이런 기회주는 진수에게 너무 감사, 함께 공부할 수 있게 해주는 수정도 사랑함
    6.ㅃ2

Screenshots or Viedeo

Link to the related issue(s)

@kimdahyee kimdahyee self-assigned this Nov 4, 2021
@kimdahyee kimdahyee changed the title Kimdahyee step1 깃헙 유저 검색 기능 구현 dahyee step1 Jan 25, 2022
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