#28 [feat] search UI and api by kongwoojin · Pull Request #29 · BCSDLab/JJBAKSA-ANDROID

kongwoojin · 2022-12-22T13:50:22Z

PR 개요

이슈 번호: #28

PR Checklist

Code convention을 잘 지켰나요?
테스트 코드를 추가했나요?

작업사항

작업사항의 상세한 설명

2022/12/22

검색 화면 완성하였습니다.
검색어 자동 완성은 아직 API에서 제공되지 않아 생략하였습니다.
커스텀 뷰(jjEditText)를 약간 수정하였습니다.
- imeOptions을 추가했습니다.
- onEditorActionListener를 추가했습니다.
검색 결과 화면은 아직 미완성입니다.
- 검색 결과에서 가게 이름, 주소 출력되도록 만들었습니다.
- 가게 사진이 아직 API에서 제공되지 않아, placeholder를 집어 넣었습니다.
- 가게 거리와 영업 시간 또한 마찬가지라 추후 추가할 예정입니다.
- 가게 아이템의 지도 아이콘 클릭 시 하단에 지도 옵션이 출력됩니다만 아직 연결은 되지 않았습니다.
- 검색 데이터를 받아올때 별도의 로딩 화면 디자인이 없어 일단은 생략하였습니다.

2022/12/26

검색 결과 화면에서 coil을 사용하도록 수정하였습니다.
네이버 지도와 카카오 맵으로 열기를 추가하였습니다.

2022/12/27

JjEditText의 imeOptions 쪽 오타를 수정했습니다.
API 에러 핸들링 코드를 추가했습니다.
UrlUtil을 추가했습니다.
남아있던 테스트 코드를 제거하였습니다.
Fragment에서 ui와 livedata 관련 코드를 onCreateView에서 onViewCreated로 옮겼습니다.

2022/12/28

google location library를 app module에서 data module로 옮겼습니다.
LocationRepository, LocationDataSource를 추가하였고, GPS 데이터를 담아 놓는 GPSData DTO를 새로 추가하였습니다.
기존의 getCurrentLocation()을 사용한 코드를 제거하고, requestLocationUpdates()를 사용하여 항상 새 위치를 요청하는 코드로 교체하였습니다.
기기의 GPS가 꺼져 있으면, 다이얼로그를 띄우도록 수정했습니다.

추가내용

작업 내용이 많아지다 보니 작업 현황 공유 차 pr을 미리 올렸습니다.
완성 후에 리뷰 요청하겠습니다!

앞으로 진행해야 하는 것

검색 결과 API 완성 후 결과 페이지 완성 (현재는 일부만 완성)
검색어 추천 API 완성 후 검색창에 연결
친구? API 완성 후 추가

Added Reviewers, Assignees, Labels, Milestone

… only once

wnehdals · 2022-12-26T00:25:06Z

app/src/main/res/layout/fragment_search_main.xml

+            android:layout_height="wrap_content"
+            android:layout_marginHorizontal="16dp"
+            android:layout_marginTop="24dp"
+            app:editText_imeOptions="actionSearch"


actionSearch 가 적용안된것 같습니다. EditText를 누르면 엔터키가 돋보기로 보여야하는데 체크표시로 보입니다.

eda2005 커밋에서 수정하였습니다!

wnehdals · 2022-12-26T00:29:02Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/search/SearchActivity.kt

