Input/Output Datei durchlesen und Eintrag herauschreiben

[Ananaskirsche]

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:

@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:

/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]

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

EDIT:

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]

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

Pfad zur ersten Datei
Zeile Inhalt

oder als Bsp.:

/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]

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):

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]

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

 

[Ananaskirsche]

Hier war mal ein Text

 

[Flown]

Du hast hier wahrscheinlich kein Java 8?

Hier hast du es in Java 7:

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]

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]

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]

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]

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]

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]

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

 

[Ananaskirsche]

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

Sollte eigentlich so funktionieren

 

[Ananaskirsche]

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

 

[Flown]

Dann lass uns doch an deiner Lösung teilhaben

 

[Ananaskirsche]

Hier der Code:

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]

Moin,

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]

War doch ein Fehler meinerseits, der Code selber funktioniert einwandfrei