Findbug: RV_RETURN_VALUE_IGNORED_BAD_PRACTICE

Question

我正在使用FindBugs从Ant中分析Eclipse中的代码。

下面的片段给出了RV_RETURN_VALUE_IGNORED_BAD_PRACTICE

方法忽略异常返回值(RV_RETURN_VALUE_IGNORED_BAD_PRACTICE) 此方法返回未检查的值。应该检查返回值，因为它可以指示异常或意外的函数执行。例如，如果文件无法成功删除(而不是抛出异常)，File.delete()方法将返回false。如果不检查结果，您将不会注意到方法调用是否通过返回一个不典型的返回值来指示意外行为。

public void export (File file) throws IOException {
    if (!file.exists()) {
        file.createNewFile();
    }

    BufferedWriter bw = null;
    try {
        bw = new BufferedWriter(new FileWriter(file.getAbsoluteFile()));
    ...

实际上，我不关心文件在哪里存在或不存在，方法应该继续执行。如果发生异常，它将被抛出export()之外

如何重写这个片段，这样就不会显示警告/错误，而不会在Findbug配置文件中禁用它？

Answer 1

在这种情况下，您不必调用file.createNewFile()，因为文件无论如何都将由FileWriter创建。

但是，必须确保文件的父文件夹存在。

Answer 2

f.createNewFile();返回boolean值

如果命名文件不存在并已成功创建，则为true；如果命名文件已存在，则为false

将方法转换为boolean result = f.createNewFile();

如果您不关心文件是否存在，您可能应该更改代码，以排除exists的检查，并直接创建BufferedWriter，如果文件不存在，BufferedWriter将创建该文件。另外，如果仍然希望使用当前代码，请添加SuppressWarnings以忽略警告。