Android双组按钮计数器干扰导致interstitial ad误触发问题

bug的核心原因非常明确：两套触发规则完全独立的按钮组，共用了同一个全局点击计数器countClicks，两组点击量被混算，必然出现跨组误触发：

• 导航组累计点到9次时，全局计数为9，此时点击任意操作组按钮，计数+1变为10，操作组的触发判断是countClicks >= 3，10远大于阈值3，直接触发广告，和你遇到的现象完全一致
• 反过来操作组点到2次时，再点一次导航组，计数到3，也会误触发操作组的3次阈值广告
• 你测试的单按钮独立计数器方案确实能解决问题，但属于过度拆分——你的业务规则本来就是按组统计累计点击，不是按单个按钮统计，给7个按钮各维护一个计数器冗余度太高，后续改阈值、加按钮要改一堆变量，维护成本很高。

另外你现有代码里firstQuote、latestQuote方法里写的if (countClicks != 3)本质是共用计数器时打的逻辑补丁，会导致计数到阈值时点击按钮无响应，等广告关了才刷新UI，交互逻辑也有问题。

只需要保留两个组级别的独立计数器，完全匹配你按组统计的业务规则，同时把重复的广告触发逻辑抽成通用方法，整体代码量更少、维护性更强。

1. 定义组级常量与计数器 #

把原来的单个全局计数器替换为两组独立的计数变量，阈值用常量定义，后续调整规则不用翻业务逻辑：

// 操作类按钮组（copy/share/first quote/latest quote）触发阈值
private static final int THRESHOLD_OPERATION = 3;
private int clickCountOperation = 0;

// 导航类按钮组（back/random/next）触发阈值
private static final int THRESHOLD_NAV = 10;
private int clickCountNav = 0;

2. 抽离通用广告触发逻辑 #

你现有7个按钮方法里的广告判断、加载、关闭回调代码重复度超过90%，抽成通用方法后，广告相关逻辑只需要维护一处：

/**
 * 检查是否满足插屏广告触发条件，满足则展示广告，否则直接执行业务逻辑
 * @param currentCount 当前点击所属组的累计点击数
 * @param threshold 当前组的广告触发阈值
 * @param normalTask 按钮点击后要执行的正常业务逻辑
 */
private void showInterstitialIfMeetThreshold(int currentCount, int threshold, Runnable normalTask) {
    // 广告未加载完成/未到触发阈值，直接执行业务逻辑
    if (mInterstitialAd == null || currentCount < threshold) {
        if (normalTask != null) normalTask.run();
        return;
    }

    // 满足条件展示广告
    mInterstitialAd.show(HomeActivity.this);
    mInterstitialAd.setFullScreenContentCallback(new FullScreenContentCallback() {
        @Override
        public void onAdDismissedFullScreenContent() {
            super.onAdDismissedFullScreenContent();
            // 加载下一条广告
            mInterstitialAd = null;
            intertitalAd();
            // 重置对应组的计数器
            if (threshold == THRESHOLD_NAV) {
                clickCountNav = 0;
            } else {
                clickCountOperation = 0;
            }
            // 广告关闭后执行被打断的业务逻辑，避免点击无响应
            if (normalTask != null) normalTask.run();
        }
    });
}

3. 改造所有按钮点击方法 #

每个按钮只需要做三件事：给所属组的计数器+1、把自身正常业务逻辑封装为Runnable、调用通用广告检查方法即可，不需要重复写广告相关代码。

导航类按钮改造示例（back方法） #

@SuppressLint("SetTextI18n")
private void back() {
    clickCountNav++;
    // 封装原本的正常业务逻辑
    Runnable clickLogic = () -> {
        if (position > 0) {
            position = (position - 1) % quotes_list.size();
            quotesTxt.setText(quotes_list.get(position));
            countTxt.setText(position + "/" + quotes_list.size());
            Log.d(TAG, quotes_list.toString());
        }
    };
    showInterstitialIfMeetThreshold(clickCountNav, THRESHOLD_NAV, clickLogic);
}

操作类按钮改造示例（latestQuote方法） #

@SuppressLint("SetTextI18n")
private void latestQuote() {
    clickCountOperation++;
    // 封装原本的正常业务逻辑，删掉之前多余的countClicks!=3判断
    Runnable clickLogic = () -> {
        position = quotes_list.size() - 1;
        quotesTxt.setText(quotes_list.get(position));
        countTxt.setText(position + "/" + quotes_list.size());
        Log.d(TAG, quotes_list.toString());
    };
    showInterstitialIfMeetThreshold(clickCountOperation, THRESHOLD_OPERATION, clickLogic);
}

其余按钮按照相同规则改造即可：

• random、next属于导航组，使用clickCountNav计数器和THRESHOLD_NAV阈值
• copy、share、firstQuote属于操作组，使用clickCountOperation计数器和THRESHOLD_OPERATION阈值

• 无冗余变量：仅2个组级计数器，完全贴合业务规则，比单按钮计数器方案少5个变量
• 维护成本极低：后续新增按钮只要归到对应组即可，调整阈值直接修改顶部常量，修改广告逻辑只需要改通用方法一处，不用在7个按钮方法里找重复代码
• 逻辑完全隔离：两组计数互不干扰，从根源上解决跨组误触发广告的问题
• 修复原有交互bug：去掉了之前为了适配共用计数器写的补丁判断，到阈值时广告关闭后自动执行点击逻辑，不会出现点击按钮无响应的问题。

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