Java file.delete() 仅返回 false

如何解决Java file.delete() 仅返回 false

我想删除“db.txt”（programPath[1]）。我检查了代码并关闭了所有 Scanners 和 BufferedReaders。当我尝试删除时，它返回 false。我试图用 VisualVM 调试我的代码，但我找不到任何错误。我在 2 种方法中使用了该文件。为了获得更好的可见性，我在打开和关闭文件 (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;

}

谁能解释一下我做错了什么？或者我怎样才能看到在哪里打开？

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

解决方法

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

使用 try-with-resources。你没有安全地关闭资源（你说你是，但你的代码片段表明情况并非如此）。当您编写 new FileReader() 时，这就是您需要 [A] 保存返回内容的确切点（您不需要，您将其传递给 BufferedReader 的构造函数），然后 [B] 关闭那个东西 在finally块中。或者，更好的是，只需将 try 与资源一起使用。我不打算向您展示如何，稍后我将在一个片段中组合多个点。 这是您问题的最可能原因。
使用旧 API。 File.delete API 只返回真或假，什么也不告诉你。因此，请使用 java.nio.file 包中的新 API。与旧 API 不同，它根本不返回布尔值（它会抛出异常，就像正确的 API 设计所指示的那样：您要求删除，如果这不是一个选项，则应抛出异常），并且因为它确实也就是说，此异常包含有关为什么无法删除的各种详细信息。那会回答你的问题。例如，您可能会收到错误消息：“IOException：只读文件系统”。现在你知道了。
糟糕的异常处理。无论它在示例中出现的频率如何，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);。
请注意，将安全凭证以原始文本形式存储在文件中并不是一种适当的安全措施。如果一家公司今天像这样存储这样的数据，并且由于某些泄漏而导致数据泄露，而您在欧洲开展业务，则将面临巨额罚款。 GDPR 需要行业标准协议。您应该获取安全凭证并使用专门为此设计的散列算法（例如 bcrypt、scrypt 或 pbkdf）对它们进行散列，并存储这些散列。或者更高级的技术，例如基于 PKI 的身份验证。
您使用的 FileReader 永远不会好，因为它使用“平台默认编码”，而您永远不希望那样。幸运的是，新的文件 API 默认为 UTF-8，这很好。
File.separator 仅用于向用户打印内容，在 java 路径中不需要它；始终使用正斜杠，java 知道它的含义，即使在 Windows 上也能正常工作。更好的是，只需使用例如构建它路径 API。

做得更好的例子：

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

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

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

Java file.delete() 仅返回 false

如何解决Java file.delete() 仅返回 false

解决方法

相关推荐