hab ein Problem (schlecht zu beschreiben)

[FanHi]

Guten Abend Leute!

Im Grunde geht es darum, aus einem Array mit 15 Elementen die aktuellen Werte jede Sekunde in eine Zeile in ein .csv file zu schreiben um aus den Werten später in Excel ein Diagramm zu machen.

1. wollte ich fragen, ob jemand vl. eine bessere Möglichkeit kennt
und 2. hab ich ein Problem mit

csvwriter.write("1;2;3;4;5;6;7;8;9;10;11;12;13;14;15\n");

es kommt die Fehlermeldung :
"Cannot refer to a non-final variable csvwriter inside an inner class defined in a different method"

nja ich hätt einfach gern ein paar Verbesserungsvorschläge und Vermutungen was falsch sein könnte.

import java.io.*;
import java.util.*;
import javax.*;




public class csvwrite{
	public static void Csvwrite(int channel,int time, int value)throws IOException{
	String filename;
	
	//die datei soll im Namen das Datum und die Uhrzeit haben. z.B. 1-6-2007-18:05.csv
	filename=(String.valueOf(Calendar.getInstance().get(Calendar.DATE))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.MONTH))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.YEAR)))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.HOUR_OF_DAY))+":"+String.valueOf(Calendar.getInstance().get(Calendar.MINUTE));
	
	boolean log=true;   //damit ich das Ganze irgendwann mal wieder abbrechen kann
	 
	
	try{
		
	BufferedWriter csvwriter= new BufferedWriter(new FileWriter(filename+".csv"));
	csvwriter.write("time;ch 1;ch 3;ch 3;ch 4;ch 5;ch 6;ch 7;ch 8;ch 9;ch 10;ch 11;ch 12;ch 13;ch 14;ch 15\n"); //kopfzeile ist immer gleich
	csvwriter.write(";;;;;;;;;;;;;;\n"); // Unter der Kopfzeile soll eine frei sein
	 
	while(log==true){  
		log=false;  //vorerst, damit er nicht ewig schreibt, kommt später weg
		
		int delay = 1000;  // TimerTick jede sekunde
		 javax.swing.Timer logTimer= new javax.swing.Timer(delay, new java.awt.event.ActionListener(){
			public void actionPerformed(java.awt.event.ActionEvent e) { 
				
				csvwriter.write("1;2;3;4;5;6;7;8;9;10;11;12;13;14;15\n");  //problemzeile 
				//-->ersetzt im moment die werte die später aus dem Array ausgelesen werden sollen 
								
			}
			
			});

		 logTimer.start();	
			
	}
	csvwriter.close();
	}
	catch (IOException e) {
		Main.Log.addlog("Error beim erstellen der Datei: "+filename+".csv");
}
}
}

danke

mfg FanHi

 

[L-ectron-X]

Zeile 21 ändern in:

final BufferedWriter csvwriter= new BufferedWriter(new FileWriter(filename+".csv"));

 

[SlaterB]

das ist so wie es da steht,
innere Klassen (deren Objeke lange lange bestehen) haben ihre Probleme mit lokalen temporären Variablen in der erzeugenden Operation,

warum genau macht nicht wirklich Sinn,
aber schreibe einfach final vor die Variable und Java ist zufrieden

----
wenn du allerdings den Writer sofort closed, dann wirst du damit so oder so keine Freude haben,
das close() darf erst nach dem letzten write ausgeführt werden,

 

[FanHi]

Danke!

Hab das final jetzt drin,
aber jetzt schimpft er, dass ich eine IOExeption nicht abfange... wo denn bitte? da steht doch mein nettes catch(..){..}!

und @ SlaterB:

das close(); steht doch ausserhalb von dem while() oder nicht?

mfg FanHi

 

[FanHi]

So, hab mich gerade angemeldet

was ich so Nebenbei mal fragen wollte: ist meine Lösung des Dateinamens wirklich die einfachste?

mfg FanHi

 

[SlaterB]

naja gut, da fragt man sich, was die ganze Schleife da überhaupt macht,
bisher sieht es so aus als würde die in einer Endlosschleife Millionen von Timer-Objekten erzeugen.., wenn log true bleibt,

die Schleife wartet nicht bis der Timer fertig ist oder so..

bei log=false wird die Schleife nach 1 ms beendet und gleich close(),
lange bevor der Timer was zu sagen hat
--------

die Fehlermeldung kommt, weil du im Timer nicht die Exception abfängst,
das ist wie gesagt ein eigenständiges Objekt, hat sehr wenig mit dem Code drumherum zu tun

---------

du machst wohl besser

while(log==true){  
      log=false;  //vorerst, damit er nicht ewig schreibt, kommt später weg 
       
      int delay = 1000;  // TimerTick jede sekunde 
      Thread.sleep(delay); // noch Exception abfangen
            
      csvwriter.write("1;2;3;4;5;6;7;8;9;10;11;12;13;14;15\n");  //problemzeile 
            //-->ersetzt im moment die werte die später aus dem Array ausgelesen werden sollen 
   }

