c - LED 闪烁顺序

Question

我有一个在线程之间共享的 volatile 无符号字符数组 LedState[9] 变量。数组中的每个索引表示一个状态。根据每种状态，LED 会以不同的顺序闪烁。一个线程设置数组中的状态，另一个线程根据数组索引使 LED 闪烁。每个状态都维护一组 ontimer 和 offtimer 数组。

无符号长TimersForBlueLedOn[] = {100,200,500,1,0,0,100,200,500};

无符号长TimersForBlueLedOff[] = {100,200,500,0,0,0,100,200,500};

在主线程中，我遍历数组中的每个状态并检查状态是否打开。 如果状态打开，我会闪烁 LED 以显示与该状态相对应的计时器值。

例如:状态 2 必须打开 500 毫秒，关闭 5 毫秒。我们继续处于状态 2，直到设置状态 3。状态 3 的 ON 计时器为 1，并且没有 OFF 计时器，这意味着 LED 应始终亮起。

状态 3 是基本状态，即状态 3 之后的任何状态，应根据计时器闪烁并应返回到状态 3。
例如，蓝色 LED 在状态 3 后打开，当设置状态 6 时，LED 应闪烁 100ms ON 和 100ms OFF。 LED 应该闪烁，直到状态 6 关闭并返回到状态 3。所以基本上它是基于优先级的。如果状态7也为ON，则在完成状态6后，应闪烁状态7，直到状态7为OFF并返回到状态3。

我的问题是闪烁看起来像闪烁，因为状态 3 始终设置。我需要无状态过渡。我无法根据下一个状态关闭状态 3。

    void TurnOnLed(ModemState state) {
    LEDState[state] = 1;
}

void TurnOffLed(ModemState state) {
    LEDState[state] = 0;
}

unsigned char CheckLedState(unsigned char state) {
    return LEDState[state];
}
void GetLedStateVar(LEDStateVar *pLS) {
    unsigned char state = pLS->State;
    pLS->LongflashCode = INVALID_VAL;
    switch(state) {
          case ModemTurnOn:
              pLS->LED = Blue;
              pLS->OnTimer  = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
          case ModemInit:
              pLS->LED = Blue;
              pLS->OnTimer  = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
          case GSMConnected:
              pLS->LED = Blue;
              pLS->OnTimer  = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
          case GPRSOn:
              pLS->LED = Blue;
              pLS->OnTimer  = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
         case ServerNotConnected:
              pLS->LED = Green;
              pLS->OnTimer = TimersForGreenLedOn[state];
              pLS->OffTimer = TimersForGreenLedOff[state];
              break;
         case SwUpdateDownload:
              pLS->LED = Blue;
              pLS->OnTimer = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
         case SwUpdateRestart:
              pLS->LED = Blue;
              pLS->OnTimer = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
         case SwUpdateNewVersion:
              pLS->LED = Blue;
              pLS->OnTimer = TimersForBlueLedOn[state];
              pLS->OffTimer = TimersForBlueLedOff[state];
              break;
}

void FlashBlueLed(LEDStateVar *pLSV) {

    if(pLSV->OnTimer == 1) {
              SetLEDBlue(1);          
      }else  {

          if(GetElapsedTime(&BlueFlashTimer) >  pLSV->OnTimer * MILLI_SECONDS) {
             if(!GetLEDBlue()) {
                 SetLEDBlue(1);
                 StartTimer(&BlueFlashTimer);
             }
          }

          if(GetElapsedTime(&BlueFlashTimer) > pLSV->OffTimer * MILLI_SECONDS) {
            if(GetLEDBlue()) {
                SetLEDBlue(0);
                StartTimer(&BlueFlashTimer);
             }
          }
    }
}   

 for(unsigned char i=0; i< FLASHSTATES; ++i) {
      LF.State = i;
      GetLedStateVar(&LF);

      //Flashcode not complete but the state has been reset
      if(i == LastBlueState || i == LastGreenState) {
          if(LF.LED == Blue) {  // BLUE LED
              FlashBlueLed(&LF);

          }else if(LF.LED == Green) {
              FlashGreenLed(&LF);
          }
      } else if(CheckLedState(i) && LF.OnTimer) {

          if(LF.LED == Blue) {  // BLUE LED
              if(LastBlueState == INVALID_VAL) {
                  FlashBlueLed(&LF);
              }
          } else if(LF.LED == Green) {  // GREEN LED
              if(LastGreenState == INVALID_VAL) {
                  FlashGreenLed(&LF);
              }
          } else if(LF.LED == Both) { //BOTH GREEN AND BLUE LED
              FlashBothLeds(&LF);
          }
      }
    }

Answer 1

目前仍缺少完整分析所需的信息，但目前可以提出这些改进建议。

1) 通过将值 0 分配给 LEDState 变量，您将写入 volatile :

void TurnOffLed(ModemState state) {
    LEDState[state] = 0;
}

你不应该这样做。 These should be read only 。无论如何，您所描述的方法中没有任何内容表明需要 volatile 。 (除非您的硬件或应用程序外部的某些其他进程正在访问 LEDState[i]。)
事实上，这个变量数组需要由程序内的各个线程进行更新，但这并不意味着它必须是 volatile 的。事实上，您的数组为每个线程都有一个专用元素就足够了，只要您严格限制对其各自线程对每个元素的访问，使用此方法更新状态就可以了。但这种方法不支持使用 volatile 。

More about volatile HERE

2) 您声明数组中的每个索引都表示一个状态。 volatile 数组只能容纳 5 个状态，但您在描述中最多提到了 7 个状态。 (后来，在评论中你说有 9 个状态。)明确定义有多少个状态，然后更改数组以支持所有状态:(注意，数组不是作为 volatile 创建的，不是有必要吗。)

#define MAX_STATES 9
unsigned char array LedState[MAX_STATES];

3) 考虑为您的状态机使用switch()语句。

... 
switch(i)  {
    case LastBlueState:
    case LastGreenState:        
        // do something
        break;
    case <some other state>
        // do something
        break;
        ...