[Java] Problema in lettura di un file di testo

[banryu79]

Originariamente inviato da ally

...si è una soluzione...io sinceramente scorrerei ogni linea del file di testo e se rispetta le clausole di appartenenza ("administrator") la scriverei direttamente sul file di destinazione senza passare per una lista...anche se questo sistema puo' creare piu' problemi nella gestione di eccezioni...

...ciao Andrea...

Sì, sono d'accordo.
Io preferisco spezzare l'operazione e avere la copia dei dati sottomano tra le due operazioni; poi uno valuta che approccio usare anche a seconda della situazione.

[Xizor]

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

class Log {
public static void main (String args[]) {

int i=0;														//indice dell'array di stringhe																
String s=null;			
List inputFile = new ArrayList();
						
try {
FileReader f = new FileReader("protezione2.txt");				//apro il file di input in lettura
BufferedReader fIN = new BufferedReader(f);

String line = fIN.readLine();

while(line != null) {
    inputFile.add(line);
    line = fIN.readLine();
}

Iterator iterator = inputFile.iterator();
while (iterator.hasNext()){
    String line = iterator.next();
    if (line.indexOf("administrator") != -1)
        iterator.remove();
}

}
catch(IOException e) {							
System.out.println("Si e' verificata un'eccezione "+e.getMessage());
}

}}

[ally]

Originariamente inviato da banryu79

Sì, sono d'accordo.
Io preferisco spezzare l'operazione e avere la copia dei dati sottomano tra le due operazioni; poi uno valuta che approccio usare anche a seconda della situazione.

...assolutamente...spezzare la cosa in metodi diversi sarebbe cosa gradita...dipende appunto dalle situazioni e dalla complessità...

...ciao Andrea...

[Xizor]

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

class Log {
public static void main (String args[]) {

int i=0;																												
String s=null;			
List inputFile = new ArrayList();
						
try {
FileReader f = new FileReader("protezione2.txt");				
BufferedReader fIN = new BufferedReader(f);

String line = fIN.readLine();

while(line != null) {
    inputFile.add(line);
    line = fIN.readLine();
}

Iterator iterator = inputFile.iterator();
while (iterator.hasNext()){
		line = (String) iterator.next();
    if (line.indexOf("administrator") != -1)
        iterator.remove();
}

}
catch(IOException e) {							
System.out.println("Si e' verificata un'eccezione "+e.getMessage());
}

}}

Ecco, va bene così?

[ally]

...se funziona si...

...ciao Andrea...

[Xizor]

Si, non ci sono errori in compilazione.
Il problema è che non so da dove ricavare l'output, le linee dove sono salvate?

[ally]

Originariamente inviato da Xizor

Si, non ci sono errori in compilazione.
Il problema è che non so da dove ricavare l'output, le linee dove sono salvate?

...le linee corrette dovrebbero essere all'interno dell'arrayList...sempre che la condizione usata per eliminare le stringhe errate non usi una logica sbagliata...e sempre che le linee eliminate siano state tolte dalla lista e non dall'iteratore usato per scorrerle...

...la cosa piu' semplice da fare è quella di scorrere l'array ottenuto e stampare a video ogni singola stringa contenuta...

...ciao Andrea...

[Xizor]

Originariamente inviato da ally

...le linee corrette dovrebbero essere all'interno dell'arrayList...sempre che la condizione usata per eliminare le stringhe errate non usi una logica sbagliata...e sempre che le linee eliminate siano state tolte dalla lista e non dall'iteratore usato per scorrerle...

...la cosa piu' semplice da fare è quella di scorrere l'array ottenuto e stampare a video ogni singola stringa contenuta...

...ciao Andrea...

String line = fIN.readLine();

while(line != null) {
    inputFile.add(line);
    line = fIN.readLine();
}

Iterator iterator = inputFile.iterator();
while (iterator.hasNext()){
		line = (String) iterator.next();
    if (line.indexOf("administrator") != -1)
        iterator.remove();
}

