Java：异常和“可能尚未初始化”

Question

我对 Java 很陌生。我正在编写一个程序来读取文件，计算其 SHA1 校验和，并将结果写入另一个文件。在出现任何错误时，我都会调用一个函数 err_exit()，它会向 stderr 打印一条消息，并通过调用具有指定退出状态的 System.exit() 来终止执行。这大概是我的main() 函数的样子：

public static void main(String[] args) {
    String in_fname = "C:/tmp/test.txt"; // not reading args yet
    String out_fname = "C:/tmp/test.sign";
    byte[] file_data;
    String hexdigest;

    try {
        file_data = readFileAsByteArray(in_fname);
    }
    catch (java.io.IOException ex) {
        file_data = new byte[] {0};  // note this line well, please
        err_exit(2, "error opening input file '" + in_fname + "'");
    }

    try {
        hexdigest = hexdigestSha1(file_data);
    }
    catch (NoSuchAlgorithmException ex) {
        hexdigest = "";  // note this line well, please
        err_exit(3, "could not compute SHA1 message digest!");
    }

    try {
        writeFileFromString(out_fname, hexdigest);
    }
    catch (java.io.IOException ex) {
        err_exit(2, "error writing output file '" + out_fname + "'");
    }

    System.exit(0); // success
}

有两行我要求你注意。这两行的存在只是为了防止编译器抱怨变量可能没有被初始化。

就编译器所知，catch 块可能会继续。实际上err_exit() 永远不会返回，因此不会传递无效值。

那么，我的问题是：处理这类事情的常用 Java 习语是什么？ try/catch 块的行有点难看；你会建议我让各种函数调用err_exit() 并且没有像这样显式的代码吗？我想我更喜欢显式检查，main() 函数是进行检查的正确位置，但我对反馈感兴趣。

如果我要使用 try/catch 块，这是消除编译器警告的好方法吗？

如果我在 Python 中执行此操作，我可能不会捕获异常，并让程序停止并返回堆栈跟踪。错误的堆栈跟踪不会让这个程序的用户感到震惊，因为那个用户就是我。我突然想到，如果我将main() 函数声明为throws Exception，那么我就无法捕获异常，它的行为类似于 Python。这是一个可怕的想法，会让思想正确的 Java 人回避我吗？

附：如果您有最喜欢的书/网页/我应该阅读的任何 Java 习语，请提及。

编辑：对于带有下划线的变量名，我深表歉意。我已经在我的真实程序中重命名了它们，但我将把它们留在这里。这实际上是因为我花了很多时间在 Python 和 C 编程；我使用的是 Python “PEP 8” 样式或通用 C 样式，请自行选择。

Answer 1

就我个人而言，我喜欢“扔空；”在调用一个根本无法返回的方法之后。

Answer 2

编译器理解 throw 永远不会返回，它不理解 err_exit() 不会返回。如果您要使用您忽略的异常重新编写，那么您的代码可能会更干净，编译器更快乐。 您还可以考虑使用 Java 命名约定。

public static void main(String[] args) {
  String inFname = "C:/tmp/test.txt"; // not reading args yet
  String outFname = "C:/tmp/test.sign";
  try {
    processFile(inFname, outFname);
  } catch (Exception e) {
    e.printStackTrace();
    throw e; // rethrow and let main() die
  }
}        

public static void processFile(String inFname, String outFname) 
    throws IOException, NoSuchAlgorithmException {
  Byte[] fileData = readFileAsByteArray(inFname);
  String hexDigest = hexdigestSha1(fileData);
  writeFileFromString(outFname, hexDigest);
}

在调用例程中，您可以捕获错误。如果您确实需要区分可能发生 IOException 的不同位置，那么您可以在例程中捕获并抛出描述性错误。

对代码的最小修复就是在设置变量的 try catch 块之外将变量初始化为 null。那么编译器会很高兴。

Answer 3

如果我确定在访问失败后我会立即退出，并且没有尝试对 readFileAsByteArray 等调用的结果执行，我可能会在申报地点。像这样的...

byte[] file_data = new byte[] {0};

...您的其余代码如下，在 catch 块中没有“虚假”分配。

Answer 4

你似乎有很强的 C 背景。我假设编译器抱怨诸如 file_data 之类的变量。只需在 try-catch 块上方添加 file_data = null。

以上几点：

1. 我认为将所有语句组合在一个块中是安全的。当抛出异常时，程序会立即跳转到正确的异常块。它看起来也不会那么难看。
2. 通常不会在 catch-block 中调用另一个方法，因为任务应该（通常并且如果可能）简短而切题。处理错误，继续或退出（可能会重新抛出异常）。
3. System.exit() 是邪恶的。如果您希望系统退出（即不可恢复的错误条件），而是抛出一个包装的 RuntimeException。 System.exit() 可能导致非干净存在。而是使用 throw new RuntimeException(e);
4. 最后，如果我可能直言不讳，Java 约定是将方法/变量名称写为 errExit，而不是 err_exit。