Probleme mit BufferedWriter und URL

[Goldarm5]

Hi,

die dargestellte Methode ist dazu da Informationen zu einer config-Datei (bei mir textdatei) hinzuzufügen. Dafür wird ein String übergeben der mit "http://eu.battle.net/api/wow/auction/data/" zu z.B. "http://eu.battle.net/api/wow/auction/data/Eredar" zusammengefügt wird. Dort wird überprüft ob es diesen WoW-Server tatsächlich gibt und falls ja und der noch nicht in der LinkedList drinne ist soll der übergebene String in die LinkedList geschrieben werden und in die config-datei. Aber stattdessen gibt es eine file-not found exception (siehe unten) in dieser Zeile:
InputStream in = url.openConnection().getInputStream();

Ich hab da schon viel rumprobiert und merkwürdigerweise scheint es zu funktionieren wenn ich statt URL url = new URL(link); von Hand URL url = new URL("http://eu.battle.net/api/wow/auction/data/Eredar"); schreibe, obwohl link und "http://eu.battle.net/api/wow/auction/data/Eredar" das gleiche enthalten. Aber dann sind da andere komische Sachen, dass er dann den Server nicht in die Datei schreibt, bzw. nur in kleinbuchstaben und mit Zahlen funktioniert es auch. Ich hab da momentan irgendwie garkeinen Plan woran das liegen könnte und wäre über jede Hilfe serh dankbar

public void addServer (String server) throws IOException {
		if (!this.serverList.contains(server)) { //Überprüfung ob LinkedListe den Server schon enthält
			StringBuilder sb = new StringBuilder();
			sb.append("http://eu.battle.net/api/wow/auction/data/");
			sb.append(server); //Zusammenfügen des teillinkes und des servers zum vollständigen link
			String link = sb.toString();
			//System.out.print(link);
			URL url = new URL(link);
			InputStream in = url.openConnection().getInputStream(); 
			InputStreamReader isr = new InputStreamReader(in); 
			BufferedReader br = new BufferedReader(isr);
			String str = br.readLine();
			if (str.length() > 70 ) { //falls es den server nicht gibt ist der String wesentlich kürzer
				this.serverList.add(server);
				File file = new File("Textdatei/config.txt"); //meine config-datei
				FileWriter fw = new FileWriter(file);
				FileReader fr = new FileReader(file);
				br = new BufferedReader(fr);
				BufferedWriter bw = new BufferedWriter(fw);
				str = br.readLine();
				while (str != null && str.length() > 4) { //der server soll ans ende geschrieben werden (letzte zeile)
					bw.write(str);
				        bw.newLine();
					str = br.readLine();
				} 
				
				sb.setLength(0);
				sb.append(server);
				bw.write(sb.toString()); //server soll geschrieben werden
				sb.setLength(0);
				
				System.out.print(sb.toString());
				bw.close();
			}
			
		}
		
	}

Gibt dann folgendes:

