RecyclerView中多次捕获NullPointerException及重复跳转GPS设置问题求助

嘿，看来你碰到两个挺闹心的定位相关问题——检测定位服务时的空指针异常，还有RecyclerView导致设置页面弹窗多次，我来帮你逐个解决：

一、解决检测定位服务时的NullPointerException #

从你提到的代码片段来看，空指针大概率出在这两个地方：

1. 获取定位服务实例时没做非空判断
2. 计算距离时用到的当前位置对象ml为null（定位服务禁用时根本拿不到位置）

修复方案： #

1. 先确保定位服务实例非空
   在检测定位服务状态前，先对LocationManager做非空校验：

   LocationManager locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
   if (locationManager == null) {
       // 处理服务获取失败的情况，比如给用户提示
       Toast.makeText(this, "定位服务不可用", Toast.LENGTH_SHORT).show();
       return;
   }
   

2. 计算距离前校验当前位置
   你的Calculdistance方法里用到了ml，如果定位服务禁用，这个对象肯定是null，直接调用方法就会炸。所以先加非空判断：

   public double Calculdistance(Pharmacien ph) {
       // 先判断当前位置和Pharmacien的经纬度是否有效
       if (ml == null || ph.getLatitude() == null || ph.getLongitude() == null) {
           // 返回一个标记值表示无法计算，比如-1
           return -1;
       }
       float[] result = new float[10];
       Location.distanceBetween(ml.getLatitude(), ml.getLongitude(), 
                               ph.getLatitude(), ph.getLongitude(), result);
       return result[0];
   }
   

二、解决设置页面弹窗多次的问题 #

这个问题很典型——你肯定是把定位检测和弹窗的逻辑放在了RecyclerView的onBindViewHolder方法里，导致每个条目绑定的时候都执行一次，弹窗次数自然和条目数一致。

修复方案： #

方案1：把检测逻辑移到页面初始化阶段（推荐）

把定位服务检测放在Activity/Fragment的onCreate或onResume里，只执行一次：

@Override
protected void onResume() {
    super.onResume();
    checkLocationService();
}

private void checkLocationService() {
    LocationManager locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
    if (locationManager != null && !locationManager.isProviderEnabled(LocationManager.GPS_PROVIDER)) {
        // 只弹出一次引导弹窗
        new AlertDialog.Builder(this)
                .setTitle("需要开启定位服务")
                .setMessage("为了显示地点距离，请开启GPS定位")
                .setPositiveButton("去设置", (dialog, which) -> {
                    Intent intent = new Intent(Settings.ACTION_LOCATION_SOURCE_SETTINGS);
                    startActivity(intent);
                })
                .setNegativeButton("取消", null)
                .show();
    }
}

方案2：加全局标记避免重复弹窗（适合需要动态检测的场景）

如果必须在Adapter里触发检测（比如下拉刷新后），可以在Activity里加一个标记变量，确保同一时间只弹一次：

private boolean isLocationDialogShown = false;

public void checkLocationService() {
    if (isLocationDialogShown) return;
    
    LocationManager locationManager = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
    if (locationManager != null && !locationManager.isProviderEnabled(LocationManager.GPS_PROVIDER)) {
        isLocationDialogShown = true;
        new AlertDialog.Builder(this)
                .setTitle("需要开启定位服务")
                .setMessage("为了显示地点距离，请开启GPS定位")
                .setPositiveButton("去设置", (dialog, which) -> {
                    Intent intent = new Intent(Settings.ACTION_LOCATION_SOURCE_SETTINGS);
                    startActivity(intent);
                    isLocationDialogShown = false; // 跳转后重置标记
                })
                .setNegativeButton("取消", (dialog, which) -> {
                    isLocationDialogShown = false; // 取消后重置标记
                })
                .show();
    }
}

然后在Adapter里调用Activity的这个方法时，就不会重复弹窗了。

额外注意点 #

别忘了检查定位权限！即使定位服务开启了，如果没有申请ACCESS_FINE_LOCATION或ACCESS_COARSE_LOCATION权限，还是拿不到位置，也可能引发空指针哦。

内容的提问来源于stack exchange，提问作者yoyo guen