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

강원대_Android_이아림_3주차과제_STEP2 #74

Open
wants to merge 26 commits into
base: arieum
Choose a base branch
from

Conversation

arieum
Copy link

@arieum arieum commented Jul 11, 2024

아직 STEP0, STEP1 피드백을 못받았지만...! 슬쩍 STEP2 PR도 보내봅니다

  • 윈도우 기반에서 지도API 가져오는 과정이 너무 험악했습니다.. 이번 프로젝트 재밌지만.. 호락호락하진 않은 느낌입니다..!!
  • ListAdapter로 상속받는 거 이제야 구현해봤는데 왜 이걸 이제야 했는가..! 싶을 정도로 RecyclerviewAdapter보다 좋았습니다..! notify()보다 더 좋은 submitlist() 메서드가 있다니..! 😮 검색결과Recyclerview는 LIstAdapter 상속으로 바꾸고 notify()를 submitlist로 바꾸는데 수월했지만 검색기록TabRecyclerview는 스파게티코드가 많이 섞여있어 섣불리 못바꾸었습니다..!!

app/src/main/java/campus/tech/kakao/map/KakaoResponse.kt Outdated Show resolved Hide resolved
Comment on lines +46 to +48
placeRepository = PlaceRepository(this)
placeRepository.reset()
placeList = placeRepository.insertInitialData()
Copy link

Choose a reason for hiding this comment

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

아직 Repository들을 View에서 직접 세팅하고 사용하고 계신데 ViewModel로 옮기셔야 합니다

app/src/main/java/campus/tech/kakao/map/MainActivity.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/MainActivity.kt Outdated Show resolved Hide resolved
}
recyclerView.adapter = resultAdapter
recyclerView.layoutManager = LinearLayoutManager(this)
resultAdapter.submitList(placeList)
Copy link

Choose a reason for hiding this comment

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

ViewModel을 적용하기가 조금 어렵다고 하셨는데 이번주까지만 MVC로 진행하시고 이번에 Step3 작업 하신 이후에는 조금씩이나마 ViewModel을 사용해서 데이터 흐름이 LocalDB -> View로 오는 흐름을 만들어보세요
실무에서는 다 이런 방식으로 설계하기 때문에 결국 부딪혀야 합니다. 강사님과 멘토 믿고 부족해도 괜찮으니 시도해보세요!

app/src/main/java/campus/tech/kakao/map/PlaceRepository.kt Outdated Show resolved Hide resolved
app/src/main/java/campus/tech/kakao/map/PlaceRepository.kt Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_main.xml Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_main.xml Outdated Show resolved Hide resolved
@mkSpace
Copy link

mkSpace commented Jul 14, 2024

아림님 키워드 검색 부분이 구현되지 않은것 같은데 맞나요??

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.

2 participants