Question

我不知道为什么我的代码不起作用（doISR）。它应该做什么：

- 有一个行车灯（6个LED）和3个按钮。我需要在正确的时间按下按钮（按钮1，如果LED 1或2打开......等）。如果正确：增加速度，如果不是：重置。

我打赌这是一个大错，但我不明白：）

   void wait(void) {
   while (!(TIFR1 & (1 << OCF1A))) // wait
   ;
   TIFR1 |= (1 << OCF1A); // delete Interrupt flag 
}

volatile bool ISRhasHappend = false;

ISR(PCINT0_vect) {
    ISRhasHappend = true;
}

int main(void) {
    OCR1A = 3125 - 1; //2 seconds
    TCCR1B |= (1 << WGM12); //turn on CTC mode
    TCCR1B |= (1 << CS12) | (1 << CS10); //Prescalemode 101 -> 1024 Takt
    DDRB = (1 << PCINT5); 
    DDRC = 0x3f; 
    PCICR = 0x3f; //Pin Change Interrupt Control Register
    PCMSK0 = (1 << PCINT0) | (1 << PCINT2) | (1 << PCINT4); 

   sei();
   doRunningLight();
   if (ISRhasHappend == true) {
        doISR();
   }
}

void doISR() {

    //while timee >0
    for(int x=3125;x>0;x=x-250){
        //if LEDs 0+1 on, and button0 pressed ...etc                                            
        if ((PORTC & (0b00000011)) && (PINB & (1 << PINB0)) || (PORTC & 
        (0b00001100)) && (PINB & (1 << PINB2)) || (PORTC & (0b00110000)) && 
      (PINB & (1 << PINB4))) {
               //All led lights up
               PORTC |= 0x3f;
               wait();
               //reduce timer if catched the light
               OCR1A = x;
           }
           else {
               //turn signal
               for (int y = 1; y < 5; y++) {
                   PORTB ^= (1 << PCINT5);
                   wait();
               }
                //back to 3124 if failed
                OCR1A = 3125 - 1;
           }
    }
    ISRhasHappend = false;
}

void doRunningLight(){

   while(1) {
       for (int i = 0; i<6; i++){
           PORTC |= (1<<i);
           wait();
           PORTC  &= ~(1<<i);
       }
   }
}

Answer 1

函数doRunningLight()永远不会返回，因此您在检查ISRhasHappend并调用doIsr之后编写的代码将永远不会运行。您需要考虑一种不同的方式来处理事件，可能不使用阻塞延迟循环。

我还会质疑你是否真的需要一个引脚更改中断。按下按钮的速度足够快，以至于您的微控制器应该能够通过读取主回路中的引脚来检测它们。

C - atmega328p - 我不明白有什么不对

1 个答案: