Perdita di risorse: "in" non viene mai chiuso

Perché Eclipse mi dà il riscaldamento "Perdita di risorse: 'in' non è mai chiuso" nel codice seguente?

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();

Perché non chiudi lo Scanner

in.close();

Come altri hanno già detto, è necessario chiamare "close" sulle classi IO. Aggiungerò che questo è un posto eccellente per usare la prova - finalmente blocca senza cattura, in questo modo:

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();
    }
}

Ciò garantisce che lo scanner sia sempre chiuso, garantendo una corretta pulizia delle risorse.

In modo equivalente, in Java 7 o versioni successive, puoi utilizzare la sintassi "try-with-resources":

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

Hai bisogno di chiamare in.close(), in un finallyblocco per assicurarti che si verifichi.

Dalla documentazione di Eclipse, ecco perché segnala questo particolare problema ( enfasi mia):

Le classi che implementano l'interfaccia java.io.Closeable (da JDK 1.5) e java.lang.AutoCloseable (da JDK 1.7) sono considerate rappresentare risorse esterne, che dovrebbero essere chiuse usando il metodo close (), quando non sono più necessarie.

Il compilatore Java di Eclipse è in grado di analizzare se il codice che utilizza tali tipi aderisce a questa politica.

...

Il compilatore contrassegnerà [violazioni] con "Perdita di risorse: 'flusso' non è mai chiuso".

Spiegazione completa qui .

E vi sta dicendo che è necessario chiudere lo scanner che si creata un'istanza System.incon Scanner.close(). Normalmente ogni lettore dovrebbe essere chiuso.

Nota che se chiudi System.in, non sarai più in grado di leggere da esso. Puoi anche dare un'occhiata alla Consoleclasse.

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: "));
    ...
}

Se stai usando JDK7 o 8, puoi usare try-catch con le risorse. Questo chiuderà automaticamente lo scanner.

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);

sopra la riga richiamerà la classe Costruttore dello scanner con l'argomento System.in e restituirà un riferimento all'oggetto appena costruito.

È connesso a un flusso di input connesso alla tastiera, quindi ora in fase di esecuzione puoi prendere l'input dell'utente per eseguire l'operazione richiesta.

//Write piece of code 

Per rimuovere la perdita di memoria:

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

Dovresti chiudere lo Scanner quando hai finito:

in.close();

l'aggiunta private static Scanner in;non risolve realmente il problema, cancella solo l'avviso. Rendere statico lo scanner significa che rimane aperto per sempre (o fino a quando la classe non viene scaricata, il che è quasi "per sempre"). Il compilatore non ti dà più avvertimenti, dal momento che gli hai detto "tienilo aperto per sempre". Ma non è quello che volevi veramente, dal momento che dovresti chiudere le risorse non appena non ne hai più bisogno.

HTH, Manfred.

In genere, le istanze di classi che si occupano di I / O dovrebbero essere chiuse dopo averle terminate. Quindi alla fine del codice potresti aggiungere in.close().

private static Scanner in;

L'ho risolto dichiarando come variabile di classe Scanner statica privata. Non sono sicuro del motivo per cui è stato risolto, ma è quello che eclipse mi ha consigliato di fare.

Lo scanner dovrebbe essere chiuso. È una buona pratica chiudere lettori, stream ... e questo tipo di oggetti per liberare risorse e evitare perdite di memoria; e farlo in un blocco finalmente per assicurarsi che siano chiusi anche se si verifica un'eccezione durante la gestione di quegli oggetti.

Ok, seriamente, almeno in molti casi, questo è in realtà un bug. Viene visualizzato anche in VS Code ed è il linter che nota che hai raggiunto la fine dell'ambito di inclusione senza chiudere l'oggetto scanner, ma non riconosce che la chiusura di tutti i descrittori di file aperti fa parte della terminazione del processo. Non c'è perdita di risorse perché le risorse vengono tutte ripulite al termine e il processo scompare, senza lasciare alcun posto in cui la risorsa possa essere trattenuta.

Scanner sc = new Scanner(System.in);

//do stuff with sc

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

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

Chiuderà Scannere spegnerà l'avviso.