Блог

Почему Eclipse выдает мне сообщение «Утечка ресурсов: «in» никогда не закрывается» в следующем коде?

 public void readShapeData() {
        Scanner in = new Scanner(System.in);
        System.out.println("Enter the width of the Rectangle: ");
        width = in.nextDouble();
        System.out.println("Enter the height of the Rectangle: ");
        height = in.nextDouble();
 

Потому что вы не закрываете свой сканер

 in.close();
 

Как говорили другие, вам нужно вызвать «close» для классов ввода-вывода. Я добавлю, что это отличное место для использования блока try — finally без подвоха, например:

 public void readShapeData() throws IOException {
    Scanner in = new Scanner(System.in);
    try {
        System.out.println("Enter the width of the Rectangle: ");
        width = in.nextDouble();
        System.out.println("Enter the height of the Rectangle: ");
        height = in.nextDouble();
    } finally {
        in.close();
    }
}
 

Это гарантирует, что ваш сканер всегда закрыт, гарантируя надлежащую очистку ресурсов.

Аналогично, в Java 7 или более поздней версии вы можете использовать синтаксис «try-with-resources»:

 try (Scanner in = new Scanner(System.in)) {
    ... 
}
 

Вам нужен call in.close() в finally блоке, чтобы убедиться, что это происходит.

Из документации Eclipse, вот почему он помечает эту конкретную проблему (выделение мое):

Классы, реализующие интерфейс java.io.Closeable (начиная с JDK 1.5) и java.lang.Считается, что автоматически закрываемые (начиная с JDK 1.7) представляют внешние ресурсы, которые следует закрыть с помощью метода close (), когда они больше не нужны.

Компилятор Eclipse Java способен анализировать, соответствует ли код, использующий такие типы, этой политике.

…

Компилятор отметит [нарушения] с помощью «Утечки ресурсов: «поток» никогда не закрывается».

Полное объяснение здесь.

Он сообщает вам , что вам нужно закрыть сканер , с помощью которого вы создали экземпляр System.in Scanner.close() . Обычно каждый ридер должен быть закрыт.

Обратите внимание, что если вы закроетесь System.in , вы не сможете читать из него снова. Вы также можете взглянуть на этот Console класс.

 public void readShapeData() {
    Console console = System.console();
    double width = Double.parseDouble(console.readLine("Enter the width of the Rectangle: "));
    double height = Double.parseDouble(console.readLine("Enter the height of the Rectangle: "));
    ...
}
 

 // An InputStream which is typically connected to keyboard input of console programs

Scanner in= new Scanner(System.in);
 

приведенная выше строка вызовет конструктор класса сканера с аргументом System.in , и вернет ссылку на вновь созданный объект.

Он подключен к потоку ввода, который подключен к клавиатуре, поэтому теперь во время выполнения вы можете использовать пользовательский ввод для выполнения требуемой операции.

 //Write piece of code 
 

Чтобы устранить утечку памяти —

 in.close();//write at end of code.
 

Если вы используете JDK7 или 8, вы можете использовать try-catch с ресурсами.Это автоматически закроет сканер.

 try ( Scanner scanner = new Scanner(System.in); )
  {
    System.out.println("Enter the width of the Rectangle: ");
    width = scanner.nextDouble();
    System.out.println("Enter the height of the Rectangle: ");
    height = scanner.nextDouble();
  }
catch(Exception ex)
{
    //exception handling...do something (e.g., print the error message)
    ex.printStackTrace();
}
 

добавление private static Scanner in; на самом деле не устраняет проблему, оно только удаляет предупреждение. Статичность сканера означает, что он остается открытым навсегда (или до тех пор, пока класс get не выгрузится, что почти равно «навсегда»). Компилятор больше не выдает вам никаких предупреждений, поскольку вы сказали ему «держите его открытым навсегда». Но это не то, чего вы действительно хотели, поскольку вы должны закрыть ресурсы, как только они вам больше не понадобятся.

Привет, Манфред.

Ладно, серьезно, по крайней мере, во многих случаях это действительно ошибка. Это также отображается в VS Code, и это линтер, замечающий, что вы достигли конца охватывающей области, не закрывая объект сканера, но не признавая, что закрытие всех открытых файловых дескрипторов является частью завершения процесса. Утечки ресурсов нет, потому что все ресурсы очищаются при завершении, и процесс завершается, не оставляя места для хранения ресурса.

Вы должны закрыть свой сканер, когда закончите с ним:

 in.close();
 

Как правило, экземпляры классов, которые имеют дело с вводом-выводом, должны быть закрыты после того, как вы закончите с ними. Итак, в конце вашего кода вы могли бы добавить in.close() .

Сканер должен быть закрыт. Рекомендуется закрывать программы чтения, потоки … и такого рода объекты, чтобы освободить ресурсы и избежать утечек памяти aovid; и делать это в блоке finally, чтобы убедиться, что они закрыты, даже если при обработке этих объектов возникает исключение.

 private static Scanner in;
 

Я исправил это, объявив in как закрытую переменную класса статического сканера. Не уверен, почему это исправлено, но это то, что eclipse рекомендовал мне сделать.

 Scanner sc = new Scanner(System.in);

//do stuff with sc

sc.close();//write at end of code.
 

 in.close();
scannerObject.close(); 
 

Он закроет Scanner и отключит предупреждение.