Auslesen - Anzeigen - Schreiben

[Mr_Klobuerste]

Hallöchen,

ich muss leider schon wieder blöde fragen stellen.

Ich habe eine Aufgabenstellung, in der ich aus einem Server Logfile, bestimmte Meldungen raus suchen will, diese vorher auf der Konsole anzeigen lassen möchte und das soeben angezeigte in eine neue Datei schreiben möchte. Zuvor wird der Benutzer gefragt welche daten er auslesen will. Zb [INFO], [SERVER], [WARNING] ect.

Der Ablauf klappt auch, er liest die daten aus, fragt nach den Meldungen, zeigt alles wieder auf der Konsole an und speichert alles in einer txt.

Aaaaaaaaber.... ich hab große Probleme mit den Zeilenumbrüchen, ich bekomm es einfach nicht hin, das er nach jeder Zeile einen bracke macht und mit einer neuen Zeile anfängt. Ich hab es schon mit verschiedenen Methoden probiert die ich im Internet gefunden habt, nur bekomme ich es nicht hin.

Vielleicht würde sich mal jemand den Code angucken. Und wenn jemand was entdeckt einen kleinen Denkanstoß geben.

Dankeschön

package datenauslesenundspeichern;
import java.io.*;

/**
 *
 * @author Mr_Klobuerste
 */
public class DatenAuslesenUndSpeichern {
 
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
 
    try {
 
         BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
 
         System.out.println("*********************");
         System.out.println("***  Datenzugriff ***");
         System.out.println("*********************");
 
         System.out.println("Nach Welchen Werten soll gesucht werden? (SEVERE | WARNING | Error)");
         String Meldung;
         Meldung = input.readLine();
         

         String logFile = "";
         StringBuilder sb = new StringBuilder();
         String replacement = "\n";
         String regex = "\n";
            BufferedWriter inFile2;
            try (BufferedReader inFile = new BufferedReader (new FileReader ("J:\\server_log2.txt"))) {
                inFile2 = new BufferedWriter (new FileWriter ("J:\\logfile2.txt"));
                logFile = inFile.readLine();
                while (logFile!= null)
                {
                   if (logFile.startsWith (Meldung, 21))
                   {
                   //sb.append(logFile).append("\n");
                   System.out.println(logFile);
                   inFile2.write(logFile);
                   }
                   logFile = inFile.readLine();
                   
                }

            }
         inFile2.close();
         
    }
 
        catch(Exception ex)
         {
  System.out.println("Feheler !!!");
 
  System.out.println(ex.getMessage() );
 
       }
 
        // TODO code application logic here
    }
}

 

[Marcinek]

Problem ist, dass unter Windows \n nicht aussreicht.

Was du suchst ist sowas:

Java: System Independent Newline Characters

 

[Bernd Hohmann]

Nimm statt des [JAPI]BufferedWriter[/JAPI] einen [JAPI]PrintWriter[/JAPI], der hat auch ein .println()

Bernd

 

[Mr_Klobuerste]

Problem ist, dass unter Windows \n nicht aussreicht.

Was du suchst ist sowas:

Java: System Independent Newline Characters

Hab es mir gerade angeschaut, auch ein paar beispiele gefunden. Werde es heute Abend testen wenn ich von Arbeit komme. Danke auf jeden fall für die Schnelle hilfe. Ich glaub das ist auch das was ich gesucht habe.

 

[Mr_Klobuerste]

