別のクラスのClassCollectionを作成することをお勧めしますか？

Carクラスがあると言いましょう：

public class Car
{
    public string Engine { get; set; }
    public string Seat { get; set; }
    public string Tires { get; set; }
}

駐車場に関するシステムを作成しているとしましょう。私は多くのCarクラスを使用するので、クラスを作成しCarCollectionますFindCarByModel。

public class CarCollection
{
    public List<Car> Cars { get; set; }

    public Car FindCarByModel(string model)
    {
        // code here
        return new Car();
    }
}

クラスを作成している場合ParkingLot、ベストプラクティスは何ですか？

オプション1：

public class ParkingLot
{
    public List<Car> Cars { get; set; }
    //some other properties
}

オプション＃2：

public class ParkingLot
{
    public CarCollection Cars { get; set; }
    //some other properties
}

ClassCollection別のものを作成することは良い習慣Classですか？

.NETのジェネリックの前は、「型付き」コレクションを作成class CarCollectionして、グループ化する必要のあるすべての型に対してなどを使用することが一般的でした。.NET 2.0では、Generics List<T>が導入され、新しいクラスが導入されました。これにより、作成できるCarCollectionなどの作成を省くことができますList<Car>。

ほとんどの場合、それList<T>は目的には十分ですが、コレクション内で特定の動作を行いたい場合があります。そうだと思われる場合は、いくつかのオプションがあります。

• List<T>たとえば、カプセル化するクラスを作成しますpublic class CarCollection { private List<Car> cars = new List<Car>(); public void Add(Car car) { this.cars.Add(car); }}
• カスタムコレクションを作成する public class CarCollection : CollectionBase<Car> {}

カプセル化アプローチを採用する場合は、少なくとも列挙子を公開して、次のように宣言する必要があります。

public class CarCollection : IEnumerable<Car>
{
    private List<Car> cars = new List<Car>();

    public IEnumerator<Car> GetEnumerator() { return this.cars.GetEnumerator(); }
}

そうしないとforeach、コレクションをやり直すことはできません。

カスタムコレクションを作成する理由は次のとおりです。

• IList<T>またはですべてのメソッドを完全に公開したくないICollection<T>
• コレクションにアイテムを追加または削除するときに追加のアクションを実行したい

それは良い習慣ですか？それはあなたがそれをしている理由に依存します。例えば、それが上に挙げた理由の1つであれば、そうです。

Microsoftは非常に定期的にそれを行っていますが、ここにかなり最近の例をいくつか示します。

• System.Web.Http.Filters.HttpFilterCollection
• System.Web.Mvc.ModelBinderProviderCollection

あなたのFindByメソッドについては、車を含むコレクションに対して使用できるように拡張メソッドに入れたいと思います：

public static class CarLookupQueries
{
    public static Car FindByLicencePlate(this IEnumerable<Car> source, string licencePlate)
    {
        return source.SingleOrDefault(c => c.LicencePlate == licencePlate);
    }

    ...
}

これにより、コレクションを照会する懸念と、車を格納するクラスを区別します。

いいえ。XXXCollectionクラスの作成は、.NET 2.0でのジェネリックの出現により、ほとんどスタイルから外れました。実際、Cast<T>()これらのカスタム形式からデータを取得するために最近使用されている気の利いたLINQ拡張機能があります。

上記のFindByCarModelの例のように、ドメイン指向のメソッドを使用してコレクションを検索/スライスすると便利ですが、ラッパーコレクションクラスを作成する必要はありません。この状況では、通常、拡張メソッドのセットを作成します。

public static class CarExtensions
{
    public static IEnumerable<Car> ByModel(this IEnumerable<Car> cars, string model)
    {
        return cars.Where(car => car.Model == model);
    }
}

必要な数のフィルターまたはユーティリティメソッドをそのクラスに追加し、どこでも使用できますIEnumerable<Car>。これには、何でも、などのICollection<Car>配列が含まれます。CarIList<Car>

永続化ソリューションにはLINQプロバイダーがあるため、頻繁に操作してreturnを実行する同様のフィルターメソッドも作成するIQueryable<T>ため、これらの操作をリポジトリにも適用できます。

.NETのイディオム（まあ、C＃）は1.1から大きく変わりました。カスタムコレクションクラスの維持は苦痛であり、CollectionBase<T>ドメイン固有のフィルターおよびセレクターメソッドのみが必要な場合、拡張メソッドソリューションでは得られないものから継承することはほとんど得られません。

他の項目のコレクションを保持するための特別なクラスを作成する唯一の理由は、価値のあるものを追加するときであると思います。それは単に、インスタンスIListまたは別のタイプのコレクションからカプセル化/継承する以上のものです。

たとえば、あなたの場合、偶数/不均等なロットスペースに駐車された車のサブリストを返す関数を追加します...そしてそれでも...多分それが頻繁に再利用される場合のみです。関数であり、一度だけ使用されますが、ポイントは何ですか？キス！

さて、あなたが多くのソート/検索メソッドを提供することを計画しているなら、はい、これはその特別なコレクションクラスでそれらが属するべき場所だからです。これは、いくつかの「検索」クエリの複雑さや、ソート/検索方法でできることを「隠す」ための良い方法でもあります。

次のオプションを使用することをお勧めします。そのため、コレクションにメソッドを追加し、リストの利点を活用できます。

public class CarCollection:List<Car>
{
    public Car FindCarByModel(string model)
    {
        // code here
        return new Car();
    }
}

そして、あなたはC＃7.0のようにそれを使用することができます

public class ParkingLot
{
    public CarCollection Cars { get; set; }=new CarCollection();
    //some other properties
}

または、次のように使用できます

public class ParkingLot
{
   public ParkingLot()
   {
      //initial set
      Cars =new CarCollection();
   }
    public CarCollection Cars { get; set; }
    //some other properties
}

-@Bryanコメントのおかげで一般的なバージョン

   public class MyCollection<T>:List<T> where T:class,new()
    {
        public T FindOrNew(Predicate<T> predicate)
        {
            // code here
            return Find(predicate)?? new T();
        }
       //Other Common methods
     }

そして、あなたはそれを使用することができます

public class ParkingLot
{
    public MyCollection<Car> Cars { get; set; }=new MyCollection<Car>();
    public MyCollection<Motor> Motors{ get; set; }=new MyCollection<Motor>();
    public MyCollection<Bike> Bikes{ get; set; }=new MyCollection<Bike>();
    //some other properties
}