Discussion :

[Sinon]

Bonjour,

Je n'arrive pas à assimiler le principe des exceptions JAVA. J'ai consulté quelques sites sur le web mais visiblement ca ne me suffit pas pour bien comprendre.

Je voudrais par exemple faire une méthode toute simple qui supprime un fichier passé en paramétre.

Classe Fichier :

Code :

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

1
2
3
4
 
public static void supprimeFichier (File pmyfile) {
   pmyfile.delete();
}

Méthode que j'appelle dans mon main() :

Code :

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

1
2
3
4
5
 
public static main(String[] args) {
   File myfile = new File("texte");
   Fichier.supprimeFichier(myfile);
}

Admettons que le fichier "texte' n'existe pas... Comment dès lors puis-je faire pour récupérer l'exception FileNotfound ? Ou faut-il l'implémenter ?

Je vous remercie pour votre aide.

[miloux32]

Plusieurs facons de faire , ca depend de l'endroit ou tu veux gerer l'exception

Solution 1 -- Exception gérée au niveau du main
Classe Fichier :

Code :

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

1
2
3
4
public static void supprimeFichier (File pmyfile) throw FileNotFoundException {
   pmyfile.delete();
}

Code :

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

1
2
3
4
5
6
7
public static main(String[] args) {
   File myfile = new File("texte");
try {   Fichier.supprimeFichier(myfile);}
catch ( FileNotFoundException e) 
{ System.out.println( "Ohh la belle exception que nous avons " +e);}
}

solution 2 -- Au niveau de la méthode elle même

On ne change pas le main

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public static void supprimeFichier (File pmyfile) 
{
try {
   pmyfile.delete();
}
catch (FileNotFoundException e2)
   {System.out.println(" une z'olie exception "+e);
  }
}

Code :

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

1
2
3
4
5
 
public static main(String[] args) {
   File myfile = new File("texte");
 Fichier.supprimeFichier(myfile);
}

Je sais plus si c'est throw ou throws qu'il faut mettre.

Ce code est donné sans doute avec des erreurs de syntaxe.

[Sinon]

Je voudrais gerer l'exception dans la méthode appelée.
Le problème c'est que ca ne marche pas. J'avais testé la meme chose que toi.

Il ne prend pas le catch (FileNotFoundException e), il me dit que :

"unreachable catch block for FileNotFoundException"

En gros, il n'arrive pas à passer dans le catch.

[miloux32]

Je voudrais gerer l'exception dans la méthode appelée.
Le problème c'est que ca ne marche pas. J'avais testé la meme chose que toi.

Il ne prend pas le catch (FileNotFoundException e), il me dit que :

"unreachable catch block for FileNotFoundException"

En gros, il n'arrive pas à passer dans le catch.

Ahahah y'a un piege sur delete si je me souviens bien !

Delete ne renvoie pas d'exception mais il renvoie un booleen !

Code :

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

1
2
3
4
5
6
7
8
public static void supprimeFichier (File pmyfile) 
{boolean del;
del=  pmyfile.delete();
if (!del) 
  {
  System.out.println("Erreur lors de la suppression du fichier");
  }
}

Enfin le plus propre ca serait de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public static void supprimeFichier (File pmyfile) 
{
if (  pmyfile.exist()) 
  {
  pmyfile.delete();
  }
else 
  {
  System.out.println("Le fichier n'existe pas");
  }
}

[adiGuba]

Salut,

"unreachable catch block for FileNotFoundException"

En gros, il n'arrive pas à passer dans le catch.

C'est normal car la méthode delete() ne gère pas d'exception mais un simple retour de valeur booléen. Si tu veux qu'il y ai des exceptions il va falloir les faire remonter toi-même...

a++

[Tuxico]

ton code?

[Sinon]

Merci miloux32 mais tu ne gères pas d'exception dans ton code. Tu fais des tests.
Or c'est vraiment les exceptions que je n'arrive pas trop à assimiler.

[adiGuba]

Or c'est vraiment les exceptions que je n'arrive pas trop à assimiler.

Mais la méthode delete() ne renvoi pas d'exception. Donc c'est un très mauvais exemple à prendre pour cela

a++

[Sinon]

Ok, j'ai vraiment le don pour trouver les emmerdes !

[Seb19]

Bonjour,

Tu peux nous montrer ton code ?

Merci.

[Tommy31]

C'est à toi de jetter l'exception dans ta méthode, de cette façon :

Code :

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

1
2
3
4
5
6
7
 
    public static void supprimeFichier(File pmyfile) throws FileNotFoundException {
 
        if (!pmyfile.delete()) {
            throw new FileNotFoundException("File " + pmyfile.getName() + " not deleted.");
        }
    }

Note que faire un if( file.exist ) avant ne garanti pas que le fichier ne sera pas supprimé entre le test et la suppression effective. C'est pourquoi il faut tester directement le delete.

[Sinon]

Code :

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

1
2
3
4
5
6
7
8
9
 
public static void supprimeFichier (File pmyfile) {
   if (!pmyfile.delete()) {
   System.out.println("suppression impossible/fichier inexistant");
   }
   else {
   System.out.println("fichier supprimé");
   }
}

J'avais écrit ça pendant ta réponse...
Merci Tommy31

[Tommy31]

T'as mis un prefixe p dans l'argument transmis, le p c'est pour parameter ou pointeur, ou autre ? c'est curieux

[Sinon]

p comme paramètre.
Avant je mettais p_myfile mais on m'a conseillé d'éviter les "_"

[Tommy31]

Moi je te conseillerais aussi de ne pas mettre le p, ça ne suit pas les conventions de nommage usuelles.

Cf: http://cyberzoide.developpez.com/java/javastyle/

[Sinon]

Ce que j'ai du mal à comprendre c'est qu'il faille également spécifier "throws FileNotFound Exception" dans la méthode...

[miloux32]

Ce n'est pas pour la même chose .

Dans la définition de la méthode c'est pour dire :
"si tu as une exception de ce type tu la fais suivre au dessus"

Dans le code c'est pour dire
"dans ce cas-là tu jettes cette exception" ( et donc avec le throws de la définition il la fait suivre au dessus )

[Sinon]

Ok !!! Je crois que je commence à comprendre ! alors c'est pour cela qu'il y a également throws IOException dans la méthode main() ? FileNotFoundException est une sous-classe de IOException... Cette dernière étant le niveau le plus haut dans la gestion des Exceptions...
j'ai bon ?

[Sinon]

Personne ne réponds...
Suis-je toujours à côté de la plaque ?

[CheryBen]

Tu es à côté de la plaque
Voici l'arbre de l'exception FileNotFoundException :

Code :

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

1
2
3
4
5
java.lang.Object
  extended by java.lang.Throwable
      extended by java.lang.Exception
          extended by java.io.IOException
              extended by java.io.FileNotFoundException