Durchlaufen von Vektoren

[JavaLehrtDemut]

Ich habe folgendes Problem: (Google und Co konnten mir auch nicht wirklich helfen)

Ich habe einen Vektor mittels while(true) und break; durchlaufen.. Funktioniert auch wunderbar AUSSER, dass auf manchen Systemen eine ConcurrentModificationException fliegt...

Nun habe ich ein Codebeispiel, welches ich für dieses Problem anwenden muss aber ich schaffe es nicht den Teil zu adaptieren

Bisher (Bsp.):

while(true) {
   tempFiles.get(i).insertData(fachFiles.get(k).getNextElement());
}

soll das werden (Bsp.):

for (Student stud : fachFiles.get(i)) {
  tempFiles.insertData(stud);
}

Eigentlich einfach, aber ich brauche bereits eine weitere Information, wo genau ich im Vector auslesen will. Das ist genau mein Problem.

Im folgenden Code ist konkret das Problem, dass ich nicht einfach mit einem Iterator i über fachFiles.get(i) laufe, sondern genau das richtige "Fach" auswähle, im unteren Code "tmp.getMnr()[5]". Dazu muss ich aber erst mein "tmp" beschreiben und da liegt mein Problem.

Wie kann ich die vorgegebe Struktur hier einsetzen OHNE, dass ich tmp schon kenne...

while (true) {
                  // Read new student from inputFiles
		tmp = inputFiles.get(index).getNextElement();
		// Get new Student from tmp and store in Fach according to
		// last digit of MatNr.
		fachFiles.get(tmp.getMnr()[5]).insertData(tmp);
}

Wäre um jede Hilfe dankbar !

 

[Der Müde Joe]

du kannst nicht änderungen an der Collection während dem durchlaufen machen

ein clone hilft schon

Vector shallowCopy = myVector.clone();

 

[JavaLehrtDemut]

Danke für die schnelle Reaktion !

Leider geht das so nicht. Eclipse sagt mir dazu "Cannot convert from Objekt to vector"

Oder habe ich deinen Vorschlag falsch verstanden ?

