[JAVA] Domanda Exceptions

[mark2x]

Col seguente codice leggo il contenuto di un file (in una lista).
Voglio che ogni errore sia gestito dal metodo chiamante (e quindi non qui).

package mypackage.begin;

import java.io.*;
import java.util.*;

public class ExceptionClass2 
    {
    public void myMethod2() throws Exception
        {   	
        List<String> lines = new ArrayList<String>();
        BufferedReader in = new BufferedReader(new FileReader("C:/my-file.txt"));            
        String str;
        while ((str = in.readLine())!=null) lines.add(str);          
        in.close();
        }
    }

Non capisco però una cosa: se nel bel mezzo della lettura "succede qualcosa" che la interrompe, la riga in.close() rimane non eseguita.
Il file rimane aperto?

[Romeo25]

Si il file rimane aperto dovresti fare il try e catch chiudere il file e lanciare l'eccezione nel catch

[mark2x]

Ti ringrazio.
Quindi nel caso di apertura risorse devo usare la seguente logica?

package mypackage.begin;

import java.io.*;
import java.util.*;

public class ExceptionClass3 
    {
    public void myMethod3() throws Exception
        {
    	BufferedReader in = null;    	
    	try
    	    {
            List<String> lines = new ArrayList<String>();
            in = new BufferedReader(new FileReader("C:/my-file.txt"));            
            String str;
            while ((str = in.readLine())!=null) lines.add(str);
    	    }
    	
    	catch(Exception e) 
    	    {
    	    // Rilancio l'eccezione (che verrà gestita dal chiamante).
    	    throw new Exception();    		
    	    }
    	
    	finally
    	    {
            if (in!=null) in.close();            
            }
        }
    }

E' ciò che si fa comunemente? E' una best practice?
Grazie.

[Romeo25]

si cosi è perfetto hai sempre la sicurezza che le risorse vengano chiuse

[mark2x]

Grazie ancora.

[floyd]

    public void myMethod3() throws Exception
        {
    	BufferedReader in = null;    	
    	try
    	    {
            List<String> lines = new ArrayList<String>();
            in = new BufferedReader(new FileReader("C:/my-file.txt"));            
            String str;
            while ((str = in.readLine())!=null) lines.add(str);
    	    }
    	finally
    	    {
            if (in!=null) in.close();            
            }
        }

meglio così

[mark2x]

Così viene rilanciata automaticamente solo l'eccezione corretta?

Tnx!

[floyd]

certo e poi il codice è più leggibile

[feua]

Puoi anche farti una classe exception personalizzata tipo

public class MyException extends Exception

Dove poi ci metti dentro tutti i metodi e le info che vuoi...

oppure, più figo ancora, puoi farti una classe personalizzata di eccezioni NON CONTROLLATE, per le quali non è obbligatorio il try-catch

public class MyException extends RuntimeException

[mark2x]

Tra qualche mese, quando sarò più "compatibile" con Java...