Discussion :

[Caxton]

Bonsoir,
Sur les conseils que j'ai pu obtenir, j'ai décider de refaire une classe qui posait problème.

Quand j'écris sous netbeans le code qui va suivre, j'ai droit à une belle erreur...

Code :

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

1
2
3
4
5
6
7
8
9
 
package jProperties;
 
import java.io.*;
import java.util.Properties;
 
public class GestionProp throws ExcErreurFichier {
 
}

Erreur :

'{' expected

Meme si j'ajoute un début de classe EXCErreurFichier, j'ai la même erreur !
Pourquoi ?

[thebloodyman]

Salut toi !

La définition throws ExcErreurFichier n'est pas valide.
Une classe ne peut déclarer un déclenchement d'exception.
Ce sont les méthodes qui le peuvent.

En gros NetBeans te dit qu'il attend l'accolade d'ouverture de la classe comme il comprend pas pourquoi il trouve quelque chose de non autorisé dans la déclaration.

[rtg57]

Bonsoir,

voila un peu de code que j'ai trouvé sur Internet:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
class MaDefinedException extends Exception
 {
  public MaDefinedException(String str)
  {
     super(str);
  }  
 }
 
 public class MaClass
 {
    public static void showMyName(String str) throws MaDefinedException
    {
          if(str.equals("What is your Name?"))
                throw new MaDefinedException("Bla bla");
    }
    public static void main(String a[])
    {
       try
       {
          showMyName("What is your Name?");
       }
       catch(MaDefinedException mde)
       {
          mde.printStackTrace();
       }
     }
 }

D'après ce que je vois, la classe ne lance pas d'exception, ce sont ses méthodes qui le font.

En espérant que cela vous aide...

[Caxton]

Ok, je comprends mieux !

Donc je devrais faire:

Fichier GestionProp.java

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
package jProperties;
 
import java.io.*;
import java.util.Properties;
 
public class GestionProp {
   public void setUrl(String url) {
      this.urlFichier = url;
   }
 
   public void create throws ErreurUrl {
      // Si le fichier existe déjà
      if(new File(url).exists()) {
         thow new ErreurUrl(url);
      }
 
      //Il n'y a pas et d'erreur alors on poursuit le traitement
   }
 
   private String url;
}

Classe d'erreur fichier ErreurUrl.java

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public class ErreurUrl extends Exception {
 
public ErreurUrl(String urlFichier) {
 
   public String getMessage() {
      message = "Le fichier demandé n'existe pas ou ne se trouve pas à l'endroit renseigné. Chemin retournant l'erreur : urlFichier";
      return message;
   }
 
   private String urlFichier;
   private String message;
}

... Enfin si j'ai bien compris !

Je fait un edit pour ajouter cette possibilité
Que se passerais-t-il si je changeais la partie erreur par:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public class ErreurUrl extends Exception {
 
   public ErreurUrl(String urlFichier) {
      message = "Le fichier demandé n'existe pas ou ne se trouve pas à l'endroit renseigné. Chemin retournant l'erreur : urlFichier";
      System.err.println(message);
   }
 
   public String getMessage() { 
      return this.message; 
   }
 
   private String urlFichier;
   private String message;
}

Je ne sait plus trop...

[jcdentons]

Bonjour,

Ce qui changerait serait l'instanciation de la variable d'instance message.
Dans votre premier essai, vous réinstanciez à chaque appel message, ceci prend plus de temps, plus de mémoire ...
Dans le second, vous instanciez une seule fois dans le constructeur ... ce qui est beaucoup plus efficace.

[darrylsite]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
public class ErreurUrl extends Exception 
{
 
   public ErreurUrl(String urlFichier) 
  {
      super("Le fichier demandé n'existe pas ou ne se trouve pas à l'endroit renseigné. Chemin retournant l'erreur : "+urlFichier);
      this.urlFichier=urlFichier;
   }
 
    public String getUrlFichier()
    {
     return urlFichier;
    }
 
   private String urlFichier;
}

[Caxton]

donc je vais opter pour la seconde méthode. Toutefois, j'ai une nouvelle question. Je parle d'utilisé deux fichiers, mais faut-il appeler le second dans le premier ou le simple fait de l'extends suffi ?

[darrylsite]

De quelles classes parles tu? de la classe UrlException et GestionProp?

Si la réponse est oui, alors il te faut les deux fichiers source java. ces deux classes doivent pouvoir exister indépendamment l'une de l'autre. C'est aussi la seule façon de faire une classe réutilisable.

Si tu veux un peu de théorie. Si les deux classes doivent se trouver dans un seul fichier (UrlException membre de GestionProp), la classe UrlException ne pourra exister sans une instance de la classe GestionProp.

Code :

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

1
2
3
 
GestionProp p = new GestionProp();
UrlException ex = p.new Urlexception();

Il serait alors impossible depuis une autre C de lancer une Exception UrlException sans y attacher une classe GestionProp.

[Caxton]

Donc, si je comprends bien, en séparant les deux, une seule classe d'erreur peut gérer facilement le même type d'erreur à plusieurs classes utilisant les mêmes points d'entrées.

Pour moi la classe GestionProp déclenche des exceptions mais il en sera de même pour, par exemple, GestionXml que je compte faire plus tard. Dans les deux cas, on a une url d'entrée et forcément des erreurs du même type. Donc UrlException doit pouvoir agir avec les deux classes.

Je ne sait pas si je suis clair.. peut être que je m'égare ou que je mélange.

[darrylsite]

C'est bien ça. C'est une classe dérivant de Exception donc tu peux l'utiliser au même titre que les autres classes d'exception que tu connais. Tu peux les lancer et les attraper dans les classes que tu veux, si ton modèle le permet.

[Caxton]

Ok !

Je suis partant pour cette seconde possibilité. Je pense attaqué cette prog dans la semaine et voir de quoi il en retourne réellement.

P.S. Je ne ferme pas le post car j'attends les tests réels pour voir si j'ai de nouvelles questions à ce sujet. Si oh combien ça fonctionne et que je n'ai pas d'autres questions, je fermerais le post avec un tag résolu.

[Caxton]

Comme je m'y attendais !!!
Comme je m'y attendais, la classe me génère une erreur quand j'écris ceci:
GestionProp.java

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
package jProperties;
 
import java.io.*;
import java.util.Properties;
 
public class GestionProp {
 
    /*
     * Méthode : Setter
     * Action : Place l'url du fichier à traiter dans une variable
     * Reçois : Une url, un chemin au format String
     */
 
    public void setUrl(String url) {
        this.urlFichier = url;
    }
 
 
    public void create throws ErreurUrl {
 
    }
 
    /*
     * Champ de la classe GestionProp
     */
 
    String urlFichier;
}

avec l'erreur suivante sur la ligne nommé " public void create throws ErreurUrl {":

'(' Expected

ErreurUrl.java

Code :

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

1
2
3
4
5
6
 
package jProperties;
 
public class ErreurUrl {
 
}

Pourquoi ?

[bého32]

Il faut que ta classe ErreurUrl hérite de la classe Throwable.
En gros, quand tu lèves une exception, "tu lèves quelque chose de Throwable", ou encore "tu throw un objet Throwable"

La classe Exception hérite déjà de Throwable. Tu peux donc faire hériter ta classe ErreurUrl de Exception.

tu peux aller voir la Javadoc pour plus d'info. Tu trouveras pour la classe Throwable :

The Throwable class is the superclass of all errors and exceptions in the Java language.

De plus, l'erreur de compilation vient du fait que tu dois mettre les parenthèses après create pour indiquer les paramètres (même si y'aura rien dans les parenthèses).