Vector ClonVektor = fachFiles.clone();
			for(Student stud : CloneVektor.get(tmp.getMnr()[5]) {

 

[Der Müde Joe]

schon mal was von cast gehört?


Vector clonVektor = (Vector)fachFiles.clone();

 

[Der Müde Joe]

Vector<String> v = new Vector<String>();
v.add("1");
v.add("2");
v.add("3");
		
for(String s : v){
	v.remove(s); //geht nicht
}
		
Vector<String> v2 = (Vector<String>)v.clone();

for(String s : v2){
	v.remove(0); //ok ((nur als Bsp)
}
}

 

[JavaLehrtDemut]

oje, ja hab ich. Da hätte ich auch drauf kommen sollen. Genauso, wie darauf, dass ich nur mein Problem geklont habe

Ich kann ja nur über das array iterieren und somit dort keine Abfragefunktion aufrufen.

Vector CloneVektor = (Vector)fachFiles.clone();
			for(Student stud : CloneVektor.get(inputFiles.get(index).getNextElement().getMnr()[5]) {
				// Read new student from inputFiles
				//tmp = inputFiles.get(index).getNextElement();
				// Get new Student from tmp and store in Fach according to
				// last digit of MatNr.
				fachFiles.get(stud.getMnr()[5]).insertData(stud);
			}

Aber den Code hattest Du wohl nicht im Sinn. Wie kann mir der Clon bei meinem Problem helfen ?

 

[Der Müde Joe]

ääähhhhmmm?
what den that?

CloneVektor.get(inputFiles.get(index).getNextElement().getMnr()[5]

hum?
und das?

fachFiles.get(stud.getMnr()[5]).insertData(stud);

von einem Student ein Nummer hohlen die ein Array ist was als Identifier für ein fachFile ist und
dann den Student irgend? hinzufügen.

Vector copy = (Vector)fach.clone();

for(Strudent s : copy){
fach.doSomethingOnOriginal(s);
}

das geht doch sicher einfacher...

 

[JavaLehrtDemut]

na ja, das ist mein OriginalCode, ich habe ihn noch mal kommentiert.

(Ein Try/Catch Block terminiert die Endlosschleife)

while (true) {
				// Auslesen in ein Student-Objekt aus den InputFiles an der stelle index, welcher mit einer
                                // for-Schleife durchlaufen wird
				tmp = inputFiles.get(index).getNextElement();
				// Jetzt lese ich die letzte Stelle der Matrikelnummer aus "tmp.getMnr()[5]" und
                                // mache das Fach auf, in das ich (zwecks Sortierung) meinen Student "tmp" schreiben muss
                                fachFiles.get(tmp.getMnr()[5]).insertData(tmp);
			}

Beim unteren Code von Dir wird wenn ich das richtig lese in das Objekt "s" nacheinander die Werte von "copy" eingelesen
und dann immer das gleiche am Original-Vektor durchgeführt.

Aber mein Problem ist ja, dass ich die Info aus "s" brauche um davon abhängig (letzte Stelle MatNr) immer ein anderes
Fach aufmachen. Java bringt mich noch zum verzweifeln

Vector copy = (Vector)fach.clone();

for(Strudent s : copy){
fach.doSomethingOnOriginal(s);
}

 

[Der Müde Joe]

ok..kapiert...mehr oder weniger..

//jeder Student studi
for(Student studi : inputFiles){  //da sind die Studies drin oder?

int lastNumberOf = studi.getLastNumber();  //die Klasse Student brauch so ne Methode
FachFile ff = fachFiles.get(lastNumberOf);  // das Fach oder so?
ff.add(studi); // add den Studi dem Fach

}

hab ich das richtig verstanden?

 

[JavaLehrtDemut]

Unglaublich, aber wahr: Ich habe es endlich geschafft ! Vielen dank für die Hilfe !!!

Zur Info:

Mit diesem Code hat es dann funktioniert:

for (Student stud : inputFiles.get(index)) {
                         //mache ich nur, weil es später in tmp stehen muss				
                         tmp=stud;
			 //ist hier benötigter code
                          fachFiles.get(tmp.getMnr()[5]).insertData(stud);
}

Allerdings fliegt mir diese Exception nicht nur einmal :x aber das sollte ich dann alleine schaffen...

Schönen Abend noch und Grüße aus Wien.[/code]

 

[bygones]

mhm... was macht ihr hier ??? warum clone... warum true und break... hab ich was uebersehen...

nutz einen iterator und aendere ueber den iterator !

 

[Guest]

mhm... was macht ihr hier ??? warum clone... warum true und break... hab ich was uebersehen...

nutz einen iterator und aendere ueber den iterator !

Der Clone wurde am ende nicht wirklich verwendet, hätte aber den Sinn gehabt (So habe ich es verstanden), dass man eine Info die man VOR dem auslesen braucht, vorher schon auslesen kann. Ist aber egal wurde nicht verwendet.

true und break hatten den Sinn, den Vektor zu durchlaufen OHNE die Anzahl der gespeicherten Elemente zu kennen.

Deswegen auch kein Iterator.

Wenn es unklar ist, schau dir mal den ersten Post an und dann den über deinem, dann siehst Du was Problem und Lösung waren.

 

[bygones]

das clone kloniert einfach mehr nicht....

mhm ich denke du hast den sinn von iteratoren nciht verstanden, da musst du nicht wissen wieviele elemente im vector sind.

Iterator it = vector.iter();
while(it.hasNext()) {
it.next(); // was damit machen
}

da ein vector nix anderes als ein array ist kannst du auch

for(int i = 0; i < vector.size(); i++) {
vector.get(i);

oder mit java 5

for(Object o : vector) {
// mach was mit o
}

 

[Der Müde Joe]

for(Object o : vector) {
// mach was mit o
}

Sowie ich verstanden habe macht er etwas mit vector und nicht mit
o was man eigentlich würde. Darum gibts eine ConcurrentModificationException.

Und um das sichtbare hat es noch irgend eine Schleife welche Elemente aus
einem anderen Vector holt, welche irgendwelche Vectoren beinhaltet.

Jedenfalls will er während der Schleife (einer Schleife) den vector verändern, was
nicht geht und ne ConcurrentModificationException schmeisst....

 

[bygones]

natuerlich geht das nicht, daher iterator und ueber den iterator aendern