Как выполнить все условия, без повторения?

Question

[Павел Дмитриев]

У меня есть 4 условия, которые должны выполняться вместе.

if ( условие 1 ) { оператор 1 } ;
if ( условие 2 ) { оператор 2 } ;
if ( ( условие 1 ) && ( условие 2 ) ) { оператор 1; оператор 2; } ;
if ( ( условие 2 ) && ( условие 1 ) ) { оператор 2; оператор 1; } ;

Но в таком виде либо 1 либо 2 условие дублируется дважды и повторяется выполнение.
Как совместить эти условия, чтобы не повторялось выполнение операторов?

Upd.:

У меня есть две кнопки, которые выполняют условия 1 и 2. При нажатии на кнопку 1 звучит звук 1, при нажатии на кнопку 2 звучит звук 2. Но вся загвоздка в том, что кнопки можно нажать одновременно. И как тогда определить какой звук первый должен звучать, какой второй? А если нажатие какой-то кнопки на миллисекунды позже, то даже просто в таком варианте

if ( условие 1 ) { оператор 1 } ;
if ( условие 2 ) { оператор 2 } ;

происходит дублирование и один звук звучит дважды.

Comments

[Андрей Мывреник]

Смысл условий не понятен. Выполнение каждого оператора должно повторяться 2 раза если оба условия верны? И смысл двух идентичных if'ов?

Answer 1

[Saboteur]

Ваш ответ такой
if ( условие 1 ) { оператор 1 } ;
if ( условие 2 ) { оператор 2 } ;

Все.

Comments

[Павел Дмитриев]

У меня есть две кнопки, которые выполняют эти условия. При нажатии на кнопку 1 звучит звук 1, при нажатии на кнопку 2 звучит звук 2. Но вся загвоздка в том, что кнопки можно нажать одновременно. И как тогда определить какой звук первый должен звучать, какой второй? А если нажатие какой-то кнопки на миллисекунды позже, то даже просто в таком варианте if ( условие 1 ) { оператор 1 } ; if ( условие 2 ) { оператор 2 } ; происходит дублирование и один звук звучит дважды.

[Saboteur]

Не морочьте голову.
У вас нет понятия какая кнопка сработала раньше, если программа УЖЕ начала выполняться. Иначе вам
тогда надо вместо условия - timestamp срабатывания кнопки, затем посортировать события и потом уже обрабатывать. А более правильно - независимые триггеры на нажатие кнопки.

То есть реальный вариант - у вас изначально неверный подход к решению.
Но если хотите потеорезировать, то вам нужно добавить ELSE
и сперва обрабатывать обе кнопки, а затем по одной.

if ( условие1 && условие2 ) { операторы 1 и 2 в удобном для вас порядке; }
else if ( условие1 ) { оператор 1; }
else if ( условие2 ) { оператор 2; }

[Павел Дмитриев]

Сергей: Такое протезирование я уже делал, в итоге дублированное звучание одного из звуков.
Можете посоветовать пример с триггером. Буду делать более правильно.

[Saboteur]

Каким образом может быть дублированное звучание? вы точно ELSE ставили?
При вышеуказанной конструкции, возможен ТОЛЬКО ОДИН блок с выполнением операторов.

а с триггером, это вам в объектно-ориентированный подход, а не функциональный.
ТО есть не не от условий опираться, а от срабатывания нажатия.

[Павел Дмитриев]

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

[Saboteur]

Павел Дмитриев: Другими словами, постановка задачи у вас изначально была неполная.
Но я не понимаю, почему у вас возникают проблемы, если каждая кнопка в своем отдельном потоке? зачем вы тогда делаете проверку условия в одном потоке, а не в потоке нужной кнопки?

P.S. Для удобства, еще можно состояние кнопок сбрасывать, например в operator1 добавить обнуление состояния кнопки2 и наоборот, чтобы не было два звука подряд - тогда та процедура, которая выполнится первой, сбросит "опоздавшую" кнопку. Нажать же кнопки ПОЛНОСТЬЮ одновременно, вряд-ли реально..

[Павел Дмитриев]

В java вообще остановка параллельных потоков это отдельная тема :) Потому наверное и проблемы, что потоки разные для каждой кнопки, а остановить потоки нормально нельзя.

Answer 2

[Василий]

if ( ( условие 1 ) && ( условие 2 ) ) { оператор 1; оператор 2; } ;
if ( ( условие 2 ) && ( условие 1 ) ) { оператор 2; оператор 1; } ;

вот этот кусок, вроде, вообще смысла не имеет, второе никогда не выполнится.

Comments

[Андрей Мывреник]

Что помешает ему выполниться?

[Василий]

gim0: да, виноват, всегда будет выполняться и первое и второе.

[Василий]

gim0: точнее, если выполняется первое, то и второе.

[Mokhirjon Naimov]

Ага не имеет смысла.

Answer 3

[Mokhirjon Naimov]

По моему надо сделать так:

if ( условие 1 && !(( условие 1 ) && ( условие 2 )) ) { оператор 1 } ;
if ( условие 2 && !(( условие 1 ) && ( условие 2 )) ) { оператор 2 } ;
if ( ( условие 1 ) && ( условие 2 ) ) { оператор 1; оператор 2; } ;
if ( ( условие 2 ) && ( условие 1 ) ) { оператор 2; оператор 1; } ;

А так 3-4 условия как парадокс! :)

Comments

[Павел Дмитриев]

Добавил апдейт к вопросу.

Answer 4

[Павел Дмитриев]

В общем сделал вот такой метод проверку и повесил на кнопки. Теперь работает как надо без дополнительных условий.

Button playOne, playTwo;
...
playOne = (Button) findViewById(R.id.playOne);
playTwo = (Button) findViewById(R.id.playTwo);
...
public void verification(Button b) {
if (b == playOne) { playSound(sound_one); } 
if (b == playTwo) { playSound(sound_two); }
}

Comments

[Saboteur]

Ну вот, я ж говорил - надо не от условий "что нажато" идти, а триггер на событие "нажатие кнопки", И условия не нужны вообще.