Не работает прибавление 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]

Проверил код

Код

void main (void)
{
podgot();
RC4 = 0;
if (PORTA & 1)
PORTB = arr_seg [0];
else
PORTB = arr_seg [1];
}

при отпущенной кнопке на индикаторе 0, при нажатой - 1.
т.е.

Код

ADCON1 = 0x07;

работает как ожидалось.

- с какой целью Вы задействовали TMR1?

после прохождения единственного "0" при нажатии, на отпускание кнопки я выделил 0.3 с.
Именно это время отсчитывает TMR1.

- почему проверяете TMR2IF?

да, это не флаг по переполнению TMR1. Спасибо за подсказку!

- где, по Вашему мнению, в цикле выполняется проверка кнопки?

командой

Код

if((knopka_prev == 0) && (DDF == 0))

чем отличается "while (DDF = 1)" от "while (DDF == 1)", и к каким последствиям приводит "while (DDF = 1)"

"=" - используется в операции присваивания
"==" - операция отношения используются для проверки равенства и неравенства выражений. По логике, надо "while (DDF == 1)"

добавьте к рыбе в посте #9 свою инициализацию из исходной программы (не счёл нужным её повторять или править несуразности, не влияющие на скорость полёта) и проверку индекса массива, предложенную sergeeff, и будет Вам рабочий код,

а менее универсально, но проще можно?

Код

if (time1 > 9)
{
time1 = 0;
}

Сообщение отредактировал loghir - May 12 2011, 15:28