LINQ 取得した値の変換方法について

Question

LINQで取得した値を置き換えたいです。

例えば、以下の例では、Addressが「青森、秋田、岩手ならば、北東北」、「山形、宮城ならば、南東北」に取得したAddressカラムの値を置き換えたいです。

何か良い方法がありましたら、ご教示いただけますと幸いです。

取得した値の変換イメージイメージ：
CompanyNm = "株式会社ABC", Address = "北海道" 　　　　　　　CompanyNm = "株式会社ABC", Address = "北海道"
CompanyNm = "株式会社DEF", Address = "青森" 　　　　　　　　CompanyNm = "株式会社DEF", Address = "北東北"
CompanyNm = "株式会社GHI", Address = "秋田" 　　　　⇒　　　CompanyNm = "株式会社GHI", Address = "北東北"
CompanyNm = "株式会社JKL", Address = "岩手" 　　　　　　　　CompanyNm = "株式会社JKL", Address = "北東北"
CompanyNm = "株式会社MNO", Address = "山形"　　　　　　　　 CompanyNm = "株式会社MNO", Address = "南東北"
CompanyNm = "株式会社PQR", Address = "宮城" 　　　　　　　　CompanyNm = "株式会社PQR", Address = "南東北"

using Microsoft.AspNetCore.Mvc;
using QuickMaster.Models;
using QuickMaster.Views;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using WebApplication1.Models;
using System.IO;

namespace WebApplication1.Controllers
{
    public class HomeController : Controller
    {


        private readonly MyContext _context;

        public HomeController(MyContext context)
        {
            this._context = context;
        }

        public ActionResult Index()
        {

            Trader[] traders = new Trader[] {
        new Trader() { Id = 1, CompanyNm = "株式会社ABC", Address = "北海道" },
        new Trader() { Id = 2, CompanyNm = "株式会社DEF", Address = "青森" },
        new Trader() { Id = 3, CompanyNm = "株式会社GHI", Address = "秋田" },
        new Trader() { Id = 4, CompanyNm = "株式会社HIJ", Address = "岩手" },
        new Trader() { Id = 5, CompanyNm = "株式会社KLM", Address = "山形" },
        new Trader() { Id = 6, CompanyNm = "株式会社OPQ", Address = "宮城" },
      };

            return View();
    }
    }
}

環境は、.NetCore 3.1 でVisual Studio、C#を使用しております。
※本当はコントローラに書くのは良くないということは分かっていますが、動作確認が容易だと判断してあえてここに記載しています。

Comments

[退会済みユーザー]

ASP.NET の話なら ASP.NET のタグをつけて、何を作っているか（Web Forms? MVC? Razor? その他?）と
開発環境（OS. Visual Studio のバージョン、.NET か Core のどっちかとそのバージョンなど）を書いてください。

Answer 1

ちなみに、regionのListを作らないで実現することはできないでしょうか。
例えば、クエリ式の中から、メソッドを呼んで、switch文で分岐処理をしたり、クエリ式のどこかで条件分岐できたりはできないという認識でよろしいでしょうか。
※クエリ式の中で、自作のメソッドを呼び出したり、if文で分岐できたら楽なのに・・と思っていたので、この質問をさせていただきました。

こういうことでしょうか？

using System;
using System.Linq;

namespace LinqSample {
	class Program {
		static void Main(string[] args) {
			var traders = new Trader[] {
				new Trader() { Id = 1, CompanyNm = "株式会社ABC", Address = "北海道" },
				new Trader() { Id = 2, CompanyNm = "株式会社DEF", Address = "青森" },
				new Trader() { Id = 3, CompanyNm = "株式会社GHI", Address = "秋田" },
				new Trader() { Id = 4, CompanyNm = "株式会社HIJ", Address = "岩手" },
				new Trader() { Id = 5, CompanyNm = "株式会社KLM", Address = "山形" },
				new Trader() { Id = 6, CompanyNm = "株式会社OPQ", Address = "宮城" },
			};

			var query =
				from trader in traders
				select new {
					Id = trader.Id,
					CompanyNm = trader.CompanyNm,
					Address = new string[] { "青森", "秋田", "岩手" }.Contains(trader.Address)
								? "北東北"
								: new string[] { "山形", "宮城" }.Contains(trader.Address)
									? "南東北"
									: trader.Address
				};

			var result = query.ToList();

			foreach (var o in result) {
				Console.WriteLine($"Id={o.Id}, CompanyNm={o.CompanyNm}, Address={o.Address}");
			}
		}
	}

	class Trader {
		public int Id { get; set; }
		public string CompanyNm { get; set; }
		public string Address { get; set; }
	}
}

