OO шаблон для разделения коллекции на группы

Question

У меня есть список MyObjects, которые мне нужно разделить на три группы:

1. Known хорошо (держать)
2. Известный плохой (отклонить)
3. Непризнанный (поднять предупреждение)

MyObject содержит различные свойства, которые должны быть изучены, чтобы определить, какой из 3-х групп поместить объект.

Моя первоначальная реализация (Java) просто берет Список в его конструкторе и выполняет сортировку там. Псевдокод:

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 
    } 
} 

Любые проблемы с этой реализацией? Есть ли лучший выбор OO?

Answer 1

Я бы предпочел статический заводский метод для фильтрации, который затем вызывает частный конструктор, который принимает три отфильтрованных списка, следуя хорошей практике кода, никогда не делая серьезной работы в конструкторе. Кроме этого, это выглядит отлично.

Answer 2

Может быть несколько подходов. Если проблема является общей или повторяющейся, вы можете определить интерфейс с методом для классификации объектов.

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

}

Таким образом, вы могли бы изменить реализацию логики легко.

Answer 3

Другая идея - использовать Цепь ответственности.

Ваш MyObjectFilterer содержит ссылку на трех объектов GoodFilterer, BadFilterer и UnrecognizedFilterer. Каждый из них содержит следующие методы: addMethod (объект MyObject), getObjects() и addFilter(). Конечно, они должны реализовать интерфейс Фильтр.

С помощью метода addFilter вы можете построить цепочку. так что GoodFilterer содержит ссылку на BadFilterer и это одна содержит ссылку на UnrecognizedFilterer

Теперь вы идете через ваш список MyObjects и вызывать метод добавления на GoodFilterer (первый в этой цепочке). Внутри метода добавления вы решаете, хорошо ли это, чем вы его сохраняете и завершаете, если не передать его на BadFilterer.

Вы держите свои три метода для получения хорошего/плохого и непризнанное, но вы будете передать это GetObjects() метод из соответствующего Filterer

Преимущество заключается в том, что логика, если это теперь есть хороший/плохой или непризнанный.

The Downside вам понадобится 3 новых класса и 1 интерфейс.

Но, как я уже сказал, это просто другая идея, что вы могли бы сделать.

Answer 4

Вы должны упростить, насколько это возможно. Просто сделайте статический метод в MyObjectFilter со следующей подписью:

public static Фильтр списка объектовMyObjects (данные списка, группа групп).

Группа перечисление с тремя значениями, и он может быть использован в качестве атрибута класса MyObject

Answer 5

я мог бы попробовать что-то вроде:

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); 
    } 
    } 
} 

вызовов КЛАССИФИЦИРОВАТЬ(), чтобы получить Multimap и извлекать каждую категорию при необходимости что-то вроде:

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

хорошая вещь об этом решения не должны создавать/публиковать методы доступа для каждой категории (если вы не хотите), и если создаются новые категории, вы также не добавляете новых аксессуаров - только новое перечисление и дополнительную классификационную логику.