Leggere un file di testo una riga alla volta

[luca93abc]

Buongiorno a tutti! Molto probabilmente la risposta alla mia domanda è scontata, ma qui io ci sto perdendo la mattina... Devo leggere un file una riga alla volta, stando fuori dalla funzione main. Ho provato in diversi modi ma ottengo sempre due letture della prima riga (penso per il principio che una chiamata ad un metodo da oggetti differenti con gli stessi parametri da risultati uguali). Il mio codice è questo:

public class GestioneFile implements IFilePartite {    private String path;


    public void setPath(String path) {
        this.path = path;
    }


    public String leggi() throws IOException{
        BufferedReader br = new BufferedReader(new FileReader(new File(path)));
        String thisLine = br.readLine();
        StringBuffer sb = new StringBuffer();
        sb.append(thisLine);
        return sb.toString();
    }
}

Poi da una classe di test ho il seguente codice:

public class TestLettura01 {    
    public static void main(String[] args) throws IOException {
        GestioneFile gf = new GestioneFile();
        gf.setPath(IFilePartite.pathCalcio);
        System.out.println(gf.leggi());
        GestioneFile gf1 = new GestioneFile();
        gf1.setPath(IFilePartite.pathCalcio);
        System.out.println(gf1.leggi());


    }
}

Il fine di tutto ciò sarebbe un'interfaccia grafica che ad ogni click mi da in output una riga del mio file di testo, ma come potete vedere sono ancora in alto mare. Mi sapreste aiutare?

[LeleFT]

Java ha una sezione apposita del forum.

Sposto.


Ciao.

[LeleFT]

Con quel tuo codice stai creando 2 istanze della classe GestioneFile.
Ciascuna avrà il suo stato (che non è condiviso con le altre per il principio di incapsulamento).
Di conseguenza, passando a ciascuna istanza la stringa del percorso, ciascuna istanza andrà a creare un BufferedReader diverso, e ciascuno di questi BufferedReader sarà posizionato all'inizio del file. Quindi, la chiamata al metodo leggi() non farà altro che prendere la riga "attualmente puntata", leggerla e portare avanti il puntatore.
Essendo due istanze diverse, con ciascuno un proprio stato, ciascuna sarà posizionata all'inizio del file, quindi ciascuna produrrà solo la prima riga del file.

Se vuoi che ad ogni chiamata venga letta una nuova riga dal file, allora crea UNA sola istanza della classe GestioneFile e solo su quella chiameri più volte il metodo leggi().


Ciao.

[andbin]

Devo leggere un file una riga alla volta, stando fuori dalla funzione main. Ho provato in diversi modi ma ottengo sempre due letture della prima riga

Semplicemente perché riapri il file (quindi ricrei FileReader/BufferedReader) ad ogni invocazione di leggi(). Visto che hai creato una classe apposita (GestioneFile) che incapsula questa logica, ti basta tenere come "stato" dell'oggetto il BufferedReader.
E più invocazioni di leggi() sullo stesso oggetto GestioneFile useranno lo stesso BufferedReader.

[LeleFT]

Semplicemente perché riapri il file (quindi ricrei FileReader/BufferedReader) ad ogni invocazione di leggi().

Azz, avevo visto il metodo leggi() e il metodo setPath() "al contrario". Avevo visto una sola riga di codice e davo per scontato che fosse un banale br.readLine(), quindi l'avevo associato al metodo leggi... c'è poco da fare: ultimamente leggo troppo velocemente.


Ciao.

[luca93abc]

Quindi dovrei creare un oggetto solo e su di esso chiamare più volte leggi()?

[andbin]

Quindi dovrei creare un oggetto solo e su di esso chiamare più volte leggi()?

Si ma devi anche fare in modo che l'oggetto GestioneFile mantenga il BufferedReader.

[luca93abc]

Quindi lo devo mettere nel costruttore?

[andbin]

Quindi lo devo mettere nel costruttore?

La costruzione del BufferedReader sì ma poi devi anche tenerlo come variabile di istanza .. non certo locale al costruttore!