Не работает прибавление 1 к переменной при нажатии на кнопку Опции

[loghir]

Доброго времени суток!
Всех с Днем Победы! Желаю чтобы агрессивные политики воевали только в парламентах по мордам друг друга!

биты конфигурации задаю вручную, RA0 сидит на +5В через резистор 4к7, кнопка на "массу", МК - pic16f72.
Срабатывание кнопки гарантировано доходит до RA0 (мерял на выводе). При "ручной" записи в порт через переменную индикация работает нормально.

Код

#include <pic.h>

#define knopka_start RA0
#define all_1 RC4
unsigned char time1 = 0;

void podgot (void)
{
ADCON1 = 0x07;
TRISA = 0b111111;
TRISB = 0b00000000;
TRISC = 0b00000000;
RBPU = 0;
PORTA = 0;
PORTB = 0b11111111;
PORTC = 0b11111111;
}

const unsigned char arr_seg [10] =
{
/* начало массива
0bABCDEFGH <– расположение сегментов по битам */
0b00000011, // «0» !
0b10011111, // «1» !
0b00100101, // «2» !
0b00001101, // «3» !
0b10011001, // «4» !
0b01001001, // «5» !
0b01000001, // «6» !
0b00011111, // «7» !
0b00000001, // «8» !
0b11110111, // «9» !
};

void main (void)
{
podgot();
all_1 = 0;
PORTB = arr_seg [time1];
if (knopka_start == 0);
{
time1 = time1 + 1;
}
}

Сообщение отредактировал loghir - May 9 2011, 06:05

[loghir]

Тут не гадать, а пробовать надо: в другом "букваре" нашел другое толкование для ";":

Точка с запятой ";" - это ограничитель оператора. Любое допустимое в Си выражение (включая пустое) завершенное точкой с запятой интепретируется как оператор, называемый еще оператором выражения. Выражение вычисляется и его значение пропадает. Если выражение не имеет побочных эффектов, mikroC может игнорировать его.

Ничего нового: зачем-то пишут книги с ошибками (или, в лучшем случае, с недосказанностями), народ за них платит
неслабые деньги, и, эквигенитально, ничего не работает в программе.
if(); - в примерах кода ";" после "if()" не встречается.
А про

Код

if()
{};

прямо написано, что так писать код нельзя.

Теперь о практике:

Код

time2 = time2++;

Ну, я на знание кашрута не претендую. Рано! Обычно пишу

Код

time2 = time2 + 1;

мне так понятнее.

И строку

Код

knopka_prev ^= 1; // изменим сохранённое состояние
               // (так будет короче и корректнее, чем knopka_prev = knopka_start;)

мне понятнее написать как

Код

knopka_prev = knopka_prev ^ 1; // если knopka_prev = 1, возвращает 0; если knopka_prev = 0, возвращает 1.

Сообщение отредактировал loghir - May 10 2011, 13:24