Input/Output Datei durchlesen und Eintrag herauschreiben

Ananaskirsche · 25. Mrz 2015

Hallo,
ich benötige eine Methode, die mir eine Reihe von Dateien nach einem String durchsucht und die Zeile in der der String steht vollständig herausschreibt. dafür habe ich mir folgende Methode gemacht:

Java:

@SuppressWarnings("resource")
	public static int SearchAndWrite(String pathToDir, String pathToOutput, String token)
	{
		//Anlegen der File-Variablen
		File dir = new File(pathToDir);
		File output = new File(pathToOutput);
		
		ArrayList<File> fileList = new ArrayList<>();
		getPaths(dir, fileList);
		
		//Erstellen der Reader und Writer
		FileReader reader = null;
		BufferedReader r = null;
		
		FileWriter writer = null;
		BufferedWriter w = null;
		
		
		for(int i = 0; i < fileList.size(); i++)
		{
			File searchFile = fileList.get(i);
			
			if(searchFile.isDirectory())
			{
				break;
			}
			
			//Init der Reader&Writer
			try
			{
				reader = new FileReader(searchFile.getAbsoluteFile());
				r = new BufferedReader(reader);
				
				writer = new FileWriter(output);
				w = new BufferedWriter(writer);
			}
			catch(IOException e)
			{
				System.err.println(e);
			}
			
			
			String buffer = null;
			int lineCounter = 0;
			boolean header = false;
			try
			{
				while((buffer = r.readLine()) != null)
				{
					lineCounter++;
					if(buffer.contains(token))
					{
						if(header == false)
						{
							w.write(searchFile.getAbsolutePath() + "\n");
                                                        header = true;
						}
						
						w.write(lineCounter + " " + buffer + "\n");
					}
				}
				
				w.flush();
				writer.flush();
			}
			catch(IOException e)
			{
				System.err.println(e);
			}
		}

Die Ausgabe ist folgende:

Code:

/home/hans/Schreibtisch/TestDir/1/Yasou
1 Test
2 TestTest
4 Kill my Test
5 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
6 1 Test
9 2 TestTest
14 4 Kill my Test
17 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
18 1 Test
21 2 TestTest
26 4 Kill my Test
29 5 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
32 6 1 Test
39 9 2 TestTest
50 14 4 Kill my Test
57 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
58 1 Test
61 2 TestTest
66 4 Kill my Test
69 5 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
72 6 1 Test
79 9 2 TestTest
90 14 4 Kill my Test
97 17 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
100 18 1 Test
107 21 2 TestTest
118 26 4 Kill my Test
125 29 5 /home/hans/Schreibtisch/TestDir/1/Hallo 2)
132 32 6 1 Test
147 39 9 2 TestTest
170 50 14 4 Kill my Test

Das Problem liegt denke ich offen, hat jemand eine Lösung dafür? Ich habe leider keine. An der Methode getPaths() liegt es nicht, die liefert alle Dateien in einem Ordner rekursiv zurück. Vielen Dank im vorraus

Ananaskirsche

Flown · 25. Mrz 2015

Also das Problem ist nicht offensichtlich. Meinst du die Ausgabe?

EDIT:

Java:

writer = new FileWriter(output);
w = new BufferedWriter(writer);

