Используйте try-with-resources или закройте этот «BufferedReader» в предложении «finally» - PullRequest
Новая
       29

Используйте try-with-resources или закройте этот «BufferedReader» в предложении «finally»

1 голос
/ 15 мая 2019

Искал способ решить эту проблему. Прочитайте все предыдущие ответы, но никто не помог мне. Может ли быть какая-либо ошибка с SonarQube? 

public class Br {

    public String loader(String FilePath){

        BufferedReader br;
        String str = null;
        StringBuilder strb = new StringBuilder();
        try {
            br = new BufferedReader(new FileReader(FilePath));
            while ((str = br.readLine()) != null) {
                strb.append(str).append("\n");
            }
        } catch (FileNotFoundException f){
            System.out.println(FilePath+" does not exist");
            return null;
        } catch (IOException e) {
            e.printStackTrace();
        }
        return strb.toString();
    }
}

Ответы [ 3 ]

5 голосов
/ 15 мая 2019

Вы не звоните br.close(), что означает риск утечки ресурса.Чтобы надежно закрыть BufferedReader, у вас есть два варианта:

с использованием блока finally: 

public String loader(String FilePath) {
    // initialize the reader with null
    BufferedReader br = null;
    String str = null;
    StringBuilder strb = new StringBuilder();
    try {
        // really initialize it inside the try block
        br = new BufferedReader(new FileReader(FilePath));
        while ((str = br.readLine()) != null) {
            strb.append(str).append("\n");
        }
    } catch (FileNotFoundException f) {
        System.out.println(FilePath + " does not exist");
        return null;
    } catch (IOException e) {
        e.printStackTrace();
    } finally {
        // this block will be executed in every case, success or caught exception
        if (br != null) {
            // again, a resource is involved, so try-catch another time
            try {
                br.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
    return strb.toString();
}

с использованием try -составление ресурсов: 

public String loader(String FilePath) {

    String str = null;
    StringBuilder strb = new StringBuilder();
    // the following line means the try block takes care of closing the resource
    try (BufferedReader br = new BufferedReader(new FileReader(FilePath))) {
        while ((str = br.readLine()) != null) {
            strb.append(str).append("\n");
        }
    } catch (FileNotFoundException f) {
        System.out.println(FilePath + " does not exist");
        return null;
    } catch (IOException e) {
        e.printStackTrace();
    }
    return strb.toString();
}
2 голосов
/ 15 мая 2019

Похоже, вы просто хотите прочитать все строки из файла.Вы можете использовать это: 

public String loader(String FilePath) {
    try(Scanner s = new Scanner(new File(FilePath).useDelimiter("\\A")) {
        return s.hasNext() ? s.next() : null;
    } catch(IOException e) {
        throw new UncheckedIOException(e);
    }
}
0 голосов
/ 15 мая 2019

Код, который вы написали, действительно пропускает ресурсы, поскольку вы не закрываете BufferedReader. Следующий фрагмент должен сделать трюк: 

public String loader(String filePath){
    String str = null;
    StringBuilder strb = new StringBuilder();
    // try-with-resources construct here which will automatically handle the close for you
    try (FileReader fileReader = new FileReader(filePath); 
         BufferedReader br = new BufferedReader(fileReader);){
        while ((str = br.readLine()) != null) {
            strb.append(str).append("\n");
        }
    }
    catch (FileNotFoundException f){
        System.out.println(filePath+" does not exist");
        return null;
    }
    catch (IOException e) {
        e.printStackTrace();
    }
    return strb.toString();
}

Если у вас все еще есть проблемы с этим кодом, то да, это ошибка SonarQubes: -)

...