是否可以在循环中断的地方将迭代器嵌套在彼此内部

Question

您认为这样的迭代器嵌套是好还是坏？

上下文：for循环版本中断，因为我在for循环中使用同义词的同时附加同义词。

// HashSet inside HashMap
HashMap<String, HashSet<String>> synonyms = new HashMap<String, HashSet<String>>();

// loops through synonyms 
Iterator word = synonyms.keySet().iterator();

while(word.hasNext()) {
    // loops through synonyms 
    Iterator line = synonyms.get(word.next().toString()).iterator();

    while(line.hasNext()) {
        // adds to Synonyms, this breaks for loop version
        addToSynonyms(word, line.next())
    }
}

或者，在for循环版本中，我可以复制HashMap

非常感谢。

下面是一个更好的例子。

import java.util.Iterator;
import java.util.HashSet;
import java.util.HashMap;


// examples of looping through HashMap<String, HashSet<String>> numbersAndWords;
// 
public class Example {
    
    HashMap<String, HashSet<String>> numbersAndWords;

    public Example() {
        numbersAndWords = new HashMap<String, HashSet<String>>();
        
        // data
        numbersAndWords.put("five", new HashSet<>());
        numbersAndWords.get("five").add(("1 2 3 4 5"));
    }

    
    /** 
     * uses two iterators to loop through numbersAndWords and adds to numbersAndWords without breaking loops
     */
    public void exampleOne() {

        // loops through HashMap Strings of words "five"
        Iterator words = numbersAndWords.keySet().iterator();
        
        while(words.hasNext()) {

            // loops through HashSet Strings of numbers "1 2 3 4 5"
            Iterator numbers = numbersAndWords.get(words.next().toString()).iterator();
            while(numbers.hasNext()) {

                // data is arbitrary reason for exmaple is this appends
                // numbersAndWords which breaks the for loop 
                // Their for I have to use an iterator but should I be using to
                // or have things in a different format
                numbersAndWords.put("three", new HashSet<>());
                numbersAndWords.get("three").add(("1, 2, 3"));
                
                numbers.next();
            }
        }
    }
    
    public static void main(String[] args) {
            Example example = new Example();
            example.exampleOne();
        }

}

目前正在考虑使用HashMap.size（）用for循环替换第一个itr，这样当我稍后追加HashMap时就不会中断。

非常感谢。

Answer 1

即使在最近的更新之后，这段代码的目的仍然有点模糊。

如果需要创建一些新数据并将其附加到原始地图numbersAndWords，并且它们在某种程度上取决于原始地图的当前状态，则必须创建并填充一个新的临时地图（使用增强的for循环，使用带有map/flatMap的流API，无论什么）。完成后，可以使用map:：putAll：

public void exampleOne() {
    Map<String, HashSet<String>> toAppend = new HashMap<>();

    for (Map.Entry<String, HashSet<String>> me : numbersAndWords.entrySet()) {
        for (String str : me.getValue()) {
            System.out.println("Appending for key: " + me.getKey() + "; value=" + str);
            toAppend.computeIfAbsent("three", k -> new HashSet<>())
                    .add("1 2 3"); // or whatever is really needed
        }
    }
    numbersAndWords.putAll(toAppend);

    System.out.println(numbersAndWords);
}

假设原始地图是这样设置的：

public Example() {
    numbersAndWords = new HashMap<String, HashSet<String>>();
        
    // data
    numbersAndWords.computeIfAbsent("five", k -> new HashSet<>()).add("1 2 3 4 5");
    numbersAndWords.computeIfAbsent("five", k -> new HashSet<>()).add("5 4 3 2 1");
    numbersAndWords.computeIfAbsent("four", k -> new HashSet<>()).add("1 2 3 4");
}

调用new Example（）时，输出如下。例一（） ：

Appending for key: four; value=1 2 3 4
Appending for key: five; value=1 2 3 4 5
Appending for key: five; value=5 4 3 2 1
{four=[1 2 3 4], five=[1 2 3 4 5, 5 4 3 2 1], three=[1 2 3]}

然而，这种硬编码值的填充是非常没有意义的，因为不能在映射中为硬编码键创建重复条目，并且将相同的值添加到集合中是无用的。