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

막차 로직 오류 수정 및 리팩토링 필요 #111

Open
DoTheBestMayB opened this issue May 25, 2023 · 1 comment
Open

막차 로직 오류 수정 및 리팩토링 필요 #111

DoTheBestMayB opened this issue May 25, 2023 · 1 comment

Comments

@DoTheBestMayB
Copy link
Collaborator

막차 로직 오류 수정

막차 시간 계산 로직이 아래와 같이 되어 있습니다.
최초 탑승지의 막차 시간 - 최초 탑승지의 소요 시간

막차1
막차2

이것은 원래의 막차 시간 로직과 다르기 때문에 수정이 필요합니다.

Untitled

막차 관련 리팩토링

  1. 막차 계산에 사용되는 GetRouteUseCaseImplGetLastTransportTimeUseCaseImpl에 구현되어 있는 코드가 매우 복잡합니다. ( A 함수에서 B 함수 호출, B 함수에서 C 함수 호출, C 함수에서 D 함수 호출...)
    이러한 depth를 줄이는 방향으로 리팩토링이 필요합니다.
  2. 현재 서울특별시_정류소정보조회 서비스 API를 이용해서 막차를 계산하고 있는데, 이 API에는 누락된 데이터가 많습니다. 더 정확한 서울특별시_노선정보조회 서비스 APIgetStaionsByRouteList를 활용하도록 수정이 필요합니다.
    https://www.data.go.kr/tcs/dss/selectApiDataDetailView.do?publicDataPk=15000193
  3. SK open API와 공공데이터 포털 API가 변경되었을 때, 어떤 부분을 수정했는지 파악하기 쉽도록 관련 test 파일 생성이 필요합니다.
  4. SK open API와 공공데이터 포털 API에 오류가 없는지 다시 한 번 점검이 필요합니다.( 현재 API 결괏값의 오류 때문에, 이것을 확인하고 처리하는 오류로 depth가 깊다.)
@DoTheBestMayB
Copy link
Collaborator Author

로딩 중일 때, 사용자의 interaction 막지 않도록 수정하기

Caution: Android includes another dialog class called ProgressDialog that shows a dialog with a progress bar. This widget is deprecated because it prevents users from interacting with the app while progress is being displayed. If you need to indicate loading or indeterminate progress, you should follow the design guidelines for Progress & Activity and use a ProgressBar in your layout, instead of using ProgressDialog.

https://developer.android.com/develop/ui/views/components/dialogs

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

No branches or pull requests

1 participant