C# 参照をする時の変数で取得する方法

Question

お世話になります。現在変数でプロパティを取得する方法がないか調べているのですが、上手くいっておりません。
アドバイスを頂けましたら有難いです。

    public static class Message
    {
        public static readonly string Message1 = "ok";
        public static readonly string Message2 = "ng";
}

他クラス
var Choice = "1"
MessageBox.Show(Message + Choice);//このように変数で指定したmessageを取得したいです。

Comments

[退会済みユーザー]

上のコードのクラス定義の中にあるのはプロパティではなくてフィールドと言います。

> MessageBox.Show(Message + Choice);//このように変数で指定したmessageを取得したいです。

そういうことはできないので別の手段を考えましょう。String の配列にして添え字で取得するとか。

[Zuishin]

var messages = new[] { "ok", "ng" };
var choice = 1;
MessageBox.Show(messages[choice - 1]);

[dodox86]

C# に限らず、変数名を文字列でアクセスしたいとする質問は頻出ですね。
teratailトップページにて：
[C# 文字列 変数 アクセス] の検索結果： 242件

[退会済みユーザー]

初心者だとListやDictionary等のコレクションクラスの存在を知らないから、そういう思考になってしまうのかなと想像したり。

Answer 1

質問のコメントにも書きましたが、質問の Message クラス定義の中にあるのはプロパティではなくてフィールドと言います。

MessageBox.Show(Message + Choice);//このように変数で指定したmessageを取得したいです。

そういうことはできないので別の手段を考えましょう。String の配列にして添え字で取得するとか。

例えば以下のように Message1 と Message2 の配列を返すパブリックプロパティを定義して、

// 静的クラスで String 型の配列を返すパブリック
// プロパティ定義して利用
public static class Message
{
    // フィールドを public にするのはお勧めしません
    private static readonly string Message1 = "ok";
    private static readonly string Message2 = "ng";

    // パブリックプロパティを定義
    public static string[] Messages
    {
        get
        {
            return new String[] { Message1, Message2 };
        }
    }
}

それを以下のように利用してはいかがですか？

MessageBox.Show(Message.Messages[0]);
MessageBox.Show(Message.Messages[1]);

以下はお勧めはしませんが・・・

Message.Messages[0] のような形にするのが気に入らないとか、どうしても Massage + index という形にしたいなら、Message クラスをインスタンスクラスとして定義してインデクサを使うとか、演算子 + をオーバーロードするとかいう方法もあります。以下のような感じ。

using System;

namespace ConsoleAppOverload
{
    internal class Program
    {
        static void Main(string[] args)
        {
            // インスタンスクラスにしてインデクサを定義
            // または演算子 + をオーバーロード
            var message = new Message();

            // インデクサ
            Console.WriteLine("-----インデクサ-----");            
            for (int index = 1; index < 4; index++)
            {
                Console.WriteLine(message[index]);
            }

            // 演算子 + オーバーロード
            Console.WriteLine("-----演算子オーバーロード-----");            
            for (int index = 1; index < 4; index++)
            {
                Console.WriteLine(message + index);
            }
        }
    }

    // インスタンスクラスにしてインデクサを定義
    // または演算子 + をオーバーロード
    public class Message
    {
        // フィールドを public にするのはお勧めしません
        private static readonly string Message1 = "ok";
        private static readonly string Message2 = "ng";

        // インデクサを定義
        public string this[int index]
        {
            get 
            {
                if (index == 1) return Message1;
                else if (index == 2) return Message2;
                else return "未定義";
            }
        }

        // 演算子 + をオーバーロード
        public static string operator +(Message msg, int index) 
        {
            if (index == 1) return Message1;
            else if (index == 2) return Message2;
            else return "未定義";
        }
    }
}

結果は:

Answer 2

とりあえず、プロパティとフィールドを間違えてると思います。
リフレクション使えば出来ると言えば出来ますけど、普通にアクセスするより面倒だし速度も非常に遅くなります。

using System;
using System.Reflection;

public static class Message
{
    public static readonly string Message1 = "ok"; // これはフィールド
    public static string Message2 { get; } = "ng"; // これがプロパティ
}

class Program
{
    static void Main()
    {
        var choice = "1";
        var fieldMessage1 = typeof(Message).GetField(
            "Message" + choice,
            BindingFlags.Static | BindingFlags.Public);

        Console.WriteLine(fieldMessage1.GetValue(null)); // okが出力される

        choice = "2";
        var propMessage2 = typeof(Message).GetProperty(
            "Message" + choice,
            BindingFlags.GetProperty | BindingFlags.Static | BindingFlags.Public);

        Console.WriteLine(propMessage2.GetValue(null)); // ngが出力される

        Console.ReadKey();
    }
}

文字列で値を持ってくるならDictionaryがありますし、別に文字列である必要が無いなら、配列やListもあります。

using System;
using System.Collections.Generic;

class Program
{
    private static readonly string[] _ArrayMessage =
    {
        "ok",
        "ng",
    };

    private static readonly Dictionary<string, string> _DictMessage = new Dictionary<string, string>()
    {
        {"Message1", "ok"},
        {"Message2", "ng"},
    };

    static void Main()
    {
        Console.WriteLine(_ArrayMessage[0]);
        Console.WriteLine(_DictMessage["Message1"]);

        Console.ReadKey();
    }
}

リフレクションの速度が非常に遅いと言われても、どれくらい遅いのかピンと来ないと思うので、BenchmarkDotNet の計測結果を載せておきます。

using BenchmarkDotNet.Attributes;
using BenchmarkDotNet.Diagnosers;
using BenchmarkDotNet.Running;
using System.Collections.Generic;
using System.Reflection;

public static class Message
{
    public static string Message1 { get; } = "ok";
    public static string Message2 { get; } = "ng";
}

//計測用テストクラス
[MemoryDiagnoser(false)]
[ShortRunJob]
public class Test
{
    private static readonly string[] _ArrayMessage =
    {
        "ok",
        "ng",
    };

    private static readonly Dictionary<string, string> _DictMessage = new Dictionary<string, string>()
    {
        {"Message1", "ok"},
        {"Message2", "ng"},
    };

    private int _Index = 0;
    private string _StringKey = "1";

    [Benchmark]
    public string UseReflection()
    {
        var prop = typeof(Message).GetProperty(
            "Message" + _StringKey,
            BindingFlags.GetProperty | BindingFlags.Static | BindingFlags.Public);

        return (string)prop.GetValue(null);
    }

    [Benchmark]
    public string UseArray() => _ArrayMessage[_Index];

    [Benchmark]
    public string UseDictionary() => _DictMessage["Message" + _StringKey];
}

class Program
{
    static void Main()
    {
        BenchmarkRunner.Run<Test>();
    }
}

（計測結果）
|        Method |        Mean |      Error |    StdDev |      Median | Allocated |
|-------------- |------------:|-----------:|----------:|------------:|----------:|
| UseReflection | 202.8417 ns | 13.9314 ns | 0.7636 ns | 203.0229 ns |      48 B |
|      UseArray |   0.0055 ns |  0.1723 ns | 0.0094 ns |   0.0000 ns |         - |
| UseDictionary |  47.3129 ns |  9.5924 ns | 0.5258 ns |  47.3259 ns |      48 B |

Comments

[退会済みユーザー]

個人的には、リフレクションが有効なケースはかなり稀で、むしろ速度やデバッグに悪影響を与える事の方が多いと思っているので、出来れば使わずに済ませるデータ構造を考えた方がよろしいかと思います。

Answer 3

リフレクションを使ってみては？

using System;
using System.Reflection;

public static class Message
{
    public static readonly string Message1 = "ok";
    public static readonly string Message2 = "ng";

    public static string GetValue(string index) {
        FieldInfo fieldInfo = typeof(Message).GetField(index,
                        BindingFlags.Public | BindingFlags.Static);
        if (fieldInfo == null) {
            throw new ArgumentOutOfRangeException();
        }
        return (string)fieldInfo.GetValue(null);
    }
}

var msg1 = Message.GetValue("Message1"); // ok
var msg2 = Message.GetValue("Message2"); // ng

"1" とか "2" を引数にするなら GetField の第1引数を "Message" + index とかにしてください。