出力。

Id=1, CompanyNm=株式会社ABC, Address=北海道
Id=2, CompanyNm=株式会社DEF, Address=北東北
Id=3, CompanyNm=株式会社GHI, Address=北東北
Id=4, CompanyNm=株式会社HIJ, Address=北東北
Id=5, CompanyNm=株式会社KLM, Address=南東北
Id=6, CompanyNm=株式会社OPQ, Address=南東北

= new string[] { "青森", "秋田", "岩手" }.Contains(trader.Address)
    ? "北東北"
    : new string[] { "山形", "宮城" }.Contains(trader.Address)
        ? "南東北"
        : trader.Address

それが設計としてよいかどうかは別として、この部分をメソッドに切り出して呼び出すことも、できます。

Comments

[abd1106]

ご回答いただき、ありがとうございました。

設計としては、あまりよくないんですね・・。

引き出しの一つとして、利用させていただこうと思います。

本当にありがとうございました。

[jhashimoto]

> 設計としては、あまりよくないんですね・・。
「よくない」という意図はなかったです。あくまでもサンプルコードなので、これだけで良し悪しの判断はできないと思います。どのようなコードが要件に沿っているかご自身で検討してみてください。

[abd1106]

ご回答いただき、ありがとうございました。

Answer 2

別回答のコメント

逆にラムダ式を使わないと、実現はできないという認識でよろしいのでしょうか。

と

ちなみに、regionのListを作らないで実現することはできないでしょうか。

例えば、クエリ式の中から、メソッドを呼んで、switch文で分岐処理をしたり、クエリ式のどこかで条件分岐できたりはできないという認識でよろしいでしょうか。
※クエリ式の中で、自作のメソッドを呼び出したり、if文で分岐できたら楽なのに・・と思っていたので、この質問をさせていただきました。

を読んでそこまでクエリ式の中で分岐やりたいなら、単純にselect句で三項演算でもできるし（このやり方が良いとは欠片も思ってないけど）

var q = from trader in traders
        select new 
        { 
            trader.Id, 
            trader.CompanyNm,  
            Address = new[] { "青森", "秋田", "岩手"}.Contains(trader.Address) ? "北東北" : new[] { "山形", "宮城" }.Contains(trader.Address) ? "南東北" : trader.Address
        };

foreach (var o in q)
{
    Console.WriteLine($"Id={o.Id}, CompanyNm={o.CompanyNm}, Address={o.Address}");
}

switch式でもできる(switch文じゃない)
ただしこれはC#のLangVersionが8.0以上じゃないとできないと思うけど。

var q = from trader in traders
        select new
        {
            trader.Id,
            trader.CompanyNm,
            Address = trader.Address switch
            {
                "青森" => "北東北",
                "秋田" => "北東北",
                "岩手" => "北東北",
                "山形" => "南東北",
                "宮城" => "南東北",
                _ => trader.Address,
            },
        };

foreach (var o in q)
{
    Console.WriteLine($"Id={o.Id}, CompanyNm={o.CompanyNm}, Address={o.Address}");
}

Comments

[abd1106]

ご回答いただき、ありがとうございました。

自分の中の引き出しの1つとさせていただきます。

ちなみに、gentaroさんなら、どういった書き方がベストだと思われますか。

よろしければ、理由も含めて教えていただけますと幸いです。

[gentaro]

> クエリ式のどこかで条件分岐

という点について、それをやるのが良くないと思うという話ね。
条件分岐の部分についてメソッド抽出するなら良いんじゃない。

[abd1106]

ご回答いただき、ありがとうございます。

実はメソッド抽出の方法は、検討致しました。ただ、どうやって、メソッドを呼び出して、値を設定すればいいのか・・？ということになってしまいました・・。

いろいろと試しましたが、以下の場合、
「匿名型のメンバー宣言子が無効です。メンバー代入、簡易名、またはメンバー アクセスを使用して、匿名型メンバーを宣言する必要があります。」
というエラーになってしまいます・・。

方法をご教示願えませんでしょうか。

例
using Microsoft.AspNetCore.Mvc;
using QuickMaster.Models;
using QuickMaster.Views;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using WebApplication1.Models;
using System.IO;

