Discussion :

[Christophe Genolini]

Bonjour,

Le code suivant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Scanner reponse = new Scanner(System.in);

déclenche un warning (ou du moins, je suppose que c'est un warning : une ampoule avec un triangle, sous Eclipse) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Resource leak: 'reponse' is never closed

C'est grave docteur ? Que dois-je faire pour corriger ça ?

Christophe

[thelvin]

C'est bien un warning.

Ce n'est pas bien grave, puisque ce n'est pas toi qui a ouvert System.in. Il n'y a donc pas de raison que ce soit toi qui le ferme. Il se fermera à la fin du programme, comme cela arrive toujours qu'on mette un Scanner dessus ou pas.

Tu as ce warning parce qu'en général, quand on utilise une ressource fermable pour construire un Scanner, il faut toujours fermer le Scanner (ou la ressource directement, mais c'est pas très bon pour l'encapsulation.)

[rg77140]

Bonjour,

Est-ce que si tu fais ceci tu as toujours le warning ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
Scanner reponse = null;
try{
reponse = new Scanner(System.in);
// ...
}finally{
    if(reponse != null){
        reponse.close();
    }
}

Romain.

[thelvin]

Hum. C'est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
Scanner reponse = new Scanner(System.in);
try{
  // ...
}finally{
  reponse.close();
}

ou en Java 1.7 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
try(Scanner reponse = new Scanner(System.in)){
  // ...
}

[adiGuba]

Salut,


+1 avec thelvin pour la meilleure façon de libérer une ressource.

Mais dans ce cas précis on est dans un des rare cas où l'on ne doit pas libérer la ressource.
En effet si on ferme System.in on ne pourra plus l'utiliser, et je doute que cela soit voulu...


Du coup dans ce cas précis, on peut utiliser une annotation pour ignorer le warning :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
	// Il ne faut pas fermer System.in !
	@SuppressWarnings("resource")
	Scanner scanner = new Scanner(System.in);

a++

[thelvin]

Il serait encore pire de le réutiliser, puisqu'on vient de le mettre dans un Scanner qui va buffuriser tout ce qu'il veut.

D'un autre côté, on a pas de raison non plus de vouloir le fermer, bien sûr, puisqu'on l'a pas ouvert.

Conclusion : en réalité on s'en fiche.