对 HashMap 的方法 containsValue 的困惑

Question

我正在尝试找到我的方法的简化版本，我想知道您是否有更好的意见。

基本上我有一个 HashMap 将键值存储为 String-String[]

我想要一个方法来确定新插入的 String[]-value 是否包含已存在于已存储 String[]-value 中的 String。

我写的“并且显然工作正常”是以下方法：

static Map<String,String[]> myMap=new HashMap<String,String[]>();

    public static boolean kijkContains(String[] syn){

for(String s:myMap.keySet()){

    String[]temp=myMap.get(s);

    for(int i=0; i<temp.length; i++){

        for(int k=0; k<syn.length; k++){

            if(temp[i].equals(syn[k])){

                return true;
            }
        }
    }
  }
return false;
}

我的疑惑是循环的数量，这显然是一种高内存消耗的方法，我想知道你是否能想到更好的版本。

我尝试过使用 Map 的 containsValue() 方法，但由于该方法将 String[] 视为值而不是读取数组，因此我不能真正将其用作比较器。

Answer 1

您的代码不会使用特别多的内存，因为您没有创建任何 String[] 的副本（您只是复制对它们的引用，这非常便宜）。

然而，您需要遍历HashMap 中的所有值，这使得这个 O(n)（或者简单地说：慢）。

如果这是一个比较少见的手术，那么这可能是可以接受的，我不会担心的。

如果插入是一种常见的操作，那么您绝对应该为此考虑一个更好的数据结构（您需要告诉我们更多关于实际用例的信息，以便我们在这里提出好的建议）。

Answer 2

对于您目前拥有的数据结构，没有比循环遍历所有这些值更好的方法了（至少您在第一次命中时很早就突破了）。

如果性能确实成为问题，您可能希望保留第二个数据结构来索引已经存在的内容。如果您只需要知道给定的字符串是否在地图中（而不是在哪里）（深），那么 HashSet<String> 可能会起作用。

这是一个内存权衡：添加的索引结构会占用额外的空间。 （顺便说一句，除了地图已经占用的空间之外，您现在所做的事情并没有使用任何空间，您只是通过引用迭代数组，没有制作额外的副本）。

Answer 3

嗯，除了循环遍历这些值之外，我不知道有什么不同的方法，但我建议使用更简单的代码：

Map<String, String[]> map = new HashMap<String, String[]>();
for (String[] strings : map.values()) {
    ArrayUtils.contains(strings, "foo");
}

ArrayUtils 是一个来自 apache commons 的帮助类，当然你也可以循环遍历字符串数组并每次调用 equals。请注意，这是区分大小写的。

Map<String, String[]> map = new HashMap<String, String[]>();
for (String[] strings : map.values()) {
    for (String string : strings) {
        if (string.equals("foo")) {
            return true;
        }
    }
}

Answer 4

您可以使用 containsValue() 方法来执行此操作。您不必迭代抛出整个 HashMap。

http://msdn.microsoft.com/en-us/library/aa989118%28v=vs.80%29.aspx

http://www.javadocexamples.com/java/util/HashMap/containsValue%28Object%20value%29.html

Answer 5

您可以通过将syn 数组复制到HashSet 中来降低时间复杂度。然后，您可以使用 O(1) 的 HashSet#contains 方法，而不是一遍又一遍地迭代 syn：

public static boolean kijkContains(String[] syn){
  if (syn == null || syn.length == 0) return false; // that was missing
  Set<String> input = new HashSet<String>();
  for (String s:syn)
    input.add(s);         // will remove duplicates, another perfo improvement

  for(String key:myMap.keySet()){
    for(String s:myMap.get(key)){  // we don't need the loop variable
      if (input.contains(s)) {
        return true;
      }
    }
  }
  return false;
}

复杂度是O(i*j*k)，我已将其减少到O(i*j+k)（i 是地图的大小，j 是值的平均大小数组和 k 同步数组的大小）

Answer 6

所以你想要的是 HashMap 的值中的每个字符串都必须是唯一的。 我认为如果您使用 HashSet 而不是 String[] 作为 Map 的值，它将起作用。

导入 java.util.*;

公共类 TestHashMap { 公共静态 void main(String args[]) {

System.out.println("Hashmap Test");
HashMap<String, HashSet> myMap = new HashMap<String, HashSet>();

HashSet<String> s1 = new HashSet<String>();
s1.add("one");
s1.add("two");
s1.add("three");

myMap.put("1", s1);
System.out.println("Value for key 1" + myMap.get("1"));

} }

我希望这是你想要的方式。