Что лучше — один длинный метод onClick для всех кнопок или много коротких методов для каждой кнопки[android]

У меня есть 12 кнопок, разделенных на 2 группы, в каждой по 6 кнопок, все кнопки реагируют на один длинный метод onClick goToCategory().

Я могу преобразовать его во множество небольших независимых методов onclick.

Моему приложению требуется слишком много времени для рендеринга изображений после нажатия/касания - около 2-3 секунд. Я запустил ddms, чтобы посмотреть, что происходит, сделал трассировку, и мое приложение натыкается на goToCategory() - по крайней мере, я думаю, что это основная проблема, вызывающая длительную задержку рендеринга. Я могу захотеть переписать длинный метод onclick.

Что лучше с точки зрения производительности?

public void goToCategory(View v){
    switch (v.getId()){
        case R.id.scientists:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.galim1);
            hero2.setBackgroundResource(R.drawable.galim2);
            hero3.setBackgroundResource(R.drawable.galim3);
            upper_category_index = "science";
            break ;
        case R.id.scientists2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.galim1);
            hero5.setBackgroundResource(R.drawable.galim2);
            hero6.setBackgroundResource(R.drawable.galim3);
            lower_category_index = "science";
            break ;
        case R.id.politics:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.pol1);
            hero2.setBackgroundResource(R.drawable.pol2);
            hero3.setBackgroundResource(R.drawable.pol3);
            upper_category_index = "politics";
            break ;
        case R.id.politics2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.pol1);
            hero5.setBackgroundResource(R.drawable.pol2);
            hero6.setBackgroundResource(R.drawable.pol3);
            lower_category_index = "politics";
            break ;
        case R.id.akins:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.akin1);
            hero2.setBackgroundResource(R.drawable.akin2);
            hero3.setBackgroundResource(R.drawable.akin3);
            upper_category_index = "akin";
            break ;
        case R.id.akins2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.akin1);
            hero5.setBackgroundResource(R.drawable.akin2);
            hero6.setBackgroundResource(R.drawable.akin3);
            lower_category_index = "akin";
            break ;
    case R.id.folk_heroes:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.folk1);
            hero2.setBackgroundResource(R.drawable.folk2);
            hero3.setBackgroundResource(R.drawable.folk3);
            upper_category_index = "folk";
        break ;
        case R.id.folk_heroes2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.folk1);
            hero5.setBackgroundResource(R.drawable.folk2);
            hero6.setBackgroundResource(R.drawable.folk3);
            lower_category_index = "folk";
            break ;
        case R.id.hans:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.han1);
            hero2.setBackgroundResource(R.drawable.han2);
            hero3.setBackgroundResource(R.drawable.han3);
            upper_category_index = "hans";
            break ;
        case R.id.hans2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.han1);
            hero5.setBackgroundResource(R.drawable.han2);
            hero6.setBackgroundResource(R.drawable.han3);
            lower_category_index = "hans";
            break ;
        case R.id.batirs:
            categories.setVisibility(View.GONE);
            hero1.setBackgroundResource(R.drawable.kabanbai);
            hero2.setBackgroundResource(R.drawable.bogenbai);
            hero3.setBackgroundResource(R.drawable.karasai);
            upper_category_index = "batirs";
            break ;
        case R.id.batirs2:
            categories2.setVisibility(View.GONE);
            hero4.setBackgroundResource(R.drawable.kabanbai);
            hero5.setBackgroundResource(R.drawable.bogenbai);
            hero6.setBackgroundResource(R.drawable.karasai);
            lower_category_index = "batirs";
            break ;

    }
}

android performance onclick dalvik ddms
person ERJAN    schedule 02.10.2015    source источник
comment
В любом случае заметной разницы в производительности не будет, особенно для такого метода, который будет вызываться очень редко.   -  person JesusFreke    schedule 02.10.2015

Ответы (2)


arrow_upward
1
arrow_downward

Я думаю, что лучше зарегистрировать onClick в xml (макете).

Найдены похожие темы:

1) Рекомендации по определению событий кнопок в Android

2) рекомендации по обработке событий пользовательского интерфейса

person KishuDroid    schedule 02.10.2015
comment
Этот ответ должен быть комментарием - person Rajesh; 02.10.2015
comment
Это не следует комментировать, потому что это правильный формат ответа. - person KishuDroid; 02.10.2015
comment
на самом деле, у меня уже ЕСТЬ методы onClick, я спрашиваю, лучше ли МНОГИЕ отдельные onClick, чем один огромный метод onClick - person ERJAN; 04.10.2015

arrow_upward
1
arrow_downward

Привязка android:onClick с функцией в XML Layout — это привязка между onClick и функция, которую он будет вызывать. Функция должна иметь один аргумент (представление), чтобы onClick работал.

https://developer.android.com/reference/android/widget/Button.html

ТАК ВНИМАТЕЛЬНО

Регистрация android:onClick — лучший способ.

Чем именно XML-атрибут android:onClick отличается от setOnClickListener ?

person IntelliJ Amiya    schedule 02.10.2015