Arraylist

[star-style]

Hallo zusammen,
ich bin gerade dabei ein Java-Code zu verstehen wobei es um ArrayLists geht.

Meine Frage generell: Kann man den folgenden Code von nicht umschreiben, damit das verständlicher wird?

public Student entferneStudMitMatNr(String matrikelNr)
{
int index = -1;
for(int i =0; i < veranstaltung.size(); i++)
{
Student stud = veranstaltung.get(i);
String matNr = stud.gibMatrikelnummer();

if(matrikelNr.equals(matNr))
{
index = i;
}
if(index > 0)
{
return varanstaltung.remove(index);
}
else
{
return null;
}
}
}

Also hier wird ja in der ArrayList welches "veranstaltung" heißt, der Student mit der matrikelNummer, welches sich im Übergabeparameter befindet entfernt. Alles schön und gut aber warum setzt man hier nach der for-Schleife das:

Student sud = veranstaltung.get(i);
String matNr = stud.gibMatrikelnummer();

Ich selbst hätte das einfach so geschrieben:

public Student entferneStudMitMatNr(String matrikelNr)
{
for(int i=0; i< veranstaltung.size(); i++)
if(veranstaltung.get(i).getMatrikelnummer.equals(matrikelNr))
{
veranstaltung.remove(veranstaltung.get(i));
}
return veranstaltung;
}

Danke für euere Hilfe..

 

[SlaterB]

in diesem Fall ginge es weitgehend, wobei dein return in Zeile 6 gehörte,
Zeile 8 entfällt, du willst schon gar nicht die Liste zurückgeben,
> veranstaltung.remove(veranstaltung.get(i));
ist auch unnötig kompliziert, im Original besser


die lokalen Variablen sind offensichtlich Spielerei,
mit dem remove muss man allgemein aufpassen beim Durchlauf einer Liste, hier mag es gehen,
wenn du aber mehrere Elemente entfernen willst wäre es gefährlich, ein i zu überspringen
(da intern aufgerückt wird, der Platz des gelöschten i nicht unbedingt erneut geprüft wird),
Iterator ginge da noch besser

 

[star-style]

also du meinst quasi:

return veranstaltung.remove(veranstaltung.get(i));

 

[illusi0n]

Hey,


also ich würde sagen (aus eigener Erfahrung ), dass die erste Variante jemand geschrieben hat, der noch relativ neu ist in der Javawelt. Das dient sicher nur der Übersichtlichkeit (sollte es ein Tutorial sein oder gar ein Arbeitsblatt gilt nur der Satz mit der Übersichtlichkeit).
Das soll heißen: Es ist natürlich möglich so wie du es geschrieben hast den Code zu optimieren, sofern du dann immer weisst (bei komplexeren Projekten z.B.) an welcher Stelle was geprüft wird.

Dies hier:

if(veranstaltung.get(i).getMatrikelnummer.equals(matrikelNr))

kann irgendwann noch länger werden, weil irgendwelche Bedingungen dazu kommen.

veranstaltung.remove(veranstaltung.get(i));

Damit solltest du immer aufpassen. Single-Elements aus 'ner Liste zu löschen mit ...(i) ist möglich - allerdings solltest du mehrere löschen wollen: Finger weg von dem simplen (i) - da in Listen immer aufgerückt wird und sich somit die Elemente verschieben und du immer etwas löschst was du eigentlich nicht löschen wolltest


[EDIT]Verdammt, wieder zu langsam ![/EDIT]

 

[AquaBall]

Und? Wenns funktioniert, dann ist's ja ok!

public Student entferneStudMitMatNr(String matrikelNr) {
	for(int i=0; i< veranstaltung.size(); i++) {
		if(veranstaltung.get(i).getMatrikelnummer.equals(matrikelNr))
			veranstaltung.remove(veranstaltung.get(i));
	}
	return veranstaltung;
}

In Punkto Verständlichkeit ist halt empfehlenswert:

• for each Schleife
• Einrückung
• sprechende Hilfsvariable.

public Student entferneStudMitMatNr(String matrikelNr) {
	for (Student stud:veranstaltung) {
		String matNr = stud.gibMatrikelnummer();
		if (matrikelNr.equals(matNr)) 
			return varanstaltung.remove(stud);
	}
	return null;
}

 

[bygones]

remove gibt bei einer List boolean zurueck obs geklappt hat, nicht das objekt