Понимаю, что тема уже неоднократно обсуждалась, но найти ничего не могу, потому спрошу здесь. Проект(ARM+Keil) содержит много сишных файлов. У каждого свой хидер. Переменные объявляю в сишниках. В хидерах объявляю прототипы функций и переменные с приставкой extern. Для обеспечения видимости переменных и фунций в других сишниках проекта просто делаю #include xxx.h. Все работает. Все кроме переменных enum. Если обьявить такую переменную в сишнике
enum {rsOFF,rsON}rele_stat=rsOFF;
а потом в хидере
extern enum {rsOFF,rsON}rele_stat;

то компилятор ругается что мол в одном месте одно и то же объявляешь.

Если убрать из хидера символные значения и оставить просто
extern enum rele_stat;
то в других сишниках, если приинклудить туда этот хидер, компилятор не знает что такое rsOFF,rsON

Если кто знает решение проблемы - прошу поделиться... Спасибо.

typedef enum {rsOFF,rsON} MyEnum_t;

extern MyEnum_t rele_stat;

Все дошло, СПАСИБО.

Что за уродство? Какие тайпдефы? Кто вас научил этим глупостям?

Делается все просто: в хедере (а не хидере!) пишешь:
enum RELE_STAT {rsOFF,rsON};

В с-файле:
RELE_STAT rele_stat = rsOFF;

если эта переменная нужна в нескольких с-файлах, то в общем хедере объявляешь ее extern
extern RELE_STAT rele_stat;

нИкакое не уродство и ничё не глупости.
А если несколько переменных надо определить? для каждого енум забивать чтоли?
Я всегда (почти всегда) так пишу енумы и всегда структуры.

Именно для нескольких так и делают. Реле много, но набор допустимых состояний у всех одинаковый - включено или выключено. Поэтому пишешь:

RELE_STAT rele1;
RELE_STAT rele2;
RELE_STAT rele3;
RELE_STAT rele4;
RELE_STAT rele5 = rsON;

Энумы - они сами могут быть именованными, поэтому нет никакой нужды в тайпдефах.

Сообщение отредактировал 777777 - Apr 2 2009, 06:56

И получаете по полной программе:



На других С компиляторах можете попробовать сами

Уж, не знаю, что Вы там неправильно написали, или транслятор у Вас - кривой... Товарищ 777777 - прав. И приведённый код нормально компилится на других трансляторах (в том же Keil).

Видите ли, дорогой товарищ, я давно выработал у себя привычку перед тем, как постить какой-либо кусок кода, даже самый мелкий, проверить его на реальном компиляторе. Этот код я проверил на VC7.1 - он, конечно, не для микроконтроллеров, но по крайней мере отсутствие на нем ошибок означает, что синтаксически все написано правильно. Разумеется, на других компиляторах могут быть проблемы, но это проблемы компиляторов, а не С. У вас, судя по выдаваемым ошибкам, просто не подключен h-файл, в которм определен этот enum.

Для C++, а не C, о чём вам тонко намекают.

А, ведь верно! В соответствии со стандартом следовало бы записать так:
enum RELE_STAT {rsOFF,rsON};
enum RELE_STAT rele_stat = rsOFF;

Или typedef-нуть, чтобы не тянуть каждый раз слово enum, о чём тут тоже уже сказали.
В С++ это не нужно, равно как и со словом struct.

Еще позволю себе добавить, что не RELE, a relay - во-первых мешать транслитерацию (RELE) и английские (OFF) слова - это плохо, во-вторых заглавными буквами обычно обзывают задефайненые константы (ну ладно, это спороный пункт), лично я перечисления и структуры начинаю с подчерка, в-третьих не хорошо сокращать одну букву. В итоге я бы написал так

С - вариант
в хереде
typedef enum { relay_off, relay_on } _relay_state;
в си-файле
_relay_state relay1_state = relay_off;
опять в хидере
extern _relay_state relay1_state;

С++ - вариант
в хереде
enum _relay_state { relay_off, relay_on } ;
в с++ -файле
_relay_state relay1_state = relay_off;
опять в хидере
extern _relay_state relay1_state;

Компилятор С может резервировать идентификаторы с подчеркиванием для своих нужд, хотя чаще используется два подчеркивания.
Это замечание примите к сведению.