C# ケース文での値の代入について

区分によって対応するプロパティに値を入れていきたいのですが、右辺は同じなのに左辺が違うだけで下記ソースのように記述するのは冗長に思えます。

なので、簡潔に書く方法があれば教えてください。

    var list = new List<Data>();
            foreach ( var data in dataList)
            {
                var model = new Data;
                switch (data.Kbn)
                {
                    case "1":
                        model.KingakuA = data.kingaku;
                        model.SuryoA = data.suryo;
                        model.TankaA = data.tanka;
                        break;
                    case "2":
                        model.KingakuB = data.kingaku;
                        model.SuryoB = data.suryo;
                        model.TankaB = data.tanka;
                        break;
                    case "3":
                        model.KingakuC = data.kingaku;
                        model.SuryoC = data.suryo;
                        model.TankaC = data.tanka;
                        break;
                    case "4":
                        model.KingakuD = data.kingaku;
                        model.SuryoD = data.suryo;
                        model.TankaD = data.tanka;
                        break;
                    default:
                        break;

                }
                list.Add(model);

            }

行動規範の内容に同意します

回答3件

直接的な回答ではないのですが、これは設計がいけてないような気がしますね。
データを横持ちにしているとこういうことが起きます。
（レコードで持つのではなくカラムで持つ）

こういうデータの持ち方は、画面表示が横になっていると、
そのまま表示できるので便利ではあるのですが、拡張性が皆無なのであまりお薦めはしません。
例えば区分に「5」が増えた場合は、KingakuEを足すという悲惨なことになっていきます。

そもそもこれって、A～Dのいずれかにしか値は入らないですよね。
AとCに入るパターンとかってのがないってことです。
そうなると、ますます横持ちに持っている意味がないような気がします。

dataListはレコードとして縦持ちに持っているので、
dataListが持っているdataの状態でいいと思います。
この状態であれば、ある程度Linqでどうとでもなりそうですし。
例えば区分「1」の合計数量とかであれば、以下のような感じでとれます。

C#
1var sum = list.Where(d => d.Kbn == "1").Sum(d => d.Suryo);

投稿2017/10/19 16:10

root_jp

総合スコア4666

ShouOkada

2017/10/21 02:21

わかりました、ありがとうございますm(__)m

行動規範の内容に同意します

ベストアンサー

解答にならないかと思いますが、

C#
1class Model {
2  KST[] abcd = new KST[4];
3}
4class KST {
5  int Kingaku;
6  int Suryo;
7  int Tanka;
8
9  KST(int kingaku, int suryo, int tanka) {
10    Kingaku = kingaku;
11    Suryo = suryo;
12    Tanka = tanka;
13  }
14}

という雰囲気の設計ではいけないのだろうかと思いました。上記のように設計してあれば、冗長さはでてこないように思います。

C#
1// どこかのstaticフィールド
2static Dictionary<string, int> Kubun2Index = new Dictionary<string, int> {
3  { "1", 0 }, {"2", 1 }, ...
4};
5...
6
7var list = new List<Data>();
8foreach (var data in dataList) {
9  if (Kubun2Index.ContainsKey(data.Kbn)) {
10    var model = new Model();
11    model.abcd[Kubun2Index[data.Kbn]] = new KST(data.kingaku, data.suryo, data.tanka);
12    list.Add(model);
13  } else {
14    Console.WriteLine("区分が不当です。無視しました。");
15  }
16}