Facendo iterator.remove() mi sa che le cancello dall'iteratore ma non sono sicuro...Come posso scorrere ArrayList dato che non è un array dotato di indici?

[ally]

...basta un ciclo for...

for(int i=0;i<inputFile.size();i++)
                System.out.println(inputFile.get(i));

...ciao Andrea...

[Xizor]

In output escono tutte le righe

[ally]

...le linee corrette dovrebbero essere all'interno dell'arrayList...sempre che la condizione usata per eliminare le stringhe errate non usi una logica sbagliata...e sempre che le linee eliminate siano state tolte dalla lista e non dall'iteratore usato per scorrerle...

...ciao Andrea...

[Xizor]

QUOTE=ally;28128253]...le linee corrette dovrebbero essere all'interno dell'arrayList...sempre che la condizione usata per eliminare le stringhe errate non usi una logica sbagliata...e sempre che le linee eliminate siano state tolte dalla lista e non dall'iteratore usato per scorrerle...

...ciao Andrea...[/quote]

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

class Log {
public static void main (String args[]) {
																											
String s=null;			
List inputFile = new ArrayList();
						
try {
FileReader f = new FileReader("protezione.txt");				
BufferedReader fIN = new BufferedReader(f);

String line = fIN.readLine();

while(line != null) {
    inputFile.add(line);
    line = fIN.readLine();
}

Iterator iterator = inputFile.iterator();
while (iterator.hasNext()){
		line = (String) iterator.next();
    if (line.indexOf("administrator") != -1)
        iterator.remove();								//inputFile.remove(line)
}

for(int i=0;i<inputFile.size();i++)
System.out.println(inputFile.get(i));

}
catch(IOException e) {							
System.out.println("Si e' verificata un'eccezione "+e.getMessage());
}

}}

inputFile.remove(line) dovrebbe cancellarle dalla lista e non dall'iteratore giusto?

[banryu79]

Originariamente inviato da Xizor

inputFile.remove(line) dovrebbe cancellarle dalla lista e non dall'iteratore giusto?

Non ha nessun senso fare una distinzione concettuale tra "le righe dell'iteratore" e "le righe della lista" poichè l'iteratore è restituito dalla lista e opera sulla lista stessa.

Il metodo remove dell'iteratore rimuove dalla collezione sottostante (cioè quella che ha fornito l'iteratore stesso) l'ultimo elemento che è stato restituito dal metodo next dell'iteratore.

Se hai dubbi su qualche metodo consulta i javadoc

Il problema è un'altro: tu nel tuo codice fai tutto tranne che alla fine scrivere la lista filtrata sul file di output...

[banryu79]

Se comunque alla fine ti stampa tutte le righe vuol dire che c'è un problema con il controllo della condizione per la rimozione.

questa dovrebbe essere giusta:

Iterator iterator = inputFile.iterator();
while (iterator.hasNext())
{
    line = (String) iterator.next();
    if (line.indexOf("Administrator") == -1)
        iterator.remove();							
}

Rimuovo la riga solo SE NON trovo la stringa "Administrator".

[Xizor]

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

class Log {
public static void main (String args[]) {
																											
String s=null;			
List inputFile = new ArrayList();
						
try {
FileReader f = new FileReader("protezione.txt");				
BufferedReader fIN = new BufferedReader(f);

String line = fIN.readLine();

while(line != null) {
    inputFile.add(line);
    line = fIN.readLine();
}

Iterator iterator = inputFile.iterator();
while (iterator.hasNext())
{
    line = (String) iterator.next();
    if (line.indexOf("Administrator") == -1)
        iterator.remove();							
}

for(int i=0;i<inputFile.size();i++)
System.out.println(inputFile.get(i));

}
catch(IOException e) {							
System.out.println("Si e' verificata un'eccezione "+e.getMessage());
}

}}