[Jidefix]

Bonjour,
il manque les parenthèses qui contiendraient les arguments s'il y en avait. Comme il n'y a en pas il suffit de rajouter des parenthèses vides:

Code :

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

1
2
3
4
5
    public void create() throws ErreurUrl {
        
    }
}

[Caxton]

Effectivement pour les parenthèses de create... Han quel noob je fait sur ce coup.

Après avoir poster le code, je suis retourné sur ErreurUrl et effectivement il manquerais un extends exception... Le noob sur ce coup.

[mode j'me lamente]Et bien ce n'est pas demain que j'arriverais à faire une classe du premier coup ![/mode j'me lamente]

[Caxton]

J'avance petit à petit.
J'ai donc construit un main, pour essayer ma classe. Si je regarde l'algorithme, je me dis la chose suivante:

Si le bouton initialisé est pressé.
Début

Créer une url statique dans le programme
Passer à la classe via setUrl() l'url du fichier

Si le fichier n'est pas présent
Début

On crée un fichier vide à l'url via la méthode create()
On inscrit les données d'initialisations dans le fichier via updateNew()
Sinon

On initialise uniquement les champs d'initialisation via updateExists()

Fin
Fin

Le souci est que j'ai volontairement créer des exceptions afin que le programmeur puisse débugger son logiciel lorsque il utilise une de mes classes et qu'un utilisateur tombe dans le piège du déplacement de fichier. Une erreur humaine est si vite arrivé !

Ce qui me chagrine, c'est que la dans mon algorytme, je sait quoi faire et d'ailleurs je vais faire tout ce que je peux pour y arrivé, mais je ne retourne aucune erreur.

Les questions

Comment faut-il s'y prendre pour que la classe d'exception que j'ai créer puisse indiquer le message bloquant, avec un Ok, sur le fait que le fichier n'existe pas ?

Comment je dois m'y prendre pour que s'il clique sur Ok, alors on procèdera à la création du fichier ?

Enfin faut-il que je vous mette la copie de ce que j'ai déjà écrit ?
Pour informations, c'est loin d'être complet !

[darrylsite]

[edit]
Au fait, c'est à la classe d'exception d'afficher le message. Mais la méthode dans la l'exception est attrapée.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 try
{
 //operation possible de lancer l'exception
}
catch(UrlException ex)
{
 //affichage du message
//autre traitement
}

[/edit]

pour ajouter la boite de dialogue bloquante avec le bouton OK comme tu le dis, il te faut d'abord choisir une bibliothèque graphique.
Si tu choisis d'utiliser par exemple swing, le composant est JOptionPane. La javadoc va te donner plus de détails.
voici un exemple,

Code :

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

1
2
3
4
5
6
7
8
9
10
 
     int choix=JOptionPane.showConfirmDialog(ParentFrame, "Creer un nouveau fichier ?", "test",JOptionPane.YES_NO_OPTION);
     if(choix==JOptionPane.YES_OPTION)
     {
      System.out.println("creation fichier");
     }
     else
     {
      System.out.println("Il fallait choisir yes! :D");
     }

ParentFrame est la Jframe dans laquelle est affichée la boite de dialogue

[Caxton]

donc je devrais alors orienté mon algorithme vers:

Si le bouton initialisé est pressé.
Début

Créer une url statique dans le programme
Passer à la classe via setUrl() l'url du fichier


Si le fichier n'est pas présent
Try
Début

On crée un fichier vide à l'url via la méthode create()
On inscrit les données d'initialisations dans le fichier via updateNew()
Fin
Catch
Début

Erreur, il faut créer le fichier pour pouvoir l'utiliser
Fin
Finally
Début

On initialise uniquement les champs d'initialisation via updateExists()
Fin

Ou je me trompe ?