課金ユーザーに広告が出る不具合を、判定の一元化で止めるまで

/**
 * 広告を出してよいかどうかを決める唯一の窓口。
 * 状態は BillingManager / AdFreeManager が持ったままにし、
 * ここでは「今この瞬間の答え」だけを合成する。
 */
class AdGate(
    private val billing: BillingManager,
    private val rewardAdFree: AdFreeManager,
) {
 
    /** 購入・復元・リワードのいずれかで広告非表示なら true */
    fun isAdFree(): Boolean =
        billing.isPurchasedAdFree() || rewardAdFree.isActive()
 
    /**
     * 広告を出してよいか。呼び出し側はこれ以外の判定を書かない。
     * reason を返すのは、出さなかった理由をログで追えるようにするため。
     */
    fun evaluate(): AdDecision = when {
        billing.isPurchasedAdFree() -> AdDecision.Suppress(Reason.PURCHASED)
        rewardAdFree.isActive()     -> AdDecision.Suppress(Reason.REWARD)
        !billing.isRestoreSettled() -> AdDecision.Suppress(Reason.RESTORE_PENDING)
        else                        -> AdDecision.Allow
    }
}
 
sealed interface AdDecision {
    data object Allow : AdDecision
    data class Suppress(val reason: Reason) : AdDecision
}
 
enum class Reason { PURCHASED, REWARD, RESTORE_PENDING }

when (adGate.evaluate()) {
    is AdDecision.Allow -> interstitialAd.show(this)
    is AdDecision.Suppress -> Unit  // 何もしない
}

// 素直だが、端末の時計を戻されると無効化されない
fun grant(durationMillis: Long) {
    prefs.edit().putLong(KEY_EXPIRES_AT, System.currentTimeMillis() + durationMillis).apply()
}
 
fun isActive(): Boolean =
    prefs.getLong(KEY_EXPIRES_AT, 0L) > System.currentTimeMillis()

class AdFreeManager(private val prefs: SharedPreferences) {
 
    fun grant(durationMillis: Long) {
        prefs.edit()
            .putLong(KEY_WALL_EXPIRES_AT, System.currentTimeMillis() + durationMillis)
            .putLong(KEY_BOOT_EXPIRES_AT, SystemClock.elapsedRealtime() + durationMillis)
            .putLong(KEY_GRANTED_BOOT_ID, bootId())
            .apply()
    }
 
    fun isActive(): Boolean {
        val wallLeft = prefs.getLong(KEY_WALL_EXPIRES_AT, 0L) - System.currentTimeMillis()
 
        // 付与時と同じ起動セッションのときだけ elapsedRealtime を信用する
        val sameBoot = prefs.getLong(KEY_GRANTED_BOOT_ID, -1L) == bootId()
        val bootLeft = if (sameBoot) {
            prefs.getLong(KEY_BOOT_EXPIRES_AT, 0L) - SystemClock.elapsedRealtime()
        } else {
            Long.MAX_VALUE  // 再起動後は壁時計側だけで判断する
        }
 
        return minOf(wallLeft, bootLeft) > 0L
    }
 
    /** 起動セッションの識別子（現在時刻 - 起動からの経過時間 ≒ 起動時刻） */
    private fun bootId(): Long =
        (System.currentTimeMillis() - SystemClock.elapsedRealtime()) / 1000L
 
    private companion object {
        const val KEY_WALL_EXPIRES_AT = "ad_free_wall_expires_at"
        const val KEY_BOOT_EXPIRES_AT = "ad_free_boot_expires_at"
        const val KEY_GRANTED_BOOT_ID = "ad_free_granted_boot_id"
    }
}

class AdGateTest {
 
    private class FakeBilling(
        private val purchased: Boolean,
        private val settled: Boolean = true,
    ) : BillingManager {
        override fun isPurchasedAdFree() = purchased
        override fun isRestoreSettled() = settled
    }
 
    private class FakeReward(private val active: Boolean) : AdFreeManager {
        override fun isActive() = active
    }
 
    @Test
    fun `復元が未確定のあいだは広告を出さない`() {
        val gate = AdGate(FakeBilling(purchased = false, settled = false), FakeReward(false))
        assertEquals(AdDecision.Suppress(Reason.RESTORE_PENDING), gate.evaluate())
    }
 
    @Test
    fun `購入済みならリワードの状態によらず出さない`() {
        val gate = AdGate(FakeBilling(purchased = true), FakeReward(false))
        assertEquals(AdDecision.Suppress(Reason.PURCHASED), gate.evaluate())
    }
 
    @Test
    fun `どの経路にも該当しなければ出す`() {
        val gate = AdGate(FakeBilling(purchased = false), FakeReward(false))
        assertEquals(AdDecision.Allow, gate.evaluate())
    }
}

#!/usr/bin/env bash
# tools/check-ad-gate.sh
# AdGate を経由しない広告表示の呼び出しを検出する
set -euo pipefail
 
VIOLATIONS=$(grep -rnE '\.(show|loadAd)\(' app/src/main/java \
  --include='*.kt' \
  | grep -vE 'AdGate\.kt' \
  | while IFS= read -r hit; do
      FILE="${hit%%:*}"
      # 同じファイル内で AdGate を経由しているか
      grep -q 'adGate' "$FILE" || echo "$hit"
    done)
 
if [ -n "$VIOLATIONS" ]; then
  echo "AdGate を経由していない広告呼び出しがあります:"
  echo "$VIOLATIONS"
  exit 1
fi
 
echo "ad gate check: OK"