0

Необходимо создать универсальный метод, который может удалять записи из Map<Key,Value>, критерии для удаления могут быть следующие: к примеру объект Key представлен классом имеющим три поля: int id, String name, String color, с помощью созданного универсального метода, мы может удалять все записи из карты если id < 20 или name=="Pen" и color=="Green", либо name=="Paper";

Подскажите пожалуйста, как лучше реализовать данный метод?

1
  • а почему без java 8? просто наиболее простое решение - передать лямбду. аналог - анонимный класс, но тогда это становится довольно громоздким 28 июн 2020 в 11:41

3 ответа 3

1

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

Если очень хотите жить на мапе, то обязательно для класса Key переопределяем метод equals, иначе два экземпляра, у которых все поля полностью совпадают, будут в мапе считаться различными ключами

Теперь вытаскиваем все ключи и ищем среди них подходящие под условия

ArrayList<Key> suited = new ArrayList<Key>();
for ( Key key : map.keySet() ) {
    if(key.id = 20 && key.name.contentEquals("Pen"))
       suited.add(key);
}

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

6
  • просят универсальный метод для удаления по разным критериям, а вы захардкодили условие удаления 28 июн 2020 в 11:37
  • @Дмитрий а что может имеется в виду под универсальностью? Вам все равно сообщать условия так или иначе придется. Можно, конечно, попробовать вытащить условие в интерфейс, но учитывая объем кода это выглядит весьма бессмысленно 28 июн 2020 в 11:41
  • да, если предположить, что такое условие всего одно. а если вы хотите вызывать метод удаления по условию многократно (из разных классов) и с разными условиями, то в вашем случае надо писать новый метод для каждого вызова 28 июн 2020 в 11:43
  • универсальность заключается в том, что у нас имеется несколько map, ключи которых различны и нам надо решить, как передать условие в метод для нахождения ключей, которые мы можем удалить
    – Evgeniy
    28 июн 2020 в 11:44
  • @Дмитрий А в каком случае не придется? Просто вместо кода, где вы пишете цикличную проверку условия, вы вызываете функцию и передаете ей другую функцию-предикат, чтоб первая функция гоняла по циклу предикат... это нормально если кода много, но менять описание цикла на описания интерфейса мне кажется бессмысленным 28 июн 2020 в 11:45
0

можно так

import java.util.HashMap;
import java.util.Map;
import java.util.Objects;
import java.util.function.Predicate;

public class Test {

    private static final Map<Object, Object> map = new HashMap<>();

    public static void main(String[] args) {
        map.put("1", 1);
        map.put(2L, "2");

        universalRemoveIf(entry -> {
            Object key = entry.getKey();
            return Objects.equals("1", key);
        });
        universalRemoveIf(entry -> {
            Object value = entry.getValue();
            return Objects.equals("1", value);
        });
    }

    public static void universalRemoveIf(Predicate<Map.Entry<Object, Object>> predicate) {
        map.entrySet().removeIf(predicate);
    }

}

up without java8

import java.util.*;

public class Test {

    private static final Map<Object, Object> map = new HashMap<>();

    public static void main(String[] args) {
        map.put("1", 1);
        map.put(2L, "2");

        System.out.println(map);
        universalRemoveIf(new DeletePred() {
            @Override
            public boolean exec(Map.Entry<Object, Object> entry) {
                Object key = entry.getKey();
                return Objects.equals(key, "1");
            }
        });
        System.out.println(map);
    }

    public static void universalRemoveIf(DeletePred predicate) {
        Set<Object> keys = new HashSet<>();
        for (Map.Entry<Object, Object> entry : map.entrySet()) {
            if (predicate.exec(entry)) {
                keys.add(entry.getKey());
            }
        }
        for (Object key : keys) {
            map.remove(key);
        }
    }

    public interface DeletePred {
        boolean exec(Map.Entry<Object, Object> entry);
    }

}
4
  • Решение отличное, но стоит условие создания метода без участия java8, к которым относятся функциональные интерфейсы
    – Evgeniy
    28 июн 2020 в 11:40
  • можно просто написать их самому) или вообще нельзя использовать интерфейсы?
    – Mike
    28 июн 2020 в 11:42
  • т.е. просто реализовать по аналогии с функциональными интерфейсами свой интерфейс и передавать в метод анонимный класс?
    – Evgeniy
    28 июн 2020 в 11:46
  • интерфейсами пользоваться можно)
    – Evgeniy
    28 июн 2020 в 11:47
0
    interface Predicate<T>{

       boolean predicate(T key)
    }
    
    void <T> removeFromWhere(HashMap<T, *> map, Predicate<T> predicate){
       for (T key : map.keySet()){
          if(predicate.predicate(key))
             map.remove(key)
       }
    }

Теперь в коде вызываем так

    removeFromWhere(map, new Predicate<Key>(){
       @Override
       boolean predicate(Key key){
          return key.id == 20 && key.name.contentEquals("Pen)
       }
    })

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.