[둘리] 1단계 자동 DI 미션 제출합니다

[hyemdooly]

안녕하세요. 베르~ 반갑습니다.
수동 DI 외에 고민해본 적이 별로 없어서 조금 어려웠네요.
ViewModelFactory와 RepositoryModule를 생성하여 일관적으로 ViewModel을 생성할 수 있도록 작성해봤습니다.
나중에는 Room을 사용하는 것 같아 Application단에서 이 Repository를 해결해보고 싶었는데 쉽지 않더라고요.
우선 현재의 요구 사항을 충족시키기위해 RepositoryModule Object를 생성하여 Repository 구현체들을 모아두었습니다!
추가 구현 사항은 리뷰 후에도 시간이 남으면 해보겠습니다.🥲
리뷰 잘 부탁드립니다. :)

ViewModelFactory

class ViewModelFactory : ViewModelProvider.Factory {
    override fun <T : ViewModel> create(modelClass: Class<T>): T {
        val constructor = modelClass.declaredConstructors.first() // 정의된 첫번째 생성자 가져오기
        require(constructor != null) { IllegalArgumentException("Unknown ViewModel Class $modelClass") }

        val types = constructor.parameterTypes // 생성자의 파라미터 타입들 모두 가져오기
        val params = mutableListOf<Any?>() // 파라미터에 담아줄 객체들을 담을 리스트

        val properties = RepositoryModule::class.java.declaredFields // RepositoryModule에 정의되어있는 모든 필드들 가져오기
        for (type in types) {
            val field = properties.first { it.type == type }
                ?: throw IllegalArgumentException("Can't find Property $type") // type과 같은 type의 property 가져오기
            field.isAccessible = true // 접근제어자 바꿔주기
            params.add(field.get(null))
        }

        return constructor.newInstance(*params.toTypedArray()) as T // ViewModel 객체 생성
    }
}

[SeongHoonC]

안녕하세요. 둘리! 잘부탁드립니다ㅎㅎ
자동 DI 잘 구현해 주었어요!
다만 테스트가 없어서 조금 아쉽네요ㅜ
리뷰 확인해주세요!

[SeongHoonC]

requireNotNull 을 활용해 보는게 어떨까요?

[hyemdooly]

오 좋아용 수정했습니다~!

[SeongHoonC]

가변 리스트와 for 문이 아닌 아닌 map 을 활용하는 것이 더 코틀린스럽지 않을까요?

[SeongHoonC]

추가로 함수 분리가 필요할 것 같아요ㅎㅎ

[hyemdooly]

알고리즘 문제 푸는 것에 뇌가 절여져서 map 쓸 생각을 못했네요...

[hyemdooly]

수정했습니다..!!! 제가 kotlin reflection이 안됐던 이유가 reflect를 import안해줘서임을 깨닫고 자바 reflection을 걷어내고 kotlin으로 바꿔줬습니다!
수업시간에 배운 내용들이긴 하지만 비슷해서 싹 바꿨어요!

[SeongHoonC]

Repository 가 추가될 때 재사용 가능하게 만들어 보면 어떨까요?

[hyemdooly]

Container로 변경하고, 프로퍼티로 map을 추가해 instance를 추가하고 가져올 수 있게 했습니다.

[SeongHoonC]

수업시간에 배운 내용을 빠르게 적용한 둘리 대단합니다!
고민해볼만한 코멘트 한 개 남겼는데 다음 미션에서 생각해 보셨으면 좋겠어요!
1단계 이만 머지하겠습니다ㅎㅎ

[SeongHoonC]

어디서나 접근할 수 있는 object 로 관리하면 instance 의 무결성을 보장할 수 없다고 생각합니다! 다음 미션에 참고해주세요!

[SeongHoonC]

이제 생성되는 저장소가 추가되어도 유연하게 대응할 수 있겠네요!