lettura e scrittura eccezione

[frakkio11299]

Questo è il mio codice:

import java.io.*;public class main1 {
	public static void main (String[]args){
		File file1=new File("prova.txt");
		BufferedWriter scrittore=null;
		try{
			if(file1.createNewFile()){
				scrittore=new BufferedWriter(new FileWriter(file1));
				scrittore.write("ciao");
				scrittore.newLine();
				scrittore.write("a tutti");
			}
			
			
//----------------------------------------------------------------------------\\			
			
			BufferedReader lettore=null;
			String frase=null;
			file1=new File("prova.txt");
			try{
				lettore=new BufferedReader(new FileReader(file1));
				while((frase=lettore.readLine())!=null){
					System.out.println(frase);
				}
			}
			catch(IOException e){}
			finally{lettore.close();}
		}
		catch(IOException e){}
		finally{try{scrittore.close();}catch(IOException e){e.printStackTrace();}}


	}
}

questo è l'errore che si genera

ciaoException in thread "main" a tuttijava.lang.NullPointerException	at main1.main(main1.java:30)

Avete delle possibili soluzioni?

[LeleFT]

1) Prima di poter aprire il file in lettura e quindi leggerlo è necessario chiuderlo, di modo che i dati vengano scritti su disco (un flush() prima della chiusura non fa male ed è consigliabile in situazioni dove viene usata la bufferizzazione).

2) Le eccezioni non vanno ammazzate: stampane sempre lo stackTrace:

} catch (Exception e) {
   e.printStackTrace();
}

Ciao.

[frakkio11299]

1) Prima di poter aprire il file in lettura e quindi leggerlo è necessario chiuderlo, di modo che i dati vengano scritti su disco (un flush() prima della chiusura non fa male ed è consigliabile in situazioni dove viene usata la bufferizzazione).

2) Le eccezioni non vanno ammazzate: stampane sempre lo stackTrace:

} catch (Exception e) {
   e.printStackTrace();
}

Ciao.

Ciao Grazie della risposta,
il file viene chiuso e scritto nel finally per sicurezza ho provato a metterlo anche prima, il flush() lo ho provato la stampa dello stack la avevo provata per vedere se mi dava delle info in piu. il problema persiste anche con le soluzioni da te proposte.

[andbin]

Oltre a quanto correttamente già detto da LeleFT, aggiungo che non è completamente corretto fare quel:

finally{lettore.close();}

Infatti il costruttore di FileReader può fallire con eccezione e in tal caso lettore resta a null e quindi nel finally ottieni un "bel" NullPointerException.

Pertanto o nel finally fai un test per vedere se lettore non è null ... oppure giri diversamente il try-catch-finally così:

BufferedReader lettore=new BufferedReader(new FileReader(file1));

try {
    // .....
} catch ( ......... ) {
    // ....
} finally {
    lettore.close();
}

Questo tra l'altro puoi farlo proprio perché hai un ulteriore try-catch più "a monte"

[LeleFT]

No, nel tuo codice il file viene chiuso DOPO che hai già cercato di leggerlo: il primo blocco try/catch racchiude al suo interno anche la lettura.

Prova a vedere questo:

import java.io.*;

public class Main1 {
    public static void main (String[]args){
        File file1 = new File("prova.txt");
        boolean leggi = false;
        FileWriter fw = null;
        BufferedWriter scrittore = null;
        try{
            fw = new FileWriter( file1 );
            scrittore = new BufferedWriter( fw );
            scrittore.write("ciao");
            scrittore.newLine();
            scrittore.write("a tutti");
            scrittore.flush();
            leggi = true;   // Indico al programma che la scrittura è andata a buon fine
        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            // Qui viene fatta la chiusura di tutti gli oggetti di scrittura
            if (scrittore != null) {
                try { scrittore.close(); } catch (Exception e) { /* Inutile stampare lo stackTrace qui */ }
            } else {
                if (fw != null) {
                   try { fw.close(); } catch (Exception e) { }
                }
            }
        }

        if ( leggi ) {   
           // Posso leggere perchè la scrittura è andata a buon fine
           FileReader fr = null;
           BufferedReader lettore = null;
           String frase=null;
           try {
               fr = new FileReader( file1 );
               lettore = new BufferedReader( fr );
               while((frase=lettore.readLine()) != null) {
                   System.out.println( frase );
               }
           } catch(IOException e){
               e.printStackTrace();
           } finally {
               if (lettore != null) {
                   try { lettore.close(); } catch (Exception e) { }
               } else {
                   if (fr != null) {
                       try { fr.close(); } catch (Exception e) { }
                   }
               }
           }
        }
    }
}

