Помогите с кодом (не могу найти ошибку)

[gosenergo]

Сразу скажу, я только учусь. Написал код бегущих светодиодов, проблема в кнопке!  при первом нажатии светодиоды (10шт) включаются, бегущий режим работает как задумано, но на повторные нажатия, которые должны привести к выключению светодиодов (остановке), ничего не происходит... они бегут без остановки((( кнопка на ноге RA2, пик 16f84a

#include <xc.h>
// CONFIG
#pragma config FOSC = HS        // Oscillator Selection bits (HS oscillator)
#pragma config WDTE = OFF       // Watchdog Timer (WDT disabled)
#pragma config PWRTE = ON       // Power-up Timer Enable bit (Power-up Timer is enabled)
#pragma config CP = OFF         // Code Protection bit (Code protection disabled)

#ifndef _XTAL_FREQ
#define _XTAL_FREQ 4000000
#endif

void pauza(void);
void nop(void);
unsigned char NAG;


void pauza(void)
{
    __delay_ms(30);
}
       
void nop(void)
{
    if(RA2==0)
        {__delay_ms(10);
        if(RA2==0&&NAG==0)
        NAG=1;
        }
    else 
        {
        if(RA2==0&&NAG==1)
        NAG=0;
        }
    }
           

void main(void)
{
    TRISB=0;
    PORTB=0;
    TRISA=0b11111100;
    PORTA=0b00000000;
    while(1)
   {
nop();
if(NAG==1)
        {
         PORTB=0b00000001;
pauza();
         PORTB=0b00000010;
pauza();
         PORTB=0b00000100;
pauza();
         PORTB=0b00001000;
pauza();
         PORTB=0b00010000;
pauza();
         PORTB=0b00100000;
pauza();
         PORTB=0b01000000;
pauza();
         PORTB=0b10000000;
pauza();
         PORTB=0b00000000;
         PORTA=0b00000001;
pauza();
         PORTA=0b00000010;
pauza();
         PORTA=0b00000000;
    }
   }
}

__eeprom.d __eeprom.i __eeprom.p1 memoryfile.xml test.X.production.cmf test.X.production.elf test.X.production.hex test.X.production.hxl test.X.production.lst test.X.production.map test.X.production.mum test.X.production.o test.X.production.rlf test.X.production.sdb test.X.production.sym

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[Lexter]

Может просто кнопку надо дольше держать нажатой? У вас её состояние при "бегущих огнях" опрашивается где-то раз в полсекнды.

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[gosenergo]

Цикл длится меньше 0.5сек, но пробовал держать дольше, не останавливаются(((

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[Lexter]

@gosenergo , извини, не сразу заметил:

NAG при нажатой кнопке всегда равен "1". Скобка перед else не там поставлена, else не выполняется. Попробуйте так:

void nop (void)

{     if (RA2==0)

        {

         __delay_ms(10);

         if (RA2==0)

                    {

                    if (NAG==0)   NAG=1

                    else    NAG=0;

                    }

        }

  }

[IMXO]

попробуйте для начала взять за правило не использовать зарезервированные имена в названии функций типа NOP() 

и проверка нажатия делается так:

void nop_Button(void)
{unsigned char us_Button=0;
 static unsigned char us_Button_old = 0;
if (!RA2) {us_Button=1;}
if ((us_Button==1)&&(us_Button_old==0)) NAG^=1;
us_Button_old = us_Button;  
}

 

[gosenergo]

10 часов назад, IMXO сказал:

//////////////////////////

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

 

Изменено 27 апреля, 2020 пользователем Falconist

[gosenergo]

Пошло по последнему коду, светодиоды включились, повторное нажатие отключились. Но не уверенно срабатывает, приходится по 5раз ловить момент нажатия((

 

Как понять действие NAG^=1

??

 

 

Изменено 27 апреля, 2020 пользователем gosenergo

[IMXO]

7 часов назад, gosenergo сказал:

не уверенно срабатывает

Как понять действие NAG^=1

тогда читаем здесь: https://forum.cxem.net/index.php?/topic/210783-помощь-с-кодом/&do=findComment&comment=3196310

 

инверсия бита 0 переменной NAG....

[gosenergo]

Разобрался))) контакт в кнопке плохой был. Все хорошо спасибо. Еще вопрос, поделитесь пожалуйста (у кого есть) примером кода С, как обрабатывать и настройку пика при работе с прерываниями по RB0 и если есть по работе с таймером, интересует пример кода по замеру времени между прерываниями?!