Вопрос по синтаксису в С

[ritmauto]

Только начал перелазить с асма на С. Вот наткнулся на проблему:

Если делать так:
--------------
if (условие1 )
действие1;
действие2;
if (условие2 && условие3 )
действие2;
---------------
то всё нормально.

действие2 это довольно большой кусок кода.
Поэтому, чтобы не вставлять его два раза, решил попробовать сделать так:
--------------
if (условие1 )
действие1;
if ((условие1 ) || (условие2 && условие3 ))
действие2;
--------------
не работает.
Похоже что связка if ((условие1 ) || (условие2 && условие3 )) как-то не так компилится. А почему?

[ut1wpr]

Только начал перелазить с асма на С. Вот наткнулся на проблему:

Если делать так:
--------------
if (условие1 )
действие1;
действие2;
if (условие2 && условие3 )
действие2;
---------------
то всё нормально.

действие2 это довольно большой кусок кода.
Поэтому, чтобы не вставлять его два раза, решил попробовать сделать так:
--------------
if (условие1)
действие1;
if ((условие1) || (условие2 && условие3))
действие2;
--------------
не работает.
Похоже что связка if ((условие1 ) || (условие2 && условие3 )) как-то не так компилится. А почему?

По первому коду у вас действие2 будет выполняться ВСЕГДА, а согласно второму правилу еще и дважды.

if(условие1) действие1;
действие2;
if(условие2 && условие3) действие2;

Вы действительно так хотите? Или забыли про операторные скобки?
Насчет "большого куска кода" - оформите в виде функции, сможете вызывать сколько угодно раз. Но с логикой советую разобраться. У вас в примере она хромает.
Рекомендую правильно форматировать текст, будет понятнее в чтении.

[Bill]

Только начал перелазить с асма на С. Вот наткнулся на проблему:

Если делать так:
--------------
if (условие1 )
действие1;
действие2;
if (условие2 && условие3 )
действие2;
---------------
то всё нормально.

действие2 это довольно большой кусок кода.
Поэтому, чтобы не вставлять его два раза, решил попробовать сделать так:
--------------
if (условие1 )
действие1;
if ((условие1 ) || (условие2 && условие3 ))
действие2;
--------------
не работает.
Похоже что связка if ((условие1 ) || (условие2 && условие3 )) как-то не так компилится. А почему?

в первом варианте действие1 выполняется вначале безусловно, в не зависимости от условия1. Разберитесь сначала с тем, что именно вам нужно. Нарисуйте алгоритм на уровне квадратиков, тогда сразу станет ясно. Или нет?
PS: можете выложить кусок ассемблерного кода здесь. Авось общими усилиями разберемся.

[hostel6psu]

все что в "действие2" загнать в функцию. тогда два раза ничего не вставится. будет только вызов ф-ии.

[ritmauto]

По первому коду у вас действие2 будет выполняться ВСЕГДА, а согласно второму правилу еще и дважды.

Код:

if (условие1) действие1; действие2; if (условие2 && условие3) действие2;

Вы действительно так хотите? Или забыли про операторные скобки?
Насчет "большого куска кода" - оформите в виде функции, сможете вызывать сколько угодно раз. Но с логикой советую разобраться. У вас в примере она хромает.
Рекомендую правильно форматировать текст, будет понятнее в чтении.

Забыл про скобки, сорри.
Вот так правильно:

if (условие1) 
{
   действие1;
   действие2;
}
if (условие2 && условие3) действие2;

Насчет "большого куска кода" - оформить в виде функции - можно.
Просто хотел понять почему не работает так:

if (условие1) действие1;

if ((условие1) || (условие2 && условие3)) действие2;

Причем! Выяснил - из ((условие1) || (условие2 && условие3)) не отрабатывает только первая часть ИЛИ. По второй части отработка нормальная.

[ut1wpr]

Выяснил - из ((условие1) || (условие2 && условие3)) не отрабатывает только первая часть ИЛИ. По второй части отработка нормальная.

Проверяйте само условие. В приведенном примере ошибки нет.

[ritmauto]

Проверяйте само условие. В приведенном примере ошибки нет.

Хм.. Так первый вариант же работает отлично. Т.е. Условие1 отрабатывается нормально.
вот думаю - не может это конкретный компилятор дурить (Keil), какие-то его установки?? С получающимся ассемблером пока не разбирался.

[Bill]

Хм.. Так первый вариант же работает отлично. Т.е. Условие1 отрабатывается нормально.
вот думаю - не может это конкретный компилятор дурить (Keil), какие-то его установки?? С получающимся ассемблером пока не разбирался.

прежде всего грешат на компилятор.

[ut1wpr]

Хм.. Так первый вариант же работает отлично. Т.е. Условие1 отрабатывается нормально.
вот думаю - не может это конкретный компилятор дурить (Keil), какие-то его установки?? С получающимся ассемблером пока не разбирался.

Не надо людям голову морочить, приведите текст условий. Смоделируем, посмотрим. Чего гадать? Условие1 - это еще не условие..

[E_C_C]

Может дело в приоритетах операций ? приоритет лог И выше чем лог ИЛИ.
Поставьте вместо условия1 единицу или true ( как там в Вашем компиляторе), действие2 должно выполниться в любом случае.

if ((1) || (условие2 && условие3)) действие2;

оффтоп - как ,черт возьми ,все путем было в Паскале , четко стройно ,алгоритмично.