Ciao.

[frakkio11299]

Oltre a quanto correttamente già detto da LeleFT, aggiungo che non è completamente corretto fare quel:

finally{lettore.close();}

Infatti il costruttore di FileReader può fallire con eccezione e in tal caso lettore resta a null e quindi nel finally ottieni un "bel" NullPointerException.

Pertanto o nel finally fai un test per vedere se lettore non è null ... oppure giri diversamente il try-catch-finally così:

BufferedReader lettore=new BufferedReader(new FileReader(file1));

try {
    // .....
} catch ( ......... ) {
    // ....
} finally {
    lettore.close();
}

Questo tra l'altro puoi farlo proprio perché hai un ulteriore try-catch più "a monte"

grazie, della risposta infatti ho messo il .close() in un altro try ho trovato l'errore adesso mando il codice corretto

[frakkio11299]

No, nel tuo codice il file viene chiuso DOPO che hai già cercato di leggerlo: il primo blocco try/catch racchiude al suo interno anche la lettura.

Prova a vedere questo:

import java.io.*;

public class Main1 {
    public static void main (String[]args){
        File file1 = new File("prova.txt");
        boolean leggi = false;
        FileWriter fw = null;
        BufferedWriter scrittore = null;
        try{
            fw = new FileWriter( file1 );
            scrittore = new BufferedWriter( fw );
            scrittore.write("ciao");
            scrittore.newLine();
            scrittore.write("a tutti");
            scrittore.flush();
            leggi = true;   // Indico al programma che la scrittura è andata a buon fine
        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            // Qui viene fatta la chiusura di tutti gli oggetti di scrittura
            if (scrittore != null) {
                try { scrittore.close(); } catch (Exception e) { /* Inutile stampare lo stackTrace qui */ }
            } else {
                if (fw != null) {
                   try { fw.close(); } catch (Exception e) { }
                }
            }
        }

        if ( leggi ) {   
           // Posso leggere perchè la scrittura è andata a buon fine
           FileReader fr = null;
           BufferedReader lettore = null;
           String frase=null;
           try {
               fr = new FileReader( file1 );
               lettore = new BufferedReader( fr );
               while((frase=lettore.readLine()) != null) {
                   System.out.println( frase );
               }
           } catch(IOException e){
               e.printStackTrace();
           } finally {
               if (lettore != null) {
                   try { lettore.close(); } catch (Exception e) { }
               } else {
                   if (fr != null) {
                       try { fr.close(); } catch (Exception e) { }
                   }
               }
           }
        }
    }
}

Ciao.

ciao grazie della risposta questa è la soluzione che ho applicato e sembra funzionare

import java.io.*;public class main1 {
	public static void main (String[]args){
		File file1=new File("prova.txt");
		BufferedWriter scrittore=null;
		try{
			if(file1.createNewFile()){
				scrittore=new BufferedWriter(new FileWriter(file1));
				scrittore.write("ciao");
				scrittore.newLine();
				scrittore.write("a tutti");
				scrittore.close();
			}
			
			
//----------------------------------------------------------------------------\\			
			
			BufferedReader lettore=null;
			String frase=null;
			file1=new File("prova.txt");
			try{
				lettore=new BufferedReader(new FileReader(file1));
				while((frase=lettore.readLine())!=null){
					System.out.println(frase);
				}
			}
			catch(IOException e){e.printStackTrace();}
			finally{lettore.close();}
		}
		catch(IOException e){e.printStackTrace();}
		


	}
}