重構的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

由於整個差異是一個id，所以一個簡單的方法可以幫助你。

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的可重構它，你使用「提取方法」重構爲： 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));