Создание фильтров метода

Question

В моем коде у меня есть List<Person>.Атрибуты к объектам в этом списке могут включать что-то вроде:

• ID
• Имя
• Фамилия

В части моего приложения я буду позволять пользователю выполнять поиск конкретного человека, используя любую комбинацию этих трех значений.На данный момент у меня есть оператор switch, просто проверяющий, какие поля заполнены, и вызывающий метод, предназначенный для этой комбинации значений.

т.е.:

switch typeOfSearch    
if 0, lookById()    
if 1, lookByIdAndName()  
if 2, lookByFirstName()

и так далее.На самом деле существует 7 различных типов.

Это дает мне один метод для каждого оператора.Это «хороший» способ сделать это?Есть ли способ, которым я должен использовать параметр или какой-то «фильтр»?Это может не иметь значения, но я кодирую это на Java.

Answer 1

Вы можете сделать что-то более элегантное с картами и интерфейсами.Попробуйте, например,

interface LookUp{
    lookUpBy(HttpRequest req);
}

Map<Integer, LookUp> map = new HashMap<Integer, LookUp>();

map.put(0, new LookUpById());
map.put(1, new LookUpByIdAndName());

...

в вашем контроллере , тогда вы можете сделать

int type = Integer.parseInt(request.getParameter(type));
Person person = map.get(type).lookUpBy(request);

Таким образом, вы можете быстронайдите метод с картой.Конечно, вы также можете использовать длинный переключатель, но я чувствую, что это более управляемо.

Answer 2

В какой-то момент вам следует взглянуть на что-то вроде Lucene , которое даст вам наилучшую масштабируемость, управляемость и производительность для этого типа поиска. Не зная объема данных, с которыми вы имеете дело, я только рекомендую это для более долгосрочного решения с большим набором объектов для поиска. Это удивительный инструмент!

Answer 3

Этот способ быстро становится неуправляемым, так как количество комбинаций быстро становится огромным.Создайте класс PersonFilter, имеющий все возможные параметры запроса, и посетите каждого человека в списке:

private class PersonFilter {
    private String id;
    private String firstName;
    private String lastName;

    // constructor omitted

    public boolean accept(Person p) {
        if (this.id != null && !this.id.equals(p.getId()) {
            return false;
        }
        if (this.firstName != null && !this.firstName.equals(p.getFirstName()) {
            return false;
        }
        if (this.lastName != null && !this.lastName.equals(p.getLastName()) {
            return false;
        }

        return true;
    }
}

Фильтрация теперь реализована с помощью

public List<Person> filter(List<Person> list, PersonFilter filter) {
    List<Person> result = new ArrayList<Person>();
    for (Person p : list) {
        if (filter.accept(p) {
            result.add(p);
        }
    }
    return result;
}

Answer 4

Если добро означает «язык делает это для меня», нет.

Если «хороший» означает «читаемый», я бы определил в Person метод match (), который возвращает true, если объект соответствует вашим критериям поиска. Также, вероятно, это хороший способ создать метод Criteria, в котором вы можете инкапсулировать критерии поиска (какие поля вы ищете и какое значение) и передать его в соответствие (критерии Criteria).