Почему CodeVision AVR игнорирует некоторые операторы в операторе «если»?

Question

Почему CodeVision AVR игнорирует некоторые операторы в операторе «если»?

0

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

   if ((key==11)^(key==13)^(key==16)^(key==17))   //this is the only 
                              // dirty hack style string needed to fix this
        {  
            switch (key)
                { 
                     case 11: { mode=0x01; break; }  //plus
                     case 13: { mode=0x02; break; }  //minus 
                     case 16: { mode=0x03; break; } //multiply
                     case 17: { mode=0x04; break; } //divide
                }  

            if (mode)
                {  
                      buffer=atof(display);  
                      firstop=1;  
                      first=1;   
                      redraw=1; 
                                      //and to delete the goto
                }   
        }

WingoPavel 11 нояб. 2014, в 16:37

Источник

0

Код не полный. пожалуйста, покажите нам определения переменных.
vlad_tepesch 11 нояб. 2014, в 17:13
0

Кстати: использование goto считается плохой практикой. Почему бы просто не поместить //some other code here в ветку else? Также ваш отступ ужасен.
vlad_tepesch 11 нояб. 2014, в 17:14
0

Я не могу поместить //some other code here в ветку else, потому что //some other code here должен выполняться в обоих случаях (if (mode) и if (! Mode) ). Во всяком случае, я попытался удалить //some other code here поскольку он не требуется текущей частью кода, и он все еще не работает. Это определения переменных
WingoPavel 11 нояб. 2014, в 17:33
0

static bit redraw=1; //this static bit first=1; //and this static bit firstop=0; //and this unsigned char mode=0x00; //and this
WingoPavel 11 нояб. 2014, в 17:35
0

Хорошо, я добавил определения переменных и изменил код для текущего состояния в моем проекте. Пожалуйста, исполни мое плохое правописание, возможно, это является причиной некоторых недоразумений.
WingoPavel 11 нояб. 2014, в 17:48
0

что с остальными определениями? давай так много вещей не хватает. как обнаружить ошибку? Я думаю, что это проблема ISR или переполнение буфера. Вы также не сказали, что не работает. Это сбой? Я не получил вещь "места". Чего-то не хватает? пожалуйста, удалите комментарии, которые больше не соответствуют.
vlad_tepesch 11 нояб. 2014, в 17:53
0

«Не работает» означает, что когда я нажимаю цифры для ввода первого операнда, затем нажимаю кнопку управления, набираю второй операнд и затем при нажатии кнопки равных ничего не происходит. По-прежнему отображается число, которое отображалось до того, как я нажал равным.
WingoPavel 11 нояб. 2014, в 18:07
0

И, наконец, я обнаружил ошибку. Это была простая логическая ошибка, я забыл, что после нажатия одинаковой клавиши режим все еще> 0. Это для дальнейшего вычисления без нажатия кнопки управления и выбора операндов. Так же, как вы нажимаете 5, затем +, затем 1 и, повторяя =, вы получите 6 , 7 , 8 и т. Д. Теперь я обновлю свой пост и покажу, что именно я изменил. Да, это довольно грязный взлом, но это только для того, чтобы убедиться, что это была ошибка.
WingoPavel 11 нояб. 2014, в 18:17
0

Теперь он работает, и я могу изменить свой код, чтобы он стал более элегантным без каких-либо взломов. Спасибо, что очень помогали, ваши вопросы помогли мне найти ошибку.
WingoPavel 11 нояб. 2014, в 18:18

Показать ещё 7 комментариев

Теги:

c++

if-statement

switch-statement

avr

1 ответ

Ещё вопросы

Код не полный. пожалуйста, покажите нам определения переменных.
Кстати: использование goto считается плохой практикой. Почему бы просто не поместить //some other code here в ветку else? Также ваш отступ ужасен.
Я не могу поместить //some other code here в ветку else, потому что //some other code here должен выполняться в обоих случаях (if (mode) и if (! Mode) ). Во всяком случае, я попытался удалить //some other code here поскольку он не требуется текущей частью кода, и он все еще не работает. Это определения переменных
static bit redraw=1; //this static bit first=1; //and this static bit firstop=0; //and this unsigned char mode=0x00; //and this
Хорошо, я добавил определения переменных и изменил код для текущего состояния в моем проекте. Пожалуйста, исполни мое плохое правописание, возможно, это является причиной некоторых недоразумений.
что с остальными определениями? давай так много вещей не хватает. как обнаружить ошибку? Я думаю, что это проблема ISR или переполнение буфера. Вы также не сказали, что не работает. Это сбой? Я не получил вещь "места". Чего-то не хватает? пожалуйста, удалите комментарии, которые больше не соответствуют.
«Не работает» означает, что когда я нажимаю цифры для ввода первого операнда, затем нажимаю кнопку управления, набираю второй операнд и затем при нажатии кнопки равных ничего не происходит. По-прежнему отображается число, которое отображалось до того, как я нажал равным.
И, наконец, я обнаружил ошибку. Это была простая логическая ошибка, я забыл, что после нажатия одинаковой клавиши режим все еще> 0. Это для дальнейшего вычисления без нажатия кнопки управления и выбора операндов. Так же, как вы нажимаете 5, затем +, затем 1 и, повторяя =, вы получите 6 , 7 , 8 и т. Д. Теперь я обновлю свой пост и покажу, что именно я изменил. Да, это довольно грязный взлом, но это только для того, чтобы убедиться, что это была ошибка.
Теперь он работает, и я могу изменить свой код, чтобы он стал более элегантным без каких-либо взломов. Спасибо, что очень помогали, ваши вопросы помогли мне найти ошибку.

Eric · Answer 1 · 2014-11-11T16-42-00.000Z

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

Вы не определяете инструкцию по умолчанию в операторе switch

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

Хорошо, я никогда не сбрасываю режим. Но это проверяется, только если я нажимаю равную кнопку, поэтому я могу изменить режим, нажав другую кнопку управления.