Какой смысл несет именованный enum?

@kzru_hunter · Регистрация: 01.02.2011

Author24 — интернет-сервис помощи студентам

Собственно зачем объявлять enum таким образом:

C++

enum suit
{
     DIAMONDS,
     HEARTS,
     SPADES,
     CLUBS
}

suit s1, s2;

когда можно просто:

C++

enum
{
     DIAMONDS,
     HEARTS,
     SPADES,
     CLUBS
}

Ведь enum просто создает константы целого типа с определенными значениями ( в данном случае: 0,1,2,3). Зачем создавать s1,s2,... ?

Добавлено через 12 минут
Всё понял, практически никакого смысла. Единственно только предупреждение компилятор выдаст при попытке в объект s1 типа suit записать целое значение, а не одну из перечисленных констант. Вообщем, делает код трудночитаемым и практически никакой пользы.

C++

enum suit
{
        DIAMONDS,
        HEARTS,
        SPADES,
        CLUBS
};
 
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
        suit s1;
        s1 = 2;
        ShowMessage(s1);
}

@pomkalk · 31.08.2011, 10:51

Затем что бы применении шире было
пример

C++

enum PrintType
{
   PT_STDOUT,
   PT_TEMPFILE
}
 
enum ShowStyle
{
  SS_TABLE,
  SS_LIST
}
 
void print(PrintType type,...);
void Show(ShowStyle style,struct _data data);

Что бы логическое разграничение между ними было, а не просто константы

@kzru_hunter · 31.08.2011, 11:27 **[ТС]**

Возможно пока не привык ещё, поэтому такая неприязнь к именованному enum.
По мне так проще воспринимать так:

C++

1	void print(int type,...);

потому как уже сразу понятно, что переменная type имеет тип int.

а так

C++

1	void print(PrintType type,...);

придется вспоминать, что из себя представляет PrintType либо нажимать Find Declaration.
если читать эту строку, то читается она так:
в функцию print передается переменная type типа PrintType, но если посмотреть этот PrintType, то это будет перечисление, а не какой-то тип. не ассоциируется у меня PrintType как тип.

@pomkalk · 31.08.2011, 12:53

Сообщение от kzru_hunter

void print(int type,...);

Мне тоже такой вариант больше нравится, но используя enum можно более информативнее описать значение данной переменной

@Maluda · 31.08.2011, 23:28

Сообщение от kzru_hunter

Зачем создавать s1,s2

Чтобы различать константы, которые одинаково называются - это РАЗ!

C++

1
2
3

enum TBorderStyle { bsNone };
enum TButtonState { bsNone };
enum TBoundStyle { bsNone };

Чтобы передавать их как параметры и делать свой код более читабельным для себя и других - это ДВА.

Сообщение от kzru_hunter

не ассоциируется у меня PrintType как тип.

Добавь впереди букву T и будет ассоциироваться.

И вообще желательно придерживаться коддинг-стандарта в той среде, в которой программируешь (или в фирме),
если в Билдере,
то enum начинается с T, далее описание - каждое слово без разделителей, последнее слово - в единственном числе, множество на основе этого enum - повторяет название enum + последнее слово во множественном числе.
Название каждого элемента в enum - это аббревиатура больших букв в названии

C++

enum TFontStyle { fsBold, fsItalic, fsUnderLine, fsStrikeOut};
typedef Set <TFontStyle,fsBold,fsStrikeOut> TFontStyles;
 
enum TMyEnumType {metFirst, metSecond, metThird};
typedef Set <TMyEnumType,metFirst,metThird> TMyEnumTypes;

У нас на фирме, например, очень жёстко придерживаются стандарта, потому что твой код могут читать другие люди и разбираться в твоём int Type, особенно когда их сотни, никто не будет.

Вот два кода для сравнения:
Как из этого кода понять, что с формой происходит ... ?

C++

TForm *AForm = new TForm(Application);
AForm->Position = 1;
AForm->FormStyle = 2;
AForm->BorderState = 5;
AForm->WindowState = 0;
AForm->Font->Style = 2;
// ... ну и так дальше

А здесь только глянул код - и всё понятно

C++