dann hast du auch keine Probleme mit close(),
allerdings würde dein Programm dann sonst nix anderes machen,
wenn das ganze nicht als Thread läuft,

und sieht ja derzeit nicht so aus als hättest du davon Ahnung

 

[SlaterB]

zum Dateinamen:
verwende da
String dateString = new SimpleDateFormat("dd.MM.yyyy").format(date);
oder ähnlich

aber im Grunde auch wieder zu schwer für deinen Wissenstand,
gibt wichtigeres

 

[FanHi]

ich bin mir nicht sicher, ob du mich verstanden hast...

 log=false;  //vorerst, damit er nicht ewig schreibt, kommt später weg

im moment ist die while schleife noch zu nichts da... später soll man die "log-funktion" per Button-klick an und ausschalten können. da brauch ich dann das. (log wird dann vom benutzer entweder true oder false gesetzt)

und das mit dem Sleep..... nja das war mir durchaus bewusst, dass das das ganze Programm anhält, was natürlich nicht sein darf. ansonsten wär ich schon auf das Sleep gekommen (kenne es von C / C++)

der Timer ist dazu da, jede sekunde eine Zeile "Array-werte" zu dem .csv file hinzuzufügen.

Gruß FanHi

 

[SlaterB]

aber der Timer macht da aus gewisser Sicht logisch keinen Sinn bzw. mag zwar sinnvoller sein als das sleep,
aber dann stimmt die Umgebung nicht,

entweder
a.) du denkst, dass die while-Schleife auf den Timer wartet,
dann wäre das aber das gleiche wie Thread.sleep()

oder
b.) du weißt, dass die while-Schleife den Timer startet und dann SOFORT selber weitermacht,
dann werden aber entweder bei eingeschalteten log sofort Millionen weitere Timer erzeugt (die Schleife ist schnell..)
oder direkt die Schleife beendet und close() aufgerufen

 

[FanHi]

--> a :?

und das Sleep funktioniert IMHO trozdem nicht.... ???:L ???:L ???:L

ich denke ich mach einfach bei meinem Button-event ein timer.start() bzw. timer.stop rein....

trozdem hab ich im moment immer noch ein problem mit meinem catch()...
da behauptet er näml. jetzt, "identifier expected"

hab gehört, dass das ein beliebter fehler bei vergessenen klammern oder ; ist... kann aber keine finden.

package logic;

import java.io.*;
import java.util.*;
import javax.*;


public class csvwrite{
	public static void Csvwrite(int channel,int time, int value)throws IOException{
	final String filename;
	
	//die datei soll im Namen das Datum und die Uhrzeit haben. z.B. 1-6-2007-18:05.csv
	 filename=(String.valueOf(Calendar.getInstance().get(Calendar.DATE))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.MONTH))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.YEAR)))+"-"+String.valueOf(Calendar.getInstance().get(Calendar.HOUR_OF_DAY))+":"+String.valueOf(Calendar.getInstance().get(Calendar.MINUTE));
	
	
	try{
		
	final BufferedWriter csvwriter= new BufferedWriter(new FileWriter(filename+".csv"));
	csvwriter.write("time;ch 1;ch 3;ch 3;ch 4;ch 5;ch 6;ch 7;ch 8;ch 9;ch 10;ch 11;ch 12;ch 13;ch 14;ch 15\n"); //kopfzeile ist immer gleich
	csvwriter.write(";;;;;;;;;;;;;;\n"); // Unter der Kopfzeile soll eine frei sein
	 
	 
		int delay = 1000;  // TimerTick jede sekunde
		 javax.swing.Timer logTimer= new javax.swing.Timer(delay, new java.awt.event.ActionListener(){
			public void actionPerformed(java.awt.event.ActionEvent e) { 
				
				csvwriter.write("1;2;3;4;5;6;7;8;9;10;11;12;13;14;15\n");  
				//-->ersetzt im moment die werte die später aus dem Array ausgelesen werden sollen 
				
			}
	Fehler-->   catch (IOException e) {
				Main.Log.addlog("Error beim erstellen der Datei: "+filename+".csv");
			}
			
			});

		 
	
	csvwriter.close();
	}
	catch (IOException e) {
		Main.Log.addlog("Error beim erstellen der Datei: "+filename+".csv");
}
}
}

wäre dir echt dankbar wenn du vl. mal deinen senf dazugeben könntest^^

 

[SlaterB]

du hast gar kein "try {" dazu,
jedenfalls nciht innerhalb der Operation actionPerformed,
sind doch nur 3 Zeilen, gut zu überblicken

 

[FanHi]

Meine Güte^^

sag doch gleich, dass ich 2 try brauch^^

danke!