(C#) 配列要素をプロパティにマップする方法

Question

オブジェクトのマッピングに関しまして質問があります。

以下のような配列要素をプロパティにマップする場合に何か良い方法はないでしょうか。

リフレクションを使用する事も検討しましたが、GetPropertiesメソッドで取得できるプロパティ一覧が、プロパティ宣言の順番でない場合がある事が調べて分かり断念しています。

お手数をお掛けしますが、よろしくお願いします。

public class Student{
    public string Name { get; set; }
    public string Age { get; set; }
    :
    :
    //stringのプロパティが大量にあるとします。
}

public static void main(string[] args){

   //データ
   List<string> list = new List<string>(){"TARO","17",....};

   //案１:コンストラクタ、もしくはプロパティを逐一更新。(これは避けたいです。)
   Student taro = new Student(list[0],list[1],....);
   Student taro = new Student(){Name=list[0], Age=list[0],...};
   
   //案２:インデックスとプロパティの順番の整合性が取れない場合がある？
   Student obj = new Student();
   foreach(var prop in typeof(Student).GetProperties().Select((item,idx)=>(item, idx))){
        prop.SetValue(obj, list[idx]); 
   }

}

Comments

[Zuishin]

方法はありますが、何のためにこんなことをするんですか？ もっと適したやり方があるかもしれません。

[gitiiii222]

外部接続してデータを取得するのにライブラリを使用しており、1番目要素はTimestamp,2番目はレコードIDという具合にstring配列を返すためです。

[gitiiii222]

公開されているAPI自体がこういった仕様のため方法を模索していました。

[Zuishin]

私の回答ではうまくいきませんか？

[Zuishin]

うまくいかない場合、どのようにうまくいかないのかを書いてください。でなければわかりません。

Answer 1

素直に言われた通り実装するなら、次のように序数をプロパティ名に変換する段階を踏めばいいと思います。
applyList に並んだ順になります。
このソースでは配列を使っていますが、既存のオブジェクトのプロパティに適用する Apply も、新しくオブジェクトを作って適用して返す ParseList も IReadOnlyList<string> 型の引数を取るので、配列でも List<string> でも使えます。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Reflection;

namespace ConsoleApp1
{
	public class Student
	{
		public string Name { get; set; }
		public string Age { get; set; }

		public static Student ParseList(IReadOnlyList<string> list)
		{
			var result = new Student();
			result.Apply(list);
			return result;
		}

		public void Apply(IReadOnlyList<string> list)
		{
			if (list.Count != applyList.Count)
			{
				throw new ArgumentException(nameof(list));
			}
			for (int i = 0; i < list.Count; i++)
			{
				properties[applyList[i]].SetValue(this, list[i]);
			}
		}

		private static readonly Dictionary<string, PropertyInfo> properties = typeof(Student)
			.GetProperties()
			.ToDictionary(a => a.Name);
		private static readonly List<string> applyList = new List<string>()
		{
			nameof(Name), nameof(Age)
		};
	}

	class Program
	{
		static void Main(string[] args)
		{
			var students = new[]
			{
				new[] { "山田太郎", "17" },
				new[] { "山田二郎", "17" },
				new[] { "佐藤花子", "16" },
			};
			foreach (var list in students)
			{
				var student = Student.ParseList(list);
				Console.WriteLine($"{student.Name}: {student.Age}");
			}
			Console.ReadKey();
		}
	}
}

なぜ追記修正依頼でこのようなことをする理由を聞いたかと言うと、データソースが List<string> ではなくデータベースや JSON や Excel や CSV ならこのようなことをする必要がないからです。

今回のソースのようにデータを直接コードに埋め込むのをハードコーディングと言いますが、ハードコーディングしたデータはコンパイルしなおさなければアップデートできませんので、良い方法ではありません。
データは Excel などで管理し、それを必要に応じて読み込む方法が推奨されます。
読み込む際には List<string> でなければならない理由はないので、このようなことをする必要がないのです。

Comments

[gitiiii222]

ありがとうございます、返信ができておりませんでした。

Answer 2

T4テンプレートを使う。
ちまちま書くのが面倒なら、ちまちま書くことをプログラミングすればいいじゃない？という方針。
リフレクションをつかうのより、T4のようなコード生成のメタプログラミングのほうがよさげ。
パフォーマンス的には最強。

Answer 3

初期化時にパラメーターを割り当てるのを避けたい理由はなんでしょうか？
コード量が多かったりパラメーター管理が面倒ということでしょうか？
個人的にはこういうケースの場合、ここを省略すると後々面倒になることが多いのでファクトリメソッドとかにして切り出してます。

それはひとまず置いといて、案を2つ考えました。

案1.Studentの値を配列で持つ

public class Student {
    public List<string> Values { get; } = new List<string>();
    public string Name { get => Values[0], set => Values[0] = value }
    public string Age { get => Values[1], set => Values[1] = value }
}

それでデータソース側にも対応するインデックスを割り当てればValues[index]で回せませんかね？
インデックスの割当管理はStudent内で行うことになります。

案2.indexとプロパティ名を対応したリストを作る

Dictionary<int, string> propertyNames = new Dictionary<int, string>();
propertyNames.Add(0, "Name");
propertyNames.Add(1, "Age");

件数にもよりますが頭で一度だけ対応リストを作っておくなら大したコストにもならないと思います。
index→propertyName→リフレクションで設定という流れです。
必要に応じてHashSetでもいいと思います。