从文件读取和删除某些行时遇到问题？

Question

该程序的目的是从文本文件中删除某些运动队及其成员，然后用新的一组值覆盖原始文件。通过将值读入一个数组，然后遍历该数组并删除组名和接下来的两行来尝试此操作，但是由于某种原因，它停止在索引之后遍历该数组。

我被困住了，所以任何帮助都会很棒

码：

private void RemoveSportsTeamButtonActionPerformed(java.awt.event.ActionEvent evt) {

    String ChosenTeam = "";
    ChosenTeam = JOptionPane.showInputDialog("What Team Do you want to remove?");

    ArrayList<String> Teamlist = new ArrayList<String>();

    if (ChosenTeam.length() > 0) {
    } else {
        Scanner Reader = null;
        try {
            Reader = new Scanner(new File("ListofSportTeams.txt"));
        } catch (FileNotFoundException ex) {

        }

        while (Reader.hasNext()) {
            Teamlist.add(Reader.next());
        }
        Reader.close();

        for (int count = 0; count < Teamlist.size(); count++) {
            {
                if (Teamlist.get(count).equals(ChosenTeam)) {
                    Teamlist.remove(count);
                    Teamlist.remove(count + 1);
                    Teamlist.remove(count + 2);
                }
            }

        }
    }

}

SportTeamList.txt =

Team1
Jeff
James
Team2
Steve
Peter

Answer 1

你应该 从来不 remove从List时迭代。你在做什么

考虑一个简单的例子，我有一个list {1,2,3,4,5}。让我们假设它的索引为0，我想删除所有大于3的数字。

0-列表项为1，保留
1-列表项为2，保留
2-列表项为3，删除。所有元素都移位了，列表现在是 {1,2,4,5}。
3-列表项为5，删除
4-列表项为 oops，不再有4

所以，我有超调的结束List，因为我带的大小是5当我开始迭代却成了4之后我删除索引的元素2，并成为3当我在索引中删除的元素3。

您可能会说：“啊哈，我可以通过while循环解决此问题”：

int i = 1;
while(i < teams.size()) {
    //do stuff
}

但这 更糟 ：

0-列表项为1，保留
1-列表项为2，保留
2-列表项为3，删除。所有元素都移位了，列表现在是 {1,2,4,5}。
3-清单项目为5，移除

因此，没有错误。看来问题已解决。但是列表包含什么？它包含{1,2,4}。但是 4
大于3。由于索引移位，它被跳过了。您现在有了一个更加隐蔽的错误。

如果您正在使用适当的增强型foreach循环，例如：

for(final String team : teams) {
    //do stuff with team
}

您将正确地获得一个ConcurrentModificationException。这只是使用增强的foreach循环而不是按索引循环的众多原因之一。

为了做你想要的使用Iterator：

final Iterator<String> iter = teams.iterator();
while(iter.hasNext()) {
    if(iter.next().equals(testString))
        iter.remove();
}

我将重申我的评论：

请 始终
使用Java命名约定。变量应始终位于中camelCase。PascalCase为课程保留。

更新

可能会更容易，因此请使用indexOf方法在中找到团队名称List并删除所需的元素

public void remove(final List<String> teams, final String name) {
    final int idx = teams.indexOf(name);
    if(idx < 0) 
        throw new IllegalArgumentException("Team " + name + " not present in list.");
    for(int i = idx + 2; i >= idx; --i) 
        teams.remove(i);
}

反向 删除这些项目非常重要
。这是由于与上述相同的问题，例如，如果删除索引处10的项目，则位于索引处的项目11将向下移动。因此，当您去删除索引处的项目时11，实际上是在删除该项目，它最初是索引12。

你可以用肮脏的把戏

for(int i = 0; i < 2; ++i)
    teams.remove(idx)

即继续在找到的索引处删除该项目，因为列表将向下随机排列以填补空白，您实际上将删除所需的项目以及位于其上方的两个项目。我认为这使代码很难阅读。您可能会忘记此技巧，然后返回代码，必须弄清楚它在做什么。