Ich hab es jetzt mit dem PrintWriter probiert und auch mit dem line.separator... ich bekomme es irgendwie nicht hin. Ich glaub ich hab mich da total verfriemelt :-(

         String logFile;
         StringBuilder sb = new StringBuilder();
         String sep = System.getProperty("line.separator");       
         PrintWriter inFile2;
         FileReader fr = null;
         BufferedReader br = null;
         
            try (BufferedReader inFile = new BufferedReader (new FileReader ("J:\\server_log2.txt"))) 
             {
                inFile2 = new PrintWriter (new FileWriter ("J:\\logfile2.txt"));
                logFile = inFile.readLine();
                while( (logFile=br.readLine()) != null )
                sb.append(logFile+sep) ;
                {
                   if (logFile.startsWith (Meldung, 21))
                   {
                   //sb.append(logFile).append("\n");
                   System.out.println(sb);
                   inFile2.write(logFile);
                   }
                   logFile = inFile.readLine();
                   
                }

            }

 

[SlaterB]

wie wäre es mit bisschen Ordnung?

> try (BufferedReader inFile = new BufferedReader (new FileReader ("J:\\server_log2.txt")))
ich weiß nicht ob du nur tausende Fehlermeldungen verschweigst oder in neuen Java-Versionen alles möglich ist,
aber halte dich doch bitte an einfache bekannte Syntax,
wie ein try/ catch-Block aussieht kann man wohl nachschlagen,

für den Anfang geht es auch ganz ohne try/catch, im Zweifel an alle Methoden 'throws Exception' ranschreiben

----

du liest aus br, aber das ist null, wird nie befüllt?

'inFile' für einen Einlesestream ist noch ganz guter Name, für einen Writer dann aber 'inFile2' ist ja ganz schlimm, wie wärs Richtung outFile?

'logFile' für einen String ist besonders nach den anderen Namen schlecht, was soll das ausdrücken?
es ist kein Stream, es ist kaum ein ganzes File, ..
'line' wäre als Name eine Richtung

-----

                while( (logFile=br.readLine()) != null )
                sb.append(logFile+sep) ;
                {

Syntax, Syntax, Syntax wiederum,
da musst du mit aller höchster Strenge drauf achten, keine wilden Zeilen zusammenbauen,

hier besteht die Schleife nur aus der folgenden Zeile, das { öffnet einen beliebigen nicht verknüpften Block,
wenn das { zur Schleife gehören soll muss es direkt beim while stehen, ganz wie in jedem Lehrbuch locker leicht zu lesen

auf Experimente, eigende Abkürzungen usw. als Anfänger lieber verzichten,
wenn es irgendwo anders steht dann erstmal untersuchen, verstehen, auf normale Syntax umbauen,

im Zweifel auch testen, es ist keine Schande in eine vermutete Schleife erstmal nur System.out.println("Hallo"); reinzuschreiben, anschauen was passiert, ob Wiederholung stattfindet,
die eingelesenen Zeilen auszugeben wäre z.B. ein sinnvollerer Schritt zum Ziel statt hunderte Verarbeitungsaktionen unklar zusammenzukleistern,

lieber wenig Code, aber wissen was man schon hat!
etwas nicht zu verstehen und zu verwenden, evtl. falsch, das ist keine Programmierung

 

[Bernd Hohmann]

Zeile 18 muss .println(...) sein

Bernd

 

[Mr_Klobuerste]

Danke für das Feedback
Dann werde ich wohl erst einmal Ordnung rein bringen. In meinem Ersten Post funktioniert es ja alles ganz gut, jedoch schreibt er halt alles nach einander ohne Absatz. Aber du hast schon recht, ich hab da einfach ein paar Beispiele rein "geklatscht" und gehofft es funktioniert. Ich werde jetzt erst einmal bissl in der API rum lesen. Danke

 

[Mr_Klobuerste]

Soweit so gut. Quellcode ist aufgeräumt und von sämtlichen Mülle befreit.

 String line;
            PrintWriter outFile;
            BufferedReader inFile;

            {
                inFile = new BufferedReader(new FileReader("J:\\server_log2.txt"));
                outFile = new PrintWriter(new FileWriter("J:\\logfile2.txt"));
                line = inFile.readLine();
                while (line != null) 
                {
                    if (line.startsWith(Meldung, 21))
                    {
                        System.out.println(line);
                        outFile.write(line);
                    }
                    line = inFile.readLine();

                }

            }
            outFile.close();

Bleibt nur noch die frage wie ich die Zeilenumbrüche hin bekomme. Da muss ich mich noch einmal belesen, ansonsten melde ich mich noch einmal. PS: geht das so? Schaut auf jeden fall übersichtlicher aus.

 

[Bernd Hohmann]

Zeile 14 muss .println(...) lauten - wie oft denn noch?

 

[SlaterB]

statt
> outFile.write(line);
die vorgeschlagene println() Methode oder zwischen den Zeilen nun noch ein write() für den Separator

 

[jwiesmann]

guckst du hier:
PrintWriter (Java Platform SE 7 )
Vielleicht kommst du ja selbst drauf

Edith sagt: man seit ihr schnell

 

[Mr_Klobuerste]

Zeile 14 muss .println(...) lauten - wie oft denn noch?

Ja sorry, hatte das vorhin mit dem anderen code probiert da hatte es nicht funktioniert. Daher hatte ich mich erst einmal darauf konzentriert die Syntax zu sortieren.

@jwiesmann

"Prints a String and then terminates the line." :idea: PS hat klick gemacht.

Ich werde das heute Abend noch einmal Compilieren, jetzt bin ich noch auf Arbeit.

 

[Mr_Klobuerste]

Zeile 14 muss .println(...) lauten - wie oft denn noch?

Vielen Dank. Hat jetzt geklappt.