Quelltext einlesen

[BaxterStockman]

hai, ich versuche einen Textstring in einer Datei suchen zu lassen, hier der Code

import java.io.*;
import java.util.*;
public class Aufgabe36 {
	public static int sucheString(String dateiname, String suchtext){
		Vector<String> cont = new Vector<String> (50,10);
		try {
			BufferedReader br = new
				BufferedReader (new 
						FileReader(dateiname));
				String zeile;
				while ((zeile=br.readLine())!=null) {         
					if (zeile.indexOf(suchtext)>=0)     
						return 1;
				                                    }	
				br.close();
		}
		catch (FileNotFoundException e){}
		catch (IOException e){}
		for (int i=(cont.size()-1);i>=0;i--){
			System.out.println((String)cont.elementAt(i));
		}
		return 0;
		
	}

public static void main (String[]args){
	int a = 0;
	sucheString("Test.java","public");
	if (a==1)
		System.out.println("Gesuchter String ist in der Datei vorhanden");
	else System.err.println("Gesuchter String ist nicht in der Datei vorhanden");
	}
}

Das Problem ist in der Test-Klasse findet er public nicht obwohl es ja drin steht in durch public class

public class Test {
public int a=1; 
public static void main (String[]args){
System.out.println("public");
}}

Hab auch versucht ob er vielleicht das nur in nem ausgegebenen Zustand findet, durch den System.out.println, aber das funktiniert auch nicht, er gibt immer die Fehlermeldung aus, die ich in Zeile 31 definiert hab, weiß jemand warum ?

 

[w0ddes]

Wie wäre es mal mit:

a = sucheString("Test.java","public");

?! Du musst dein a doch auch verändern, wenn du es danach abfragen willst! So wie du es hast geht dein

return 1

in's Nirvana

 

[BaxterStockman]

ok, macht Sinn, hab ich gemacht, ändert trotzdem nichts

 

[SlaterB]

schlecht sind deine leeren catch-Blöcke,
wenn die Datei nicht gefunden wird bekommst du davon nichts mit,

baue überhaupt mehr Logging ein, gib jede Zeile mit System.out.println() aus
edit: hast du ja irgendwie schon, nur mit dem Vector cont der immer leer ist?

 

[turing]

Doch, damit gehts:

public static void main (String[]args){
    int a = sucheString("Test.java","public");
    if (a==1)
        System.out.println("Gesuchter String ist in der Datei vorhanden");
    else System.err.println("Gesuchter String ist nicht in der Datei vorhanden");
    }
}

 

[henpara]

edit:
thx java-rocks , hier stand blödsinn.

Dann als Tipp:

if (!datei.exists) {
System.out.println("Datei nicht vorhanden, Methode abgebrochen");
return 0
}

einfügen. Ansonsten die Frage, ob du irgendwelche Exceptions geworfen bekommst?

Und den Tipp von Slater berücksichtigen, lieber am Anfang mehr Ausgaben, wo die Methode gerade ist, oder mit Debugger arbeiten, um Fehler sicher aufzuspüren.

 

[javaRocks]

@henpara

ich denke nicht das es um einen import geht. So wie ich das verstehe, will er den Quelltext der Test.java Klasse einlesen (parsen) und feststellen, ob das Wort: public vorkommt. Nun fragt er warum immer die Meldung kommt: "Gesuchter String ist nicht in der Datei vorhanden" obwohl doch die Test.java Klasse mit 'public class ...' anfängt.

@BaxterStockman
Bitte korregieren, wenn ich damit falsch liege...

 

[ARadauer]

catch (FileNotFoundException e){}
catch (IOException e){}

mach das nicht...

 

[freez]

Mach mal als ersten Befehl in deiner Main()

System.out.println(new File(".").getAbsolutePath()+"\n\n");

und prüfe, ob du im richtigen Ordner bist. Oft steht der aktuelle Pfad ein paar Ebenen höher, z.B. direkt in dem Projektordner und nicht direkt im Ordner des Package.

 

[BaxterStockman]

@javarocks: genau so hab ich mir das vorgestellt, ich will den Quelltext der Testklasse Java einlesen und da "public" finden

@freez: in der Tat befinde ich mich wohl in der falschen Directory, da nur das überpackage angeben wird, wie kann ich das ändern?

@turing: nee funktionier immer noch nicht mein Code sieht genauso aus wie Deiner jetzt

public static void main (String[]args){
	System.out.println(new File(".").getAbsolutePath()+"\n\n");
	int a = sucheString("Test.java","public");
	if (a==1)
		System.out.println("Gesuchter String ist in der Datei vorhanden");
	else System.err.println("Gesuchter String ist nicht in der Datei vorhanden");
	}

 

[BaxterStockman]

logsicher weise hab ich den oben auf geändert

import java.io.*;
import java.util.*;
public class Aufgabe36 {
	public static int sucheString(String dateiname, String suchtext){
		int a = 0; 
		Vector<String> cont = new Vector<String> (50,10);
		try {
			BufferedReader br = new
				BufferedReader (new 
						FileReader(dateiname));
				String zeile;
				while ((zeile=br.readLine())!=null) {         
					if (zeile.indexOf(suchtext)>=0){
						a=1;
						break;
					}
						
				                                    }	
				br.close();
		}
		catch (FileNotFoundException e){}
		catch (IOException e){}
		for (int i=(cont.size()-1);i>=0;i--){
			System.out.println((String)cont.elementAt(i));
		}
		return a;
		
	}

warum soll ich die catchblöcke weglassen? Das stand so in ner Musteraufgabe in nem Buch ?

 

[henpara]

warum soll ich die catchblöcke weglassen?

du sollst sie nicht weglassen, die Blöcke sind sinnvoll. aber du machst ja garnichts, wenn der Fehler auftritt, wirst also nicht informiert.

Entweder entsprechende sys.outs formulieren oder e.stacktrace aufrufen in den Blöcken, dann weißt du auch ganz genau, wenn was schiefgeht, was es ist (zB file not found).

Deine Exception wird zwar erkannt, aber es passiert nix, du wirst also nicht mal drauf aufmerksam gemacht, daß die exception aufgetreten ist.

 

[turing]

Dann gebe den relativen Pfad (oder auch den absoluten Pfad) in deinem Parameter "dateiname" explizit an.