RecyclerView嵌入Facebook原生广告时替换原Item的问题求助

你遇到的核心问题是广告视图直接覆盖了原RecyclerView Item，这是因为你没有区分「原数据Item」和「插入的广告Item」的位置关系——当前逻辑把广告位当成原数据的一部分，导致原数据的position被广告占用，最终原Item被替换。下面是具体的修复方案：

步骤1：修正总Item数计算，区分原数据与广告数量 #

广告是插入到列表中，而非替换原Item，所以总Item数需要是「原数据数量 + 插入的广告数量」。添加广告数量计算方法并修改getItemCount：

// 计算需要插入的广告总数：第1个位置1个，之后每14个原Item插1个
private int getAdCount() {
    if (nameArraylist.isEmpty()) return 0;
    return 1 + (nameArraylist.size() / 14);
}

@Override
public int getItemCount() {
    // 总Item数 = 原数据数量 + 广告数量
    return nameArraylist.size() + getAdCount();
}

步骤2：修改getItemViewType，正确判断广告位 #

原来的逻辑直接用原position判断，导致广告占用原Item位置。现在需要先判断当前position是否属于广告插入位：

@Override
public int getItemViewType(int position) {
    // 第1个位置（position=0）展示广告
    if (position == 0) {
        return SINGLE_AD_TYPE;
    }
    // 减去第一个广告的偏移，计算相对于原数据的位置
    int adjustedPos = position - 1;
    // 每14个原Item后插一个广告：即每15个展示Item（14原+1广告）的最后一个是广告
    if (adjustedPos % 15 == 14) {
        return SECOND_AD_TYPE;
    }
    // 其余是原数据Item
    return DEFAULT_CONTENT;
}

步骤3：修正原数据索引的获取逻辑 #

因为插入了广告，原数据的真实索引需要减去前面已插入的广告数量：

@Override
public void onBindViewHolder(final RecyclerView.ViewHolder holder, final int position) {
    if (holder instanceof SingleAdViewHolder) {
        // 实现SingleAdViewHolder的广告绑定逻辑（和SecondAdViewHolder类似即可）
        onBindAdViewHolder(holder);
    } else if (holder instanceof SecondAdViewHolder) {
        final SecondAdViewHolder adViewHolder = (SecondAdViewHolder) holder;
        // 优化：不要每次onBind都创建新NativeAd，建议在ViewHolder中缓存并在回收时销毁
        final NativeAd nativeAd = new NativeAd(context, context.getResources().getString(R.string.fb_banner_ad));
        nativeAd.setAdListener(new AdListener() {
            @Override
            public void onError(Ad ad, AdError adError) {
                Log.d(TAG, "广告加载失败: " + adError.getErrorMessage());
                // 加载失败可以隐藏广告布局
                adViewHolder.itemView.setVisibility(View.GONE);
            }

            @Override
            public void onAdLoaded(Ad ad) {
                adViewHolder.itemView.setVisibility(View.VISIBLE);
                adViewHolder.nativeAdTitle.setText(nativeAd.getAdTitle());
                adViewHolder.nativeAdSocialContext.setText(nativeAd.getAdSocialContext());
                adViewHolder.nativeAdBody.setText(nativeAd.getAdBody());
                adViewHolder.nativeAdCallToAction.setText(nativeAd.getAdCallToAction());
                
                NativeAd.Image adIcon = nativeAd.getAdIcon();
                NativeAd.downloadAndDisplayImage(adIcon, adViewHolder.nativeAdIcon);
                adViewHolder.nativeAdMedia.setNativeAd(nativeAd);
                
                AdChoicesView adChoicesView = new AdChoicesView(context, nativeAd, true);
                adViewHolder.adChoicesContainer.removeAllViews();
                adViewHolder.adChoicesContainer.addView(adChoicesView);
                
                List<View> clickableViews = new ArrayList<>();
                clickableViews.add(adViewHolder.nativeAdTitle);
                clickableViews.add(adViewHolder.nativeAdCallToAction);
                nativeAd.registerViewForInteraction(adViewHolder.nativeAdContainer, clickableViews);
            }

            @Override
            public void onAdClicked(Ad ad) {}
            @Override
            public void onLoggingImpression(Ad ad) {}
        });
        nativeAd.loadAd();
    } else {
        MyViewHolder myViewHolder = (MyViewHolder) holder;
        // 计算原数据的真实索引：减去前面插入的广告数量
        int adOffset = 1 + (position - 1) / 15;
        int realDataPos = position - adOffset;
        
        // 绑定原数据内容
        TempModel model = nameArraylist.get(realDataPos);
        myViewHolder.textView.setText(model.getName());
        Picasso.get().load(model.getImage()).into(myViewHolder.imageView);
        
        // 你的选中样式逻辑：注意要用真实数据索引，而非展示position
        if (row_index == realDataPos) {
            ViewGroup.MarginLayoutParams layoutParams = (ViewGroup.MarginLayoutParams) myViewHolder.cardView.getLayoutParams();
            layoutParams.setMargins(0, 0, 0, 0);
            ViewGroup.MarginLayoutParams layoutParamsCard = (ViewGroup.MarginLayoutParams) myViewHolder.mCv_round_img.getLayoutParams();
            layoutParamsCard.setMargins(19, 0, 0, 0);
            myViewHolder.cardView.setRadius(0);
            myViewHolder.mView.setVisibility(View.VISIBLE);
        } else {
            // 恢复默认样式
            ViewGroup.MarginLayoutParams layoutParams = (ViewGroup.MarginLayoutParams) myViewHolder.cardView.getLayoutParams();
            layoutParams.setMargins(16, 16, 16, 16);
            ViewGroup.MarginLayoutParams layoutParamsCard = (ViewGroup.MarginLayoutParams) myViewHolder.mCv_round_img.getLayoutParams();
            layoutParamsCard.setMargins(0, 0, 0, 0);
            myViewHolder.cardView.setRadius(8);
            myViewHolder.mView.setVisibility(View.GONE);
        }
    }
}

