アーキテクチャのコンポーネントViewModelsは、アクティビティ/フラグメントからのパラメータを持つダガーを挿入します。

Question

私は、ダガーを注入しながら新しいアーキテクチャコンポーネントViewModelをアプリケーションに追加しようとしています。 googleがhereと表示したコードに基づいています。私はそれぞれViewModelタイプのためにViewModelFactoryを避けようとしているので、Map<Class<? extends ViewModel>, Provider<ViewModel>> creatorsを受け取るViewModelFactoryを使用しました。これは@Singletonスコープの依存関係を持つViewModelsで動作します。しかし、私のViewModelsの1つは、そのフラグメントに由来する依存性を持っています。これは、そのフラグメントのモジュールです：

@Module 
public abstract class DownloadIssueDialogFragmentModule { 

    @Binds 
    abstract DialogFragment dialogFragment(DownloadIssueDialogFragment dialogFragment); 

    @Provides 
    @FragmentScope 
    static Issue provideIssue(DownloadIssueDialogFragment dialogFragment) { 
     return dialogFragment.getIssue(); 
    } 
} 

そして、私のViewModelModule：

@Module 
public abstract class ViewModelModule { 

    @Binds 
    abstract ViewModelProvider.Factory bindViewModelFactory(ViewModelFactory factory); 

    @Binds 
    @IntoMap 
    @ViewModelKey(DownloadIssueViewModel.class) 
    abstract ViewModel bindDownloadIssueViewModel(DownloadIssueViewModel viewModel); 

} 

短剣は、それがIssueを提供することができないと言います。 Map<Class<? extends ViewModel>, Provider<ViewModel>>はコンパイル時に作成されたようです。しかし、私は、その断片の範囲内でのみパラメータを知っているでしょう。どうすればこれを達成できますか？

ありがとうございます。

EDIT：最後に

私は別のアプローチを行ってきました。今度はViewModelごとにファクトリを作成し、ViewModelをインジェクトするのではなく、ファクトリを挿入します。

私はこのライブラリを作成しました：AutoViewModelFactory

自動的に工場を生成するには。これまでに私が見つけた最高のソリューションです。

Answer 1

Android ArchitectureのコンポーネントViewModelsは、Fragmentよりもスコープが大きく（永続的なライフサイクルを読みやすく）、ViewModelをFragmentのフィールドに依存させないようにする必要があります。

ただし、Issueが実行時にのみ認識され、フラグメント内のロジックによって生成されている場合は、ホルダーパターンを使用して小さな依存性サイクルの問題をエスケープすることができます。

これはsome other Dagger 2 StackOverflow questionsで議論されてきたが、あなたは、単に公共のアクセサ/ミューテータでJava Beanを定義します

class IssueHolder { 
    private Issue issue; 

    @Inject 
    IssueHolder() {} //empty explicit constructor as required by Dagger 2   

    public void setIssue(@Nullable Issue issue) { 
     this.issue = issue; 
    } 

    @Nullable 
    public Issue getIssue() { 
     return issue; 
    } 
} 

次に、あなたがあなたのViewHolderがIssue上IssueHolderのではなく、直接に依存することができます：

@Inject IssueHolder issueHolder; 

public void doSomething() { 
    if (issueHolder.get() == null) { 
     throw new IllegalStateException("Expected IssueHolder to be set by IssueFragment at this point"); 
    } 
    //TODO: the logic you want here 
} 

どのようなパターンと同様に、このホルダーパターンは、容易に退化する可能性があるため、控えめに使用する必要があります。可能であれば、サイクルの可能性を排除するようにモジュールと依存関係を設計するのが最善の解決策です。