Question

实际问题将进一步解决:)，谢谢。

我是Java的新手（几乎通过400页的书）。

我还不熟悉API。

这是我阅读.txt文件并检查是否存在已存储在.txt文件中的任何已收集数据的最佳方法。如果是这种情况，数据将从数据集合中删除，并且将添加.txt中尚未找到的数据。

一些变数：

public String[] names;
public int[] levels;
public int[] IDs;

public ArrayList<String> line = new ArrayList<String>();
public ArrayList<RSNPC> monsterList = new ArrayList<RSNPC>();
public ArrayList<String> monstersToAdd = new ArrayList<String>();

检查现有.txt文件的方法：

    private void checkForLine() {
     try{
         // Create file 
        File file = new File(getCacheDirectory() + "output.txt");
        RandomAccessFile out = new RandomAccessFile(file, "rw");
        for(int i = 0; i < file.length(); i++){
            line.add(out.readLine());
        }
        for(String monster : monstersToAdd){    
            if(line.contains(monster)){
                monstersToAdd.remove(monster);
            }
        }
        //Close the output stream
        out.close();
     }catch (Exception e){//Catch exception if any
         System.err.println("Error: " + e.getMessage());
         }
     }

然后最终保存checkForLine（）定义的信息（已经不存在于文件中的信息）的方法：

private void saveToFile() {
     try{
         // Create file 
        BufferedWriter out = new BufferedWriter(new FileWriter(getCacheDirectory() + "output.txt"));
        for(String a : monstersToAdd){
            out.write(a);
            out.newLine();
            log("Wrote " + a + "to file");
        }
         //Close the output stream
         out.close();
         }catch (Exception e){//Catch exception if any
         System.err.println("Error: " + e.getMessage());
         }
     }

执行顺序：

        getNPCS();
    getNames(monsterList);
    getLevels(monsterList);
    getIDs(monsterList);
    combineInfo();
    checkForLine();
    saveToFile();

问题但是，它没有正确检查.txt文件以获取信息。我可以看到，因为它只是一次又一次地保存它所观察到的任何内容，而不是将任何东西排序。这是我用我有限的知识思考的唯一方法，但它没有用。

对于那些想知道的人：这是一个名为RSbot的机器人脚本，可以播放名为RuneScape的游戏。我实际上并没有使用机器人，但是我想在练习中这样做。

我可以粘贴整个脚本，如果这样可以进一步清理。

我真的很感激任何帮助，当然记得选择我使用过的答案（如果有人帮忙的话）;）。

感谢。

Answer 1

for(String monster : monstersToAdd){    
    if(line.contains(monster)){
        monstersToAdd.remove(monster);
    }
}

如果ConcurrentModificationException为line.contains(monster)，则{p>会抛出true，而monstersToAdd包含monster。 The only safe way to remove an element from a collection while iterating is to use Iterator：

for(Iterator<String> iter = monstersToAdd.iterator(); iter.hasNext();){
    String monster = iter.next();
    if (line.contains(monster)) iter.remove();
}

修改

@trutheality指出

实际上，完成同样事情的一种更简单的方法是monstersToAdd.removeAll(line);

所以你可以用一行代码替换for循环。

Answer 2

一个可能的问题是，当您“保存”时，您似乎正在覆盖同一个文件。我建议你在一个文件中读取并写入另一个文件进行测试运行。

为了附加到文件，您有几个选项：

让你的两个函数共享'RandomAccessFile`，这样在第一个完成读取文件之后，光标位于文件的末尾，第二个函数可以从那里继续写入，附加到文件< / LI>
在第二个函数中打开RandomAccessFile并将光标移动到文件的末尾（例如，通过阅读其中的所有内容，直到没有其他行）。

Java - 将ArrayList中的字符串与.txt文件中的所有文本进行比较

2 个答案:

修改