求助：从URL读取笑话的Android APP点击按钮崩溃（AsyncTask实现）

嘿，我来帮你排查下这个点击按钮就崩溃的问题！结合你说的用AsyncTask从URL读笑话的场景，常见的坑大概有这几个，你可以逐一排查：

Android 系统要求，任何需要访问网络的应用必须在AndroidManifest.xml中声明网络权限。如果没加这个权限，点击按钮触发网络请求时会直接崩溃。

• 解决方案：在AndroidManifest.xml的根节点下添加：

<uses-permission android:name="android.permission.INTERNET" />

AsyncTask的doInBackground方法是在子线程执行的，如果在这里直接更新进度条、修改TextView等UI组件，会触发CalledFromWrongThreadException导致崩溃。

• 正确做法：把UI操作放到对应的主线程回调里：

private class JokeTask extends AsyncTask<Void, Integer, List<String>> {
    @Override
    protected void onPreExecute() {
        super.onPreExecute();
        // onPreExecute在主线程执行，适合显示进度条
        progressBar.setVisibility(View.VISIBLE);
    }

    @Override
    protected List<String> doInBackground(Void... voids) {
        // 这里只做网络请求、数据解析，绝对不能碰UI
        List<String> jokes = new ArrayList<>();
        // 你的URL读取逻辑
        // 如果需要更新进度，调用publishProgress(当前进度值)
        return jokes;
    }

    @Override
    protected void onProgressUpdate(Integer... values) {
        super.onProgressUpdate(values);
        // 这里在主线程执行，适合更新进度条进度
        progressBar.setProgress(values[0]);
    }

    @Override
    protected void onPostExecute(List<String> jokes) {
        super.onPostExecute(jokes);
        // 这里在主线程执行，适合隐藏进度条、显示笑话
        progressBar.setVisibility(View.GONE);
        // 将jokes展示到UI组件中
    }
}

如果AsyncTask是MainActivity的非静态内部类，会持有Activity的强引用。当Activity意外销毁（比如旋转屏幕）但AsyncTask还在运行时，会导致内存泄漏，严重时触发崩溃。

• 解决方案：
  • 使用静态内部类+弱引用持有Activity：

  private static class JokeTask extends AsyncTask<Void, Integer, List<String>> {
      private WeakReference<MainActivity> activityRef;
  
      public JokeTask(MainActivity activity) {
          activityRef = new WeakReference<>(activity);
      }
  
      @Override
      protected void onPostExecute(List<String> jokes) {
          MainActivity activity = activityRef.get();
          // 先判断Activity是否存活，再更新UI
          if (activity != null && !activity.isFinishing()) {
              activity.progressBar.setVisibility(View.GONE);
              // 显示笑话逻辑
          }
      }
  }
  

  • 在Activity销毁时取消正在运行的AsyncTask：

  @Override
  protected void onDestroy() {
      super.onDestroy();
      if (jokeTask != null && jokeTask.getStatus() == AsyncTask.Status.RUNNING) {
          jokeTask.cancel(true);
      }
  }
  

如果URL读取时出现IOException（比如网络不通、URL无效），没有捕获异常的话会直接抛出导致崩溃。

• 解决方案：在doInBackground中添加异常捕获：

@Override
protected List<String> doInBackground(Void... voids) {
    List<String> jokes = new ArrayList<>();
    try {
        // 你的网络请求逻辑，比如用HttpURLConnection
        URL url = new URL("你的笑话API地址");
        HttpURLConnection connection = (HttpURLConnection) url.openConnection();
        // 读取数据、解析笑话的逻辑
    } catch (IOException e) {
        e.printStackTrace();
        // 返回null或空列表，在onPostExecute中处理错误提示
        return null;
    }
    return jokes;
}

然后在onPostExecute中处理错误情况：

@Override
protected void onPostExecute(List<String> jokes) {
    MainActivity activity = activityRef.get();
    if (activity != null && !activity.isFinishing()) {
        activity.progressBar.setVisibility(View.GONE);
        if (jokes == null || jokes.isEmpty()) {
            Toast.makeText(activity, "加载失败，请检查网络", Toast.LENGTH_SHORT).show();
        } else {
            // 显示笑话
        }
    }
}

如果你的应用targetSdkVersion是31（Android 12）及以上，AsyncTask已经被官方弃用，虽然仍能运行，但可能出现兼容性问题导致崩溃。

• 替代方案（推荐）：用Coroutines（Kotlin）或ExecutorService+Handler（Java）实现异步操作：
  • Kotlin Coroutines示例：

  fun loadJokes(count: Int) {
      lifecycleScope.launch {
          progressBar.visibility = View.VISIBLE
          // 切换到IO线程执行网络请求
          val jokes = withContext(Dispatchers.IO) {
              getJokesFromUrl(count)
          }
          progressBar.visibility = View.GONE
          if (jokes.isNotEmpty()) {
              // 显示笑话
          } else {
              Toast.makeText(this@MainActivity, "加载失败", Toast.LENGTH_SHORT).show()
          }
      }
  }
  

  • Java Executor示例：

  private void loadJokes(int count) {
      progressBar.setVisibility(View.VISIBLE);
      ExecutorService executor = Executors.newSingleThreadExecutor();
      Handler handler = new Handler(Looper.getMainLooper());
  
      executor.execute(() -> {
          // 子线程执行网络请求
          List<String> jokes = getJokesFromUrl(count);
          // 回到主线程更新UI
          handler.post(() -> {
              progressBar.setVisibility(View.GONE);
              if (jokes != null && !jokes.isEmpty()) {
                  // 显示笑话
              } else {
                  Toast.makeText(MainActivity.this, "加载失败", Toast.LENGTH_SHORT).show();
              }
          });
      });
  }
  

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