namespace WebApplication1.Controllers
{
public class HomeController : Controller
{

private readonly MyContext _context;

public HomeController(MyContext context)
{
this._context = context;
}

public ActionResult Index()
{

Trader[] traders = new Trader[] {
new Trader() { Id = 1, CompanyNm = "株式会社ABC", Address = "北海道" },
new Trader() { Id = 2, CompanyNm = "株式会社DEF", Address = "青森" },
new Trader() { Id = 3, CompanyNm = "株式会社GHI", Address = "秋田" },
new Trader() { Id = 4, CompanyNm = "株式会社HIJ", Address = "岩手" },
new Trader() { Id = 5, CompanyNm = "株式会社KLM", Address = "山形" },
new Trader() { Id = 6, CompanyNm = "株式会社OPQ", Address = "宮城" },
};

var query1 = from m in _context.Trader
select new
{
m.Id,
m.CompanyNm,
m.Address = changeAddress(m.Address);
};

return View();

}
public string changeAddress(string args)
{
if (args == "青森" || args == "秋田" || args == "岩手")
{
args = "北東北";
}
else if (args == "山形" || args == "宮城")
{
args = "南東北";
}

return args;

}

}

[gentaro]

✕m.Address = changeAddress(m.Address);
○Address = changeAddress(m.Address)

[abd1106]

自分の理想の形になりました！

ありがとうございました。

Answer 3

こうですかね。

traders = traders.Select(t=>{
  if ((new string[] { "青森", "秋田", "岩手" }).Any(s=>s==t.Address)) { t.Address = "北東北"; }
  if ((new string[] { "山形", "宮城"        }).Any(s=>s==t.Address)) { t.Address = "南東北"; }
  return t;
}).ToArray();

Comments

[abd1106]

ご回答いただき、ありがとうございます。

確かに、これで実現できることを確認致しました。

ラムダ式を使えば、できるんですね。大変勉強になりました、ありがとうございました。

逆にラムダ式を使わないと、実現はできないという認識でよろしいのでしょうか。

Answer 4

一案ですが、北東北/南東北を別のデータソースに持たせれば、結合できます。

using System;
using System.Linq;

namespace LinqSample {
	class Program {
		static void Main(string[] args) {
			var traders = new Trader[] {
				new Trader() { Id = 1, CompanyNm = "株式会社ABC", Address = "北海道" },
				new Trader() { Id = 2, CompanyNm = "株式会社DEF", Address = "青森" },
				new Trader() { Id = 3, CompanyNm = "株式会社GHI", Address = "秋田" },
				new Trader() { Id = 4, CompanyNm = "株式会社HIJ", Address = "岩手" },
				new Trader() { Id = 5, CompanyNm = "株式会社KLM", Address = "山形" },
				new Trader() { Id = 6, CompanyNm = "株式会社OPQ", Address = "宮城" },
			};

			var regions = new Region[] {
				new Region() { Address = "青森", RegionName = "北東北" },
				new Region() { Address = "秋田", RegionName = "北東北" },
				new Region() { Address = "岩手", RegionName = "北東北" },
				new Region() { Address = "山形", RegionName = "南東北" },
				new Region() { Address = "宮城", RegionName = "南東北" }
			};
			
			var query =
				from t in traders
				join r in regions
				on t.Address equals r.Address into gj
				from tr in gj.DefaultIfEmpty()
				select new {
					Id = t.Id,
					CompanyNm = t.CompanyNm,
					Address = tr?.RegionName ?? t.Address    // Traderに結合されるRegionがない場合はAddressを返す
				};

			var result = query.ToList();

			foreach (var o in result) {
				Console.WriteLine($"Id={o.Id}, CompanyNm={o.CompanyNm}, Address={o.Address}");
			}
		}
	}

	class Trader {
		public int Id { get; set; }
		public string CompanyNm { get; set; }
		public string Address { get; set; }
	}
	
	class Region {
		public string Address { get; set; }
		public string RegionName { get; set; }
	}
}

出力。

Id=1, CompanyNm=株式会社ABC, Address=北海道
Id=2, CompanyNm=株式会社DEF, Address=北東北
Id=3, CompanyNm=株式会社GHI, Address=北東北
Id=4, CompanyNm=株式会社HIJ, Address=北東北
Id=5, CompanyNm=株式会社KLM, Address=南東北
Id=6, CompanyNm=株式会社OPQ, Address=南東北

Comments

[abd1106]

ご回答いただき、ありがとうございます。

regionのListを作成して、joinするという選択肢がありませんでしたので、勉強になりました。

ちなみに、regionのListを作らないで実現することはできないでしょうか。
例えば、クエリ式の中から、メソッドを呼んで、switch文で分岐処理をしたり、クエリ式のどこかで条件分岐できたりはできないという認識でよろしいでしょうか。
※クエリ式の中で、自作のメソッドを呼び出したり、if文で分岐できたら楽なのに・・と思っていたので、この質問をさせていただきました。

[jhashimoto]

コードブロック使えないので、回答の方に書きます。