如何使用MVP实现PhoneStateListener

Question

我正在MVP中重构我的一个旧应用程序。 我已经重构了大多数逻辑，现在我坚持遵循以下逻辑。 在我的一项活动中，我实现了如下的PhoneStateLister。

private class CallStateListener extends PhoneStateListener {

    public void onCallStateChanged(int state, String incomingNumber) {

        switch (state) {
            case TelephonyManager.CALL_STATE_IDLE:
                deactivateSpeaker();
                if (callCount == 0) {
                    doCall();
                } else {
                    checkForHangUp();
                }
                break;
            case TelephonyManager.CALL_STATE_OFFHOOK:
                checkAutoSpeaker();
                break;
            case TelephonyManager.CALL_STATE_RINGING:
                break;
            default:
                break;
        }
    }

    private void checkAutoSpeaker() {
        if (preferenceManager.isAutoSpeaker()) activateSpeaker();
    }

    private void activateSpeaker() {
        final Handler mHandler = new Handler();
        mHandler.postDelayed(new Runnable() {
            @Override
            public void run() {
                audioManager.setMode(AudioManager.MODE_IN_CALL);
                audioManager.setSpeakerphoneOn(true);
            }
        }, 1000);
    }

    private void deactivateSpeaker() {
        audioManager.setMode(AudioManager.MODE_NORMAL); //Deactivate loudspeaker
    }

}

我很难决定将该逻辑放在MVP中的哪个位置。 Activity应该处理这个PhoneStateListener还是演示者应该处理这个？ 请帮我解决。 谢谢。

Answer 1

以MVP模式设置侦听器总是有些棘手。 我很确定您的问题没有“正确”或“错误”的答案，因为它在很大程度上取决于您的个人喜好和建筑风格。 但是按照我的一般规则，将尽可能多的业务逻辑移交给演示者并尽可能使其他所有内容（包括侦听器 ）保持愚蠢总是很聪明的。

这就是为什么我建议您将一个接口添加到您的侦听器类，并在此保留对演示者的（弱）引用。 在该示例中， 侦听器唯一要做的就是侦听状态更改，然后将信息转发给演示者 （在其中分发）。

public class CallStateListener extends PhoneStateListener {
    private final WeakReference<CallStateListenerInterface> presenterInterface;
    public CallStateListener(CallStateListenerInterface presenterInterface){
        this.presenterInterface = new WeakReference<>(presenterInterface);
    }
    public void onCallStateChanged(int state, String incomingNumber){
        presenterInterface.get().onCallStateListenerCallStateChanged(state, incomingNumber);
    }
    public interface CallStateListenerInterface {
        void onCallStateListenerCallStateChanged(int state, String incomingNumber);
    }
}

演示者需要实现CallStateListenerInterface （当然），并在呼叫状态更改时处理所有业务逻辑。

public class myPresenter implements MVPInterface, CallStateListenerInterface {

    /*your business logic*/

    public void onCallStateListenerCallStateChanged(int state, String incomingNumber){
        switch (state) { 
           case TelephonyManager.CALL_STATE_IDLE: 
                deactivateSpeaker(); 
                if (callCount == 0) { 
                    doCall(); 
                } else { 
                    checkForHangUp(); 
                } 
               break; 
           case TelephonyManager.CALL_STATE_OFFHOOK: 
               checkAutoSpeaker(); 
               break; 
           case TelephonyManager.CALL_STATE_RINGING: 
               break; 
           default: 
               break; 
        }
    }
}

从这里开始，需要与视图交互的所有内容都将被调用，但是将这些调用保持在简单的一行指令中，并使所有复杂的事情由演示者决定。

如果您不想为每个侦听器类添加新接口 ，也可以将整个MVPInterface传递给CallStateChangedListener （并在合同的中心位置声明所有内容）。

如果以这种方式实现侦听器，则您将获得一个非常干净的体系结构，该体系结构应易于测试。 希望对您有所帮助，如果您有任何疑问，请告诉我:)

小替代实施：

正如评论者所指出的：在我的原始建议中，演示者从TelephonyManager类访问静态电话状态整数。 这不是最干净的设计，因为演示者应与任何Android特定代码完全分开。 由于没有真正的超级干净的方法可以解决此问题，因此我只能建议从TelephonyManager状态到保存在演示者中的（自定义的）电话状态（最好是静态常量文件）之间进行映射。 基本上，这意味着CallStateListener中的onCallStateChanged应该看起来像这样：

public void onCallStateChanged(int state, String incomingNumber){
    int stateValue = myPresenter.PRESENTER_CALL_STATE_IDLE;
    if (state == TelephonyManager.CALL_STATE_RINGING){
        stateValue = myPresenter.PRESENTER_CALL_STATE_RINGING;
    } else if (state == TelephonyManager.CALL_STATE_OFFHOOK){
        stateValue = myPresenter.PRESENTER_CALL_STATE_OFFHOOK;
    }
    presenterInterface.get().onCallStateListenerCallStateChanged(stateValue , incomingNumber);
}

通过此实现，您可以将TelephonyManager状态移出演示者并使用自己定义的状态。 （再次：我真的不确定这是否值得付出努力，但我想至少提供替代方案）

Answer 2

我相信您应该将侦听器留在片段/活动部分。 我有两个解决方案。 第一个是将开关/案例留在您的侦听器中，并向演示者添加一些逻辑：

public void onCallStateChanged(int state, String incomingNumber) {
    switch (state) {
        case TelephonyManager.CALL_STATE_IDLE:
            mPresenter.onCallStateIDLE();
            break;
        case TelephonyManager.CALL_STATE_OFFHOOK:
            mPresenter.onCallStateOffHook();
            break;
        case TelephonyManager.CALL_STATE_RINGING:
            mPresenter.onCallStateRinging();
            break;
        default:
            break;
    }
}

并在您的演示者中：

void onCallStateIDLE(){
    mView.deactivateSpeaker();
    if (callCount == 0) {
        mView.doCall();
    } else{
        mView.checkForHangUp();
    }
}

另一个选择应该是将开关/外壳移到演示器中。 棘手的部分应该是如何处理TelephonyManager常数。 如您所知，您应该将演示者保留为纯Java代码。

您应该有一个View界面，您可以在其中执行类似的操作

interface MyActivityContract{
    interface View{
        int CALL_STATE_IDLE = TelephonyManager.CALL_STATE_IDLE;
        //etc etc the same with the rest of constants
    }
    interface Presenter{}
}

然后在您的听众中，您应该对主持人有一个参考

private class CallStateListener extends PhoneStateListener {
    public void onCallStateChanged(int state, String incomingNumber) {
        mPresenter.onCallStateChanged(state, incomingNumber);
    }
}

最后你的主持人可能是这样的

public void onCallStateChanged(int state, int incomingNumber){
    switch (state) {
        case MyActivityContract.View.CALL_STATE_IDLE:
            mView.deactivateSpeaker();
            if (callCount == 0) {
                mView.doCall();
            } else {
                mView.checkForHangUp();
            }
            break;
        case MyActivityContract.View.CALL_STATE_OFFHOOK:
            if (preferenceManager.isAutoSpeaker()){
                mView.activateSpeaker();
            }
            break;
        case MyActivityContract.View.CALL_STATE_RINGING:
            break;
        default:
            break;
    }
}

这样，您的演示者中就没有任何android参考，但您的合同中却有它。