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;

}

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

P.S. 我有权删除/修改文件

java.util.scanner bufferedreader

使用 try-with-resources。您没有安全地关闭资源（您说您是，但您的代码片段表明情况并非如此）。当你写时，这正是你需要 [A] 保存返回的内容的确切点（你不需要，你把它传递给 BufferedReader 的构造函数），然后 [B] 在 finally 块中关闭该东西。或者，更好的是，只需使用资源即可尝试。我不打算向你展示如何做，我稍后会把多个点组合在一个片段中。这是您问题的最可能原因。new FileReader()
使用旧 API。File.delete API 只返回 true 或 false，不会告诉你任何信息。因此，请使用包中的新 API。与旧的 API 不同，它根本不返回布尔值（它会抛出一个异常，就像正确的 API 设计所规定的那样：你请求删除，如果这不是一个选项，则应该抛出一个异常），并且因为它这样做了，这个异常包含了关于为什么它不能删除的各种细节。这将回答您的问题。例如，您可能会收到错误：“IOException： Read-only filesystem”。现在你知道了。java.nio.file
令人遗憾的异常处理。无论它在示例中出现多少次，都是不好的，在代码中永远是不可接受的。正确的“我只是不想处理这种方法”总是在你的方法中添加一个子句（记住：可以而且应该写到！），如果这不是一个选项，那么“走开，编译器”对 catch 块的看法不是，而是：。这为您提供了比 e.printStackTrace 更多的调试信息，并方便地阻止了 e.printStackTrace 导致的“错误雪球”（只需 e.printStackTrace 将打印错误，然后继续运行代码。该代码极有可能也会失败，因为变量未处于预期状态。如果您编写的所有代码都只是捕获异常、打印并继续，那么单个错误将导致一连串的、完全不相关的错误，导致 1 个问题打印出 183 个单独的堆栈跟踪，并且除了第一个之外，所有跟踪都是完整的红鲱鱼。这一切都可以通过适当的错误处理来防止，也可以通过更合理的默认值来防止（这些是：add ，如果不能，则 .catch (Something e) {e.printStackTrace();}throwspsv mainthrows Exceptione.printStackTracethrow new RuntimeException("uncaught", e);throwsthrow 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
}

Java file.delete（）仅返回 false

Java file.delete() returns false only

评论

评论

评论

Java file.delete（） 仅返回 false

Java file.delete() returns false only

评论

评论

评论

Java file.delete（）仅返回 false