C#のLinqを使い独自クラスに実装したIDに重複が無いかどうかを調べたい

Question

実現したいこと

C#のLinqを使い、以下のようなことをしたいです

• 実装時に、IDの重複が無いかどうかを調べたい
• 重複が無ければ、何もしない
• 重複があれば、そのIDと該当するクラス名をLogに出力したい

解決方法

        var duplicationCustomers = CreateCustomers().GroupBy(x => x.Id).Where(x => x.Count() > 1).SelectMany(x => x);

        foreach (var customer in duplicationCustomers )
        {
            Debug.LogFormat("{0}:{1}", customer.Id, customer.GetType().Name);
        }

前提条件

こういったクラスを多数作った際にIdの重複が無いかをチェックしたいです。

class Program
{
    interface ICustomer
    {
        string Id { get; }
        void Action();
    }

    public class CustomerA : ICustomer
    {
        public string Id => "A";
        public voic ACtion()
        {
        	//個別処理
        }
    }

    public class CustomerB : ICustomer
    {
        public string Id => "B";
        public voic ACtion()
        {
        	//個別処理
        }
    }

    public class CustomerC : ICustomer
    {
        public string Id => "C";
        public voic ACtion()
        {
        	//個別処理
        }
    }

    static void Main(string[] args)
    {
    	string id = GetId();

    	ICustomer _customer = CreateCustomer(id)

    	if(_customer != null)
    		_customer.Action();
    }

    static string GetId()
    {
    	return ("サーバーから取得したID");
    }

    static ICustomer CreateCustomer(string id)
    {
        //swich(id)
        //{
        //    case id = "A":
        //    	return new CustomerA();
        //    	break;
        //    case id = "B":
        //    	return new CustomerB();
        //    	break;
        //    case id = "C":
        //    	return new CustomerC();
        //   	 break;
        //        ・
        //        ・
        //        ・
        //    各CustomerクラスにIDを紐付け、Customerクラスが増えたときに、
        //    Caseが増えるのを避けたい
        //    case id = "D":
        //    	return new CustomerD();
        //    	break;
        //    default:
        //    //エラー処理
        //    	break;
        //}

        //そのため、switchCaseを使わずに、以下のような実装を検討している
        return
            CreateCustomers().FirstOrDefault(x => x.id == id);
    }
}

Reflectionの機能を用いた以下のようなメソッドにより、ICustomerを継承しているクラスの一覧を取得した後に

    private static IEnumerable<ICustomer> CreateCustomers()
    {
        return Assembly.GetExecutingAssembly().GetTypes()
            .Where(x => x.IsClass && x.BaseType == typeof(ICustomer))
            .Select(x => (ICustomer)System.Activator.CreateInstance(x));
    }

Linqを使ってうまく書きたいのですが、うまい書き方があれば教えてください。
よろしくお願いいたします。

環境

C# 4.0

Comments

[ozwk]

クラスにIDが紐付いているならstaticかattributeにしたほうがよくて、インスタンスに紐付けるならICustomerにプロパティを実装すればいいと思うのですが、これなにがしたいんですか?

[Zuishin]

ICustomer は名前だけ見るとインターフェースのようですが、BaseType でインターフェースは取得できません。クラスであれば I の頭文字は誤解を招くので止めましょう。

[Zuishin]

またクラスに定数の ID を付与するのであれば、重複のあるなしは設計時にチェックするものであって実行時ではないはずです。重複しない ID のつけ方を調べるのが良いのではないでしょうか。今はどうやってつけていますか？

[necos]

ozwkさん　本来やりたいことを追記しました。

[necos]

Zuishinさん　インターフェースの指摘の件、ありがとうございます。修正しておきます。

[necos]

ID付与方法の件は、設計時にチェックしています。今回は、実装時の転記ミス等による重複チェックを目的としてます。

Answer 1

おそらく Action() がそれぞれ違うのでクラスを大量に作っているのではないかと思いますが、これはインスタンスで解決すべき案件だと思います。
以下のように Customer のそれぞれのインスタンスにそれぞれの Action を指定することができます。

interface ICustomer
{
	string ID { get; }
	void Action();
}

class Customer : ICustomer
{
	public string ID { get; }

	public Customer(string id, Action<Customer> action)
	{
		ID = id;
		this.action = action;
	}

	Action<Customer> action;
	public void Action()
	{
		if (action != null) action(this);
	}
}

var customer1 = new Customer("customer1", customer => Console.WriteLine($"私の ID は {customer.ID} です。"));
customer1.Action();

重複のチェックですが、設計時であるならばわざわざ Customer を使うまでもありません。
使用する ID のリストがあるはずなので、それを使ってください。
例えば次のように Customer を作っている場合

List<string> Ids = new List<string>()
{
	"customer1", "customer2", "customer3"
};

var customer1 = new Customer(Ids[0], customer =>
{
	Console.WriteLine("私は customer1");
});
var customer2 = new Customer(Ids[1], customer =>
{
	Console.WriteLine("私は customer2");
});
var customer3 = new Customer(Ids[2], customer =>
{
	Console.WriteLine("私は customer3");

});

次のようにチェックできます。

Console.WriteLine(Ids.Distinct().Count() == Ids.Count() ? "重複はありません" : "重複がありました");

Comments

[necos]

