@HiltViewModel annotated class should contain exactly one @Inject annotated constructor

Question

I keep getting this error:

public final class MainViewModel extends androidx.lifecycle.ViewModel { ^ @HiltViewModel annotated class should contain exactly one @Inject annotated constructor. [Hilt] Processing did not complete. See error above for details.

Here is my MainViewModel:

@HiltViewModel
class MainViewModel @Inject constructor(
    repository: DefaultRepository
) : ViewModel() {
    val items = repository.getItems().asLiveData()
}

Did you use multiple constructors in your ViewModel? Otherwise it should work — 0xAliHn, Jul 03 '21 at 09:11
@0xAliHn I provided the entire ViewModel code here. There are no other constructors — Atick Faisal, Jul 03 '21 at 09:57
`Run with --stacktrace` with this try getting more stack trace that might help, the details you shared are not helping, because your `viewmodel` is correct, there might be some other issues. — Anand, Jul 03 '21 at 10:28
This might sound weird but I had a package name called `..data.default` which was containing the`DefautRepository`. This `default` in the package name was causing the issue. I renamed the package now everything is working fine. — Atick Faisal, Jul 03 '21 at 11:04
Are you trying to inject any variables inside your viewmodel? In that case just remove those and add them to the constructor, that should fix your issue — Catalin, Aug 01 '21 at 08:57
Same issue as @AtickFaisal, I had `throws` in the package name and changing it fixed the issue — Tyler Turnbull, Sep 29 '21 at 21:11
same issue as @TylerTurnbull and Atick Faisal, i had **new** in the package name, renaming it fixed the issue — konnovdev, Dec 06 '21 at 05:17
I have faced the same issue by creating . package. Just rename `case` into `usecase` — Vahe Gharibyan, Apr 15 '23 at 18:56

score 16 · Answer 1 · answered Sep 06 '22 at 19:58

16

When it happened to me, it was because someone removed the injected constructor from class I was injecting.

It was:

@HiltViewModel
class MyViewModel : ViewModel()

But needed to be:

@HiltViewModel
class MyViewModel @Inject constructor() : ViewModel()

answered Sep 06 '22 at 19:58

HTMLlama

1,464
2
11
16

score 2 · Answer 2 · answered Sep 18 '21 at 09:33

2

In my case, I was specifying default values for some of my primary constructor parameters. Getting rid of the default values fixed the issue.

answered Sep 18 '21 at 09:33

heyheyhey

1,216
8
12

score 1 · Answer 3 · answered Jan 05 '22 at 09:14

1

Make sure you are using this inject annotation

import javax.inject.Inject

instead of this one

com.google.inject.Inject

answered Jan 05 '22 at 09:14

AncientMethods

256
1
9

score 1 · Answer 4 · answered Apr 28 '22 at 22:11

In my case the problem was that one of the packages was called "case", which is a name used by some conventions (like of SQL, or even Java).

@Atick Faisal (see comments above) had also another problem with package name, so I guess it might worth a check if you don't find the problem.

score 1 · Answer 5 · answered Aug 09 '22 at 14:30

Well, I think I figured it out after wasting 3 goddam hours on this. Apparently, you cannot name your package by keywords

I was using enum, and a few people in the comments were using case, default etc.

If only the error message was a bit clearer (－‸ლ).

NB: This is my assumption and it makes sense too.

score 0 · Answer 6 · answered Jan 20 '22 at 14:15

0

Make sure in your module you have @InstallIn(SingletonComponent::class) instead of @InstallIn(ApplicationComponentManager::class)

answered Jan 20 '22 at 14:15

Strahinja Vučković

1

@HiltViewModel annotated class should contain exactly one @Inject annotated constructor

6 Answers6