Discussion :

[gege22mars]

Bonjour, je developpe une appli client/server avec du corba. Je me pose juste une question de bonne pratique de developpement lors d'un appel d'une fonction qui peut lever des exceptions.

Prenons par exemple une methode saveFile() . Le client fait une demande de sauvegarde de fichier au serveur via un proxy. Le serveur applique sa fonction saveFile.

Est il preferable que le serveur catch son exception, la logue et renvoie un etat success true/false (ou code erreur ) de la methode au client,

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
 
//SERVER
   public boolean saveFile(String name) {
        File file = new File(name);
        boolean success = false;
        try {
            BufferedWriter bw = new BufferedWriter(new FileWriter(olbFile));
            bw.write(olv.toOLBString());
            bw.close();
            success = true;
        } catch (IOException e) {
           System.out.println(e);
        }
        return success;
    }

ou le serveur se contente de faire un throws de son exception et c'est le client qui se charge de la catcher et de la loguer.

Code :

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

1
2
3
4
5
6
7
8
 
//SERVER
   public void saveFile(String name) throws IOException  {
        File file = new File(name);
        BufferedWriter bw = new BufferedWriter(new FileWriter(olbFile));
        bw.write(olv.toOLBString());
        bw.close();
    }

Quel serait la facon de faire la plus "propre" ?? merci pour vos avis.

[CyberChouan]

Le traitement correct des exceptions IO est expliqué dans la (un indice : le mot-clé "finally")

[gege22mars]

En fait ma question ne porte pas sur l exception IO, (jaurais pu prendre n importe quelle exception) mais plutot sur le mecanisme d'envoi d erreur qu'il est preferable d utiliser dans une archi client server. En gros en cas d erreur, faut il que le serveur renvoie une exception au client, ou doit il se contenter de renvoyer un code retour d erreur exemple valeur -1 pour une fonction qui doit renvoyer un entier et qui echoue.

[in]

Je n'ai pas de vrai réponse, mais je dirais que ça dépend de ce que tu veux faire au niveau client ... en, gros quel est le besoin ? (il vaut mieux répondre au besoin que de répondre à des critères "esthétiques" du code)

Y-a-t'il une utilité pour le client à recevoir un objet Exception ? Quels traitements supplémentaires ferais tu ? afficher un message d'erreur précis ?

Si tu n'en a pas vraiment l'utilité, contente toi de gérer les erreurs par un code retour ... après selon les méthodes que tu appelles l'intérêt n'est peut être pas le même ?

Pour le saveFile() le client a-t'il besoin savoir si ok ou ko ou il a aussi besoin de savoir pourquoi ça na pas marché, à quel endroit ... ?