Comment puis-je fermer un fichier après l'interception d'une IOException en java?

 BufferedReader fileIn = null;
 try {
       fileIn = new BufferedReader(new FileReader(filename));
       //etc.
 } catch(IOException e) {
      fileArrayList.removeall(fileArrayList);
 } finally {
     try {
       if (fileIn != null) fileIn.close();
     } catch (IOException io) {
        //log exception here
     }
 }
 return fileArrayList;

Peu de choses sur le code ci-dessus:

• fermer doivent être dans une enfin, sinon il ne sera pas fermé à la fin du code normalement, ou si une autre exception est levée par ailleurs IOException.
• Généralement vous avez un statique d'utilité méthode pour fermer une ressource comme ça pour qu'il vérifie la valeur null et attrape toutes les exceptions (dont vous ne voulez jamais faire quoi que ce soit d'autre que le journal dans ce contexte).
• Le retour appartient après l'essai, de sorte que la principale ligne de code et l'interception des exceptions avoir un retour de la méthode sans redondance.
• Si vous mettez le retour à l'intérieur de la enfin, il permettrait de générer un avertissement du compilateur.

OriginalL'auteur Yishai

Il se plaint de la symbole n'étant pas là, car il n'est pas. C'est dans le bloc try. Si vous voulez faire référence à fileIn, vous aurez besoin de la déclarer en dehors de l'essayer.

Cependant, il sonne vraiment comme vous voulez placer le fermer dans un bloc finally plutôt: vous devez fermer le fichier indépendamment de la réussite ou de l'échec avant de revenir.

public static ArrayList readFiletoArrayList(String fileName, ArrayList fileArrayList)
{
    fileArrayList.removeAll(fileArrayList);

    BufferedReader fileIn = null;
    try {
        //open the file for reading
        fileIn = new BufferedReader(new FileReader(fileName));

        //add line by line to array list, until end of file is reached
        //when buffered reader returns null (todo). 
        while(true){
                fileArrayList.add(fileIn.readLine());
            }
    }catch(IOException e){
        fileArrayList.removeAll(fileArrayList); 
    }finally{
        if(fileIn != null) fileIn.close();
    }
    return fileArrayList;
}

OriginalL'auteur Yuliy

Déplacer la déclaration du bloc try:

public static ArrayList readFiletoArrayList(String fileName, ArrayList fileArrayList)
{
    fileArrayList.removeAll(fileArrayList);

    BufferedReader fileIn = null;
    try {
        //open the file for reading
        fileIn = new BufferedReader(new FileReader(fileName));

        //add line by line to array list, until end of file is reached
        //when buffered reader returns null (todo). 
        while(true){
                fileArrayList.add(fileIn.readLine());
            }
    }catch(IOException e){
        fileArrayList.removeAll(fileArrayList);
        fileIn.close(); 
        return fileArrayList; //returned empty. Dealt with in calling code. 
    }
}

Mais, vous aurez toujours besoin de faire attention à ce que fileIn a effectivement été initialisé avant d'essayer de le fermer:

if (fileIn != null)
    fileIn.close();

OriginalL'auteur Kaleb Pederson

Il est préférable de ne pas traiter avec null - le général idiome pour l'acquisition de ressources et de la libération en Java :

final Resource resource = acquire();
try { use(resource); }
finally { resource.release(); }

donc:

public static List<String> readFiletoArrayList(String fileName,
        List<String> fileArrayList, String charsetName) {
    fileArrayList.clear(); //why fileArrayList.removeAll(fileArrayList) ?
    try {
        InputStream file = new FileInputStream(fileName);
        try {
            InputStreamReader reader = new InputStreamReader(file, charsetName);
            BufferedReader buffer = new BufferedReader(reader);
            for (String line = buffer.readLine(); line != null; line = buffer
                    .readLine()) {
                fileArrayList.add(line);
            }
        } finally {
            try {
                file.close();
            } catch (IOException e) {
                e.printStackTrace(); //you do not want this to hide an
                //exception thrown earlier so swallow it
            }
        }
    } catch (IOException e) {
        fileArrayList.clear(); //returned empty. Dealt with in client
    }
    return fileArrayList;
}

Voir mes points ici

Si vous utilisez un lecteur, vous doit spécifier l'encodage comme je le fais ici. Si vous voulez lire des octets oublier le lecteur. Aussi, si vous utilisez readLine() vous devez oublier les caractères de fin de ligne - si c'est un souci de considérer l'omission de la BufferedReader entièrement.

OriginalL'auteur Mr_and_Mrs_D