Почему светодиоды сами по себе меняют цвет?

Question

[Wasya UK]

При нажатии на кнопку, микроконтроллер запускает анимацию светодиодов, но потом они просто сменяют одна другую в цикле? ( В данном вопросе я новичок, прошу сильно не ругать ) Вроде они должны заменяться после нажатия кнопки, но после первого нажатия уже на нее не реагируют, и работают на автомате. Что я делаю не правильно?

void main(void)
{    
    int buttonStatus = 0;
                                     
    // PORT C
    LED_STRIP_PORT &= ~(1<<LED_STRIP_PIN);
    LED_STRIP_DDR |= (1<<LED_STRIP_PIN); 

    while (1)  
    {           
        // check ws2812 animation type by pressing the button
        if (PINB.0 == 1) {                 
            if (buttonStatus < 3) {
                buttonStatus++;
                PORTC.0 = 1;
            } else {
                buttonStatus = 0;
                PORTC.0 = 0; 
            }
        }
    
        switch (buttonStatus) {  
            // first ws2812 animation   
            case 1:
                startFistAnimation(); 
            break;   
            // second ws2812 animation
            case 2:
                startSecondAnimation();
            break;
            // third ws2812 animation
            case 3:
                startThirdAnimation();    
            break;
            // turn off ws2812
            default:
                PORTC.0 = 0; 
            break;
        }
    } 
}

Comments

[tsarevfs]

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

[Wasya UK]

YES

[Wasya UK]

Даже не представляю в чем проблема. Вроде кнопка получается постоянно зажатой

[Wasya UK]

Евгений Андриенко, void startFistAnimation(){

[Wasya UK]

void startSecondAnimation(){
    unsigned int i,j;  
    unsigned int n[10][3] = {
        {100,10,200},
        {200,10,60},
        {20,10,240},
        {100,240,200},
        {100,10,200},
        {10,10,240},
        {10,240,240},
        {150,240,30},
        {1,10,200},
        {240,10,240},
    }; 
    
    for (j = 0; j < 10; j++){
        for(i = 0; i < LED_COUNT; i++)
        {
          rgb_color rgb;
          rgb.red = n[j][0];
          rgb.green = n[j][1];
          rgb.blue = n[j][2];
          colors[i] = rgb;
        }

        writeScreen(colors, LED_COUNT);

        delay_ms(150);
    }    

}

[Wasya UK]

Тут можно посмотреть полный код

Answer 1

[Роман]

У Вас нет никакой задержки в цикле while (1), поэтому при нажатии на кнопку переменная buttonStatus успевает увеличиться сразу до трех. Возможно в этом проблема.

Comments

[Wasya UK]

не, не помогло

[Wasya UK]

Даже такое пробовал

if (PINB.0 == 1) { 
            PINB.0 = 0; 
            PORTB.0 = 0;  
            
            delay_ms(200);
                          
            if (buttonStatus < 3) {
                buttonStatus++;
                PORTC.0 = 1;
            } else {
                buttonStatus = 0;
                PORTC.0 = 0; 
            }   
        }

[Роман]

А выводить какой-нибудь дебаг есть возможность?
что за микроконтроллер?

[Wasya UK]

Atmega 8, но как менять анимацию, если пока она не завершиться, то не будет проверки на нажатие кнопки? Нужно держать кнопку, тогда номер увеличится и анимация измениться.

Answer 2

[Ilia Zhitenev]

используйте прерывания. Когда происходит изменение состояние на ножке с кнопкой - изменяете цвет. Например по каждому нажатию. В основном цикле уберите изменение цвета.

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