Vector wird überschrieben

[filth]

Hallo,
mein Vektor wird anscheinend überschrieben:

Vector<String[]> logData = new Vector<String[]>();
		String[] logEntry = new String[5];
		int i = 0;
		
		// read file
		try {
			BufferedReader in = new BufferedReader(new FileReader(file.getAbsolutePath()));
			String zeile = null;
			
			// collect the log data and group by loops
			while ((zeile = in.readLine()) != null) 
			{				
				logEntry[i] = zeile;
							
				// 4 loops collected, add to vector
				if (i == 3) 
				{
					i = 0;
					logData.add(logEntry);
					System.out.println(logData.get(0)[1]);
										
				}								
				else
				{
					i++;
				}
			}
		} catch (IOException e) {
			e.printStackTrace();
		}

Ausgabe:
Loop:2,Delay:30,LeanL:3.0,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3
Loop:2,Delay:30,LeanL:0,LeanR:5.0,TPS:0.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3
Loop:2,Delay:30,LeanL:2,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3

Da ich aber immer das gleiche Element ausgebe (System.out.println(logData.get(0)[1]) sollte es sich doch nicht ändern. Was ist da los?

 

[Marco13]

Es wird immer derSELBE Array verwendet... direkt NACH Zeile 19
logEntry = new String[5];
zu machen könnte schon das Problem lösen...


EDIT: Falls ich das Problem richtig verstanden habe ... wobei ich mit so beim zweiten Draufschauen nicht mehr so sicher bin... ???:L

 

[filth]

Ja es ist immer das gleiche Array, aber die Zellen des Arrays werden immer neu beschrieben bevor es angehangen wird.

Oder andersrum: der Eintrag 1 im Vektor dürfte sich doch unter keinen Umständen ändern. Der tut es aber!

 

[nrg]

mir würde es persönlich helfen die inputdatei bzw den gewünschten output zu kennen.
grüße

 

[filth]

Also die Inputdatei sieht so aus:

Loop:1,$GPRMC,185104.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:3.0,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3
Loop:2,Delay:30,LeanL:3.0,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3
Loop:3,Delay:20,LeanL:4.0,LeanR:1.0,TPS:1.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100
Loop:4,Delay:40,LeanL:4.9,LeanR:2.0,TPS:4.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300
Loop:1,$GPRMC,185105.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:0,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3
Loop:2,Delay:30,LeanL:0,LeanR:5.0,TPS:0.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3
Loop:3,Delay:20,LeanL:0,LeanR:1.0,TPS:0.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100
Loop:4,Delay:40,LeanL:1,LeanR:2.0,TPS:0.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300
Loop:1,$GPRMC,185106.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:1,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3
Loop:2,Delay:30,LeanL:2,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3
Loop:3,Delay:20,LeanL:3,LeanR:1.0,TPS:1.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100
Loop:4,Delay:40,LeanL:5,LeanR:2.0,TPS:4.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300

Im Vektor soll sich jeweils ein Array drin befinden. Das jeweilige Array hat je 4 Felder, in der Loop1 bis Loop4 drin steht

 

[eRaaaa]

Ja es ist immer das gleiche Array, aber die Zellen des Arrays werden immer neu beschrieben bevor es angehangen wird.

Richtig und genau da liegt ja das Problem. Du änderst halt immer das selbe Array ! D.h. der Vector zeigt an jeder Stelle auf dieses eine Array (wenn du nun das eine Array[0] änderst, zeigen alle auf diesen einen Platz!)

	Vector<String[]> logData = new Vector<String[]>();
	String[] lala = {"hallo","welt"};
	logData.add(lala);
	lala[0] = "tschüß";
	lala[1] = "alter wert";
	logData.add(lala);
	for (String[] string : logData) {
	    for (String string2 : string) {
		System.out.println(string2);
	    }
	}
//tschüß
//alter wert
//tschüß
//alter wert

 

[filth]

Aaah arbeitet der Vektor dann also mit Pointern? Ich hatte gedacht die Werte werden dort abgelegt.... aaaaah

danke!!

Wie sollte man das dann besser lösen?

 

[nrg]

Naja du gibst ja immer genau loop2 aus, weil das ja in deinem Array immer den Index 1 hat:

Loop:1,$GPRMC,185104.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:3.0,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3//array index 1 vektorindex 0
[B]Loop:2,Delay:30,LeanL:3.0,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3//array index 2 vektorindex 0[/B]
Loop:3,Delay:20,LeanL:4.0,LeanR:1.0,TPS:1.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100//array index 3 vektorindex 0
Loop:4,Delay:40,LeanL:4.9,LeanR:2.0,TPS:4.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300//array index 4 vektorindex 0
Loop:1,$GPRMC,185105.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:0,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3//array index 1 vektorindex 1
[B]Loop:2,Delay:30,LeanL:0,LeanR:5.0,TPS:0.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3//array index 2 vektorindex 1[/B]
Loop:3,Delay:20,LeanL:0,LeanR:1.0,TPS:0.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100//array index 3 vektorindex 1
Loop:4,Delay:40,LeanL:1,LeanR:2.0,TPS:0.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300//array index 4 vektorindex 1
Loop:1,$GPRMC,185106.000,A,5122.4537,N,00655.4834,E,43.98,285.43,291209,,,A*56^$GPGGA,185106.000,5122.4537,N,00655.4834,E,1,6,1.34,80.3,M,47.5,M,,*66|Delay:0,LeanL:1,LeanR:2.0,TPS:3.9,ADC1:0.0,ADC2:0.0,ADC3:0.0,ADC4:0.0,Freq:33.3//array index 1 vektorindex 2
[B]Loop:2,Delay:30,LeanL:2,LeanR:5.0,TPS:3.0,ADC1:0,ADC2:0,ADC3:0,ADC4:0,Freq:50.3//array index 2 vektorindex 2[/B]
Loop:3,Delay:20,LeanL:3,LeanR:1.0,TPS:1.0,ADC1:2,ADC2:1,ADC3:3,ADC4:3,Freq:100//array index 3 vektorindex 2
Loop:4,Delay:40,LeanL:5,LeanR:2.0,TPS:4.9,ADC1:3,ADC2:4,ADC3:5,ADC4:5,Freq:300//array index 4 vektorindex 2

weiß leider immernoch net was die ausgabe sein soll . oder war die Ausgabe einfach zum Test gedacht? Dein Array bzw deine List wird auf jedenfall wie gewünscht gefüllt (so wie ich das jetz verstanden hab). edit: oder wohl doch nicht ganz, wie uns eRa grad aufgeklärt hat . im Vektor steht ja nur die Referenz auf das array. hab ich jetz auch net dran gedacht. war zu sehr auf die Ausgabe fixiert.

ahja:
[Java]
String[] logEntry = new String[4];
[/code]
reicht.

grüße
nrg

 

[eRaaaa]

Wie sollte man das dann besser lösen?

Erstell halt einfach immer neue String-Arrays, nachdem du ein Array hinzugefügt hast:

logEntry = new String[5];
//bzw [4]