Java file.delete() 仅返回 false

Question

我想删除“db.txt”（programPath[1]）。我检查了代码并关闭了所有 Scanners 和 BufferedReaders。当我尝试删除时，它返回 false。我尝试使用 VisualVM 调试我的代码，但我找不到任何错误。我以两种方法使用了该文件。为了获得更好的可见性，我在打开和关闭文件 (TODO) 时标记了该位置。这是 第一种方法：

public static void changePassword(boolean isLogged, String[] programPath, File db) {
    if (!isLogged) {
        login(isLogged, programPath, db);
    }

    File tempFile = new File(programPath[2]);
    File tempDbFile = new File(programPath[0] + File.separator + "tempDb.txt");


    try {
        BufferedReader tempFileScanner = new BufferedReader(new FileReader(tempFile));
        BufferedReader dbScanner = new BufferedReader((new FileReader(db))); // TODO db open
        //Scanner dbScanner = new Scanner(db);
        FileWriter tempDbWriter = new FileWriter(tempDbFile);

        if (!tempDbFile.exists()){
            try {
                tempDbFile.createNewFile();
            } catch (IOException e) {
                e.printStackTrace();
            }
            System.out.println("created");

        }


        String[] lineToRemove = tempFileScanner.readLine().split("\\:");
        tempFileScanner.close();

        String line;
        while((line = dbScanner.readLine()) != null) {
            System.out.println("Line: " + line);
            System.out.println("Line to remove: " + lineToRemove[0] + ":" + lineToRemove[1]);

            if(line.equals(lineToRemove[0] + ":" + lineToRemove[1])){
                tempDbWriter.write(lineToRemove[0] + ":" + newPass + System.getProperty("line.separator"));
                continue;
            }
            tempDbWriter.write(line + System.getProperty("line.separator"));
            System.out.println("Written: " + line);
        }
        tempDbWriter.close();
        dbScanner.close(); // TODO db closed

    } catch (Exception e) {
        e.printStackTrace();
    }

    if(db.delete()){
        System.out.println("File deleted");
    } else if(!db.delete()){
        System.out.println("File not deleted. W H Y ?");
    }


}

第二种方法：

public static boolean checkCredentials(String[] programPath, File db){

    Scanner scan = new Scanner(System.in); // for getting user input

    System.out.println("Please insert your card number: ");
    String cardNr = scan.nextLine();
    System.out.println("Please insert your PIN: ");
    String pin = scan.nextLine();

    try {
        Scanner dbScanner = new Scanner(db); // TODO db open
        //now read the file line by line...

        while (dbScanner.hasNextLine()) {
            String[] line = dbScanner.nextLine().split("\\:");
            System.out.println("CardNr: " + cardNr);
            System.out.println("Pin: " + pin);

            if(line[0].equals(cardNr) && line[1].equals(pin)) {
                String[] loggedInAs = {line[0], line[1]};
                tempFile.createTempFile(programPath, loggedInAs);
                return true;
            }
        }
        dbScanner.close(); // TODO db close
    } catch(FileNotFoundException e) {
        //handle this
    }

    return false;

}

谁能解释我做错了什么？或者如何查看打开的位置？

附：我有权删除/修改文件

Answer 1

另一个答案更全面。您的特定问题的一个可能罪魁祸首是：

if(line[0].equals(cardNr) && line[1].equals(pin)) {
    String[] loggedInAs = {line[0], line[1]};
    tempFile.createTempFile(programPath, loggedInAs);
    return true;

这意味着您无需关闭文件即可返回。如果你对资源使用 try，那不会有问题。

try(Scanner dbScanner = new Scanner(db)) {
     // TODO db open
    //now read the file line by line...

现在 dbScanner 将关闭，即使您通过 return 退出该方法，或异常退出。

您应该对要关闭的所有对象执行此操作。他们是autoclosable

Answer 2

很多很多东西。大多数情况下，您正在编写 Java 代码，就好像它仍然是 1992 年一样。

1. 使用 try-with-resources。您没有安全地关闭资源（您说是，但您的代码 sn-p 表明情况并非如此）。当您编写new FileReader() 时，正是您需要 [A] 保存返回的内容（您不需要，将其传递给 BufferedReader 的构造函数），然后 [B] 关闭那个东西 in finally 块。或者，更好的是，只需将 try 与资源一起使用。我不打算向你展示如何，稍后我将在一个 sn-p 中组合多个点。 这是您问题的最可能原因。

2. 使用旧 API。 File.delete API 只返回 true 或 false，什么也不告诉你。因此，请使用 java.nio.file 包中的新 API。与旧的 API 不同，它根本不返回布尔值（它会抛出异常，就像正确的 API 设计要求的那样：您要求删除，如果这不是一个选项，则应该抛出异常），并且因为它确实也就是说，此异常包含有关为什么无法删除的各种详细信息。这将回答你的问题。例如，您可能会收到错误消息：“IOException: Read-only filesystem”。现在你知道了。

3. 可悲的异常处理。无论它在示例中出现的频率如何，catch (Something e) {e.printStackTrace();} 都不好，并且在代码中绝不可以接受。正确的“我只是不想处理这种方法”总是在你的方法中添加一个throws 子句（记住：psv main 可以而且应该写到throws Exception！），如果那不是选项，“走开，编译器”对 catch 块的处理不是e.printStackTrace，而是：throw new RuntimeException("uncaught", e);。这为您提供了比 e.printStackTrace 更多的调试信息，并方便地停止 e.printStackTrace 导致的“错误雪球”（只是 e.printStackTrace 将打印错误然后继续运行代码。该代码极有可能也会失败, 因为变量未处于预期状态。如果您编写的所有代码都只是捕获异常、打印并继续，那么单个错误将导致一连串进一步的、完全不相关的错误，导致 1 个问题打印出 183 个单独的堆栈跟踪，并且所有这些除了第一个都是完整的红鲱鱼。这一切都可以通过适当的错误处理来防止，并且也可以通过更合理的默认值来防止（那些是：添加throws，如果你不能，throw new RuntimeException("Uncaught", e);。

4. 请注意，将安全凭证以原始文本形式存储在文件中并不是适当的安全措施。如果一家公司今天会存储这样的数据，并且由于一些泄漏，数据会泄露，而你在欧洲开展业务，那么将面临巨额罚款。 GDPR 需要行业标准协议。您应该获取安全凭证并使用专门为此设计的散列算法（例如 bcrypt、scrypt 或 pbkdf）对它们进行散列，并存储这些散列。甚至更高级的技术，例如基于 PKI 的身份验证。

5. 您使用的 FileReader 永远不会好，因为它使用“平台默认编码”，而您永远不会想要这样。幸运的是，新的文件 API 默认为 UTF-8，这很棒。

6. File.separator 仅用于向用户打印内容，在 java 路径中不需要它；始终使用正斜杠，java 知道它的含义，甚至在 Windows 上也可以使用。更好的是，只需使用例如构建它路径 API。

做得更好的例子：

Path p = Paths.get(programPath[0], "tempDb.txt"));
try (BufferedReader br = Files.newBufferedReader(p)) {
   // read your stuff
}