Klassen Doppelte Elemente aus Container entfernen

[thprogmas]

Hallo,
Hier nun mein erster Beitrag, da ich im Forum nichts passendes gefunden habe.
Ich habe einen Konstruktor einer Klasse, die selber eine "selbstdefinierte Collection" werden soll.
Diesem Konstruktor wird eine Collection übergeben.
Also: public xxxCollection (Collection<xxx> yyy)
Jetzt soll, falls die übergebene Collection doppelte Elemente besitzt eine Exception geworfen werden.
Weiterhin habe ich noch einen default constructor, der einfach ein neues HashSet initialisiert (um beim adden später Doppelbelegungen zu vermeiden).

Jetzt meine Frage: Wie handle ich diese übergebene Collection bzw. wie soll ich denn durch eine abstrakte Klasse durch iterieren, ohne zu wissen, welche Spezifikation sie hat?
Kann ich die übergebene Collection einfach irgendwie dumm in ein HashSet casten?

Vielen Dank schon mal für jede Antwort!

 

[Flown]

Also wenn du wissen möchtest, ob die übergebene Collection ein Set ist dann mach einfach:

if(yyy instanceof Set) {
  Set<xxx> set = (Set<xxx>) yyy;
}

Sonst musst du dir das Set selber zusammenbauen:

Set<xxx> seen = new HashSet<>();
for(xxx element : yyy) {
  if(!seen.add(element)) {
    throw new IllegalArgumentException("Duplicate elements");
  }
}

 

[MarzAttak]

Bin nicht der große Collection-Fachmann, aber wenn es darum geht, eine Exception zu schmeißen wenn doppelte Elemente vorhanden sind, würde ich selbst folgendes mal ausprobieren:
Im Konstruktor ein HashSet anlegen, dieses mit Hilfe von .addAll() mit der übergebenen Collection füllen und danach die beiden vergleichen, ob sie dieselbe Länge haben. Das Test-Hashset müsste doppelte Einträge eigentlich von sich aus rausschmeißen und dann kürzer sein. In diesem Fall dann die Exception auslösen.
Falls ich Murks erzähle, erschlagt mich nicht... ich wollte nur helfen.

 

[mrBrown]

Im Konstruktor ein HashSet anlegen, dieses mit Hilfe von .addAll() mit der übergebenen Collection füllen und danach die beiden vergleichen, ob sie dieselbe Länge haben. Das Test-Hashset müsste doppelte Einträge eigentlich von sich aus rausschmeißen und dann kürzer sein. In diesem Fall dann die Exception auslösen.

So würd ich's auch am ehesten machen - dem im Standard-Kostruktor erstellen HashSet alle adden (dann ist auch gleichzeitig ne Defensive Kopie angelegt), und danach die Größen vergleichen, sind sie ungleich die Exception schmeißen

 

[Flown]

Warum Längen vergleichen, wenn man on-the-fly die Exception erzeugen kann (fail-fast? & extra Bonus -> man kann sogar das Element identifizieren)

 

[MarzAttak]

Klar, add() liefert ein boolean zurück. Das ist performanter

 

[mrBrown]

Warum Längen vergleichen, wenn man on-the-fly die Exception erzeugen kann (fail-fast? & extra Bonus -> man kann sogar das Element identifizieren)

Weil man dann nicht per Hand iterieren muss - wenn man natürlich noch ausgeben möchte, welches Schuld war, führt kein Weg dran vorbei