LED 闪烁顺序

Question

我有一个在线程之间共享的 volatile unsigned char 数组 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 毫秒，关闭毫秒。我们继续处于状态 2 直到设置状态 3。状态 3 的 ON 定时器为 1，没有 OFF 定时器，这意味着 LED 应该一直亮着。

状态 3 是基本状态，即状态 3 之后的任何状态，应根据计时器闪烁并返回状态 3。
例如，Led blue 在状态 3 之后打开，当设置状态 6 时，LED 应闪烁 100ms ON 和 100ms OFF。 LED 应该闪烁直到状态 6 关闭并返回状态 3。所以基本上它是基于优先级的。如果状态 7 也为 ON，则在完成状态 6 后，它应该闪烁状态 7，直到状态 7 关闭，然后应该返回状态 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 个状态。 （后来，在 cmets 你说有 9 个状态。）明确定义有多少个状态，然后更改数组以支持所有状态：（注意，数组不是创建为 volatile，不是必须是.)

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

3) 考虑为您的state machine 使用switch() 语句。

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