Vector.clear() oder neu initialisieren

[redbomber]

Hi zusammen,
ich habe eine Frage.
Ich habe eine Methode in der ein Vector initialisiert und mit Objekten befüllt zurück gegeben wird.
In dieser Methode kann eine Exception auftreten, die ich mit einem try catch finally block abfange.

im finally block wird der Vector zurück gegeben.

Wenn eine Exception auftritt möchte ich einen leeren Vector zurück geben.

Jetzt nur eine allgemeine Frage:
Was ist nun "korrekter".
Soll ich im catch-block den Vector neu initiallisieren?
Oder soll ich Vector.clear() aufrufen ?

Bsp:

public Vector method(){
Vector vector = new Vector();

   try{
     // do something
   } catch{
     vector = new Vector();
     // oder besser:
     // vector.clear()
   } finally {
     return vector;
   }
}[/Java]

 

[AmunRa]

ich würde clear machen, da du ja kein neues Objekt brauchst aber das ist nur geschmackssache.

Aber warum verwendest du eigentlich Vector und nicht ArrayList?

 

[xehpuk]

Hi,

ich denke nicht, dass es hier ein "korrekter" gibt.

Der Code sieht ja folgendermaßen aus:
[JAVA=789] /**
* Removes all of the elements from this Vector. The Vector will
* be empty after this call returns (unless it throws an exception).
*
* @since 1.2
*/
public void clear() {
removeAllElements();
}[/code]
[JAVA=602] /**
* Removes all components from this vector and sets its size to zero.
*
* <p>This method is identical in functionality to the {@link #clear}
* method (which is part of the {@link List} interface).
*/
public synchronized void removeAllElements() {
modCount++;
// Let gc do its work
for (int i = 0; i < elementCount; i++)
elementData = null;

elementCount = 0;
}[/code]
Der Unterschied ist also, dass das Array (mit seiner Kapazität) wiederverwendet wird, während bei einem neuen Vector ein neues Array angelegt wird.

Ich präferiere das Leeren. Da hat der GC nicht so viel aufzuräumen und zum anderen setze ich bei Methoden mit einer Liste als Rückgabe gern die am Anfang initialisierte Liste final.

Es gibt aber ein paar Dinge zu bemängeln:

• Das return sollte nicht in den finally-Block.
• Alles so abstrakt wie möglich halten.
• Einen generischen Typ verwenden.

Meine Variante sieht also so aus:

	public Collection<?> method() {
		final ArrayList<?> list = new ArrayList<Object>();
		try {
			// do something
		} catch (Exception e) {
			list.clear();
		}
		return list;
	}

 

[tfa]

Ich würde eine neue, leere Liste erzeugen. Man hilft dem GC am besten, indem man Objekte so kurzlebig hält wie möglich. Objektwiederverwertung ist eher schädlich (kommt natürlich auch auf den Einzelfall an).
Wenn man hier nur clear() macht, könnte man einen leeren Vector haben, der intern Platz für 100000 Elemente hat. Wer weiß, wie lange der im Speicher rum liegt.