使用Dagger 2注入子类的最佳方法是什么?

时间:2016-05-08 21:38:22

标签: android dependency-injection dagger dagger-2

我最近开始使用Dagger 2来管理我的应用程序的依赖注入。为了使一些类可测试,我开始创建我需要注入的许多类。我能够这样做,但是,注入这些新类的过程看起来有点复杂。让我们举一个例子:

我想测试一下我的RecyclerViewAdapter中的方法onCreateViewHolder()。为此,我创建了一个Factory,它根据给定的LayoutType返回ViewHolder:

public class ViewHolderFactor {

    public RecyclerView.ViewHolder getViewHolder(ViewGroup parent, int viewType) {
        LayoutInflater inflater = this.getLayoutInflater(parent.getContext());
        View view;
        switch (LayoutType.fromInteger(viewType)) {
            case SMALL_VERTICAL:
                view = inflater.inflate(R.layout.rsc_util_item_small, parent, false);
                return new ViewHolder.ItemViewHolder(view);
            case LARGE_VERTICAL:
                view = inflater.inflate(R.layout.rsc_util_item_large, parent, false);
                return new ViewHolder.ItemViewHolder(view);
        }
        return null;
    }

    private LayoutInflater getLayoutInflater(Context context) {
        return LayoutInflater.from(context);
    }

}

通过将上面的代码移到一个单独的类,我可以使用:

执行单元测试
@RunWith(JUnit4.class)
public class TestViewHolderFactor extends TestCase {

    ViewHolderFactor viewHolderFactor;

    @Test
    public void testGetViewHolder () {
        this.viewHolderFactor = Mockito.mock(ViewHolderFactor.class);
        ViewGroup viewGroup = Mockito.mock(ViewGroup.class);
        Mockito.when(viewHolderFactor.getViewHolder(viewGroup, LayoutType.SMALL_VERTICAL.toInteger())).thenCallRealMethod();
        Context context = Mockito.mock(Context.class);
        Mockito.when(viewGroup.getContext()).thenReturn(context);
        LayoutInflater layoutInflater = Mockito.mock(LayoutInflater.class);
        Mockito.when(viewHolderFactor.getLayoutInflater(context)).thenReturn(layoutInflater);
        Mockito.when(layoutInflater.inflate(R.layout.rsc_util_item_small, viewGroup, false)).thenReturn(Mockito.mock(View.class));
        RecyclerView.ViewHolder result = viewHolderFactor.getViewHolder(viewGroup, LayoutType.SMALL_VERTICAL.toInteger());
        assertNotNull(result);
    }

}

问题是:现在,为了使应用程序正常工作,我还必须注入包含实例的outter类(在创建ViewHolderFactor之前我没做过的事情)。最后,我会有一个Dagger配置:

@Module
public class ModuleBusiness {
    @Provides
    public CharacterUIService provideCharacterService(Picasso picasso, NotificationUtil notificationUtil, ViewHolderFactor viewHolderFactor) {
        return new CharacterUIService(picasso, notificationUtil, viewHolderFactor);
    }
}

其中CharacterUIService是创建包含ViewHolderFactory的RecyclerViewAdapter的新实例的类。

public class 
    private ViewHolderFactor 
    @Inject
    public CharacterUIService(ViewHolderFactor viewHolderFactor) {
        this.mViewHolderFactor = viewHolderFactor;
    }

    // ...
}

我需要来自ViewHolderFactor的成员才能注入它。

我担心需要创建新的全局变量并增加构造函数中传递的参数数量。有没有更好的方法来注入这些子类,或者我认为这是一个很好的做法,以便允许单元测试?

1 个答案:

答案 0 :(得分:1)

  

我可以将其视为允许单元测试的好习惯吗?

首先 - 在我个人看来 - 我认为你过度测试了。在您提供的测试用例中,您基本上是在测试 android框架本身。如果要测试工厂,则应测试是返回小项还是大项,而不是视图不为空。

但是,你的,似乎是允许进行单元测试的合理方法。

  

其中CharacterUIService是创建RecyclerViewAdapter的新实例的类

您可能想要超越CharacterUIService的角色。 “这创造了一个新的实例”听起来像是应该由匕首处理的东西,因为你已经在使用它了。

  

还必须将持有实例的outter类注入工厂

为什么呢? ViewHolderFactor(原文如此!)本身没有依赖关系。即使它确实如此,是什么阻止你用匕首创造并注射它?

class ViewHolderFactor {
    @Inject
    ViewHolderFactor() { // allow for constructor injection
    }
}

class YourAdapter {
    ViewHolderFactor mFactor;

    @Inject // allow for constructor injection
    YourAdapter (ViewHolderFactor factor) {/**/}
}
// now dagger knows how to create that adapter

只需创建让dagger创建该适配器?

通常,如果您支持构造函数注入,则可以删除@Provides providesSomething()方法。只需删除整个方法。你在构造函数上有一个@Inject注释,所以请使用它,让dagger为你编写代码。

// CharacterUIService can be constructor injected.
@Provides // DAGGER WILL DO THIS FOR YOU
public CharacterUIService provideCharacterService(Picasso picasso, NotificationUtil notificationUtil, ViewHolderFactor viewHolderFactor) {
    // REMOVE ALL OF THIS.
    return new CharacterUIService(picasso, notificationUtil, viewHolderFactor);
}

因此,虽然我认为你通过使事情变得可测试做得很好,但是你应该再次看好匕首,因为你显然混合了很多东西。仔细研究构造函数注入并使用它,它可以为您节省大量的工作。