버스 리팩토링 회고

[Yang-Seungmin]

버스 리팩토링 결과에 따른 문제점

1. 코드 중복

• BusArrivalInfo.kt
• BusRemainTime.kt
• BusRoute.kt
• BusMapper.kt
• 그 외 Bus data source 및 repository 등...

2. 버스 타이머 로직 개선 필요

• GetBusTimerUseCase.kt
• 앞선 코드 중복으로 인한 코드 중복 발생
• java.time.LocalTime과 시스템 시간을 이용하여 남은 시간을 계산, 이로 인해 시스템 시간이 달라질 경우 버스 타이머의 버스 출발 시간이 달라짐
  • 대부분의 사람이 시스템 시간을 자동으로 해놓지만, 수 초의 차이로 버스 출발 시간이 1분 늦춰 보여지는 경우가 있음(ex. 13:31 출발)
• 버스 남은 시간을 API에서 가져오는 트리거(60초마다, 어떤 버스 남은 시간이 0초가 된 이후 +1초 후)를 이해하기 힘듦

해결 방안

1. BusArrivalInfo및 BusRemainTime의 코드 중복 최소화

• sealed class를 이용하여 버스 종류를 분리하여 묶을 필요가 있는가?
• 두 클래스에서 각 버스별 요구하는 정보가 거의 유사함, 따라서 BusArrivalInfo 클래스 하나에 버스 정보 관련 프로퍼티를 추가하는 방법
• 위 방법은 시내버스에만 존재하는 busNumber를 어떻게 할 지에 고민 필요