das musst du auf jedenfall vor der ersten Schleife machen, sonst wird dein Ausgabefile immer wieder überschrieben. Und eine Warning zu unterdrücken mit "resources" macht man nicht, da es meistens auf MemoryLeaks hinweist. Also bitte schließe auch die Streams (InputFiles nach jeder inneren Schleife und OutputFile nach der ganzen Methode!

Ananaskirsche · 25. Mrz 2015

Als "offensichtliches Problem" war die Ausgabe gemeint. Sie sollte eigentlich wie folgt aussehen:

Code:

Pfad zur ersten Datei
Zeile Inhalt

oder als Bsp.:

Code:

/home/hans/Schreibtisch/TestDir/1/Datei1.txt
1 Test ist der Token
4 Hier wieder Test in der Zeile
10 Hier ist wieder Test in der Zeile

Das mit der "resource" annotation habe ich noch nicht verstanden, ich werde mal google fragen.

Zitat von Flown
"(...)das musst du auf jedenfall vor der ersten Schleife (...)"

Welche Schleife ist gemeint? Die while- oder die for-Schleife?
Und wann soll ich die Reader und Writer schließen? Innerhalb der for-Schleife oder außerhalb?

Flown · 25. Mrz 2015

Also ich persönlich würde die Files während des durchlaufens schon mal filtern, ob sie ein Directory sind.

Hier mal ein Code mit meiner Implementierung von getPaths (wie immer gilt: ACHTUNG Java 8):

Java:

public static int searchAndWrite(String pathToDir, String pathToOutput, String token) {
  List<File> fileList = getPaths(pathToDir);
  int filesWithToken = 0;
  try (PrintWriter writer = new PrintWriter(new FileWriter(pathToOutput))) {
    for (File f : fileList) {
      try (BufferedReader reader = new BufferedReader(new FileReader(f))) {
        int lineNumber = 0;
        boolean header = true;
        for (String line = reader.readLine(); line != null; line = reader.readLine()) {
          lineNumber++;
          if (line.contains(token)) {
            if (header) {
              filesWithToken++;
              writer.println(f.getAbsolutePath());
              header = false;
            }
            writer.format("%d %s%n", lineNumber, line);
          }
        }
      }
    }
    writer.flush();
  } catch (IOException ex) {
    ex.printStackTrace();
  }
  return filesWithToken;
}

public static List<File> getPaths(String root) {
  try {
    return Files.walk(Paths.get(root)).filter(p -> !Files.isDirectory(p)).map(Path::toFile).collect(Collectors.toList());
  } catch (IOException ex) {
    ex.printStackTrace();
  }
  return Collections.emptyList();
}

Ananaskirsche · 25. Mrz 2015

Ich werde mal morgen deinen Code ausprobieren und dann mich wieder hier hin wenden. Danke schonmal für deine Bemühungen 🙂

Ananaskirsche · 26. Mrz 2015

Hier war mal ein Text

Flown · 26. Mrz 2015

Du hast hier wahrscheinlich kein Java 8?

Hier hast du es in Java 7:

Java:

public static List<File> getPaths(String root) {
  try {
    final List<File> files = new ArrayList<>();
    Files.walkFileTree(Paths.get(root), new SimpleFileVisitor<Path>() {

      @Override
      public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
        files.add(file.toFile());
        return super.visitFile(file, attrs);
      }

    });
    return files;
  } catch (IOException ex) {
    ex.printStackTrace();
  }
  return Collections.emptyList();
}

Ananaskirsche · 26. Mrz 2015

Ich habe mir derweil Java 8 installiert, aber er findet in der Methode getPaths() keine Variable p

return Files.walk(Paths.get(root)).filter(p -> !Files.isDirectory(p)).map(Path::toFile).collect(Collectors.toList());

Zudem bekomme ich einen Syntax error bei Path::toFile

Flown · 26. Mrz 2015

Nachdem ich beide Codes getestet habe, kann ich nur sagen das es an dir liegt. Runtimekonfiguration etc.

Wenn du dir Java8 geholt hast, hast du auch die richtigen Pfade gesetzt, deine IDE umgestellt und dein Projekt darauf eingestellt.

Ananaskirsche · 26. Mrz 2015

Ja umgestellt hab ich, auch die Java 7 Variante funktioniert leider nicht und ich weiß leider auch nicht warum. Diesmal sagt Eclipse dass die "BasicFileAttributes" nicht in eine Variable gespeichert werden konnten. Tut mir nochmal leid, wenn ich dich damit nerve, aber ich habe wirklich keine Ahnung warum und wieso.

Flown · 26. Mrz 2015

Bitte den ganzen Stacktrace posten.

Habs jetzt auch noch mit Java 7 + 8 probiert und beide male kein Problem. Also wie gesagt am Code liegts nicht.

Ananaskirsche · 26. Mrz 2015

Ich habe das Programm mit Fehler (welcher von Eclipse angezeigt wird) kompiliert und ausgeführt. Ich bekomme keinen Fehler bei der Ausführung aber die Ausgabedatei ist leer.

Flown · 26. Mrz 2015

Richtige Pfade für deine Funktion mitgeliefert? Ich wiederhole mich nur ungern, aber bei mir funktioniert das.

Ananaskirsche · 26. Mrz 2015

Ja eigentlich schon,
pathToDir = "/home/hans/Schreibtisch/TestDir"
pathToOutput = "/home/hans/Schreibtisch/Ausgabe.txt"
token ="Test"