步骤4：修复onCreateViewHolder中的重复return错误 #

你原来的代码中SINGLE_AD_TYPE分支有重复的return语句，删掉多余的即可：

@Override
public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    if (viewType == SINGLE_AD_TYPE) {
        View adLayoutOutline = LayoutInflater.from(context)
                .inflate(R.layout.item_facebook_native_ad_outline, parent, false);
        ViewGroup vg = adLayoutOutline.findViewById(R.id.ad_container);
        LinearLayout adLayoutContent = (LinearLayout) LayoutInflater.from(parent.getContext())
                .inflate(R.layout.item_facebook_native_ad, parent, false);
        vg.addView(adLayoutContent);
        return new SingleAdViewHolder(adLayoutOutline);
        // 删掉下面这句重复的return
        // return new SingleAdViewHolder(view);
    } else if (viewType == SECOND_AD_TYPE) {
        View adLayoutOutline = LayoutInflater.from(context)
                .inflate(R.layout.item_facebook_native_ad_outline, parent, false);
        ViewGroup vg = adLayoutOutline.findViewById(R.id.ad_container);
        LinearLayout adLayoutContent = (LinearLayout) LayoutInflater.from(parent.getContext())
                .inflate(R.layout.item_facebook_native_ad, parent, false);
        vg.addView(adLayoutContent);
        return new SecondAdViewHolder(adLayoutOutline);
    } else {
        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.play_row_item, parent, false);
        return new MyViewHolder(view);
    }
}

额外优化建议 #

1. 缓存NativeAd避免内存泄漏：在onViewRecycled方法中销毁广告资源，避免重复创建导致的内存问题。
2. 合并广告ViewHolder：SingleAdViewHolder和SecondAdViewHolder逻辑一致，可以合并为一个AdViewHolder减少冗余代码。
3. 广告加载失败处理：隐藏广告布局，避免空布局影响用户体验。

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