Ошибка № 29 ожидала выражения
Следующий фрагмент кода, который представляет собой двумерный массив, выдает ошибку #29, ожидающую выражения.
typedef enum
{
BATTERY_POW = 0,
USB_POW = 1,
END_STATE = 2
} BMTState_e;
typedef enum //event enums
{
NO_EVENT = 0,
BOOT_EVENT =1,
//I/O events
POW_GOOD_LOW =2,
POW_GOOD_HIGH = 3,
VBUS_POW_LOW = 4,
VBUS_POW_HIGH =5
}BMTEvent_e;
Структура определяется как
typedef struct //state machine definition
{
void (*funcPtr)();
BMTState_e nextState;
}BMTAction_t;
BMTState_e BMTGlobal_State ; //global state
Функция определяется как
void BMTTest()
{
//do nothing for time being
}
BMTAction_t action[END_STATE][END_EVENT]={
[BATTERY_POW][NO_EVENT] = {BMTTest,BATTERY_POW}
[BATTERY_POW][BOOT_EVENT] = {BMTTest,BATTERY_POW},
[BATTERY_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
[BATTERY_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW}
[BATTERY_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
[BATTERY_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW},
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW_}
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
[USB_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
[USB_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW}
[USB_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
[USB_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW}
};
void BMT_HandleEvent(BMTEvent_e event)
{
BMTAction_t stAction;
if(event != NO_EVENT)
{
stAction.funcPtr = action[BMTGlobal_State][event].funcPtr;
stAction.nextState = action[BMTGlobal_State][event].nextState;
printf("current State =%d, event = %d, nextState = %d",BMTGlobal_State, event,stAction.nextState );
if(NULL!= stAction.funcPtr)
stAction.funcPtr();
BMTGlobal_State = stAction.nextState;
}
}
int main()
{
BMTEvent_e = BOOT_EVENT;
if(retVal)
{
BMTGlobal_State = BATTERY_POW;
}
else // PG is low so check VBUS signal
{
retVal = GPIO_Read_Pin(USB_VBUS_PWR_PIN);
if(retVal)
{
BMTGlobal_State = USB_POW;
}
else
{
BMTGlobal_State = CRADLE_POW;
}
}
event = BOOT_EVENT;
while (1)
{
BMT_HandleEvent(event);
}
}
Идея состоит в том, чтобы выполнить конечный автомат на основе полученных событий. В 2D массиве перечисляется текущее состояние и все возможные события для этого состояния. После получения события будет вызван указатель функции и состояние перейдет в следующее состояние.
Я использую MicroC/OS2 с компилятором / инструментами GreenHills. Буду признателен за ваш ответ.
Я последовал рекомендации Кейта
BMTAction_t action[END_STATE][END_EVENT] = {
{{NULL,BATTERY_POW}}, //0
{{BMTProcess_BatteryPowBoot,BATTERY_POW}},// 1
{{BMTProcess_PowGoodLow,BATTERY_POW}},//2
{{BMTProcess_PowGoodHigh,BATTERY_POW}}, //3
{{BMTProcess_VBUSPowerLow,BATTERY_POW}}, //4
{{BMTProcess_VBUSPowerHigh,BATTERY_POW}},//5
};
Компилятор выдает ошибку в строке
"{{BMTProcess_PowGoodHigh,BATTERY_POW}}, //3"
"error # 146 too many initializer values"
4 ответа
Это неправильный синтаксис для назначенного инициализатора.Вы можете указать только один индекс за раз.
Я был не прав; допускается несколько обозначений. На самом деле ваш код компилируется без ошибок, используя gcc -std=c99 -pedantic -Wall -Wextra
после того, как я добавлю несколько объявлений.
typedef enum { BATTERY_POW, END_STATE } BMTState_e;
typedef enum { BOOT_EVENT, POW_GOOD_LOW, POW_GOOD_HIGH, END_EVENT } BMTEvent_e;
void BMTTest(void);
typedef struct //state machine definition
{
void (*funcPtr)();
BMTState_e nextState;
}BMTAction_t;
BMTAction_t action[END_STATE][END_EVENT]={
[BATTERY_POW][BOOT_EVENT] = {BMTTest,BATTERY_POW},
[BATTERY_POW][POW_GOOD_LOW] = {BMTTest,BATTERY_POW},
[BATTERY_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW}
};
Скорее всего, ваш компилятор не понимает обозначенные инициализаторы. (Они были добавлены в язык по стандарту 1999 года, и даже сегодня не все компиляторы поддерживают их.)
Если этого не произойдет, вам нужно удалить выражения в скобках (и убедиться, что элементы расположены в правильном порядке). Вероятно, что-то вроде:
BMTAction_t action0[END_STATE][END_EVENT] = {
{{ BMTTest, BATTERY_POW }},
{{ BMTTest, BATTERY_POW }},
{{ BMTTest, BATTERY_POW }}
};
Ответ Кит помог мне понять правильную инициализацию. Мне пришлось перечислить все возможные события и связанный указатель функции для данного состояния внутри {{ }}.
Это похоже на инициализацию элементов row[0][0] и row [0][1]. Мне пришлось перечислить все элементы внутри двойных фигурных скобок верхнего уровня, что-то вроде
{ **{{**values/function/events associated with row [0][0],{values/function/events associated with row [0][1] }},
//now do the same for row 1.
**{{**values/function/events associated with row [1][0],{values/function/events associated with row [1][1] **}}** }; //end of the array
BMTAction_t action[END_STATE][END_EVENT] = {
//STATE = BATTERY_POW
{{NULL,BATTERY_POW}, //0
{BMTProcess_BatteryPowBoot,BATTERY_POW},// 1
{BMTProcess_PowGoodLow,BATTERY_POW},//2
{BMTProcess_PowGoodHigh,BATTERY_POW}, //3
{BMTProcess_VBUSPowerLow,BATTERY_POW}, //4
{BMTProcess_VBUSPowerHigh,BATTERY_POW}},//5
//STATE = USB_POW
{{NULL,USB_POW}, //0
{BMTProcess_BatteryPowBoot,USB_POW},// 1
{BMTProcess_PowGoodLow,USB_POW},//2
{BMTProcess_PowGoodHigh,USB_POW}, //3
{BMTProcess_VBUSPowerLow,USB_POW}, //4
{BMTProcess_VBUSPowerHigh,USB_POW}}//5
};
Вы понимаете, что ваш код включает в себя следующее:
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW_}
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
Даже я был бы смущен этим. Вы имели в виду первым NO_EVENT
?
Используемая вами инициализация будет работать с компилятором greenhills, если вы используете -c99
вариант.
Мне пришлось внести несколько небольших изменений в ваш код, чтобы он скомпилировался:
- Добавить
END_EVENT
перечисление - Добавьте несколько запятых между элементами
action
декларация. - Исправлена опечатка
USB_POW_
вUSB_POW
Вот модифицированный код:
#include <stdio.h>
typedef enum {
BATTERY_POW = 0,
USB_POW = 1,
END_STATE = 2
} BMTState_e;
typedef enum {
NO_EVENT = 0,
BOOT_EVENT =1,
//I/O events
POW_GOOD_LOW =2,
POW_GOOD_HIGH = 3,
VBUS_POW_LOW = 4,
VBUS_POW_HIGH =5,
END_EVENT = 6
} BMTEvent_e;
typedef struct {
void (*funcPtr)();
BMTState_e nextState;
} BMTAction_t;
void BMTTest()
{
//do nothing for time being
}
BMTAction_t action[END_STATE][END_EVENT]={
[BATTERY_POW][NO_EVENT] = {BMTTest,BATTERY_POW},
[BATTERY_POW][BOOT_EVENT] = {BMTTest,BATTERY_POW},
[BATTERY_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
[BATTERY_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW},
[BATTERY_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
[BATTERY_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW},
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
[USB_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
[USB_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW},
[USB_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
[USB_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW}
};
int main()
{
return 1;
}
Вот вывод компилятора без -c99
:
$ ccppc test.c
"test.c", line 30: error #29: expected an expression
[BATTERY_POW][NO_EVENT] = {BMTTest,BATTERY_POW},
^
"test.c", line 31: error #29: expected an expression
[BATTERY_POW][BOOT_EVENT] = {BMTTest,BATTERY_POW},
^
"test.c", line 32: error #29: expected an expression
[BATTERY_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
^
"test.c", line 33: error #29: expected an expression
[BATTERY_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW},
^
"test.c", line 34: error #29: expected an expression
[BATTERY_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
^
"test.c", line 35: error #29: expected an expression
[BATTERY_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW},
^
"test.c", line 36: error #29: expected an expression
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
^
"test.c", line 37: error #29: expected an expression
[USB_POW][BOOT_EVENT] = {BMTTest,USB_POW},
^
"test.c", line 38: error #29: expected an expression
[USB_POW][POW_GOOD_LOW] = {BMTTest,USB_POW},
^
"test.c", line 39: error #29: expected an expression
[USB_POW][POW_GOOD_HIGH] = {BMTTest,BATTERY_POW},
^
"test.c", line 40: error #29: expected an expression
[USB_POW][VBUS_POW_LOW] = {BMTTest,BATTERY_POW},
^
"test.c", line 41: error #29: expected an expression
[USB_POW][VBUS_POW_HIGH] = {BMTTest,USB_POW}
^
И с -c99
:
$ ccppc -c99 test.c