Java delete（）只返回false

我想删除“db.txt”（programPath[1]）。我检查了代码，关闭了所有的扫描器和缓冲阅读器。当我尝试删除时，它返回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;

}

有人能解释我做错了什么吗？或者我怎样才能看到哪里开门

---

另外，我有权删除/修改文件

很多很多事情。大多数情况下，您编写java代码的时候好像还是1992年

使用try with资源。您没有安全地关闭资源（您说是，但代码段表明情况并非如此）。当您编写

new FileReader（）

时，这正是您需要[A]保存返回的内容（您不需要，将其传递给BufferedReader的构造函数），并[B]在finally块中关闭该内容。或者，更好的方法是使用try with资源。我还不打算向您展示如何在一个片段中组合多个点这是问题最可能的原因

使用旧的API。File.delete API只返回true或false，不告诉您任何信息。因此，请使用

java.nio.file

包中的新API。与旧的API不同，它根本不返回布尔值（它会引发异常，就像正确的API设计所规定的：您请求删除，如果这不是一个选项，则应引发异常），因为它会这样做，所以此异常包含有关无法删除的各种详细信息。这将回答你的问题。例如，可能会出现错误：“IOException:只读文件系统”。现在你知道了

糟糕的异常处理。无论它在示例中出现的频率有多高，

catch（Something e）{e.printStackTrace（）；}

都是不好的，在代码中永远都是不可接受的。正确的“我只是不想处理这种方法”总是在方法中添加一个

throws

子句（请记住：

psv main

可以并且应该写入

throws Exception

！），如果这不是一个选项，那么在catch块上使用的“just gow away，compiler”就不是

e.printStackTrace

，它是：

抛出新的RuntimeException（“uncaught”，e）。这为您提供了比e.printStackTrace更多的调试信息，并方便地阻止了e.printStackTrace导致的“雪球般的错误”（仅e.printStackTrace将打印错误，然后继续运行代码。该代码也极有可能失败，因为变量未处于预期状态。如果您编写的所有代码仅捕获异常、打印并继续，则单个错误将导致进一步的、完全不相关的错误级联，从而导致1个问题打印出183个独立的堆栈跟踪，除第一个之外，所有这些跟踪都是完全的误导。正确的错误处理可以防止这些情况发生，而更合理的默认设置也可以防止这些情况发生（这些默认设置是：添加
抛出
，如果不能，则
抛出新的RuntimeException（“Uncaught”，e）；

请注意，在文件中以原始文本形式存储安全凭据不是适当的安全措施。如果一家公司今天要存储这样的数据，并且由于数据泄露，而您在欧洲开展业务，将面临巨额罚款。GDPR需要行业标准协议。您应该使用ha对安全凭据进行哈希处理为此专门设计的sh算法（如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-with-resources，则不会有问题

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

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

---

您应该对要关闭的所有对象执行此操作。它们是可自动关闭的。首先，检查您是否有删除文件的权限。我有。另一个类正在删除文件，我没有问题。我编辑了问题。您在哪一行编写了文件。delete（）抱歉，我似乎找不到它。第一个方法，在最后一行。有一个if语句；该语句检查文件是否被删除。你能举一个简单的例子吗？我相信StackOverflow要求其用户对新手友好。@maksimov我相信这是一个很好的答案，涵盖了许多需要涵盖的主题。这其中哪一部分非常有用“你觉得什么不好？@Andreas虽然答案本身可能很有启发性，但可能不那么苛刻：“你编写java代码的时候好像还是1992年”，“糟糕的异常处理”。人们来这里寻求帮助，而不是判断。如果我理解正确：当我使用try（）时函数对象是可自动关闭的，对吗？@n1c0t1n3

autoclosable

是一个接口。您找到的任何派生自它的类都是广告：是的，我需要关闭。如果您觉得某个对象是一个资源，只需将其粘贴到try-with-resources块中。如果它不是，编译器会告诉您它不是。