at javafx.fxml.FXMLLoader$MethodHandler.invoke(Unknown Source)
at javafx.fxml.FXMLLoader$ControllerMethodEventHandler.handle(Unknown Source)
at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEventImpl(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEvent(Unknown Source)
at javafx.event.Event.fireEvent(Unknown Source)
at javafx.scene.Scene$ClickGenerator.postProcess(Unknown Source)
at javafx.scene.Scene$ClickGenerator.access$8100(Unknown Source)
at javafx.scene.Scene$MouseHandler.process(Unknown Source)
at javafx.scene.Scene$MouseHandler.access$1500(Unknown Source)
at javafx.scene.Scene.impl_processMouseEvent(Unknown Source)
at javafx.scene.Scene$ScenePeerListener.mouseEvent(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$350(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler$$Lambda$144/16893725.get(Unknown Source)
at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(Unknown Source)
at com.sun.glass.ui.View.handleMouseEvent(Unknown Source)
at com.sun.glass.ui.View.notifyMouse(Unknown Source)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$145(Unknown Source)
at com.sun.glass.ui.win.WinApplication$$Lambda$36/3326003.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
Caused by: java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at sun.reflect.misc.Trampoline.invoke(Unknown Source)
at sun.reflect.GeneratedMethodAccessor1.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at sun.reflect.misc.MethodUtil.invoke(Unknown Source)
... 36 more
Caused by: java.io.FileNotFoundException: http://eu.battle.net/api/wow/auction/data/
at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source)
at auctionspackage.Config.addServer(Config.java:57)
at guipackage.Controller.addServer(Controller.java:26)
... 45 more

 

[Major_Sauce]

Morgen,

ich habe bei dir folgendes gesehen:

			File file = new File("Textdatei/config.txt"); //meine config-datei

das ist nicht unbedingt all zu schick, denn was ist wenn die Datei gar nicht existiert ?
Ich hoffe du hast irgendwo sichergestellt dass es die Datei gibt.

Die Fehlermeldung lässt auf Zeile 57 in der Config.java schließen, wäre vielleicht besser gewesen wenn du die ganze Classe hier gepostet hättest, am bessten so dass die Zeilen auch mit den Zeilenangaben deiner IDE zusammenpassen, dann könnte man auch mal schauen wo der Fehler denn überhaupt liegt.

mfg Major

 

[Enceladus271]

Es kann keinen Unterschied machen ob du den String über den StringBuilder zusammensetzt oder komplett als einzelnen String angibst. Es kann dann nur sein, dass in dem Parameter den du übergibst, etwas falsches steht. Ist es möglich dass dort unsichtbare Steuerzeichen vorkommen? Oder Steuerzeichen die man bei print nicht sofort bemerkt (z.B. Zeilenumbruch am Ende)? Prüfe das doch mal indem du den zusammengesetzten String per equals mit dem String vergleichst den du dort erwartest.

Die Exception die du gepostest hast tritt in der Klasse Config auf. Hierzu können wir erst was sagen wenn du den Code dieser Klasse postest.

Und zwei weitere Dinge:
1. Auch Reader müssen per close geschlossen werden. Am besten verwendet man try-catch oder ab Java 8 try-with-resources.
2. Deine Mechanik ans Ende der Datei zu schreiben wird nicht funktionieren. Wenn du ans Ende einer Datei schreiben willst bietet die Klasse FileWriter einen geeigneten Konstuktor dafür: new FileWriter( file, true );

 

[Goldarm5]

Morgen,

ich habe bei dir folgendes gesehen:

			File file = new File("Textdatei/config.txt"); //meine config-datei

das ist nicht unbedingt all zu schick, denn was ist wenn die Datei gar nicht existiert ?
Ich hoffe du hast irgendwo sichergestellt dass es die Datei gibt.

Die Fehlermeldung lässt auf Zeile 57 in der Config.java schließen, wäre vielleicht besser gewesen wenn du die ganze Classe hier gepostet hättest, am bessten so dass die Zeilen auch mit den Zeilenangaben deiner IDE zusammenpassen, dann könnte man auch mal schauen wo der Fehler denn überhaupt liegt.

mfg Major

Die Datei existiert definitiv, und wenn nicht im Konstruktor wird das geprüft und notfalls erstellt In welcher Zeile genau die Exception geworfen wird steht oben : InputStream in = url.openConnection().getInputStream();

Es kann keinen Unterschied machen ob du den String über den StringBuilder zusammensetzt oder komplett als einzelnen String angibst. Es kann dann nur sein, dass in dem Parameter den du übergibst, etwas falsches steht. Ist es möglich dass dort unsichtbare Steuerzeichen vorkommen? Oder Steuerzeichen die man bei print nicht sofort bemerkt (z.B. Zeilenumbruch am Ende)? Prüfe das doch mal indem du den zusammengesetzten String per equals mit dem String vergleichst den du dort erwartest.

Die Exception die du gepostest hast tritt in der Klasse Config auf. Hierzu können wir erst was sagen wenn du den Code dieser Klasse postest.

Und zwei weitere Dinge:
1. Auch Reader müssen per close geschlossen werden. Am besten verwendet man try-catch oder ab Java 8 try-with-resources.
2. Deine Mechanik ans Ende der Datei zu schreiben wird nicht funktionieren. Wenn du ans Ende einer Datei schreiben willst bietet die Klasse FileWriter einen geeigneten Konstuktor dafür: new FileWriter( file, true );

Danke, ich werd mal eine Ausgabe einbauen, die das mit equlas mal nachprüft. Und doch es funktioniert ja, denn wenn ich den Link von Hand eingebe: URL url = new URL("http://eu.battle.net/api/wow/auction/data/Eredar");
Dann funktioniert es ja zumindest begrenzt, dann nimmt er zwar anfangs Eredar nicht an, aber eredar und alles was ne Zahl ist und dann manchmal wenn ich danach Eredar schreibe auch das.

 

[Goldarm5]

So es funktioniert nun endlich die Exception kam scheinbar, weil ich meine Methode bei 2 Events vom Button ausgelöst habe, daher hat er also scheinbar URL url = new URL(link); mit link = "http://eu.battle.net/api/wow/auction/data/" + Eredar gemacht, anschließend das TextField gecleared und dann ja nochmal die Methode aufgerufen, aber diesmal mit link = "http://eu.battle.net/api/wow/auction/data/", da das TextField ja leer war. Danke an den Tipp mit der equals Überprüfung, das ist mir nämlich erst aufgefallen, als bei

 			if (link.equals("http://eu.battle.net/api/wow/auction/data/Eredar")) {
				System.out.print("true");
			} else {
				System.out.print("false");
			}

truefalse als Ausgabe kam...
Wer sichs nochmal angucken möchte ich werd auch mal die komplette Klasse posten:

public class Config {
	
	private List<String> serverList;
	private List<String> itemList;
	
	public Config () throws IOException {
		this.serverList = new LinkedList<String>();
		this.itemList = new LinkedList<String>();
		File file = new File("Textdatei/config.txt");
		if (!file.exists()) {
			file.createNewFile();
		} else {
			FileReader fr = new FileReader(file);
			BufferedReader br = new BufferedReader(fr);
			String str = br.readLine();
			while (str != null && str.length() > 2) {
				if (str.matches("[0-9]+")) {
					this.itemList.add(str);
				} else {
					this.serverList.add(str);
				}
				str = br.readLine();
			}
//			if (str != null) {
//				String[] strArray;
//				strArray = str.split("[ ]");
//				for (String s: strArray) {
//					this.itemList.add(s);
//				}
//			}
//			str = br.readLine();
			
			br.close();
		}
	}
	
	public void addServer (String server) throws IOException {
		if (!this.serverList.contains(server)) {
			StringBuilder sb = new StringBuilder();
			sb.append("http://eu.battle.net/api/wow/auction/data/");
			sb.append(server);
			String link = sb.toString();
			if (link.equals("http://eu.battle.net/api/wow/auction/data/Eredar")) {
				System.out.print("true");
			} else {
				System.out.print("false");
			}
			URL url = new URL(link);
			InputStream in = url.openConnection().getInputStream(); 
			InputStreamReader isr = new InputStreamReader(in); 
			BufferedReader br = new BufferedReader(isr);
			String str = br.readLine();
			if (str.length() > 70 ) {
				this.serverList.add(server);
				File file = new File("Textdatei/config.txt");
				FileWriter fw = new FileWriter(file, true);
				BufferedWriter bw = new BufferedWriter(fw);
				sb.setLength(0);
				sb.append(server);
				bw.newLine();
				bw.write(sb.toString());
				sb.setLength(0);
				
				//System.out.print(sb.toString());
				bw.close();
			}
			br.close();
		}
		
	}
	
	public void addItem (String itemId) throws IOException {
		if (!this.itemList.contains(itemId)) {
			this.itemList.add(itemId);
			File file = new File("Textdatei/config.txt");
			FileWriter fw = new FileWriter(file);
			BufferedWriter bw = new BufferedWriter(fw);
			for (String s: this.itemList) {
				bw.write(s);
				bw.newLine();
			}
			for (String s: this.serverList) {
				bw.write(s);
				bw.newLine();
			}
			
			
			
			bw.close();
			
		}
	}
	
	

}