[챗GPT로 코딩 공부] 프로젝트 코드·성능 개선

GPT를 이용한 프로젝트 코드·성능 개선

프롬프트: 코틀린으로 안드로이드 개발을 하고 있어. API 통신을 통해 특일 정보를 받아오는 코드야. 특일의 종류는 holiday, anniversary, solar term, etc 4종류야. usecase 클래스에서 API 요청으로 받아온 종류별 특일 정보를 하나의 flow로 변환하여 반환해. 이 코드를 더 개선할 수 있을까?

interface HolidayRepository {
    suspend fun getHoliday(year: Int, month: Int): Flow
    suspend fun getAnniversary(year: Int, month: Int): Flow
    suspend fun getSolarTerm(year: Int, month: Int): Flow
    suspend fun getEtc(year: Int, month: Int): Flow
}

class GetHolidayUseCase @Inject constructor(
    private val holidayRepository: HolidayRepository
) {
    private val TAG = this@GetHolidayUseCase::class.java.simpleName

    suspend operator fun invoke(year: Int, month: Int): Flow =
        combine(
            holidayRepository.getHoliday(year, month),
            holidayRepository.getAnniversary(year, month),
            holidayRepository.getSolarTerm(year, month),
            holidayRepository.getEtc(year, month),
        ) { holiday, anniversary, solarTerm, etc ->
            mutableListOf().apply {
                if (holiday is EntityWrapper.Success) addAll(holiday.data)
                else Log.d(TAG, "failed holiday, ${(holiday as EntityWrapper.Fail).throwable}")
                if (anniversary is EntityWrapper.Success) addAll(anniversary.data)
                else Log.d(TAG, "failed anniversary, ${(anniversary as EntityWrapper.Fail).throwable}")
                if (solarTerm is EntityWrapper.Success) addAll(solarTerm.data)
                else Log.d(TAG, "failed solarTerm, ${(solarTerm as EntityWrapper.Fail).throwable}")
                if (etc is EntityWrapper.Success) addAll(etc.data)
                else Log.d(TAG, "failed etc, ${(etc as EntityWrapper.Fail).throwable}")
            }
        }
}

이미 잘 구성된 코드이지만, 몇 가지 개선할 수 있는 방법이 있습니다. 코드의 가독성과 유지 보수성을 높이는 것이 주된 목적입니다. - 중복 코드 최소화: 현재 종류별로 if-else 로직이 반복되고 있습니다. 이를 더 간결하게 처리할 수 있는 방법을 모색할 수 있습니다. - 데이터 처리 로직 분리: 데이터 처리 로직을 별도의 함수로 분리하여, 종류별 데이터 처리를 한 곳에서 관리하게 할 수 있습니다. - 에러 로깅 개선: 현재는 실패 시 로그를 출력하고 있지만, 이를 더 효율적으로 관리할 수 있는 방법을 고민해 볼 수 있습니다. 아래는 개선된 코드 예시입니다:

class GetHolidayUseCase @Inject constructor(
    private val holidayRepository: HolidayRepository
) {
    private val TAG = this@GetHolidayUseCase::class.java.simpleName

    suspend operator fun invoke(year: Int, month: Int): Flow =
        combine(
            holidayRepository.getHoliday(year, month),
            holidayRepository.getAnniversary(year, month),
            holidayRepository.getSolarTerm(year, month),
            holidayRepository.getEtc(year, month),
        ) { results ->
            results.flatMap { result ->
                when (result) {
                    is EntityWrapper.Success -> result.data
                    is EntityWrapper.Fail -> {
                        Log.d(TAG, "failed to fetch data, ${result.throwable}")
                        emptyList()
                    }
                }
            }
        }

    // Optional: 데이터 처리 로직을 별도의 함수로 분리
    private fun handleResult(result: EntityWrapper): List {
        return when (result) {
            is EntityWrapper.Success -> result.data
            is EntityWrapper.Fail -> {
                Log.d(TAG, "failed to fetch data, ${result.throwable}")
                emptyList()
            }
        }
    }
}

내일배움캠프가 여러분의 러닝메이트가 되어드리겠습니다

취업 준비, 어디서부터 시작해야 할지 모르겠다면?