丁寧な回答、ありがとうございました。
私の説明不足もあったようで、質問の趣旨と少し離れてしまった様に感じましたので、
ベストアンサーは他の方の回答とさせていただきました。
しかし、内容としては大変参考になりました。
ありがとうございます。

[Zuishin]

見る人の頭にはてなマークの浮かびそうな(実際に浮かんだ)クラスベースの言語らしくない設計を見直すべきと思ったので、趣旨と離れているのは承知の上でより好ましい実装を書きました。
おそらく JavaScript 流でしょうか。

質問者さんの好みに合わない回答は、質問者向けではなくこの質問を見て参考にするほかの人向けだと思ってください。そのような回答をすることはよくあります。

[necos]

補足回答ありがとうございます。
一応、わたしの質問の趣旨は伝わっていたということで、少し安心しました。

そもそも今回の実装の背景には

1
処理をCustmer毎に、とにかくファイルを分けたい
2
実際はActionが10種類程度存在する
3
送られてくるIDをキーとして、処理を動的に振り分けたい

といったものがあります。

そのためには、IDと個別の処理(==class)をなんらかの形で紐付ける必要があり、かつCustomerは定期的に増えて行くので、手を加える箇所を極力少なくする意図で、このような形をとりました。

Answer 2

C# では原則としてクラスのフィールドの公開はしないことになってます。
C# ではマーカーインターフェイスが推奨されませんのでインターフェイスには少なくとも一つのプロパティかメソッドが必要です。
よってこんな感じになるはずです。

class Program
{
    static void Main(string[] args)
    {
        var duplicationCustomers = CreateCustomers()
            .GroupBy(c => c.Id)
            .Where(g => g.Count() > 1)
            .SelectMany(g => g);
    }

    private static IEnumerable<ICustomer> CreateCustomers()
    {
        return Assembly.GetExecutingAssembly().GetTypes()
            .Where(x => x.IsClass && x.BaseType == typeof(ICustomer))
            .Select(x => (ICustomer)System.Activator.CreateInstance(x));
    }

    interface ICustomer
    {
        string Id { get; }
    }

    class CustomerA : ICustomer
    {
        public string Id => "A";
    }

    class CustomerB : ICustomer
    {
        public string Id => "B";
    }

    class CustomerC : ICustomer
    {
        public string Id => "C";
    }
}

Logはお使いの実装で

Comments

[necos]

回答ありがとうございます。
ご提示いただきました内容で、やりたいことが実現できましたので、
hihijijiさんの回答を今回のベストアンサーといたしました。

Answer 3

私も、KSwordOfHasteさんと同じく重複チェックにはGroupByとAny(Count() > 1)を使います。

// クラスの定義
public class ICustomer
{
    string id = "Base";
    public virtual string ID => id;
}

public class CustomerA : ICustomer
{
    string id = "A";
    public override string ID => id;
}

public class CustomerB : ICustomer
{
    string id = "B";
    public override string ID => id;
}

public class CustomerC : ICustomer
{
    string id = "C";
    public override string ID => id;
}

// 判定部分
private static bool IsDuplicate()
    => Assembly.GetExecutingAssembly().GetTypes()
            .Where(x => x.IsClass && x.BaseType == typeof(ICustomer))
            .Select(x => (ICustomer)System.Activator.CreateInstance(x))
            .GroupBy(x => x.ID)
            .Any(x => x.Count() > 1);

Comments

[necos]

回答ありがとうございます。
GroupByとAny(Count() > 1)で、重複チェックができるのですね。
ありがとうございました。

Answer 4

GroupBy
https://msdn.microsoft.com/ja-jp/library/bb534304(v=vs.110).aspx

を使ってみたらいいんじゃないでしょうか？同じキーを持つ要素があるかどうかはIGroupingインターフェースのCountが1以上かどうかでチェックできますし。

Comments

[necos]

回答、ならびにURLの添付、ありがとうございます。
GroupByで、Countをみれば、重複しているかどうかがわかる
ということですね！
試してみます。

Answer 5

解決済みではありますが、ちょっとだけ。
switch や FirstOrDefault が文中にありますが、このような場合は Dictionary を使うべきです。
Dictionary であればキーの一意性も確認できますし、毎回の検索が早くなります。

public class Strategy
{
  public Strategy()
  {
    var customers = this
      .GetType()
      .Assembly
      .GetTypes()
      .Where(x => x.IsClass && !x.IsAbstract && typeof(ICustomer).IsAssignableFrom(x))
      .Select(x => Activator.CreateInstance(x) as ICustomer);
#if 例外版
    // 重複時 ArgumentException
    _table = customers.ToDictionary(x => x.ID); 
#else // ログ版
    var dict = new Dictionary<string, ICustomer>();
    foreach (var c in customers)
    {
      if (!dict.ContainsKey(c.ID))
      {
        dict.Add(c.ID, c);
      }
      else
      {
        // ログ
      }
    }
    _table = dict;
#endif
  }

  private readonly Dictionary<string, ICustomer> _table;

  public void DoAction(string id)
  {
    ICustomer customer;
    if (_table.TryGetValue(id, out customer))
    {
      customer.Action();
    }
  }
}

public class Progmram
{
  private static _strategy = new Strategy();
  public static void Main()
  {
    _strategy.DoAction(GetId());
  }
}