Витік ресурсу: "in" ніколи не закривається

Чому 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();

Як казали інші, вам потрібно зателефонувати "закрити" на заняттях вводу-виводу. Додам, що це відмінне місце для використання спроби - нарешті, блокувати без улову, наприклад:

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)) {
    ... 
}

Вам потрібен дзвінок in.close()у finallyблоці, щоб переконатися, що він відбувається.

З документації Eclipse ось чому вона позначає саме цю проблему ( наголос моя):

Класи, що реалізують інтерфейс java.io.Closeable (починаючи з JDK 1.5) та java.lang.AutoCloseable (починаючи з 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: "));
    ...
}

Якщо ви використовуєте 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();
}

// 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.

Закінчивши сканер, слід закрити його:

in.close();

додавання private static Scanner in;насправді не вирішує проблему, воно лише очищає попередження. Якщо статичний сканер статичним, це означає, що він залишається відкритим назавжди (або до тих пір, поки клас не розвантажиться, що майже “назавжди”). Компілятор більше не попереджує вас, оскільки ви сказали йому "тримати його відкритим назавжди". Але це не те, що ви насправді хотіли, оскільки вам слід закрити ресурси, як тільки вони вам більше не знадобляться.

HTH, Манфред.

Як правило, екземпляри класів, що мають справу з введенням / виведенням, слід закривати після того, як ви закінчите з ними. Тож у кінці коду ви можете додати in.close().

private static Scanner in;

Я виправив це, оголосивши приватною змінною класу статичного сканера. Не впевнений, чому це виправлено, але саме це я рекомендував зробити затемнення.

Сканер слід закрити. Це хороша практика - закривати Readers, Streams ... і подібні об’єкти, щоб звільнити ресурси та витік пам'яті; і робить це в блоці нарешті, щоб переконатися, що вони закриті, навіть якщо під час обробки цих об’єктів виникає виняток.

Добре, серйозно, принаймні у багатьох випадках це насправді помилка. Це також відображається у VS Code, і це лінтер, який зауважує, що ви досягли кінця охоплюючої області, не закриваючи об'єкт сканера, але не визнаючи, що закриття всіх дескрипторів відкритих файлів є частиною завершення процесу. Немає витоку ресурсів, оскільки всі ресурси очищаються при припиненні, і процес зникає, не залишаючи місця для зберігання ресурсу.

Scanner sc = new Scanner(System.in);

//do stuff with sc

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

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

Він закриє Scannerта закриє попередження.