先直接给出结论：`onCreate()` 仅仅是一个起点，它不应成为所有初始化逻辑的默认执行入口——尤其是在这些逻辑依赖于定位权限、异步 API 或延迟回调的场景下。 从实际开发经验来看，你当前代码的核心问题可以归纳为：**在权限尚未获取、位置数据尚未返回之前，就把操作代码全部塞入了 `onCreate()`。** 举个典型例子，在 `onCreate()` 里直接调用 `fetchLastLocation()`，紧接着又执行 `getLastKnownLocation()` 和 `suppMapFragment.getMapAsync(this)`。此时定位权限还未确认、位置数据尚未返回，`currentLocation` 几乎必定为空。等到 `onMapReady()` 回调被触发时，再试图调用 `currentLocation.getLatitude()`——空指针异常几乎必然发生。 --- ### ✅ 正确做法：将地图初始化逻辑解耦并统一管理 关键点在于：把所有依赖定位状态的操作，全部从 `onCreate()` 中移出。`onCreate()` 只负责完成组件的必要初始化，例如创建 `client`、`LManager` 这些基础对象。而地图加载与位置获取，则封装成一个可安全重入的统一入口方法： ```ja va private void setupMapAndLocation() { // 1. 先检查权限是否已经授予，否则发起请求 if (ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_FINE_LOCATION) != PackageManager.PERMISSION_GRANTED) { ActivityCompat.requestPermissions(this, new String[]{Manifest.permission.ACCESS_FINE_LOCATION}, REQUEST_CODE); return; // 等待 onRequestPermissionsResult 回调 } // 2. 权限已就绪，尝试获取最近一次已知位置 Task task = client.getLastLocation(); task.addOnSuccessListener(location -> { if (location != null) { currentLocation = location; initMap(); // ✅ 位置有效，立即初始化地图 } else { // 没有记录的位置 → 启动实时定位或降级处理（后面会详细说明） requestNewLocation(); } }).addOnFailureListener(e -> { Log.e("Location", "Failed to get last location", e); showLocationError("无法获取位置，请检查设置"); }); } ``` 同时，`onCreate()` 应精简到接近“纯配置”的程度： ```ja va @Override protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); client = LocationServices.getFusedLocationProviderClient(this); LManager = (LocationManager) getSystemService(LOCATION_SERVICE); // ❌ 把 fetchLastLocation()、getLastKnownLocation()、getMapAsync() 等依赖性的调用全部移除 suppMapFragment = (SupportMapFragment) getSupportFragmentManager() .findFragmentById(R.id.maps); // ✅ 等到权限就绪后，由统一入口来驱动后续流程 setupMapAndLocation(); } ``` --- ### ? `onMapReady()` 职责重构：专注渲染，不负责数据获取 `onMapReady()` 的职责应当非常纯粹：仅负责地图 UI 层的渲染逻辑。它无需再次检查权限或调用 `getLastKnownLocation()`——这些前置条件已在 `setupMapAndLocation()` 中落实。修正后的写法如下： ```ja va @Override public void onMapReady(@NonNull GoogleMap googleMap) { this.mMap = googleMap; // ✅ 仅处理渲染逻辑：此时 currentLocation 应由 setupMapAndLocation 保证就绪 if (currentLocation == null) { showLocationPlaceholder(); // 例如显示一个“定位中...”的提示 return; } LatLng latLng = new LatLng(currentLocation.getLatitude(), currentLocation.getLongitude()); MarkerOptions markerOptions = new MarkerOptions() .position(latLng) .title("您在此处！"); googleMap.moveCamera(CameraUpdateFactory.newLatLngZoom(latLng, 12)); googleMap.addMarker(markerOptions); googleMap.getUiSettings().setMyLocationButtonEnabled(true); // ✅ 安全启用定位图层（权限已在入口处确认过） if (ActivityCompat.checkSelfPermission(this, Manifest.permission.ACCESS_FINE_LOCATION) == PackageManager.PERMISSION_GRANTED) { googleMap.setMyLocationEnabled(true); } } ``` --- ### ⚠️ 几个关键细节决定了稳定性 * **`onRequestPermissionsResult` 必须触发重试**：用户授权后不要只调用 `fetchLastLocation()`，而应重新调用 `setupMapAndLocation()` 这个完整入口，确保地图初始化链路完整重启。 * **空位置的降级策略**：如果 `getLastLocation()` 返回 null，不要等待，主动用 `requestLocationUpdates()` 获取实时位置，或者显示友好的引导提示，让用户开启 GPS。 * **避免重复 `getMapAsync()`**：在 `onCreate()` 中调用一次 `suppMapFragment.getMapAsync(this)` 就足够了。`setupMapAndLocation()` 中无需重复调用——`onMapReady()` 会自动触发。 * **线程安全**：在类似 `Hospitals()` 的网络解析逻辑中，如果需要调用 `googleMap.addMarker()`，务必在主线程执行（你已经用 `handler.post()` 处理了线程安全问题，这很好），切勿在后台线程直接操作 UI。 --- ### ✅ 说到底，这是异步编程思维方式的转变 Android 的权限模型与位置 API 本质上是异步的。如果强行用同步思维去编写初始化逻辑，崩溃只是时间问题。正确的实践可以总结为四步： 1. **识别依赖项** —— 例如权限、网络、传感器； 2. **将业务逻辑封装成独立函数** —— 例如 `setupMapAndLocation()`； 3. **在所有可能就绪的入口点统一调用这个函数** —— 包括 `onCreate`、`onRequestPermissionsResult`、`onSuccess` 回调； 4. **让 UI 层只消费已经验证好的数据** —— 不做状态判断。 遵循这一模式，你的应用不仅能彻底摆脱“首次启动崩溃”和“GPS 关闭时闪退”的问题，代码的可维护性与可扩展性也会显著提升。