Внедрение Interstial AdMob для различных видов деятельности

Question

В моем приложении есть активность, содержащая кнопку 25+. Я использую промежуточную рекламу для каждого из них - это приложение для больших историй, так что я думаю, что оно не так раздражает! - В любом случае я использую этот код

public void clkBtnPart1 (View v) {

        Intent intentP1 = new Intent(PartsActivity.this,
                part1.class);
        startActivity(intentP1);

        // Interstital Ad
        if (mPublisherInterstitialAd.isLoaded()) {
            mPublisherInterstitialAd.show();
        } else {
            Log.d("TAG", "The interstitial wasn't loaded yet.");
        }
        // End Int Ad
    }

, и я повторял его в течение +25 раз для каждой кнопки - в том же упражнении - поэтому у меня большой код, и я думаю, что это может замедлить мое приложение

Q1: Как я могу оптимизировать структуру рекламного кода?

Q2: Есть ли какая-то выгода от этой строки

else {
            Log.d("TAG", "The interstitial wasn't loaded yet.");
        }

или я могу безопасно удалить ее?

ОСНОВНАЯ цель для обоих вопросов: (1) Cleaner Code (2) Более быстрое приложение

Answer 1

Брат, вы правы, это нехорошая практика. Вы можете установить для атрибута onClick в XML одинаковую функцию для всех и использовать такой код для краткости и простоты понимания и чтения

switch (id){
        case R.id.button1 :
            Intent intentP1 = new Intent(PartsActivity.this,
                    part1.class);
            startActivity(intentP1);
            loadInterstitial();
            break;
        case R.id.button2 :
            Intent intentP2 = new Intent(PartsActivity.this,
                    part2.class);
            startActivity(intentP2);
            loadInterstitial();
            break;
        case R.id.button3 :
            Intent intentP3 = new Intent(PartsActivity.this,
                    part3.class);
            startActivity(intentP3);
            loadInterstitial();
            break;
    }