Linq-extension-Methode

Ich Häufig zu begrenzen Wählt durch Felder wie publishStart, publishEnd, active

Habe ich diese Felder in verschiedenen Tabellen. So werden nur die Zeilen ausgewählt werden sollte, sind 

a: active == true;
b: publishStart < now;
c: publishEnd > now;

So, zum Beispiel: 

db.myTable.SingleOrDefault(a => (a.ID == _theID 
          //now the active and start-end part:            
                      && ((a.publishEnd > DateTime.Now) || (a.publishEnd == null))
                      && ((a.publishStart <= DateTime.Now) || (a.publishStart == null))
                      && a.active == true));

Dies ist etwas langatmig, so dass ich Frage mich, ob es möglich ist, zu erstellen (Erweiterung?)-Methode wie:

db.myTable.SingleOrDefault(a => (a.ID == _theID).isActive()

wo die isActive() bietet die 3 Zeilen in der obigen Schnipsel.

Wie könnte ich dies tun?
Gibt es einen besseren Weg, um clean-up-code?

  • Diese sind alle in separaten Tabellen, richtig? Nicht eine einzige Tabelle, die ausgesetzt ist, die durch den Kontext? Ich Frage, weil Sie angeben, in der Präambel, dass die Eigenschaften sind auf separaten Tabellen, die bei der Verwendung von nur einem Kontext in den filter. Wenn Sie alle auf die gleiche Tabelle, es ist einfach, wenn Sie sich auf unterschiedlichen Tabellen, verändert Sie die richtige Antwort ungemein.
  • In diesem Fall würden Sie benötigen, rufen Sie IsActive vor SingleOrDefault so dass Sie können filter der aktiven Elemente, bevor Sie nur eine, es sei denn, es ist immer nur ein Element mit der entsprechenden ID.
InformationsquelleAutor reinhard | 2013-01-08
  1. 13

    Definieren, eine Erweiterung, die Sie brauchen eine statische Klasse. Sie können diese in beliebiger namespace, die Sie wollen, nur denken Sie daran, es in Ihr aufzuhalten der Benutzung. 

    public static class Extensions
{
    public static IQueryable<T> Active<T>(this IQueryable<T> source)
        where T : YourEntityType
    {
        return source.Where(a => ((a.publishEnd > DateTime.Now) || (a.publishEnd == null))
                          && ((a.publishStart <= DateTime.Now) || (a.publishStart == null))
                          && a.active == true);
    }
}

    Bemerken YourEntityType dort. Dieser wird verwendet um sicherzustellen, dass die Methode Kenntnis von der Existenz publishStart, publishEnd, und active. Dies sollte entweder in der Klasse, die diese Felder oder einen Vertrag (interface), die Sie definiert.

    Würden Sie dann nennen das so:

    var item = db.myTable.Active().SingleOrDefault(...);

    Mehr auf Erweiterung Methoden hier: http://msdn.microsoft.com/en-us/library/bb383977.aspx

    Da es viele Kommentare aufspringen alle über dem Platz, ich werde fügen Sie hier eine kurze Erklärung der interface-Lösung...

    Es ist unklar in der Frage, ob oder nicht es ist eine Allgemeine Implementierung für die drei gefilterten Felder oder eine Schnittstelle zu definieren. Wenn nicht, für die oben genannten arbeiten, werden Sie nicht entweder:

    1. Eine Basisklasse implementieren diese Felder. In diesem Szenario würden Sie ersetzen YourEntityType mit YourBaseEntityType.
    2. Eine Schnittstelle zu definieren, die Felder. In diesem Szenario würden Sie benötigen, um Ihre Klassen implementieren die Felder. Wenn die Klassen werden automatisch generiert (z.B. entity framework-Modell/db-first), dann können Sie implementieren, partielle Klassen, dass Sie die Schnittstelle implementieren. In diesem Fall ersetzen Sie YourEntityType mit IYourContract.
    • Gibt es einen Grund für die Verwendung IQueryable<T> statt IEnumerable<T>?
    • Das wird nicht funktionieren, in einem Aufruf SingleOrDefault was der OP fordert.
    • Ich benutze lieber IQueryable<T> wie ich habe Polymorphismus in der Vergangenheit Probleme mit IEnumerable<T>. Da gibt es Erweiterungen für beide IEnumerable<T> und IQueryable<T>, die eine IEnumerable<T> führen kann, in die falschen Erweiterungen genannt werden, die weiter in der Kette, wodurch die Aufzählung eher als latente Abfragen.
    • Es ist die Frage, ob Sie möchten, um die Durchführung der Abfrage auf die Datenbank, oder die Rückkehr, die alle Elemente und filtern in C# - code. Filterung in der DB ist fast immer vorzuziehen, wenn möglich, aber es ist nicht immer möglich.
    • Es gibt keine müssen machen dies eine generische Methode und dann beschränken Sie den parameter Typ auf einen Typ, es sei denn, alle Typen mit diesen drei Eigenschaften modelliert werden, die mit der Vererbung - sowas würde ich wirklich nicht empfehlen zu tun, in den meisten Fällen.
    • Nein, das kompilierte assembly nicht das gleiche. C# - generics sind keine templates. Es können funktionell gleichwertig, wenn die Art versiegelt ist (oder keine abgeleiteten Typen sind überhaupt benutzt), aber es ist nicht gleichwertig IL.
    • In meinem Beispiel YourEntityType kann eine Basis-Entität oder eine Schnittstelle, in dem Fall ist es völlig gültig zu sein, die generischen. Wenn die OP verwendet die tatsächliche Einheit geben, dann ja, ich Stimme zu.
    • danke - ich bin mit entity framework-erste, und ich weiß nicht was du meinst mit " dann können Sie implementieren, partielle Klassen, dass Sie die Schnittstelle implementieren. In diesem Fall ersetzen Sie YourEntityType mit IYourContract." Ein kleines code-Beispiel bitte?
    • wo ist die extension-Methode Beispiel, Sie nur zeigte die lange Lambda-syntax
    • Der code innerhalb der Methode nichts zu tun hat Witz-Erweiterung Methoden (es ist eigentlich der OP-code). Die Erweiterung die Methode ist, die Active Methode selbst. Beachten Sie die this Stichwort, indem der erste parameter. Ein static Methode in einer static Klasse kann gemacht werden in eine extension-Methode durch hinzufügen thia auf den ersten parameter. Das heißt, Sie können rufen Sie die statische Methode als ob es ist ein - Instanz-Methode der erste parameter. Ich habe einen link zu meiner Antwort zum weiteren Lesen.

    InformationsquelleAutor Paul Fleming
  2. 4

    Nur definieren Sie eine Schnittstelle wie diese

    public interface IHaveAActivityPeriod 
{
    Boolean active { get; }

    DateTime? publishStart { get; }

    DateTime? publishEnd { get; }
}

    und fügen Sie alle relevanten Klassen.

    public class Foo : IHaveAActivityPeriod { [...] }

public class Bar : IHaveAActivityPeriod { [...] }

    Nun können Sie mit dieser Erweiterung Methode

    public static class Extensions
{
    public static Boolean IsActive(this IHaveAActivityPeriod item)
    {
        var now = DateTime.Now;

        return item.active &&
               (item.publishStart <= now)
               (!item.publishEnd.HasValue || (item.publishEnd > now));
    }
}

    auf jeder Instanz die IHaveAActivityPeriod.

    var foo = new Foo();

var isFooActive = foo.IsActive();

var bar = new Bar();

var isBarActive = bar.IsActive();

    Ich völlig verpasst die Möglichkeit zu konstruieren, die eine Erweiterung Methode, die die Filterung der Sequenz anstatt sich auf eine einzelne Person auf einmal. Nimm einfach die extension-Methode von flem Antwort ein Wurf in die Schnittstelle, wie Typ-Einschränkung.

    public static class Extensions
{
    public IQueryable<T> IsActive<T>(this IQueryable<T> sequence)
        where T : IHaveAActivityPeriod
    {
        return source.Where(item =>
                   item.active &&
                   (item.publishStart <= now) &&
                   (!item.publishEnd.HasValue || (item.publishEnd > now));

    }
}
    • Dies wird nicht wandeln sql, und würde daher erfordern die volle Bewertung für die Filterung.
    • Das ist wahr, aber es ist keine wirklich gute Lösung, ist übersetzbar in SQL (ohne einen benutzerdefinierten Anbieter) und so beschlossen wir einen LINQ to Objects-Lösung.
    InformationsquelleAutor Daniel Brückner
  3. 3
    public static class Extensions
{
    public static IEnumerable<MyClass> isActive(this IEnumerable<MyClass> list)
    {
        return list.Where(a =>  
               ((a.publishEnd > DateTime.Now) || (a.publishEnd == null))
                 && ((a.publishStart <= DateTime.Now) || (a.publishStart == null))
                 && a.active == true);
    }
}
    InformationsquelleAutor I4V
Schreibe einen Kommentar