Zwei Methoden in einen Schritt zusammenfassen

[paradox.]

Hallo,

ich habe zwei Methoden geschrieben, einmal zum auslesen einer CSV Datei aus dem Internet, die anderen um alle doppelt vorkommenden Datensätze aus der CSV Datei zu entfernen und die CSV Datei neu zu schreiben.. Hier mal beide Methoden, funktionieren beide einwandfrei.

Spoiler

public static void Lesen() {
	String speicher = "";
	BufferedWriter bw = null;
	try
	{
	String eingabeurl = EingabeURL();
	URL url = new URL(eingabeurl);
	Reader einlesen = new InputStreamReader( url.openStream() );
	BufferedReader in = new BufferedReader( einlesen );
	String dateiausgabe = "Ausgabe.csv";
	bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
	for ( String aktuellerDatensatz; ( aktuellerDatensatz = in.readLine() ) != null; ){
	//Aktuellen Datensatz in CSV-Datei schreiben
	bw.write(aktuellerDatensatz);
	//Neue Zeile
	bw.newLine();
	}
	//String schreiben  
    bw.write(speicher); 
    //Puffer leeren
    bw.flush();
    //BufferWriter schliessen
    bw.close();
    in.close();
	}
	catch ( MalformedURLException e ) {
	System.out.println( "MalformedURLException: " + e );
	}
	catch ( IOException e ) {
	System.out.println( "IOException: " + e );
	}
	
}

public static void DoppelteDSloeschen() throws FileNotFoundException {
	FileReader datei = null;
	BufferedReader buff = null;
	datei = new FileReader("Ausgabe.csv");
	buff = new BufferedReader(datei);
	Set<String> zeilen = new LinkedHashSet<>();
	String speicher = "";
	
	try {
	   String line;
	  while((line = buff.readLine()) != null) {
		   if(!zeilen.contains(line)) {
		   zeilen.add(line);}}
	       buff.close();
	       String dateiausgabe = "Ausgabe.csv";
	       for (String aktuellerDatensatz: zeilen ){
	       BufferedWriter bw = null;
	       bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
		   bw.write(aktuellerDatensatz);
		   bw.newLine();
		   bw.write(speicher);
		   bw.flush();
		   bw.close();
		   
	       }
		   
	 } catch (IOException e) {
	     System.err.println("Fehler:" + e);
	 } finally {
	     try {
	         buff.close();
	         datei.close();
	     } catch (IOException e) {
	         System.err.println("Fehler:" + e);
	     }
	 }
	
}

Jedoch will ich jetzt beide Methoden in einem Schritt abwickeln, sprich in einer Methode, jedoch funktioniert dann gar nichts mehr.. Hier mal der Code wenn ich meine Methoden zusammenfasse..

Spoiler

public static void Zusammengefasst() throws FileNotFoundException {
	String speicher = "";
	BufferedWriter bw = null;
	try
	{
	String eingabeurl = EingabeURL();
	URL url = new URL(eingabeurl);
	Reader einlesen = new InputStreamReader( url.openStream() );
	BufferedReader in = new BufferedReader( einlesen );
	String dateiausgabe = "Ausgabe.csv";
	bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
	for ( String aktuellerDatensatz; ( aktuellerDatensatz = in.readLine() ) != null; ){
	//Aktuellen Datensatz in CSV-Datei schreiben
	bw.write(aktuellerDatensatz);
	//Neue Zeile
	bw.newLine();
	}
	//String schreiben  
    bw.write(speicher); 
    //Puffer leeren
    bw.flush();
    //BufferWriter schliessen
    bw.close();
    in.close();
	}
	catch ( MalformedURLException e ) {
	System.out.println( "MalformedURLException: " + e );
	}
	catch ( IOException e ) {
	System.out.println( "IOException: " + e );
	}
	FileReader datei = null;
	BufferedReader buff = null;
	datei = new FileReader("Ausgabe.csv");
	buff = new BufferedReader(datei);
	Set<String> zeilen = new LinkedHashSet<>();
	speicher = "";
	
	try {
	   String line;
	  while((line = buff.readLine()) != null) {
		   if(!zeilen.contains(line)) {
		   zeilen.add(line);}}
	       buff.close();
	       String dateiausgabe = "Ausgabe.csv";
	       for (String aktuellerDatensatz: zeilen ){
	       bw = null;
	       bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
		   bw.write(aktuellerDatensatz);
		   bw.newLine();
		   bw.write(speicher);
		   bw.flush();
		   bw.close();
		   
	       }
		   
	 } catch (IOException e) {
	     System.err.println("Fehler:" + e);
	 } finally {
	     try {
	         buff.close();
	         datei.close();
	     } catch (IOException e) {
	         System.err.println("Fehler:" + e);
	     }
	 }
}

