2

新人の方にenumが持つ全ての値をforeachで取得しようとしてたら、
エラーが出たと質問されました。

public enum TopCities
{
    Tokyo,

    Yokohama,

    Osaka,

    Nagoya,

    Sapporo,

    Kobe,

    Kyoto
}

#endregion

#region Public Methods and Operators

public void LoopThroughAllTopCities()
{
    foreach (string city in Enum.GetValues(TopCities))
    {
        ltlResult.Text += city + "\r\n";
    }
}

#endregion

#region Methods

protected void Page_Load(object sender, EventArgs e)
{
    LoopThroughAllTopCities();
}

#endregion

このエラーが出てコンパイルできないと聞かれました。

TopCities' is a 'type' but is used like a 'variable'

この場合Enum.GetValuesの引数はSystem.Typeなので、typeof(TopCities)を渡すべきです。

画像の説明をここに入力

foreach (var city in Enum.GetValues(typeof(TopCities)))

もしくは

foreach (var city in Enum.GetNames(typeof(TopCities)))

で都市名の取得もできますと答えました。

Enum.GetValuesEnum.GetNamesの違いは指定した値を持つ指定した列挙体にある定数の名前を取得する代わりに、定数の値の配列を取得することです。

そこで思ったのですが、どのようなの場合にどちらのメソッドが推奨されるのかを聞きたいです。

1
  • ケースバイケースのような気がしますが、値は重複する場合があるので、名前で収得して値は名前からParseで得るのがいいような気がします。(まあ例の場合は重複しないので、そんな必要もないわけですが)、あるいは既に回答されてるType.GetFields のメソッドを使用するMSDNの例
    – BLUEPIXY
    Commented 2015年2月14日 1:44

3 件の回答 3

5

名称だけでよいのであればEnum.GetNames、数値と名称が必要であればEnum.GetValues、各フィールドでリフレクションを行うのであればType.GetFieldsを使用べきかと思います。

質問の例でEnum.GetValuesを使用するとEnum.ToStringの呼び出しコストがかかるのでEnum.GetNamesが適切です。しかし

public enum TopCities
{
    [DisplayName("東京")]
    Tokyo,
    [DisplayName("横浜")]
    Yokohama,
}

のように属性が適用されていてそれを参照したいのであればType.GetFieldsとなります。

4

そもそもenumには重複した値を設定できます。

enum Value {
    One = 1,
    Alias = Value.One,
    Two = 2,
}

この例ではGetNames()は "One", "Alias", "Two" を返しますし、GetValues()は 1, 1, 2 を返します。
名前の通りですので、後は適材適所で選択してください。

0

まったく違うメソッドです。
GetValues()はenumを返しGetNames()はstringを返します。
あなたが必要な方を使うことをオススメします。
詳しくはMSDNを調べてください。

0

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。