主持人是否知道MVP模式中的活动/上下文是个坏主意？

时间：2015-12-16 03:32:40

标签： android mvp android-sharedpreferences

我已经玩了几个星期的MVP模式，我已经到了需要上下文来启动service并访问Shared Preferences。

我已经读过MVP的目的是将视图与逻辑分离，并且context中的Presenter可能会破坏该目的（如果我错了，请纠正我）。

目前，我的LoginActivity看起来像这样：

LoginActivity.java 

public class LoginActivity extends Activity implements ILoginView {

    private final String LOG_TAG = "LOGIN_ACTIVITY";

    @Inject
    ILoginPresenter mPresenter;
    @Bind(R.id.edit_login_password)
    EditText editLoginPassword;
    @Bind(R.id.edit_login_username)
    EditText editLoginUsername;
    @Bind(R.id.progress)
    ProgressBar mProgressBar;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_login);
        MyApplication.getObjectGraphPresenters().inject(this);
        mPresenter.setLoginView(this, getApplicationContext());
    }

    @Override
    public void onStart() {
        mPresenter.onStart();
        ButterKnife.bind(this);
        super.onStart();
    }

    @Override
    public void onResume() {
        mPresenter.onResume();
        super.onResume();
    }

    @Override
    public void onPause() {
        mPresenter.onPause();
        super.onPause();
    }

    @Override
    public void onStop() {
        mPresenter.onStop();
        super.onStop();
    }

    @Override
    public void onDestroy() {
        ButterKnife.unbind(this);
        super.onDestroy();
    }

    @OnClick(R.id.button_login)
    public void onClickLogin(View view) {
        mPresenter.validateCredentials(editLoginUsername.getText().toString(),
                editLoginPassword.getText().toString());
    }

    @Override public void showProgress() { mProgressBar.setVisibility(View.VISIBLE); }

    @Override public void hideProgress() {
        mProgressBar.setVisibility(View.GONE);
    }

    @Override public void setUsernameError() { editLoginUsername.setError("Username Error"); }

    @Override public void setPasswordError() { editLoginPassword.setError("Password Error"); }

    @Override public void navigateToHome() {
        startActivity(new Intent(this, HomeActivity.class));
        finish();
    }
}

Presenter Interface ILoginPresenter.java 

public interface ILoginPresenter {
    public void validateCredentials(String username, String password);


    public void onUsernameError();

    public void onPasswordError();

    public void onSuccess(LoginEvent event);

    public void setLoginView(ILoginView loginView, Context context);

    public void onResume();

    public void onPause();

    public void onStart();

    public void onStop();
}

最后，我的演示者：

LoginPresenterImpl.java 

public class LoginPresenterImpl implements ILoginPresenter {

    @Inject
    Bus bus;

    private final String LOG_TAG = "LOGIN_PRESENTER";
    private ILoginView loginView;
    private Context context;
    private LoginInteractorImpl loginInteractor;

    public LoginPresenterImpl() {
        MyApplication.getObjectGraph().inject(this);
        this.loginInteractor = new LoginInteractorImpl();
    }

    /**
     * This method is set by the activity so that way we have context of the interface
     * for the activity while being able to inject this presenter into the activity.
     *
     * @param loginView
     */
    @Override
    public void setLoginView(ILoginView loginView, Context context) {
        this.loginView = loginView;
        this.context = context;

        if(SessionUtil.isLoggedIn(this.context)) {
            Log.i(LOG_TAG, "User logged in already");
            this.loginView.navigateToHome();
        }
    }

    @Override
    public void validateCredentials(String username, String password) {
        loginView.showProgress();
        loginInteractor.login(username, password, this);
    }

    @Override
    public void onUsernameError() {
        loginView.setUsernameError();
        loginView.hideProgress();
    }

    @Override
    public void onPasswordError() {
        loginView.setPasswordError();
        loginView.hideProgress();
    }

    @Subscribe
    @Override
    public void onSuccess(LoginEvent event) {
        if (event.getIsSuccess()) {
            SharedPreferences.Editor editor =
                    context.getSharedPreferences(SharedPrefs.LOGIN_PREFERENCES
                            .isLoggedIn, 0).edit();
            editor.putString("logged_in", "true");
            editor.commit();

            loginView.navigateToHome();
            loginView.hideProgress();
        }
    }

    @Override
    public void onStart() {
        bus.register(this);
    }

    @Override
    public void onStop() {
        bus.unregister(this);

    }

    @Override
    public void onPause() {

    }

    @Override
    public void onResume() {
    }
}

如您所见，我将Activity中的上下文传递到我的Presenter，以便我可以访问Shared Preferences。我非常担心将上下文传递给我的演示者。这是一件好事吗？或者我应该采取其他方式吗？

