我在使用 try-and-catch 时经常遇到一些问题：
1) 一些变量需要在 try 括号内声明，否则它们将不在范围内
2) 最终，即使我的 return 语句最终也必须在 try 括号内，但该方法不会返回任何内容。
解决此类问题的正确方法是什么。
导致此问题的方法示例如下。它需要处理 FileNotFoundException 和处理 IOException。我怎样才能最优雅地做到这一点？
public static String getContents (File file) {
BufferedReader reader = new BufferedReader(new FileReader(file));
String contents = new String();
while (reader.ready())
contents += reader.readLine();
return contents;
}
如果异常处理不需要在方法中发生，还有一个选择——在getContents方法中添加一个throws子句以使方法抛出异常：
public static String getContents (File file)
throws IOException, FileNotFoundException {
这样，调用方法的代码将处理Exceptions 而不是方法本身。如果将s 扔给调用它的方法，则该方法中不需要try/catch块。Exception
这可能是也可能不是处理这种情况的理想方式，具体取决于该方法的预期行为方式。
编辑
再三考虑，让方法抛出异常可能是个好主意。我认为 D.Shawley 的评论总结得很好——“异常处理应该意味着只处理有意义的异常。”
在这种情况下，该getContents方法似乎获取了指定 的内容File，并将 a 返回String给调用者。
如果要在getConents方法中执行异常处理，则传达已发生错误的唯一方法是返回某种预先确定的值，例如null向调用者通知发生了错误。
但是，通过让方法本身向调用者抛出异常，调用者可以选择做出相应的反应：
try {
String contents = getContents(new File("input.file"));
} catch (IOException ioe) {
// Perform exception handling for IOException.
} catch (FileNotFoundException fnfe) {
// Inform user that file was not found.
// Perhaps prompt the user for an alternate file name and try again?
}
与其让setContents方法提出自己的协议来通知发生了错误，不如将IOException和FileNotFoundException返回给方法调用者可能会更好，因此可以在可以采取适当的替代操作的地方执行异常处理地方。
只有在可以进行一些有意义的处理时才应执行异常处理。
您可以按以下方式处理它：
StringBuilder contents = new StringBuilder();
BufferedReader reader;
try {
reader = new BufferedReader(new FileReader(file));
while (reader.ready()) {
contents.append(reader.readLine());
}
} catch (FileNotFoundException fne) {
log.warn("File Not Found", fne);
} catch (IOException ioe) {
log.warn("IOException", ioe);
}
return contents.toString();
不过，在上述情况下，您可能应该使用 StringBuilder 而不是 String，这在性能方面要好得多。
public static String getContents (File file) {
String contents = new String();
BufferedReader reader = null;
try {
reader = new BufferedReader(new FileReader(file));
while (reader.ready())
contents += reader.readLine();
}
catch (FileNotFoundException ex) {
// handle FileNotFoundException
}
catch (IOException ex) {
// handle IOException
}
finally {
if (reader != null) {
try {
reader.close();
}
catch (IOException ex) {
// handle IOException
}
}
}
return contents;
}
我添加了一个finally块来关闭你的BufferedReader，尽管你没有在你的代码中这样做。我还建议您使用StringBuilder而不是String串联，但有人已经指出了这一点。的声明和初始化只是因为我添加了块而在块reader之外；否则，可以在块内声明引用。tryfinallyreadertry
而且我没有处理异常，我认为这与您的问题无关。
您在这里缺少的是一个简单的实用程序，它大大简化了清理工作：
public static void closeAndLog(Closable c) {
if ( c == null )
return;
try {
c.close()
} catch ( IOException e) {
LOGGER.warn("Failed closing " + c +, e);
}
}
这样你的代码就可以变成：
public static String getContents (File file) throws IOException {
BufferedReader r = null;
try {
r = new BufferedReader(...);
// do stuff
} finally {
closeAndLog(r);
}
}
您可以尝试将您的 return 语句移动到一个finally块中。
关于变量范围，我不确定是否有更优雅的方法来做到这一点。通常我会尝试考虑在发生错误时我的返回值是什么，然后将变量分配给该值。
关于 return 语句，如果你使用我上面的建议，你可以在 try / catch 块之后返回。
因此，如果我使用 null 返回值来指示错误，我会这样做
public static String getContents (File file) {
String contents = null;
try {
BufferedReader reader = new BufferedReader(new FileReader(file));
contents = new String();
while (reader.ready())
contents += reader.readLine();
} catch (Exception e) {
// Error Handling
}
return contents;
}
IMHO you have two ways to deal properly with the exceptions (here, IOException and FileNotFoundException) :
To have good advice about exceptions, see also : Why do Java people frequently consume exceptions silently?
从 Java 7 开始，您可以使用try-with-resources来确保您的资源正确且“自动”关闭。您所需要的只是一个实现了java.lang.AutoCloseable它的对象BufferedReader。文档中实际上有以下示例：
try (BufferedReader br = new BufferedReader(new FileReader(path))) {
return br.readLine();
}