使用MVP架构在运行时检查权限的最佳方法是什么?

时间:2016-12-06 18:17:39

标签: android android-permissions mvp

我正在开发一个Android应用程序,我必须在运行时请求权限。我想知道使用Model-View-Presenter架构实现它的最佳方法。

我最初的想法是让演示者调用负责权限的组件(比如PermissionHandler),并相应地更新视图。

问题是检查权限的代码与Activity类紧密结合。以下是涉及需要活动或上下文的一些方法:

  • ContextCompat.checkSelfPermission()
  • ActivityCompat.shouldShowRequestPermissionRationale()
  • ActivityCompat.requestPermissions()
  • onRequestPermissionsResult()(回调)

这意味着我必须将一个活动对象传递给演示者,我不太喜欢,因为我听说让演示者免于Android代码是有利于测试的。

由于这个原因,我考虑在视图级别(在一个活动中)处理权限,但是我认为这会损害使视图仅对UI更新负责的目的,而没有业务逻辑。

我不确定解决这个问题的最佳方法是尽可能保持代码的解耦和可维护性。有什么想法吗?

3 个答案:

答案 0 :(得分:9)

我要做的是:

视图将实现:

public Activity getViewActivity();

演示者将实施:

public void requestPermissions();
public void onPermissionsResult();

requestPermissions内,演示者将执行:getViewActivity().checkSelfPermission; getViewActivity.requestPermissions(); etc.

该视图会调用onRequestPermissionsResult回调presenter.onPermissionsResult();

这样所有的逻辑都将在演示者中实现。

在我看来,你的演示者是分离的:它不依赖于任何视图实现(它只取决于视图界面)。

“我听说让您的演示者免受Android代码的攻击,这对测试非常有用。”我不明白这一部分。如果代码是好的,可以毫无问题地进行测试。

答案 1 :(得分:1)

如果您仍希望能够模拟权限访问/请求,您仍然可以创建类似PermissionHandler的内容,但只能在视图类中引用它。例如 -

接口:

public interface PermissionsHandler {
    boolean checkHasPermission(AppCompatActivity activity, String permission);
    void requestPermission(AppCompatActivity activity, String[] permissions, int requestCode);
}

生产实施:

public class PermissionsHandlerAndroid implements PermissionsHandler {
    @Override
    public boolean checkHasPermission(AppCompatActivity activity, String permission) {
        return ContextCompat.checkSelfPermission(activity, permission) == PackageManager.PERMISSION_GRANTED;
    }

    @Override
    public void requestPermission(AppCompatActivity activity, String[] permissions, int requestCode){
        ActivityCompat.requestPermissions(activity, permissions, requestCode);
    }
}

模拟类(例如,测试并确保您的活动正确处理onRequestPermissionsResult

public class PermissionsHandlerMocked implements PermissionsHandler {
    @Override
    public boolean checkHasPermission(AppCompatActivity activity, String permission) {
        return false;
    }

    @Override
    public void requestPermission(AppCompatActivity activity, String[] permissions, int requestCode){
        int[] grantResults = new int[permissions.length];
        for (int i = 0; i < permissions.length; i++) {
            grantResults[i] = PackageManager.PERMISSION_GRANTED
        }
        activity.onRequestPermissionResult(requestCode, permissions, grantResults);
    }
}

然后在你的活动中:

PermissionsHandler permissionsHandler;

@Override
protected void onCreate(@Nullable Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    permissionsHandler = Injection.providePermissionsHandler();
    //or however you choose to inject your production vs mocked handler. 
}

//method from your view interface, to be called by your presenter
@Override
void requestLocationPermission() {
    permissionsHandler.requestPermision((AppCompatActivity) this, new String[]{Manifest.permission.ACCESS_COARSE_LOCATION}, REQUEST_CODE_LOCATION};
}

fobo66,您始终可以使视图实现更多通用方法,例如 checkLocationPermissionGranted()requestLocationPermission()。然后,您的视图实现可以根据需要引用活动,并且您的演示者永远不必触摸活动参考。

答案 2 :(得分:0)

权限请求和状态是View(片段或活动)的职责,这取决于用户发出请求或授予权限的操作。我通过MVP管理权限,如下所示(请阅读外部存储示例):

我的合同

interface View {
    ...
    void requestReadPermission();
    boolean areReadPermissionGranted();
    void showPermissionError();
    void hidePermissionError();
    ...
}

interface Presenter {
    ...
    void setReadPermissions(boolean grantedPermissions);
    ...
}

interface Model {
    ...
}

我的视图实现。 (在这种情况下为片段,但它可以是Activity或其他任何内容,Presenter只会期望响应)。

public class MyView extends Fragment implements Contract.View {

...
Contract.Presenter presenter;

@Override
public void onRequestPermissionsResult(int requestCode, @NonNull String[] permissions, @NonNull int[] grantResults) {
    super.onRequestPermissionsResult(requestCode, permissions, grantResults);
    boolean grantedPermissions =  (grantResults.length > 0) && (grantResults[0] == PackageManager.PERMISSION_GRANTED);
    presenter.setReadPermissions(grantedPermissions);
}



@Override
public void showPermissionError() {
    // Show not permission message
}

@Override
public void hidePermissionError() {
    // Hide not permission message
}

@Override
public void requestReadPermission() {
    this.requestPermissions(new String[]{Manifest.permission.READ_EXTERNAL_STORAGE}, 1);
}
@Override
public boolean areReadPermissionGranted() {
    return ContextCompat.checkSelfPermission(getContext(), Manifest.permission.READ_EXTERNAL_STORAGE) == PackageManager.PERMISSION_GRANTED;
}
...

演示者实现

public class MyPresenter implements Contract.Presenter {

...
Contract.View view;

public void doSomethingThatRequiresPermissions() {

    ...
        if ( !view.areReadPermissionGranted() ) {
            view.requestReadPermission();
            view.showPermissionError();
        } else {
            view.hidePermissionError();
            doSomethingWithPermissionsGranted();
        }
    ...
}

@Override
public void setReadPermissions(boolean grantedPermissions) {
    if( grantedPermissions ){
        view.hidePermissionError();
        doSomethingThatRequiresPermissions();

    } else {
        view.showPermissionError();
    }
}

public void doSomethingWithPermissionsGranted(){
    ...
}

然后您可以像进行单元测试

Contract.View mockedView;
@Test
public void requestAlbumListWithoutPermissions() {
    when(mockedView.areReadPermissionGranted()).thenReturn(false);
    presenter.doSomethingWithPermissionsGranted();
    verify(mockedView).showPermissionError();
    verify(mockedView).requestReadPermission();
}