TForm *AForm = new TForm(Application);
AForm->Position = poDesigned;
AForm->FormStyle = fsStayOnTop;
AForm->BorderState = bsDialog;
AForm->WindowState = wsMaximized;
AForm->Font->Style = fsItalic;
// ... ну и так дальше

Добавлено через 5 минут
Кстати чего далеко ходить

Сообщение от kzru_hunter

C++

enum suit
{
 DIAMONDS,
 HEARTS,
 SPADES,
 CLUBS
};
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
 : TForm(Owner)
{
 suit s1;
 s1 = 2;
 ShowMessage(s1);
}

C++

enum TSuitType {
        stDiamonds, stHearts, stSpades, stClubs
};
 
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner): TForm(Owner) {
        TSuitType ASuitType = stDiamonds;
        ShowMessage(ASuitType);
}

Добавлено через 6 минут
Ах, да и, совсем забыл, в Билдере можно получать текстовое значение enum, что очень удобно при сохранении в различные конфиги, а также при разработке различных скриптов.

@kzru_hunter · 01.09.2011, 11:42 **[ТС]**

Сообщение от Maluda

Чтобы различать константы, которые одинаково называются - это РАЗ!

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

Сообщение от Maluda

Чтобы передавать их как параметры и делать свой код более читабельным для себя и других - это ДВА.

это можно сделать и c неименованным enum.
все примеры ниже можно применить и к неименованному enum.

C++

enum {poDesigned, fsStayOnTop, bsDialog, wsMaximized, fsItalic};
TForm *AForm = new TForm(Application);
AForm->Position = poDesigned;
AForm->FormStyle = fsStayOnTop;
AForm->BorderState = bsDialog;
AForm->WindowState = wsMaximized;
AForm->Font->Style = fsItalic;

C++

enum { stDiamonds, stHearts, stSpades, stClubs };
 
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner): TForm(Owner) {
        int ASuitType = stDiamonds;
        ShowMessage(ASuitType);
}

C++

1 2	enum { fsBold, fsItalic, fsUnderLine, fsStrikeOut}; typedef Set <int,fsBold,fsStrikeOut> TFontStyles;

LK · 01.09.2011, 17:36

давайте исходить из этого: если enum присутствует - это намного лучше, чем было бы наличие его отсутствия

, а пить-не пить - дело вкуса

.

@Maluda · 14.01.2013, 19:22

Сообщение от kzru_hunter

это можно сделать и c неименованным enum.

Это делается для удобства, ещё раз повторю

C++

enum TSuitType {
        stDiamonds, stHearts, stSpades, stClubs
};
 
void GetSuit(const TSuitType &ASuitType) {
    // some actions with TSuitType
}

kzru_hunter предлагает писать просто вот так

C++

enum {
        stDiamonds, stHearts, stSpades, stClubs
};
 
void GetSuit(const unsigned int &ASuitType) {
    
}

Но ИМХО гораздо удобнее передавать enum в качестве параметра.
Интуитивно понятнее, гораздо проще разобраться для того, кто будет использовать твой код.
Также компилятор предупредит, если будет передан не enum, autocomplete будет предлагать enum,
если в параметре функции он прописан, ну и т.д.

@Avazart · 14.01.2013, 19:34

Сообщение от kzru_hunter

По мне так проще воспринимать так:

C++

1	void print(int type,...);

И что тут простого ? А главное что хорошего ? Магические цифры ? Или сишный #define ? или то что пользователь может задать 999 и потом искать эту ошибку долго и нудно?

Ставить int там где должен быть enum очень плохой тон...

@Maluda · 14.01.2013, 19:35

C++

void GetSuit(const TSuitType &ASuitType);
 
GetSuit(stDiamonds);  // компилятор пропустит
 
GetSuit(0);               // компилятор выдаст предупреждение

@pomkalk 365 / 247 / 24 Регистрация: 03.04.2011 Сообщений: 558 Записей в блоге: 1
	31.08.2011, 12:53	4
	Сообщение от kzru_hunter void print(int type,...); Мне тоже такой вариант больше нравится, но используя enum можно более информативнее описать значение данной переменной 0

LK Заблокирован
	01.09.2011, 17:36	7
	давайте исходить из этого: если enum присутствует - это намного лучше, чем было бы наличие его отсутствия , а пить-не пить - дело вкуса . 0

Какой смысл несет именованный enum?

Решение