Grüße

 

[reibi]

Hi
würde es gerne mal testen
Dafür fehlt mir aber einiges.

zB die Methode EingabeURL()
Falls das eine Methode ist, wie ich vermute!

Grundsätzlich mal folgendes merken:
Alles in Java fängt mit einem Kleinbuchstaben an! Außer Klassennamen!
Durch diese Konventionen wird ein Code lesbar!


Also ich brauche den ganzen Code und eine URL zum testen.

Dann kann ich vielleicht ne Aussage machen

 

[paradox.]

Hi
würde es gerne mal testen
Dafür fehlt mir aber einiges.

zB die Methode EingabeURL()
Falls das eine Methode ist, wie ich vermute!

Grundsätzlich mal folgendes merken:
Alles in Java fängt mit einem Kleinbuchstaben an! Außer Klassennamen!
Durch diese Konventionen wird ein Code lesbar!


Also ich brauche den ganzen Code und eine URL zum testen.

Dann kann ich vielleicht ne Aussage machen

Jap ist ne Methode. Vermute es bringt dir nicht viel, da ich auf ein Geräte, des im Netzwerk angeschlossen ist zugreife.

public static String EingabeURL() {
	String eingabeurl = "http://<ip-adresse-des-geraets>/sonstiger/pfad/<dateiname>.csv";
	return eingabeurl;

Würde dann ungefähr so aussehen, wie gesagt außerhalb des Netzwerks kann man da leider nicht drauf zugreifen..

 

[paradox.]

Hab mein Code mal bissl überarbeitet, funktioniert aber leider immer noch nicht..

Spoiler

public static void lesen() {
	String speicher = "";
	BufferedWriter bw = null;
	try
	{
	String eingabeurl = eingabeURL();
	URL adresse = new URL(eingabeurl);
	Reader einlesen = new InputStreamReader( adresse.openStream() );
	BufferedReader in = new BufferedReader( einlesen );
	String einlesenneu = inputStreamReaderInString(in);
	String dateiausgabe = "Dateiausgabe.csv";
	bw = new BufferedWriter(new FileWriter(dateiausgabe, true)); 
	Set<String> zeilen = new LinkedHashSet<>();
	while((einlesenneu = in.readLine()) != null) {
		   if(!zeilen.contains(einlesenneu)) {
		   zeilen.add(einlesenneu);}}
	       in.close();
	       for (String aktuellerDatensatz: zeilen ){
	       bw = new BufferedWriter(new FileWriter(dateiausgabe, true));
		   bw.write(aktuellerDatensatz);
		   bw.newLine();
		   bw.write(speicher);
		   bw.flush();
		   bw.close();
	}}
	catch ( MalformedURLException e ) {
	System.out.println( "MalformedURLException: " + e );
	}
	catch ( IOException e ) {
	System.out.println( "IOException: " + e );
	}
}

 

[Harry Kane]

Ist dir klar, daß du für jede Zeile aus deinem Set<String> eine ev. vorhandene Datei mit dem Namen dateiausgabe überschreibst?
Im übrigen ist "es funktioniert gar nicht mehr" eine sehr schlechte Fehlerbeschreibung.

 

[paradox.]

Ist dir klar, daß du für jede Zeile aus deinem Set<String> eine ev. vorhandene Datei mit dem Namen dateiausgabe überschreibst?
Im übrigen ist "es funktioniert gar nicht mehr" eine sehr schlechte Fehlerbeschreibung.

ich wollte eine CSV Datei von einem Link einlesen da die Duplikate entfernen & diese dann ohne Duplikate in eine neue Datei speichern. Vorher war dies in 2 Methoden da hat es geklappt.

Hab ne endlosschleife drinnen.. Aufjedenfall funktioniert jetzt gar nichts mehr.

 

[Beatsleigher]

while ((line = reader.readLine()) != null) //**

Sent from my GT-I9505 using Tapatalk

 

[paradox.]

das Problem liegt darin dass beide Methoden seperat von einander sehr gut funktionieren. Jedoch will ich jetzt beide einem Button auf meiner GUI hinterlegen und dann funktioniert meine Methode, die die Duplikate entfernt leider nicht mehr..

 

[Beatsleigher]

Vielleicht packst du da mal ein Thread.sleep(500); rein. Das kann Wunder bewirken

Sent from my GT-I9505 using Tapatalk