Sergey Grishin
Sergey Grishin

Reputation: 462

Dagger: Can't provide Map<java.lang.Class<? extends android.arch.lifecycle.ViewModel> without an @Provides-annotated method

I've tried, to create several components. First will store main parts of app, like ViewModel's Fabric, Context, other settings. Other components -- component per screen. So, f.ex. I have FirstScreen. I've tried to create component with its ViewModel:

@Subcomponent(modules = [StoreModule::class])
@StoreScope
interface StoreComponent {

    fun inject(activity: MainActivity)

    fun inject(fragment: StoreFragment)

    @Subcomponent.Builder
    interface Builder {

        fun build(): StoreComponent

    }

}

So, StoreViewModel, as its dependency, StoreRepository, builds in module of StoreComponent:

@Module
abstract class StoreModule {

    @Binds
    @IntoMap
    @ViewModelKey(StoreViewModelImpl::class)
    abstract fun getStoreViewModel(viewModel: StoreViewModelImpl): ViewModel

    @Binds
    @StoreScope
    abstract fun getStoreRepository(repository: StoreRepositoryImpl): StoreRepository

}

And this is AppComponent:

@Component(modules = [
    GsonModule::class,
    RetrofitModule::class,
    ViewModelsFactoryModule::class,
    CiceroneModule::class
])
@Singleton
interface AppComponent {

    fun getStoreComponentBuilder(): StoreComponent.Builder

}

Here buildes ViewModel's Factory:

@Module
abstract class ViewModelsFactoryModule {

    @Target(AnnotationTarget.FUNCTION,
            AnnotationTarget.PROPERTY_GETTER,
            AnnotationTarget.PROPERTY_SETTER
    )
    @Retention(AnnotationRetention.RUNTIME)
    @MapKey
    internal annotation class ViewModelKey(val value: KClass<out ViewModel>)


    @Binds
    abstract fun getViewModelFactory(factory: ViewModelFactory): ViewModelProvider.Factory

}

But, when I try to build project, I have en error, that dagger can't provide Map<java.lang.Class<? extends android.arch.lifecycle.ViewModel> into ViewModelFactory-class without @Provides-annonated method.

error: [com.sagrishin.smartreader.di.components.StoreComponent.inject(com.sagrishin.smartreader.presentation.fragments.StoreFragment)] java.util.Map<java.lang.Class<? extends android.arch.lifecycle.ViewModel>,javax.inject.Provider<android.arch.lifecycle.ViewModel>> cannot be provided without an @Provides-annotated method.
public abstract interface AppComponent {
                ^
      java.util.Map<java.lang.Class<? extends android.arch.lifecycle.ViewModel>,javax.inject.Provider<android.arch.lifecycle.ViewModel>> is injected at
          com.sagrishin.smartreader.presentation.viewmodels.factory.ViewModelFactory.<init>(creators)
      com.sagrishin.smartreader.presentation.viewmodels.factory.ViewModelFactory is injected at
          com.sagrishin.smartreader.presentation.fragments.StoreFragment.viewModelsFactory
      com.sagrishin.smartreader.presentation.fragments.StoreFragment is injected at
          com.sagrishin.smartreader.di.components.StoreComponent.inject(fragment)

UPD: viewmodel factory:

typealias ViewModelsProvidersMap =
        Map<Class<out ViewModel>, @JvmSuppressWildcards Provider<ViewModel>>


@Singleton
class ViewModelFactory : ViewModelProvider.Factory {

    private val creators: ViewModelsProvidersMap

    @Inject
    constructor(creators: ViewModelsProvidersMap) {
        this.creators = creators
    }

    override fun <T : ViewModel> create(modelClass: Class<T>): T {
        var creator: Provider<out ViewModel>? = creators[modelClass]
        if (creator == null) {
            for ((key, value) in creators) {
                if (modelClass.isAssignableFrom(key)) {
                    creator = value
                    break
                }
            }
        }
        if (creator == null) {
            throw IllegalArgumentException("unknown model class $modelClass")
        }
        try {
            return creator.get() as T
        } catch (e: Exception) {
            throw RuntimeException(e)
        }

    }
}

Upvotes: 3

Views: 1122

Answers (4)

USMAN osman
USMAN osman

Reputation: 1030

Your dependency version might be different, for instance your core/compiler versions are 2.22 or some other but your android versions are 2.16 this might be causing the exception as well, just do this:
from this

