CodeVision AVR в вопросах и ответах

[Аlex]

Потому, что после выполнения каждого условия, программа попадает на следующее, и оно выполняется.
Используйте switch-case или else if.

[unalex]

мне кажется изза того, что при приеме нового бита переменная temp сбрасывается, попробуйте объявить ее как static

[ibiza11]

+1 Alex и +1 unalex оба правы.

Код: Выделить всё

// USART Receiver interrupt service routine
interrupt [USART_RXC] void usart_rx_isr(void)
{
   static unsigned char temp;
   switch(temp)
   {
      case 0:
         ru = UDR;
         temp = 1;
         break;
      case 1:
         gu = UDR;
         temp = 2;
         break;
      case 2:
         bu = UDR;
         temp = 0;
         break;
      default:
         break;
   }
}

[Аlex]

temp у него явно не локальная, т.к. в области видимости ф-ии, моим зрением, не обнаруживается

Код: Выделить всё

// USART Receiver interrupt service routine
interrupt [USART_RXC] void usart_rx_isr(void)
{
if(temp == 0){temp = 1; ru = UDR; };
if(temp == 1){temp = 2; gu = UDR; };
if(temp == 2){temp = 0; bu = UDR; };
}

Из этого следует, что она не может сбрасываться

[unalex]

мыслю абстрактно - не вседа в форум пастят весь кусок функции

каждая посылка заносятся в переменную ru

а вот одну мою опечаточку - бит != байт заметил только что

[ibiza11]

В общем, думаю автор уже разобрался.

PS/ Вопрос про точку с запятой в конце строки:

if(temp == 0){temp = 1; ru = UDR; } ;

Это допустимо? разве круглые скобки не завершают оператор if?

[Совесть]

Этот вопрос вы могли бы проверить сами, в течении пары минут. В данном случае все корректно.

[unalex]

если компилятор не ругается - это не значит корректно
точка с запятой там не нужна

[Совесть]

"если компилятор не ругается - это не значит корректно" - а что не так в этой записи? чему она противоречит? вашему стилю написания программы?

[ibiza11]

Этот вопрос вы могли бы проверить сами, в течении пары минут.

Мог бы - не спрашивал бы. Не было компилятора под рукой. Сейчас проверил - ни ошибок ни предупреждений.
Но смысл так писать? Новый стандарт Си?)))

[Совесть]

В чем отличие от используемого ВАМИ стандарта? В каком из стандартов Си данная запись будет ошибочна?

[Аlex]

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

Код: Выделить всё

if(....)
   ....
else
   ;

Это если не хочется ставить НОП, и не хочется else сносить. Компилятором он (else) будет отброшен.
Так что стандарт не запрещает его ставить ни после скобок, ни до, ни стопицот раз подряд.

[ibiza11]

В чем отличие от используемого ВАМИ стандарта? В каком из стандартов Си данная запись будет ошибочна?

Вы случайно по национальности не еврей? Я вопросы задал, Вы, не ответив на мои, мне свои задаете.

[Совесть]

Ок. Отвечу.
"Но смысл так писать?" - это всего лишь стиль записи, кому как удобнее. Каждый выбирает для себя свой, либо подстраивается под требуемый.
"Новый стандарт Си?)))" - чтобы ответить на этот вопрос, надо узнать ваши познания в стандартах Си, т.е. относительно чего он может быть новым. Ради чего и были заданы два наводящих вопроса.
Теперь вы на мои ответите?

[ibiza11]

Спасибо.
На Ваши тоже отвечу.

В чем отличие от используемого ВАМИ стандарта? В каком из стандартов Си данная запись будет ошибочна?

От используемого мной стандарта отличий нет.
Согласно используемому мной стандарту ошибок при добавлении точки с запятой после закрывающихся фигурных скобок не будет (этого я не знал до сегодняшнего дня).
В каком конкретно стандарте Си данная запись будет ошибочна я не знаю, поскольку знаком только со стандартом С89 и стараюсь писать код не выходящий за рамки вышеуказанного стандарта. Но поскольку остальные стандарты поддерживают C89, а в С89 эта запись не является ошибочной, то думаю и в других не вызовет ошибок.

Вообще говоря стандарт не запрещает использовать в этом месте разделительную точку с запятой, но и нет ни одного указания к применению подобной конструкции, кроме раздела 3.6.3 Expression and null statements стандарта, в котором точка с запятой рассматривается как пустой оператор. Смысла ставить пустой оператор в том месте спорного кода я не вижу. (Может быть Вы видите?) Из этого можно сделать вывод, что автор тех спорных строк кода Valek3306 либо прочитал стандарт C89 и нашел в нем недосказанный момент и использует его повсеместно, поскольку ему так удобнее или привычнее, либо плодит эту конструкцию по незнанию, думая, что именно так и нужно и по-другому работать не будет. Не знаю как Вам, но мне кажется более достоверным второй вариант. Поэтому и был задан соответствующий вопрос, дабы обратить внимание в первую очередь автора на возможные пробелы в знаниях синтаксиса языка.

[Совесть]

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

[ibiza11]

Согласен. Речь велась о допустимости. Выяснили, что допустимо. Осталось выяснить, если возможно, почему именно так пишет автор.

[unalex]

"если компилятор не ругается - это не значит корректно" - а что не так в этой записи? чему она противоречит? вашему стилю написания программы?

ну конечно, проще напихать точек с запятой, а компилятор их сам уберет в процессе оптимизации


если вы считаете это правильным подходом, то флаг вам в руки

[ELcat]

Главное не писать так:

Код: Выделить всё

if (a==0); b=1;

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

[unalex]

Спойлер

коммент удален, чтобы дискуссия не переросла во флуд