这是在 Java 中传递输入文件的坏习惯吗？

Question

我有一个主类，它希望通过命令行参数提供一个输入文件名； 如果这不是真的，则程序退出并显示错误消息。

我们假设存在一个名为 SudokuReducer 的类。 确定有输入文件后，主函数会将输入文件（不仅仅是文件名）传递给 SudokuReducer 的一个实例。

我想知道的是，这是不好的形式/做法吗？ 像这样将整个扫描放在 try/catch 中是错误的吗？ 因为如果我想在 try/catch 之外的“main”中声明 SudokuReducer 实例，而不是在其中，我不能，因为它无法识别传递了什么“fileInput”，因为它在“try”中的范围有限'

有没有更好的方法来做到这一点？ 这是我所拥有的：

import java.io.File;

public class MainMV {

File inputFile;

public static void main(String args[]) {

    // check if file is entered and handle exception
    try {
        if (args.length > 0) {
            File inputFile = new File(args[0]);

            System.out.println("Processing file");
            SudokuReducer reducer = new SudokuReducer(inputFile);

        } else {
            System.out.println("No file entered.");
            System.exit(1);
        }

    } catch (Exception e) {
        System.out.println("File failed to open.");
        System.exit(1);
    }

}

}

Answer 1

回答标题中的问题：不，如果该方法需要File来完成其工作，这不是坏习惯。

另一种选择是传递一个String ； 这是一个糟糕的选择，因为它没有传达参数应该代表某种File 。

也许更好的选择是将InputStream传递给该方法，因为 a) 清楚地表明它将用于输入（而不是您将写入的File ）； b) 它更灵活，因为它不必引用磁盘上的文件。

---

要回答问题中的问题：不，像这样将所有内容都包含在一次 try/catch 中并不是一个很好的做法。 很难区分失败的模式：您的代码中可能会出现许多不同的问题，最好分别处理这些问题，例如提供特定的失败消息。

构建代码的更好方法是这样的：

if (args.length == 0) {
  System.out.println("No file entered.");
  System.exit(1);
}

File inputFile = new File(args[0]);
System.out.println("Processing file");
try {
  SudokuReducer reducer = new SudokuReducer(inputFile);
  // Do something with reducer.
} catch (IOException e) {
  e.printStackTrace();
  System.out.println("File failed to open.");
  System.exit(1);
}

请注意，它具有处理特定错误的小块，而不是将错误处理与导致错误的事物分开的大块。

另外，请注意，它没有捕获Exception ：除非必须这样做，否则您真的不想这样做，因为您没有正确处理它会捕获的必须以特殊方式处理的异常（即InterruptedException ）。 捕获最具体的异常类型。

Answer 2

根据我的理解，Java 将参数作为对象的引用按值传递，这对我来说非常令人困惑。

链接到按引用传递与按值传递的解释。

链接到 Java 实现的解释

根据生成 SudokuReducer 类实例所需的文件信息量，此操作的开销可能很大。 如果是这种情况，您将需要在 main 方法中逐行解析您的输入。

 try {
     SudokuReducer reducer = SudokuReducer.makeSudokuReducer(args[0])
  }
 catch(Exception e) { 
     System.out.println("No file entered.");
     System.exit(1);
 }

这是逐行读取文件的示例有很多方法可以做到这一点，但我能想到的最有效的方法是使用 Java 8 的 Stream 和 Files 类。

方法签名将如下所示：

public static SudokuReducer makeSudokuReducer(String filename) {
    //Open file
    //Parse input line by line
    //Use information to create a new instance of your class
    //Return the instance of this class
}

您将能够在任何地方调用此静态方法以根据文件名生成类的新实例。