IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Java Discussion :

Fermeture de fichier sur exception


Sujet :

Java

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre Expert
    Avatar de Clorish
    Profil pro
    Inscrit en
    Juin 2003
    Messages
    2 474
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2003
    Messages : 2 474
    Par défaut Fermeture de fichier sur exception
    Hello,

    Je me pose uen question metaphysique : Lorsqu'on manipule un fichier via les bufferwriter (par exemple) il faut encadrer son code par un try/catch avec une clause finally pour fermet le fichier (close). Or cette methode lance egalement une exception IOException.

    Existe t il un moyen propre de gerer ca sans faire un 2e bloc try/catch dans le finally ... ce qui est assez hard en somme.

  2. #2
    Expert éminent
    Avatar de adiGuba
    Homme Profil pro
    Développeur Java/Web
    Inscrit en
    Avril 2002
    Messages
    13 938
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Développeur Java/Web
    Secteur : Transports

    Informations forums :
    Inscription : Avril 2002
    Messages : 13 938
    Billets dans le blog
    1
    Par défaut
    Salut,

    Perso j'utilise plutôt un try/finally à l'intérieur d'un try/catch, par exemple :
    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
    	try {
    		// Ouverture du fichier :
    		BufferedWriter writer = new BufferedWriter(new FileWriter("..."));
    		try {
    			// Traitements sur le fichier...
    			...
    		} finally {
    			// Fermeture du fichier.
    			writer.close();
    		}
     
    	} catch (IOException e) {
    		// Gestion de l'exception
    		e.printStackTrace();
    	}
    On a une gestion commune de toutes les exceptions, et le scope de la variable "writer" est bien limité (pas besoin de la déclarer dans le bloc supérieur ni de la tester dans le finally).


    J'y vois toutefois 2 défauts :
    • Le double niveau de bloc (try/finally + try/catch) pas forcément très lisible (vivement les closures!)
    • Si une exception survient dans le try/finally, on fait le close() avant le traitement de l'exception. Or le close() peut remonter une autre exception qui "cacherait" alors la première. Mais bon ce cas est toutefois assez rare...



    a++

  3. #3
    Membre Expert
    Avatar de CheryBen
    Inscrit en
    Mai 2005
    Messages
    1 599
    Détails du profil
    Informations personnelles :
    Âge : 43

    Informations forums :
    Inscription : Mai 2005
    Messages : 1 599
    Par défaut
    Salut, bouh! c'est dans la FAQ

    Ca n'évite pas complètement les 2 blocs, c'est plutôt
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
     
    try {
        try {
             fichier.write();
        }
        finally {
            fichier.close();
        }
    }
    catch(IOException e) {
        e.printstacktrace();
    }
    EDIT : grillé par adiGuba, la FAQ vivante

  4. #4
    Membre Expert
    Avatar de Clorish
    Profil pro
    Inscrit en
    Juin 2003
    Messages
    2 474
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2003
    Messages : 2 474
    Par défaut
    Merci pour vos reponses ....
    J'aurais justement voulu eviter les doubles blocs
    Je trouve que le java deviens trop vite illisible .....

    M'enfin bon on fera avec.
    Quand aux closures a mon avis c'est pas pour tout de suite. deja que la ou je bosse ils en sont encore a la 1.4 .....

  5. #5
    Membre Expert
    Avatar de gifffftane
    Profil pro
    Inscrit en
    Février 2007
    Messages
    2 354
    Détails du profil
    Informations personnelles :
    Localisation : France, Loire (Rhône Alpes)

    Informations forums :
    Inscription : Février 2007
    Messages : 2 354
    Par défaut
    Je suis assez d'accord, cela conduit à un code assez lourd ; il me semble que le close aurait dû être protégé par une RuntimeException.

    Moi, ma petite combine consiste à tout englober dans une méthode qui rejette les IOExceptions, tout en faisant le close dans un finally. Ainsi :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
     
    void manipulation() throws IOException
    {
     try
     {
      jeManipule();
     }
     finally
     {
      io.close();
     }
    }
    Mes 2 centimes.

  6. #6
    Membre Expert
    Avatar de Patriarch24
    Homme Profil pro
    Ingénieur développement logiciels
    Inscrit en
    Septembre 2003
    Messages
    1 047
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 42
    Localisation : France

    Informations professionnelles :
    Activité : Ingénieur développement logiciels
    Secteur : Industrie

    Informations forums :
    Inscription : Septembre 2003
    Messages : 1 047
    Par défaut
    En utilisant l'API commons io d'apache, on écrit ce code :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    InputStream in = null;
    try {
        in = new BufferedInputStream(new FileInputStream(f));
        // utilise in
    } catch (IOException e) {
        log.error(e);
    } finally {
        IOUtils.closeQuietly(in); // ferme in sans exception !
    }

  7. #7
    Expert éminent
    Avatar de adiGuba
    Homme Profil pro
    Développeur Java/Web
    Inscrit en
    Avril 2002
    Messages
    13 938
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Développeur Java/Web
    Secteur : Transports

    Informations forums :
    Inscription : Avril 2002
    Messages : 13 938
    Billets dans le blog
    1
    Par défaut
    Citation Envoyé par Patriarch24 Voir le message
    En utilisant l'API commons io d'apache, le code devient :
    Ben perso je trouve cela nettement moins propre : on ignore carrément une exception !

    Et surtout avec un flux bufférisé comme ici !
    les opérations d'écriture peuvent être effectuées réellement lors du close(), donc en cas de problème tu passerais carrément à coté !


    a++

Discussions similaires

  1. Fermeture Excel fichier sur serveur [VB6]
    Par ellimac60 dans le forum Excel
    Réponses: 2
    Dernier message: 19/07/2013, 10h00
  2. Fermeture du fichier Access sur ouverture requete
    Par jj4822 dans le forum Access
    Réponses: 8
    Dernier message: 05/06/2009, 21h50
  3. Gestion des permissions des fichiers sur NTFS
    Par gnu_thomas dans le forum MFC
    Réponses: 3
    Dernier message: 22/01/2004, 08h22
  4. [VB6] ouvrir fichier sur glisser déplacer
    Par bhaal76 dans le forum VB 6 et antérieur
    Réponses: 6
    Dernier message: 15/04/2003, 17h04
  5. [] [Réseau] Transfert fichier sur un FTP
    Par CYFL dans le forum VB 6 et antérieur
    Réponses: 5
    Dernier message: 26/02/2003, 17h33

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo