プチクイズ 3 - C++の列挙型

Q.

以下のC++コードのリファクタリングすべき点を述べよ．

enum { ELEM1, ELEM2 } ENUM_1;
enum { SUN, MON, TUE, WED, THU, FRI, SAT };

void ShowEnumVal(int e) {
    cout << e << endl;
}

A.

列挙体に関しての2つのDON'Tを直す必要がある．
１つはplain vanilla enumを使用している点，２つは関数がEnum型ではなくintを引数の型としている点である．
上記コードはコンパイルもランタイムも問題なく動くが，非常にリスクの大きいコードである．
なぜなら関数ShowEnumVal()は本来，オブジェクトとして全くことなる(として扱うべき)２つのEnumを受け入れてしまう点である．各enumはtypedefを用いてコンパイルに明示的に違う型のように扱ってもらうべきであり，関数ShowEnumVal()も，その関数が扱うべきenumオブジェクトだけを受け入れるべきである．
上記２点を修正したコードは以下の通り．

typedef enum { ELEM1, ELEM2 } ENUM_1;
typedef enum { SUN, MON, TUE, WED, THU, FRI, SAT } DAY_OF_WEEK;

void ShowEnum1Val( ENUM_1 e ) {

 cout << e << endl;

}