Sollte eigentlich so funktionieren

Ananaskirsche · 26. Mrz 2015

Danke für deine Ausdauer Flown, habe das Problem jetzt anderweitig gelöst 😉

Flown · 26. Mrz 2015

Dann lass uns doch an deiner Lösung teilhaben

Ananaskirsche · 26. Mrz 2015

Hier der Code:

Java:

public static int searchAndWrite(String pathDir, String pathToOutput, String token)
	{
		File output = new File(pathToOutput);
		File dir = new File(pathDir); 
		
		ArrayList<File> files = new ArrayList<File>();
		getPaths(dir, files);
		
		if(!output.exists())
		{
			try
			{
				output.createNewFile();
				FileWriter wrt = new FileWriter(output);
				wrt.write(getCurrentTimeAndDate() + "\n");
				
				wrt.flush();
				wrt.close();
			}
			catch(IOException e)
			{
				e.printStackTrace();
				return 1;
			}
		}
		else
		{
			try
			{
				FileWriter wrt = new FileWriter(output, true);
				wrt.write("\n\n\n" + getCurrentTimeAndDate() + "\n");
				
				wrt.flush();
				wrt.close();
			}
			catch(IOException e)
			{
				e.printStackTrace();
				return 1;
			}
		}
		
		
		
		
		for(int i = 0; i < files.size(); i++)
		{
			FileReader reader = null;
			BufferedReader r = null;
			
			FileWriter writer = null;
			BufferedWriter w = null;
			
			try 
			{
				reader = new FileReader(files.get(i).getAbsolutePath());
				r = new BufferedReader(reader);
				
				writer = new FileWriter(output.getAbsolutePath(), true);
				w = new BufferedWriter(writer);
			} 
			catch (FileNotFoundException e) 
			{
				e.printStackTrace();
				return 1;
			}
			catch (IOException e) 
			{
				e.printStackTrace();
				return 1;
			}
			
			
			boolean header = true;
			int lineNumber = 0;
			String buffer = null;
			
			
			try
			{
				while((buffer = r.readLine()) != null)
				{
					lineNumber++;
					if(buffer.contains(token))
					{
						if(header)
						{
							w.write(files.get(i).getAbsolutePath() + "\n");
							header = false;
						}
						
						w.write(lineNumber + " " + buffer + "\n");
					}
				}
				
				
				r.close();
				reader.close();
				
				w.flush();
				writer.flush();
				
				w.close();
				writer.close();
			}
			catch(IOException e)
			{
				e.printStackTrace();
				return 1;
			}
		}
		
		return 0;
	}
	
	
	private static ArrayList<File> getPaths(File file, ArrayList<File> list)
	{
        if (file == null || list == null || !file.isDirectory())
            return null;
        File[] fileArr = file.listFiles();
        for (File f : fileArr) 
        {
            if (f.isDirectory()) 
            {
                getPaths(f, list);
            }
            if(f.isFile())
            {
            	list.add(f);
            }
        }
        return list;
    } 
	
	
	
	private static String getCurrentTimeAndDate()
	{
		SimpleDateFormat formatter = new SimpleDateFormat("dd:MM:yyyy - HH:mm:ss");
		Date currentTime = new Date();
		return formatter.format(currentTime).toString();
	}

Ein letztes Problem ist geblieben: Der Code erkennt keine Strings in .cs (C#-Source)-Datein

VfL_Freak · 26. Mrz 2015

Moin,

Ananaskirsche hat gesagt.:
Ein letztes Problem ist geblieben: Der Code erkennt keine Strings in .cs (C#-Source)-Dateien

Was genau bedeutet das ???:L
Fehlermeldungen ???:L
Wo genau im Code ???:L

Fragen über Fragen ..... :noe:

Gruß Klaus

Ananaskirsche · 26. Mrz 2015

War doch ein Fehler meinerseits, der Code selber funktioniert einwandfrei

Input/Output Datei durchlesen und Eintrag herauschreiben

Aktives Mitglied

Administrator

Aktives Mitglied

Administrator

Aktives Mitglied

Aktives Mitglied

Administrator

Aktives Mitglied

Administrator

Aktives Mitglied

Administrator

Aktives Mitglied

Administrator

Aktives Mitglied

Aktives Mitglied

Administrator

Aktives Mitglied

Top Contributor

Aktives Mitglied

Ähnliche Themen