使用AsyncTask的ViewPager仅显示最后一页问题排查

嘿，我看到你遇到了ViewPager加载全部5个页面但只有最后一页显示内容的问题，咱们来一步步拆解问题根源，然后给你修复方案：

核心问题分析 #

• 全局position变量被覆盖：你在Adapter里定义了全局的position变量，而ViewPager会连续调用instantiateItem预加载多个页面，后面的调用会覆盖这个全局变量的值，最终所有页面的onPostExecute都使用了最后一次的position，导致所有页面都显示最后一页的内容。
• AsyncTask使用错误：你在instantiateItem里调用vpTask.get()会阻塞主线程，完全违背了AsyncTask的异步设计初衷，而且获取资源数组的操作是轻量的，根本不需要异步执行。
• 冗余的资源获取：你在doInBackground里重新获取了资源数组，覆盖了传入的参数，这不仅多余，还让之前的参数传递失去意义。
• 重复添加View：你在onPostExecute里手动调用container.addView(result)，但instantiateItem的职责就是返回View并由ViewPager负责添加，重复操作会引发异常。

修复后的代码实现 #

我们可以去掉不必要的AsyncTask，把资源初始化移到构造方法里，确保每个页面的position独立传递：

public class CustomPagerAdapter extends PagerAdapter {
    private Context context;
    private LayoutInflater layoutInflater;
    private String[] businessNames;
    private String[] businessSlogans;
    private TypedArray businessAds;
    private TypedArray businessLogos;

    public CustomPagerAdapter(Context context) {
        this.context = context;
        // 构造方法里初始化资源，避免重复获取
        layoutInflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        businessNames = context.getResources().getStringArray(R.array.store_names);
        businessSlogans = context.getResources().getStringArray(R.array.store_slogans);
        businessAds = context.getResources().obtainTypedArray(R.array.ads);
        businessLogos = context.getResources().obtainTypedArray(R.array.ad_logo);
    }

    @Override
    public int getCount() {
        return 5;
    }

    @Override
    public boolean isViewFromObject(@NonNull View view, @NonNull Object object) {
        return view == object;
    }

    @NonNull
    @Override
    public Object instantiateItem(@NonNull ViewGroup container, int position) {
        // 加载布局
        View itemView = layoutInflater.inflate(R.layout.ad_view, container, false);
        
        // 绑定当前position对应的数据
        ImageView businessLogo = itemView.findViewById(R.id.business_logo);
        TextView businessName = itemView.findViewById(R.id.business_name);
        TextView businessSlogan = itemView.findViewById(R.id.business_slogan);
        ImageView adContent = itemView.findViewById(R.id.ad_content);

        adContent.setImageDrawable(businessAds.getDrawable(position));
        businessName.setText(businessNames[position]);
        businessSlogan.setText(businessSlogans[position]);
        businessLogo.setImageDrawable(businessLogos.getDrawable(position));

        // 添加View到容器，返回该View作为标识
        container.addView(itemView);
        return itemView;
    }

    @Override
    public void destroyItem(@NonNull ViewGroup container, int position, @NonNull Object object) {
        // 直接移除View即可，不需要强制转换为ConstraintLayout（除非你的布局根是这个）
        container.removeView((View) object);
    }

    // 记得在Adapter销毁时回收TypedArray，避免内存泄漏
    @Override
    public void destroy() {
        super.destroy();
        businessAds.recycle();
        businessLogos.recycle();
    }
}

额外提示 #

如果你的广告图片是网络图片，建议使用Glide、Picasso等成熟的图片加载库来异步加载，它们会自动处理线程切换、缓存和内存管理，比自己写AsyncTask可靠得多。

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