在地图中查找匹配的键和值

Question

在这个问题中，我必须在两个映射中找到所有匹配的键/值映射，然后将其返回到新映射中，但是我遇到了一些问题。 我的想法是从两个映射图中找到所有匹配的键，然后使用这些键将其引用为值。 如果值匹配，我会将键/值放入映射中。 我试图找出为什么它只是添加了所有共同的钥匙。 仅当其对应值也匹配时才添加这些键。 谢谢。

提示：

编写一个方法相交，将两个字符串映射为整数的整数作为参数，然后返回一个新映射，其内容为两者的交集。 此处将两个地图的交集定义为两个地图中都存在的一组键和值。 因此，如果某些键K在第一个和第二个映射中都映射到值V，则将其包括在结果中。 如果在两个映射中都不存在K作为键，或者在两个映射中K均未映射为相同的值V，请从结果中排除该对。 例如，考虑以下两个地图：

{Janet=87, Logan=62, Whitaker=46, Alyssa=100, Stefanie=80, Jeff=88, Kim=52, Sylvia=95}
{Logan=62, Kim=52, Whitaker=52, Jeff=88, Stefanie=80, Brian=60, Lisa=83, Sylvia=87}

在前面的地图上调用方法将返回以下新的地图（键/值对的顺序无关紧要）：

{Logan=62, Stefanie=80, Jeff=88, Kim=52}

我的代码：

// we need to store the keys, then get the values in common, then put the key/map into map
public static Map<String, Integer> intersect(Map<String, Integer> first, Map<String, Integer> second) {
    Map<String, Integer> output = new HashMap<String, Integer>(first); // combined output
    Set<String> keyFirst = new HashSet<String>(); // stores all the keys for first
    for (String key: first.keySet()) { // goes through each key in input
        keyFirst.add(key); // adds all keys from first into keyFirst
    }

    // goes through each key in common and checks to see if they reference to the same value
    Iterator<String> keyFirstItr = keyFirst.iterator();
    while (keyFirstItr.hasNext()) {
        String keyTemp = keyFirstItr.next();
        if (first.get(keyTemp) == second.get(keyTemp)) { // If same key, same value mapped
            output.put(keyTemp, first.get(keyTemp)); // add key value to map
        }
    }
    return output;
}

Answer 1

您正在将所有值从第一个传递到输出，方法是将其传递给构造函数。

Map<String, Integer> output = new HashMap<String, Integer>(first); // you are passing first to the constructor.

您无需创建另一个Set，keySet（）方法将返回set，因此不需要以下几行。

Set<String> keyFirst = new HashSet<String>(); // stores all the keys for first
    for (String key: first.keySet()) { // goes through each key in input
        keyFirst.add(key); // adds all keys from first into keyFirst
}

这是正确的实现。

// we need to store the keys, then get the values in common, then put the key/map into map
public static Map<String, Integer> intersect(Map<String, Integer> first, Map<String, Integer> second) {
    Map<String, Integer> output = new HashMap<String, Integer>(); // combined output

    // goes through each key in common and checks to see if they reference to the same value    
    Iterator<String> keyFirstItr = first.keySet().iterator();
    while (keyFirstItr.hasNext()) {
        String keyTemp = keyFirstItr.next();
        if (first.get(keyTemp).equals(second.get(keyTemp))) { // If same key, same value mapped
            output.put(keyTemp, first.get(keyTemp)); // add key value to map
        }
    }
    return output;
}

Answer 2

对于此练习，更简单的解决方案是跳过迭代器并使用for循环，如下所示。 对于map1中的每个名称，我们检查它是否存在于map2中，以及值是否匹配。 然后将K和V添加到新地图：

public static Map intersect(Map<String, Integer> map1, Map<String, Integer> map2){
    Map<String, Integer> newMap = new HashMap<>();
    for (String name : map1.keySet()){
        if(map2.containsKey(name) && map1.get(name).equals(map2.get(name))){
            newMap.put(name, map1.get(name));
        }
    }
    return newMap;        
}