рефакторинга Java код 8 поток

Question

Я осуществил следующий код с помощью Java 8.

Map<String, String> coMap = getHashMap(); 

String newCoName = coMap.entrySet() 
        .stream() 
        .filter(coEntry -> coEntry.getValue().equals(newcoId)) 
        .map(coEntry -> coEntry.getKey()) 
        .collect(Collectors.joining()); 


String oldCoName = coMap.entrySet() 
        .stream() 
        .filter(coEntry -> coEntry.getValue().equals(oldcoId)) 
        .map(coEntry -> coEntry.getKey()) 
        .collect(Collectors.joining()); 

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

Answer 1

Поскольку вся разница - это идентификатор, простой способ делает это для вас.

String getName(int id) { // supposed id is an integer 
    return coMap.entrySet() 
      .stream() 
      .filter(coEntry -> coEntry.getValue().equals(id)) 
      .map(coEntry -> coEntry.getKey()) 
      .collect(Collectors.joining()); 
} 

Answer 2

большинство IDE, можно реорганизовать его для вас с помощью "Extract Method" рефакторинга: http://refactoring.com/catalog/extractMethod.html или вы можете сделать это вручную

Answer 3

Вы можете использовать этот вспомогательный метод:

public static String join(Map<String, String> map, String value) { 
    return map.keySet().stream() 
      .filter(key -> Objects.equals(map.get(key), value)) 
      .collect(Collectors.joining()); 
} 

Здесь некоторый пример кода, используя метод:

Map<String, String> coMap = new HashMap<>(); 
    coMap.put("A", null); 
    coMap.put("B", "V"); 
    coMap.put("C", "V"); 
    coMap.put("D", "Z"); 
    coMap.put("E", "Z"); 
    coMap.put("F", null); 

    System.out.println("Vs: " + join(coMap, "V")); 
    System.out.println("Zs: " + join(coMap, "Z")); 

А вот выход:

Vs: BC 
Zs: DE 

Answer 4

Большей проблемой, чем повторять один и тот же код дважды выполняет один и тот же код дважды.

Было бы более эффективным для запуска одного Stream трубопровода производить свой вывод:

Map<String,String> keysByValue = 
      coMap.entrySet() 
       .stream() 
       .collect(Collectors.groupingBy(Map.Entry::getValue, 
               Collectors.mapping(Map.Entry::getKey, 
                    Collectors.joining()))); 

Это даст вам для каждого значения исходной Map (не только два значения исходного кода является поиск), которые имеют это значение.

Затем вы можете извлечь из Map необходимые вам данные:

String newCoName = keysByValue.get(newcoId); 
String oldCoName = keysByValue.get(oldcoId); 

ввода пробы и вывод:

Map<String,String> coMap = new HashMap<>(); 
coMap.put("a","foo"); 
coMap.put("b","foo"); 
coMap.put("c","bar"); 
coMap.put("d","bar"); 
Map<String,String> keysByValue = ... // same as the code above 
String newValueKeys = keysByValue.get("foo"); 
String oldValueKeys = keysByValue.get("bar"); 
System.out.println (newValueKeys); 
System.out.println (oldValueKeys); 

Выход:

ab 
cd 

Answer 5

Другой способ использует с FunctionalInterface Predicate , ваш фильтр условий будет динамическим

public static Predicate<Map.Entry> getPredicate(String col) { 
    return p -> p.getValue().equals(col); 
} 

public static String getName(HashMap<String, String> coMap, Predicate<Map.Entry> predicate) { 
    return coMap.entrySet() 
      .stream() 
      .filter(predicate) 
      .map(coEntry -> coEntry.getKey()) 
      .collect(Collectors.joining()); 
} 

Вызов в вашем коде:

getName(coMap, getPredicate(newcoId)); 
    getName(coMap, getPredicate(oldcoId));