Multiple если заявления

Question

гипотетически допустим я использую один, если заявление для учета десяти возможных нажатий кнопок, которые посылают значение события на слушателя событий:

public boolean onTouch(int v) { //this is my only listener for all ten buttons 
    if(event.getAction() == MotionEvent.ACTION_DOWN){ 
    if(v==button_one){pool.play(bass1, 1f,1f, 1, 0, 1f);} 
    if(v==button_two){pool.play(bass2, 1f,1f, 1, 0, 1f);} 
    if(v==button_three){pool.play(bass3, 1f,1f, 1, 0, 1f);} 
    if(v==button_four){pool.play(snare1, 1f,1f, 1, 0, 1f);} 
    if(v==button_five){pool.play(snare2, 1f,1f, 1, 0, 1f);} 
    if(v==button_six){pool.play(snare3, 1f,1f, 1, 0, 1f);} 
    if(v==button_seven){pool.play(hh1, 1f,1f, 1, 0, 1f);} 
    if(v==button_eight){pool.play(hh2, 1f,1f, 1, 0, 1f);} 
} 
return false; 
} 

было бы более эффективно классифицировать их? скажем ... одно событие onClick для ловушек и одно для басов и одно для hi-шляп, так что при нажатии кнопки программа не должна проверять каждую инструкцию if, только те, которые находятся внутри слушателя для уволенного мероприятие?

Answer 1

Я бы не стал беспокоиться о том, чтобы сломать это до тех пор, пока вы не узнаете, что это проблема. Переключатель поможет уменьшить повторное if заявление:

public boolean onTouch(int v) { 
    if(event.getAction() == MotionEvent.ACTION_DOWN) { 
    switch (v) { 
    case button_one: pool.play(); break; 
    case button_two: pool.play(..); break; 
    ... 
    } 
    } 
return false; 
} 

Answer 2

А что-то вроде этого?

HashMap<int,int> soundMap = new HashMap<int,int>(); 
soundMap.put(button_one, bass1); 
soundMap.put(button_two, bass2); 
soundMap.put(button_three, bass3); 
soundMap.put(button_four, snare1); 
soundMap.put(button_five, snare2); 
soundMap.put(button_six, snare3); 
soundMap.put(button_seven, hh1); 
soundMap.put(button_eight, hh2); 

Имеет HashMap как переменный класс, и есть отображение инициализировано в OnCreate или что-то. Тогда вы можете просто использовать это для вашего слушателя:

public boolean onTouch(int v) { 
    if(event.getAction() == MotionEvent.ACTION_DOWN) { 
     pool.play(soundMap.get(v), 1f, 1f, 1, 0, 1f); 
    } 
    return false; 
} 

Преимущество этого в том, что если у вас есть, чтобы добавить дополнительные кнопки в будущем, вам нужно только изменить метод инициализации карты с новым звуковым отображением; слушатель не потребует никаких изменений.

Answer 3

В этом случае не стоит беспокоиться о производительности; время, затраченное на условное утверждение, будет полностью затмевано кодом обработки событий вокруг него. Первые и третьи законы исполнения - это измерение, измерение, измерение, и я скептически отношусь к шансам найти разницу.

Я не могу не заметить, что единственное, что меняется, это первый аргумент для pool.play. Есть ли связь между bass1, bass2 и т. Д. И соответствующими значениями v?

Answer 4

Я не думаю, что это необходимо, но я бы определенно рекомендовал выключатель и корпус. Вы можете заказать как наиболее распространенные, наименее распространенные, но такие небольшие изменения, которые не будут замечены. Это не занимает очень много времени, чтобы пройти через те заявления if.

Answer 5

Не знаю, сколько раз это if работает, но для меня это кажется неэффективным. Этот булевский тест очень быстрый. Во всяком случае, если вы хотите, чтобы сделать его как можно более эффективным, я вижу два варианта:

1. Простой один: использовать else if вместо if.
2. Сложные один: использовать массив ActionIf объектов, чтобы сделать то, что вы хотите:

public interface ActionIf { 
    public void go(); 
} 

public class ActionBass1 implements ActionIf { 
    @Override 
    public void go() { 
     pool.play(bass1, 1f,1f, 1, 0, 1f); 
    } 
} 

public class ActionBass2 implements ActionIf { 
    @Override 
    public void go() { 
     pool.play(bass2, 1f,1f, 1, 0, 1f); 
    } 
} 

... 

public ActionIf[] actions = {new ActionBass1(), new Action Bass2(), ...); 

public boolean onTouch(int v) { //this is my only listener for all ten buttons 
    if(event.getAction() == MotionEvent.ACTION_DOWN && v >= 0 && v <= (button_eight-button_one)){ 
     actions[button_one+v].go(); 
    } 
    return false; 
} 

Answer 6

Просто, чтобы добавить к богатой вене советов и мнения здесь, и согласованию со всеми ранее о " это действительно проблема производительности », я бы пошел на структуру, которая делает код проще всего читать и обслуживать вас, и всех, кому может понадобиться его поддерживать. При обслуживании, подумайте также о его расширении. Что происходит через 3 месяца, когда вы хотите добавить еще 5 звуковых колодок?

Уменьшение минимального количества строк может дать вам почти неизмеримое повышение производительности и сохранить несколько байтов в вашем APK, но в большинстве случаев я бы обменял его на удобочитаемость.

Все, что сказал, мне нравится решение kcoppocks. для меня это короткое сладкое и элегантное, и как бы я это сделал, но, если ваш уровень опыта отличается, и вы не можете просто взглянуть на него и сказать «да, я понял», а затем сохранить ваше ifs или лучше, переключатель.