OO Muster eine Sammlung in Gruppen zum Teil

Question

Ich habe eine Liste von MyObjects die ich in drei Gruppen zu unterteilen müssen:

1. gut bekannt (halten)
2. schlecht bekannt (ablehnen)
3. unerkannte (erhöhen alert)

MyObject enthält verschiedene Eigenschaften, die das Objekt in setzen, die der 3 Gruppen zu prüfen ist, um festzustellen.

Meine erste Implementierung (Java) nimmt nur eine Liste in ihrem Konstruktor und führt die Triage dort durch. Pseudo-Code:

class MyObjectFilterer { 
    public MyObjectFilterer(List<MyObject> list) { 
    // triage items here 
    } 
    public List<MyObject> getGood() { 
    // return sub-list of good items 
    } 
    public List<MyObject> getBad() { 
    // return sub-list of bad items 
    } 
    public List<MyObject> getUnrecognized() { 
    // return sub-list of unrecognized items 
    } 
} 

Probleme mit dieser Implementierung? Gibt es eine bessere OO-Wahl?

Answer 1

Ich würde wahrscheinlich eine statische Factory-Methode bevorzugen, um die Filterung durchzuführen, die dann einen privaten Konstruktor aufruft, der die drei gefilterten Listen nimmt und der guten Code-Praxis folgt, niemals ernsthaft in einem Konstruktor zu arbeiten. Ansonsten sieht das gut aus.

Answer 2

Es kann mehrere Ansätze geben. Wenn das Problem generisch/wiederholend genug ist, könnten Sie eine Schnittstelle mit einer Methode zum Klassifizieren der Objekte definieren.

interface Selector { 
    public boolean isGood(MyObject myObject); 
    public boolean isBad(MyObject myObject); 
    public boolean isUnknown(MyObject myObject); 

}

diese Weise können Sie leicht die Logik Implementierung ändern könnte.

Answer 3

Eine andere Idee wäre die Kette der Verantwortung.

Ihre MyObjectFilterer enthält einen Verweis auf drei Objekte GoodFilterer, BadFilterer und UnrecognizedFilterer. Jede von ihnen enthält die folgenden Methoden: addMethod (MyObject Objekt), getObjects() und addFilter(). Natürlich müssen sie eine Schnittstelle implementieren Filterer.

Mit der addFilter Methode können Sie die Kette bauen. so dass die GoodFilterer enthält einen Verweis auf die BadFilterer und dieser enthält zum UnrecognizedFilterer

Jetzt können Sie durch die Liste gehen von MyObjects und rufen Sie die Add-Methode auf dem GoodFilterer eine Referenz (erster in dieser Kette). Innerhalb der Add-Methode entscheiden Sie, ob das gut ist, als Sie es behalten und die Arbeit beenden, wenn Sie es nicht weiterleiten an die BadFilterer.

Sie behalten Ihre drei Methoden für die gut/schlecht und unerkannt bekommen, aber Sie werden diese getObjects auf die() übergeben Methode des filterer

Der Nutzen entspricht, dass die Logik, wenn dies ein guter/schlechter oder Unerkannter ist jetzt getrennt.

Der Nachteil wäre, dass Sie 3 neue Klassen und 1 Schnittstelle benötigen.

Aber wie ich schon sagte, das ist nur eine andere Idee, was Sie tun könnten.

Answer 4

Sie sollten vereinfachen, wie es möglich ist. Machen Sie einfach statische Methode in MyObjectFilter mit folgender Signatur:

öffentliche statische Liste filterMyObjects (List data, Group group).

Gruppe Aufzählung mit drei Werten, und es kann ich vielleicht

Answer 5

als Attribut von MyObject-Klasse verwendet werden versuchen, so etwas wie:

enum MyObjectStatus { 
    GOOD, BAD, UNRECOGNIZED; 
} 

class MyObjectFilterer { 
    private MyObjectStatus getStatus(MyObject obj) { 
    // classify logic here, returns appropriate enum value 
    } 

    // ListMultimap return type below is from Google Guava 
    public ListMultimap<MyObjectStatus, MyObject> classify(List<MyObject> objects) { 
    ListMultimap<MyObjectStatus, MyObject> map = ArrayListMultimap.create(); 
    for(MyObject obj: objects) { 
     map.put(getStatus(obj), obj); 
    } 
    } 
} 

Anruf Klassifizieren() eine Multimap zu bekommen, und jede Kategorie extrahieren wie mit so etwas wie benötigt:

List<MyObject> good = map.get(GOOD); 
List<MyObject> bad = map.get(BAD); 
List<MyObject> unknown = map.get(UNRECOGNIZED); 

Eine nette Sache über diese Lösung Sie müssen nicht erstellen/Zugriffsmethoden für jede Kategorie veröffentlichen (es sei denn, Sie möchten), und Wenn neue Kategorien erstellt werden, fügen Sie auch keine neuen Zugriffsmethoden hinzu - nur die neue Enumeration und die zusätzliche Klassifikatorlogik.