    implementation 'com.google.dagger:dagger:2.22'
    kapt 'com.google.dagger:dagger-compiler:2.22'
    // For android
    implementation 'com.google.dagger:dagger-android:2.16'
    implementation 'com.google.dagger:dagger-android-support:2.16'
    // if you use the support libraries
    kapt 'com.google.dagger:dagger-android-processor:2.16'

to this

 //Dagger 2
    implementation 'com.google.dagger:dagger:2.22'
    kapt 'com.google.dagger:dagger-compiler:2.22'
    // For android
    implementation 'com.google.dagger:dagger-android:2.22'
    implementation 'com.google.dagger:dagger-android-support:2.22'
    // if you use the support libraries
    kapt 'com.google.dagger:dagger-android-processor:2.22'

Upvotes: 0

yash786
yash786

Reputation: 1151

No need to use @JvmSuppressWildcards. As the day of my Post my Kotlin version
 is : '1.3.40'

 Use dagger dependency in build.gradle

 apply plugin: 'kotlin-kapt'

 //Dagger dependencies
implementation 'com.google.dagger:dagger:2.22.1'
kapt 'com.google.dagger:dagger-compiler:2.22.1'

implementation 'com.google.dagger:dagger-android:2.22'
implementation 'com.google.dagger:dagger-android-support:2.22' // if you use the support libraries
kapt 'com.google.dagger:dagger-android-processor:2.22'


And for Java use the following dagger dependency

 //Dagger dependencies
implementation 'com.google.dagger:dagger:2.22.1'
annotationProcessor 'com.google.dagger:dagger-compiler:2.22.1'

implementation 'com.google.dagger:dagger-android:2.22'
implementation 'com.google.dagger:dagger-android-support:2.22' // if you use the support libraries
annotationProcessor 'com.google.dagger:dagger-android-processor:2.22'

Upvotes: 0

Juan Mendez Escobar
Juan Mendez Escobar

Reputation: 2757

You don't have to switch to Java, and simply use kotlin 1.3.31 to have it fixed. https://github.com/google/dagger/issues/1478#issuecomment-486712176

include the following Jvm annotation

Map<Class<out ViewModel>, @JvmSuppressWildcards Provider<ViewModel>>

Upvotes: 1

Aaron Thompson
Aaron Thompson

Reputation: 1889

I fixed the same issue I was having by switching the ViewModelFactory.kt and ViewModelsFactoryModule.kt to java.

ViewModelFactory.java

@Singleton
public class ViewModelFactory implements ViewModelProvider.Factory {
    private final Map<Class<? extends ViewModel>, Provider<ViewModel>> creators;

    @Inject
    ViewModelFactory(Map<Class<? extends ViewModel>, Provider<ViewModel>> creators) {
        this.creators = creators;
    }

    @NonNull
    @SuppressWarnings("unchecked")
    @Override
    public <T extends ViewModel> T create(@NonNull Class<T> modelClass) {
        Provider<? extends ViewModel> creator = creators.get(modelClass);
        if (creator == null) {
            for (Map.Entry<Class<? extends ViewModel>, Provider<ViewModel>> entry : creators.entrySet()) {
                if (modelClass.isAssignableFrom(entry.getKey())) {
                    creator = entry.getValue();
                    break;
                }
            }
        }
        if (creator == null) {
            throw new IllegalArgumentException("unknown model class " + modelClass);
        }
        try {
            return (T) creator.get();
        } catch (Exception e) {
            throw new RuntimeException(e);
        }
    }
}

ViewModelsFactoryModule.java

@Module
abstract class ViewModelModule {

    @NonNull
    @Binds
    @IntoMap
    @ViewModelKey(StoreViewModelImpl.class)
    abstract ViewModel bindLauncherViewModel(StoreViewModelImpl viewModel);

    @NonNull
    @Binds
    abstract ViewModelProvider.Factory bindViewModelFactory(ViewModelFactory viewModelFactory);
}

Upvotes: 0

Related Questions