Programma funzionante ma...

[Darèios89]

Ho fatto un esercizio, sembrerebbe giusto, poichè l'output mi risulta corretto:
Mi genera una NullPointerException, vorrei capire perchè, anche se potrei risolvere gestendola, ma vorrei capire se è dovuta ad un errore nel mio programma.

"Si realizzi un programma Java che preso in input un file contenente una sequenza
di stringhe, una per riga, verifichi se tali stringhe siano palindrome o meno.
Utilizzare una funzione ricorsiva per verificare che le stringhe siano palindrome.
Input
Il file di input contiene una sequenza di stringhe, una per riga, terminanti con il
simbolo “ ;” (punto e virgola). Ciascuna stringa ha una lunghezza massima di 10
caratteri ed è composta solo da caratteri minuscoli.
Output
Il file di output contiene, in corrispondenza delle righe contenenti stringhe
palindrome, la parola “ palindroma” . In corrispondenza di tutte le altre righe
contiene il simbolo “ -” ."

Ecco il codice:

import java.io.*;

public class Esercizio2
{
	public static void main(String [] args) throws Exception
	{
		
		FileReader reader=null;
		BufferedReader buff=null;
		PrintWriter out=null;
		
		try{
			reader=new FileReader("input.txt");
			buff=new BufferedReader(reader);
			out=new PrintWriter("OUTPUT.txt");
			
			String s="";
			String inverse="";
			String output="";
			
			while(buff!=null)
			{
				s=buff.readLine();
				int i=s.length()-2;    //Siccome le stringhe sono del tipo "aab;" non mi interessa la virgola
				inverse=inverti(s,output,i);
				if(inverse.equals(s.substring(0,s.length()-1))) //Confronto l'inversa con la stringa di partenza privata del ; finale
					out.print("palindroma");
				else out.print('-');
				out.print('\r');
				out.print('\n');
			}
		
		    
		    }
		    

		    catch(IOException e)
		    {
		    }
		    
		      finally{
			    if(reader!=null)
				    reader.close();
			    if(buff!=null)
				    buff.close();
			    if(out!=null)
				    out.close();
				}
		    
		    
		  
		    
	}
	
	
	public static String inverti(String s, String output, int i)
	{
		if(i<0)return output;
		
		else if(i>=0){
			output+=s.charAt(i);
		i--;}
		return inverti(s,output,i);
	}
}

[andbin]

Originariamente inviato da Darèios89

			buff=new BufferedReader(reader);

			
			while(buff!=null)
			{
				s=buff.readLine();
				int i=s.length()-2;    //Siccome le stringhe sono del tipo "aab;" non mi interessa la virgola

Per il ciclo non devi testare che buff sia diverso da null ..... perché è sempre diverso da null visto che hai istanziato e assegnato il BufferedReader appena sopra!!
Quindi stai facendo un ciclo teoricamente "infinito" perché la condizione è sempre vera. Quando non ci sono più righe (fine del file) il readLine() restituisce null. Il fatto è che poi invochi s.length() e su un null ovviamente lancia NullPointerException (quindi dal ciclo si esce artificiosamente per questo motivo).

Per la lettura "a righe" con un BufferedReader l'idioma tipico è:

String linea;

while ((linea = buff.readLine()) != null) {
    ..... usa linea ......
}

Punto.


Altra cosa, meno grave, ma da segnalarti. Quando hai classi di I/O "wrappate" in altre come:

reader=new FileReader("input.txt");
buff=new BufferedReader(reader);

Quello che devi chiudere è sempre solo quello più "esterno", ovvero se hai il BufferedReader allora questo, altrimenti se BufferedReader non fosse stato costruito (per causa di qualunque eccezione), allora il FileReader.

Pertanto invece di fare:

if(reader!=null)
    reader.close();
if(buff!=null)
    buff.close();

Dovresti come minimo fare:

if (buff != null) {
    buff.close();
} else if (reader != null) {
    reader.close();
}

Insomma, quello che hai di più "esterno" in quel momento lo chiudi e basta. Non tutti quelli nella "catena" di I/O.

[Darèios89]

Ah ok ok, ti ringrazio.