In esecuzione dice :

Exception in thread "main" java.lang OutOfMemoryError: Java heap space

cos'è?

[banryu79]

Originariamente inviato da Xizor

In esecuzione dice :

Exception in thread "main" java.lang OutOfMemoryError: Java heap space

cos'è?

E' un messaggio di errore della JVM; significa che la memoria allocata dal SO alla JVM è terminata (!).

Non so come mai: ma il file protezione.txt quanto grande è?

Comunque ho provato il codice che hai costruito fino a qui e postato con un file di input così fatto:

Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,576,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,576,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.29,Security,Accesso/fine sess. ,538,MAGA02$,SERVER
Operazioni riuscite,06/07/2009,12.07.28,Security,Accesso/fine sess. ,540,FAR15$,SERVER
Operazioni riuscite,06/07/2009,12.07.28,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.28,Security,Accesso/fine sess. ,576,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.24,Security,Accesso/fine sess. ,540,MAGA02$,SERVER
Operazioni riuscite,06/07/2009,12.07.20,Security,Accesso/fine sess. ,538,cbibiani,SERVER

e ottengo (correttamente) questo output:

Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,576,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.30,Security,Accesso/fine sess. ,576,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.28,Security,Accesso/fine sess. ,540,Administrator,SERVER
Operazioni riuscite,06/07/2009,12.07.28,Security,Accesso/fine sess. ,576,Administrator,SERVER

[Xizor]

Grazie 1000

Allora quelle righe sono soltanto una piccolissima parte del file "protezione.txt" , le ho prese come esempio.
Compilando il codice per queste poche righe in effetti funziona correttamente come testato da te.

Quando ho provato ad eseguirlo sul file intero mi ha dato che messaggio di errore. Il file protezione intero è grande 32 MB

Se esiste un metodo per evitare quel problema bene, altrimenti bene lo stesso perchè l'algoritrmo funziona. Sapevo che Java aveva una quantità impressionante di classi e metodi, infatti questi non li conoscevo. Grazie infinite lo stesso

[banryu79]

Originariamente inviato da Xizor

Se esiste un metodo per evitare quel problema bene, altrimenti bene lo stesso perchè l'algoritrmo funziona.

Certo che esiste: basta istruire la JVM passandole un parametro apposito che ne fissa esplicitamente la memoria massima che può richiedere al sistema operativo.
Io con ste cose non ci azzecco: non mi ricordo mai di default quant'è: a te basterebbe specificare l'ordine di grandezza successivo (es: se è 32M ne chiedi 64M, se è 64M ne chiedi 128M e così via).

Originariamente inviato da Xizor

Sapevo che Java aveva una quantità impressionante di classi e metodi, infatti questi non li conoscevo. Grazie infinite lo stesso

Beh, oserei dire che usare Java per programmare senza avvalersi del JDK ha poco senso...

Trovi tutto online (con la possibilità di scaricarti in locale la documentazione che ti interessa); due sono le risorse più utili, secondo me:
1) i javadoc del JDK
2) Quella cosa chiamata "The Really Big Index" (compendio di tutorial della Sun su quasi tutto quello che vorresti sapere del linguaggio Java e del JDK).

[ally]

Originariamente inviato da banryu79

Non ha nessun senso fare una distinzione concettuale tra "le righe dell'iteratore" e "le righe della lista" poichè l'iteratore è restituito dalla lista e opera sulla lista stessa.

Il metodo remove dell'iteratore rimuove dalla collezione sottostante (cioè quella che ha fornito l'iteratore stesso) l'ultimo elemento che è stato restituito dal metodo next dell'iteratore.

Se hai dubbi su qualche metodo consulta i javadoc

Il problema è un'altro: tu nel tuo codice fai tutto tranne che alla fine scrivere la lista filtrata sul file di output...

...mio grosso errore...grazie per il chiarimento...

...ciao Andrea...