了解 Dagger 2 中的范围

我在 Dagger 2 中遇到了与范围相关的错误，我正在尝试了解如何解决它。

我有一个CompaniesActivity这表明公司。当用户选择一个项目时，所选公司的员工会显示在EmployeesActivity。当用户选择一名员工时，她的详细信息显示在EmployeeDetailActivity.

class Company {
    List<Employee> employees;
}

Class CompaniesViewModel包含公司和选定的公司（或null):

class CompaniesViewModel {
    List<Company> companies;
    Company selected;
}

CompaniesActivity有参考CompaniesViewModel:

class CompaniesActivity extends Activity {

    @Inject
    CompaniesViewModel viewModel;

    @Override
    protected void onCreate(Bundle b) {
        //more stuff
        getComponent().inject(this);
        showCompanies(viewModel.companies);
    }

    //more stuff

    private onCompanySelected(Company company) {
        viewModel.selected = company;
        startActivity(new Intent(this, EmployeesActivity.class));
    }

}

Class EmployeesViewModel包含员工和选定的员工（或null):

class EmployeesViewModel {
    List<Employee> employees;
    Employee selected;
}

EmployeesActivity有参考EmployeesViewModel:

  class EmployeesActivity extends Activity {

        @Inject
        EmployeesViewModel viewModel;

        @Override
        protected void onCreate(Bundle b) {
            //more stuff
            getComponent().inject(this);
            showEmployees(viewModel.employees);
        }

        //more stuff

        private onEmployeeSelected(Employee emp) {
            viewModel.selected = emp;
            startActivity(new Intent(this, EmployeeDetailActivity.class));
        }

    }

最后，在EmployeeDetailActivity，我从视图模型中选择 Employee 并显示她的详细信息：

  class EmployeeDetailActivity extends Activity {

        @Inject
        EmployeesViewModel viewModel;

        @Override
        protected void onCreate(Bundle b) {
            //more stuff
            getComponent().inject(this);
            showEmployeeDetail(viewModel.selected); // NullPointerException
        }
    }

I get NullPointerException因为EmployeesViewModel实例在EmployeesActivity不一样EmployeeDetailActivity并且，在第二个中，viewModel.selected is null.

这是我的匕首模块：

@Module
class MainModule {

    @Provides
    @Singleton
    public CompaniesViewModel providesCompaniesViewModel() {
        CompaniesViewModel cvm = new CompaniesViewModel();
        cvm.companies = getCompanies();
        return cvm;
    }

    @Provides
    public EmployeesViewModel providesEmployeesViewModel(CompaniesViewModel cvm) {
        EmployeesViewModel evm = new EmployeesViewModel();    
        evm.employees = cvm.selected.employees;
        return evm;
    }

}

注意CompaniesViewModel是单例（@Singleton) but EmployeesViewModel不是，因为每次用户选择公司时都必须重新创建它（员工列表将包含其他项目）。

I could set公司员工至EmployeesViewModel每次用户选择一个公司，而不是创建一个新实例。但我想CompaniesViewModel是一成不变的。

我该如何解决这个问题？任何建议将不胜感激。

不幸的是，我认为在这种情况下你滥用了 DI 框架，并且你遇到的问题是“代码味道”——这些问题暗示你做错了什么。

应使用 DI 框架将关键依赖项（协作者对象）注入顶级组件，并且执行这些注入的逻辑应完全独立于应用程序的业务逻辑。

第一眼看上去一切都很好——你使用 Dagger 来注入CompaniesViewModel and EmployeesViewModel into Activity。如果这些是真正的“对象”，这本来可以很好（尽管我不会这样做）。但是，就您而言，这些是“数据结构”（因此您希望它们是不可变的）。

对象和数据结构之间的区别并非微不足道，而是非常重要。这篇博文总结得很好。

现在，如果您尝试使用 DI 框架注入数据结构，您最终会将框架转变为应用程序的“数据提供者”，从而将部分业务功能委托给它。例如：看起来像EmployeesViewModel独立于CompaniesViewModel，但这是一个“谎言” - 中的代码@Provides方法将它们逻辑地联系在一起，从而“隐藏”依赖性。在这种情况下，好的“经验法则”是，如果 DI 代码依赖于注入对象的实现细节（例如调用方法、访问字段等），这通常表明关注点分离不充分。

两个具体建议：

1. 不要将业务逻辑与 DI 逻辑混合。在您的情况下 - 不要注入数据结构，而是注入提供对数据的访问权限的对象（不好），或者在抽象数据时公开所需的功能（更好）。
2. 我认为您在多个屏幕之间共享视图模型的尝试并不是一个非常稳健的设计。最好为每个屏幕都有一个单独的视图模型实例。如果您需要在屏幕之间“共享”状态，那么根据具体要求，您可以使用 1) Intent extras 2) Global object 3) Shared prefs 4) SQLite 来实现