EDIT实施了Jahnold的第三个偏好

所以让我们忽略界面和实现，因为它几乎就是整个事情。所以现在我injecting将共享偏好的界面添加到我的演示者中。这是AppModule

的代码

AppModule.java 

@Module(library = true,
    injects = {
            LoginInteractorImpl.class,
            LoginPresenterImpl.class,
            HomeInteractorImpl.class,
            HomePresenterImpl.class,

    }
)
public class AppModule {

    private MyApplication application;

    public AppModule(MyApplication application) {
        this.application = application;
    }

    @Provides
    @Singleton
    public RestClient getRestClient() {
        return new RestClient();
    }

    @Provides
    @Singleton
    public Bus getBus() {
        return new Bus(ThreadEnforcer.ANY);
    }

    @Provides
    @Singleton
    public ISharedPreferencesRepository getSharedPreferenceRepository() { return new SharedPreferencesRepositoryImpl(application.getBaseContext()); }

    }
}

我获取上下文的方式来自MyApplication.java

当应用程序开始时，我确保用这行代码创建这个Object图：

objectGraph = ObjectGraph.create(new AppModule(this));

这可以吗？我的意思是我现在不必将活动的上下文传递给我的演示者，但我仍然有应用程序的上下文。

3 个答案:

答案 0 :(得分：70)

自从您提出这个问题以来已经过了一段时间，但我认为无论如何都要提供一个答案是有用的。我强烈建议演示者不应该有Android Context（或任何其他Android类）的概念。通过将Presenter代码与Android系统代码完全分离，您可以在JVM上对其进行测试，而不会出现模拟系统组件的复杂情况。

为实现这一点，我认为你有三种选择。

从视图访问SharedPreferences

这是我最不喜欢的三个，因为访问SharedPreferences是而不是一个视图操作。但是，它确实使Activity中的Android系统代码远离Presenter。在您的视图界面中有一个方法：

boolean isLoggedIn();

可以从演示者处调用。

使用Dagger注入SharedPreferences

由于您已经使用Dagger注入事件总线，因此可以将SharedPreferences添加到ObjectGraph中，这样就可以获得使用ApplicationContext构造的SharedPreferences实例。这是你得到它们而不必将Context传递给你的演示者。

这种方法的缺点是你仍在传递Android系统类（SharedPreferences），并且当你想测试Presenter时必须模拟它。

创建SharePreferencesRepository接口

这是我从Presenter中访问SharedPreferences数据的首选方法。基本上，您将SharedPreferences视为模型并具有存储库接口。

您的界面类似于：

public interface SharedPreferencesRepository {

    boolean isLoggedIn();
}

然后，您可以具体实现：

public class SharedPreferencesRepositoryImpl implements SharedPreferencesRepository {

    private SharedPreferences prefs;

    public SharedPreferencesRepositoryImpl(Context context) {

        prefs = PreferenceManager.getDefaultSharedPreferences(context);
    }

    @Override
    public boolean isLoggedIn() {

        return prefs.getBoolean(Constants.IS_LOGGED_IN, false);
    }

}

然后将SharedPreferencesRepository接口与Dagger一起注入Presenter。这样，在测试期间可以在运行时提供非常简单的模拟。在正常操作期间，提供了具体实施。

答案 1 :(得分：4)

前一段时间回答了这个问题，并且假设MVP的定义是他的代码中使用的OP，@ Jahnold的回答非常好。

然而，应该指出的是，MVP是一个高级概念，并且可以有许多遵循MVP原则的实现 - 有不止一种方法可以使猫皮肤化。

There is another implementation of MVP，基于Activities in Android are not UI Elements的想法，{{3}}将Activity和Fragment指定为MVP演示者。在此配置中，MVP演示者可以直接访问Context。

顺便说一句，即使在前面提到的MVP实现中，我也不会使用Context来访问演示者中的SharedPreferences - 我仍然会为{定义一个包装类{1}}并将其注入演示者。

答案 2 :(得分：1)

大多数域元素（如DB或网络）都需要构建Context。无法在View中创建Thay，因为View无法了解Model。然后必须在Presenter中创建它们。它们可以由Dagger注入，但它也使用Context。所以Context用于Presenter xP

黑客就是如果我们想避免在Presenter中使用Context，那么我们就可以创建从Context创建所有这些Model对象而不保存它的构造函数。但在我看来，这是愚蠢的。 Android中的新JUnit可以访问Context。

另一个hack是使Context可以为空，并且在域对象中应该有机制在上下文中为null时提供测试实例。我也不喜欢这个黑客。

相关问题
最新问题