+
+    private val requestPermission =
+        registerForActivityResult(ActivityResultContracts.RequestMultiplePermissions()) { results ->
+            for (result in results) {


82번라인에서 시작되어 for문을 타면 ACEESS_FINE_LOCATION, ACCESS_COARSR_LOCATION 둘중 하나만 있어도 GPS를 받아오는것 같습니다.

ACEESS_FINE_LOCATION은 어차피 ACCESS_COARSR_LOCATION와 같이 허용되고, ACCESS_COARSR_LOCATION만 허용되었을 경우, ACEESS_FINE_LOCATION를 다시 요청하도록 만들어서 둘 중 하나만 있어도 GPS를 받아오도록 만들었습니다!

wnehdals · 2022-12-26T00:34:25Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/search/SearchMainFragment.kt

+        requireActivity().onBackPressedDispatcher.addCallback(this, onBackPressedCallback)
+    }
+
+    override fun onCreateView(


onCreateView에서는 어떤 layout xml파일을 로드할지 결정하여 로드하여 화면에 보여주고 있습니다.
onViewCreate 함수 안은 layout xml파일이 이미 로드가 완료된 상태여서 거기서 binding.~ 관련된 변수들을 사용하는게 더 안전해보입니다.

58e2c5a 커밋에서 수정했습니다!

wnehdals · 2022-12-26T00:36:26Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/search/SearchActivity.kt

+            for (result in results) {
+                when (result.value) {
+                    true -> {
+                        getGPSLocation()


위치정보가 반드시 있어야 한다면 위치정보를 얻지못하였을 경우 액티비티를 종료하는 코드가 필요해보입니다.

155b8f0 커밋에서 수정했습니다!

wnehdals · 2022-12-26T00:42:35Z

data/src/main/java/com/jjbaksa/data/repository/ShopRepositoryImpl.kt

+    private val shopLocalDataSource: ShopLocalDataSource
+) : ShopRepository {
+    override suspend fun getTrendings(): TrendingResult? {
+        val resp = shopRemoteDataSource.getTrendings().body()


통신 오류가 났을 경우 api 응답 json 형태가 TrendingResult의 형태가 아닐경우에 대한 예외처리가 없습니다.
getSearchHistory
searchShop 함수도 동일하게 예외처리가 없습니다.
viewmodel의 runcatching으로 이동될 수 있지만 서버에서 보내주는 오류메시지가 필요할 수 있을것 같습니다.

api 호출 시 오류나도록 호출해보고 그 응답값을 커버할 수 있도록 코드를 작성하면 좋을 것 같습니다.

e277c5c 31f32b6 2945f9a cf27079 e2b8a03 커밋에서 수정했습니다!

wnehdals · 2022-12-26T08:52:20Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/search/SearchActivity.kt

+    }
+
+    @SuppressLint("MissingPermission")
+    fun getGPSLocation() {


GPS 해제하는 코드가 혹시 있나요? 없다면 GPS를 사용하지 않은곳으로 이동하면 해제하는 코드가 필요해 보입니다.

따로 callback을 만들어서 호출하는 것이 아니라 필요할 때 일회성으로 호출하는 코드라 따로 GPS를 해제하는 코드가 필요 없는것으로 알고 있었는데 혹시 제가 잘못 알고 있는것인가요?

https://developer.android.com/training/location/retrieve-current.html
마지막으로 저장된 위치를 가져오는것은 정확한 위치를 의미하지 않은것 같습니다.
서울에서 쩝쩝박사 앱으로 검색을 한 후 부산을 가서 GPS를 끄고 쩝쩝박사에서 검색한다면 마지막 위치인 서울 위치를 가젼올것 같습니다.
위치정보를 2~3번 호출받아 정확한 위치를 사용하는게 정확성이 높아보입니다.
https://developer.android.com/training/location/request-updates

c5d84bd 5f9b000 0bffd4a d937f0e ad0c0a2 5ec2028 05749b8 커밋에서 수정했습니다!

wnehdals · 2022-12-26T08:58:02Z

app/src/main/java/com/jjbaksa/jjbaksa/ui/search/SearchShopListFragment.kt

+
+        binding.buttonSearchShopListNaverMap.setOnClickListener {
+            try {
+                val naverMapUrl = "nmap://map?" +


UrlUtil 혹은 생각나는 이름으로 Object 파일 만들어서 makeUrl 이란 함수로 필요한 url을 만드는 Object가 있으면 좋아보입니다.

bb1446e 커밋에서 수정했습니다!

wnehdals · 2022-12-26T09:01:20Z

data/src/main/java/com/jjbaksa/data/model/shop/TrendingResp.kt

@@ -0,0 +1,5 @@
+package com.jjbaksa.data.model.shop
+
+data class TrendingResp(


BaseResp를 상속하여 구현하면 에러코드도 담을 수 있어 보입니다.

마찬가지로 e277c5c 31f32b6 2945f9a cf27079 e2b8a03 커밋에서 수정했습니다!

kongwoojin added 28 commits November 15, 2022 22:04

#28 [fix] Fix top padding when hasTitle is false

1bd3189

#28 [add] Add trending api

aeda745

#28 [add] Add ShopRepository and ShopDataSource

640b750

#28 [add] Add GetTrendingsUseCase

40cf587

#28 [add] Add search history database

ac064e4

#28 [add] Add search history methods in datasource

9281a09

#28 [add] Add search history methods in repository

535c4d8

#28 [add] Add AddSearchHistoryUseCase and GetSearchHistoryUseCase

dd9f9cd

#28 [add] Create SearchActivity and fragments

fa8c0ad

#28 [feat] Create search main page

e68a4c1

#28 [add] Add location permission request

9f8dbff

#28 [mod] Search immediately when click trending item

1cd0505

#28 [fix] Fix back press not work correctly on result page

a99b3eb

#28 [add] Add play-services-location library

b1adb5d

#28 [fix] Fix id naming convention

2862b39

#28 [add] Add get current GPS location code

b44dc96

#28 [add] Add shops api

3ff5453

#28 [add] Add SearchShopsUseCase

cc6b4ed

#28 [add] Create shop list fragment in search page

3a26b4e

#28 [add] Set default input type of jjEditText to text

21bae2d

#28 [add] Add imeOptions and OnEditorActionListener in jjEditText

6da306d

#28 [add] Set imeOptions of search edittext to actionSearch

7cc4f1b

#28 [add] Do search if soft keyboard enter pressed

51c9edd

#28 [fix] Fix shop list fragment not finished when backpressed called…

9e0fb90

… only once

#28 [chore] Fix ktlint in JjEditText

8bcd1f0

#28 [fix] Fix typo

5407a0e

#28 [add] Add naver map and kakao map intent

083baf4

#28 [add] Use coil to load shop image

1152d75

wnehdals reviewed Dec 26, 2022

View reviewed changes

kongwoojin added 11 commits December 26, 2022 19:15

#28 [fix] Fix imeOptions typo

eda2005

#28 [add] Add RespResult and ErrorType class

e277c5c

#28 [mod] Return trending data using RespResult

31f32b6

#28 [add] Show toast when error occurred

2945f9a

#28 [mod] Return shops data using RespResult

cf27079

#28 [add] Show toast when error occurred in shop list

e2b8a03

#28 [add] Add UrlUtil

bb1446e

#28 [chore] Remove unused test code

45e7990

#28 [chore] Move livedata observing and ui related code to onViewCreated

58e2c5a

#28 [chore] Fix ktlint

6b34b8e

#28 [add] Make activity finish if failed to get current location data

155b8f0

kongwoojin changed the title ~~Feature/#28 feat search UI and api~~ #28 [feat] search UI and api Dec 27, 2022

kongwoojin added 9 commits December 28, 2022 00:27

#28 [chore] Move google location library to data module

c5d84bd

#28 [add] Create LocationDataSource

5f9b000

#28 [add] Create LocationRepository

0bffd4a

#28 [add] Create GPSData

d937f0e

#28 [feat] Get real current location

ad0c0a2

#28 [add] Add GetCurrentLocationUseCase

5ec2028

#28 [add] Replace get location code to new one

05749b8

#28 [add] Show alert dialog if device loation turned off

4f889c2

#28 [fix] Don't call getGPSLocation() twice

37bb744

		@@ -0,0 +1,5 @@
		package com.jjbaksa.data.model.shop

		data class TrendingResp(

Conversation

kongwoojin commented Dec 22, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR 개요

PR Checklist

작업사항

작업사항의 상세한 설명

2022/12/22

2022/12/26

2022/12/27

2022/12/28

추가내용

앞으로 진행해야 하는 것

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

kongwoojin commented Dec 22, 2022 •

edited

Loading