Buon modello di progettazione per l'esecuzione di un processo su una raccolta di dati?

Question

Best spiegato con il codice di credo, questo è solo un semplice esempio:

public class MyPOJO {

    public String name;
    public int age;

    public MyPOJO(String name, int age) {
        this.name = name;
        this.age = age;
    }
}

public class MyProcessor {

    public List<MyPOJO> process(List<MyPOJO> mypojos) {
        List<MyPOJO> temp = new ArrayList<MyPOJO>; 
        for (int i=0; i <moypojos.size(); i++) {
            if (filterOne(mypojos[i])) continue;
            if (filterTwo(mypojos[i])) continue;
            if (filterThree(mypojos[i])) continue;
            temp.add(mypojos[i];
        }
    }

    public boolean filterOne(MyPOJO mypojo) {
        // in practice filters aren't so basic
        return (mypojo.age < 21);
    }
    // assume implementations for the other filter methods
}

Yikes che è brutto. Fondamentalmente ho una collezione e mi piacerebbe passare attraverso un setaccio di sorta di continuare solo elaborazione degli oggetti che soddisfano determinati criteri. La mia ipotesi è che c'è un modello migliore per questo che un sacco di metodi che restituiscono booleani.

Answer 1

È possibile avere l'elenco dei IFilters.

in questo modo

boolean filtersResult = false;
for (IFilter filter : filters) {
    filterResult = filter.process(mypojos[i])

    if (filterResult)
        break;
}

Answer 2

Si consiglia di implementare i filtri in modo tale che essi prendono una collezione, e restituire un insieme filtrato:

public interface IFilter<E> {
  public Collection<E> filter(Collection<E> input);
}

In questo modo è possibile concatenare insieme filtra molto banalmente. Il rovescio della medaglia è che per grandi collezioni è più lento e richiede più spazio; ma il codice è molto più leggibile.

Answer 3

Perché non usare Bean-Query ? può rendere il codice leggibile.

List<MyPOJO> result=selectBean(MyPOJO.class).where(
                                                not(
                                                  anyOf(
                                                      value("aga",lessThan(21)),
                                                      value("age",greaterThan(50))
                                                  )
                                                )
                                            .executeFrom(myPojos).