迭代器似乎正在丢失数据 java

Question

我有一个返回哈希图列表的方法（数据来自 ResultSet）。 在尝试使用迭代器循环数据并将其转换为字符串数组时，我发现代码不起作用 - 在循环时似乎丢失了一半的数组数据。

当交换它以使用 for 循环而不是迭代器时，数据不会丢失。我尝试同时使用 Iterator() 和 ListIterator() 均无济于事

这是我的原始代码（不起作用）：

public String[][] getLayoutEdges() throws SQLException {
    ArrayList<String[]> returnArray = new ArrayList<>();

    List<HashMap> layoutEdges = db.getLayoutEdgesFromDatabase();
    ListIterator<HashMap> edgesIterator = layoutEdges.listIterator();

    while(edgesIterator.hasNext()) {
        ArrayList<String> tmpList = new ArrayList<>();
        tmpList.add(edgesIterator.next().get("fromnode").toString());
        tmpList.add(edgesIterator.next().get("tonode").toString());
        tmpList.add(edgesIterator.next().get("distance").toString());

        String[] tmpStr = new String[tmpList.size()];
        returnArray.add(tmpList.toArray(tmpStr));

    } 

    String[][] rtn = new String[returnArray.size()][returnArray.size()];
    return returnArray.toArray(rtn);
}

下面是有效的代码：

public String[][] getLayoutEdges() throws SQLException {
    ArrayList<String[]> returnArray = new ArrayList<>();

    List<HashMap> layoutEdges = db.getLayoutEdgesFromDatabase();

    for(HashMap tmp : layoutEdges) {
        ArrayList<String> tmpList = new ArrayList<>();
        tmpList.add(tmp.get("fromnode").toString());
        tmpList.add(tmp.get("tonode").toString());
        tmpList.add(tmp.get("distance").toString());

        String[] tmpStr = new String[tmpList.size()];
        returnArray.add(tmpList.toArray(tmpStr));
    }

    String[][] rtn = new String[returnArray.size()][returnArray.size()];
    return returnArray.toArray(rtn);
}

谁能告诉我为什么迭代器不工作？我可以使用 for 循环，但我一生都无法弄清楚为什么我编写的原始迭代器代码没有保留我的所有数据。想知道我是否遗漏了一个步骤，或者我是否尝试使用错误的解决方案。

Answer 1

因为这几行：

tmpList.add(edgesIterator.next().get("fromnode").toString());
tmpList.add(edgesIterator.next().get("tonode").toString());
tmpList.add(edgesIterator.next().get("distance").toString());

在调用 edgesIterator.next() 的那一刻，它会将光标移动到下一个元素。

Answer 2

在您的第一个 sn-p 中，您为循环的每次迭代调用了三次 edgesIterator.next()，我很确定您不想这样做。只是不要这样做：

while (edgesIterator.hasNext()) {
    HashMap tmp = edgesIterator.next();
    tmpList.add(tmp.get("fromnode").toString());
    tmpList.add(tmp.get("tonode").toString());
    tmpList.add(tmp.get("distance").toString());
    String[] tmpStr = new String[tmpList.size()];
    returnArray.add(tmpList.toArray(tmpStr));
}

顺便说一句，如果您可以避免在 API 中使用原始类型，它通常会更简洁 - 我在这里谈论的是 HashMap 的使用。

无论如何，我通常也会支持您的代码的第二个版本 - 除非您出于某种原因需要显式使用迭代器，否则让增强的 for 循环的语法糖自动为您完成。