Hashmap und ContainsValue()

[guradon]

Hallo,

ich habe ein Problem und schon gegoogelt und bei sun.com in der doku gelesen, aber finde gerade keine Lösung dafür, also hier mal die Frage:

Map<String,Spieler> Spielerliste = new HashMap<String,Spieler>();
		
public String Spielerregistrieren(String Vorname,String Nachname,String Kontaktdaten)
	{
	Spieler neuerSpieler= new Spieler(Vorname, Nachname, Kontaktdaten);
	SpielerID = neuerSpieler.getSpieler_ID();	

	if((Spielerliste.containsKey(SpielerID))||(Spielerliste.containsValue(neuerSpieler))){
	// abbruch der Erstellung des Spielers
	return "Spieler bereits vorhanden oder ID bereitsvergeben!";
	}
	else{
	Spielerliste.put(SpielerID,neuerSpieler);
    // Spieler wurde zur Liste hinzugefügt!
	 return "Spieler wurde erfolgreich registriert mit ID:"+SpielerID+"!";    
	}}

Das ganze funktioniert soweit, dass alle Einträge ausgegeben werden und bei gleichem Key das neue Objekt nicht in die Hashmap geschrieben wird. Aber wenn ich ein Objekt mit den gleichen Daten(Value) erzeuge, wird es trotz der containsValue() Abfrage in die Hashmap geschrieben. In der vorliegenden Version habe ich noch die ID in der Klasse Spieler drin stehen, aber ich habe es bereits ohne diese Variable getestet und trotzdem geht es nicht.

public class Spieler {
	String Vorname,Nachname,Kontaktdaten;//Kontaktdaten: Telefon,Adresse,Email 	
	 
	static int Spieler_counter; 
	// dient dazu die ID nach Anzahl der spieler festzulegen wird beim erstellen jedes Spielers erhöht.
	/**
	 * @param vorname
	 * @param nachname
	 * @param spieler_ID
	 */
public Spieler(String vorname, String nachname, String kontaktdaten) {
	    //vorher prüfen ob Spieler in der Liste vorhanden ist.
		super(); // Ka warum der beim automatischen erstellen eines Konstruktors hiern super() schreibt
		Vorname = vorname;
		Nachname = nachname;
		Kontaktdaten = kontaktdaten;
	this.Spieler_increase();
		
	}

Muss ich die containsValue() Methode irgendwie überlagern, damit die Values bekannt sind ?

 

[SlaterB]

Gegenfrage: welchen guten Grund hast du denn bisher implementiert,
so dass Java erkennen kann, dass zwei Spieler gleich sind?

z.B. eine equals-Operation?
erstelle zwei Spieler mit gleichen Daten und führe
System.out.println("gleich?: "+spieler1.equals(spieler2));
aus,
wenn das nicht geht, dann die Map bestimmt auch nicht

 

[guradon]

Hey Danke für den Hinweis!
Ich habe jetzt in der Spieler Klasse eine equals Funktion geschrieben, die geht auch mit einzelnen Elementen, ich wollte jetzt mit folgender containsValue Funktion das ganze prüfen:

public boolean containsValue(Spieler spieler)
         {System.out.println("Spieler bereits vorhanden oder ID bereitsvergeben!");
          Iterator it = Spielerliste.entrySet().iterator(); 
           while (it.hasNext()) {
	      Map.Entry entry = (Map.Entry)it.next();
	      if(entry.equals(spieler)){return true;}
            
        }return false;}

Das klappt aber wieder nicht muss ich beim verwenden von entry noch was beachten?
Die equals Funktion:

public boolean equals(Spieler spieler){
if((this.Vorname.equals(spieler.Vorname))&&
(this.Nachname.equals(spieler.Nachname))&&
(this.Kontaktdaten.equals(spieler.Kontaktdaten)))
{
 System.out.println("Spielerbereits vorhanden");return true;}
else{System.out.println("Spieler nicht vorhanden");
return false;}}

 

[SlaterB]

ein Map.Entry ist ein Map.Entry, ein Map.Entry kann niemals deinem Spieler gleich sein,
vergleiche entry.getValue().equals(spieler) oder ähnliches,

aber map bietet doch schon eine Operation containsValue()?

hier der Code aus HashMap, den baust du gerade nach :

    /**
     * Returns <tt>true</tt> if this map maps one or more keys to the
     * specified value.
     *
     * @param value value whose presence in this map is to be tested.
     * @return <tt>true</tt> if this map maps one or more keys to the
     *         specified value.
     */
    public boolean containsValue(Object value) {
	if (value == null) 
            return containsNullValue();

	Entry[] tab = table;
        for (int i = 0; i < tab.length ; i++)
            for (Entry e = tab[i] ; e != null ; e = e.next)
                if (value.equals(e.value))
                    return true;
	return false;
    }

 

[guradon]

Hallo,

Ja da hast du Recht.
Muss ich ContainsValue so überschreiben, dass meine eigene equals Funktion verwendet wird, wie kann ich das machen ?

 

[SlaterB]

es wird die equals-Version des übergebenen Objekts genommen,
gibt Spielerliste.containsValue(neuerSpieler) ein und fertig, so hattest du es ja schon am Anfang

spielerMap wäre übrigens ein besserer Name, Variablen immer klein!

 

[guradon]

Ok, das Problem ist aber das es mit der obigen equals Funktion nicht geht. Also wenn ich wie du gesagt hast
spielerMap.containsValue(neuerSpieler) aufrufe und die obige equals Funktion verwende, bekomme ich immer nur false. Hab ich bei der euals Funktion was falsch gemacht?

 

[SlaterB]

stimmt, da war noch was falsch in equals, der Parameter muss zwischen Object sein, so gehts:

public class Test
{

    public static void main(String[] args)
        throws Exception
    {

        Map<String, Spieler> map = new HashMap<String, Spieler>();

        Spieler sp1 = new Spieler("a", "b", "c");
        Spieler sp2 = new Spieler("a", "b", "c");
        System.out.println("gleich?: " + sp1.equals(sp2));

        map.put("key", sp1);
        System.out.println("contains: " + map.containsValue(sp2));
    }
}


class Spieler
{
    String Vorname, Nachname, Kontaktdaten;// Kontaktdaten: Telefon,Adresse,Email

    public Spieler(String vorname, String nachname, String kontaktdaten)
    {
        Vorname = vorname;
        Nachname = nachname;
        Kontaktdaten = kontaktdaten;

    }

    public boolean equals(Object o)
    {
        if (!Spieler.class.isInstance(o))
        {
            return false;
        }
        Spieler spieler = (Spieler)o;
        if ((this.Vorname.equals(spieler.Vorname)) && (this.Nachname.equals(spieler.Nachname))
            && (this.Kontaktdaten.equals(spieler.Kontaktdaten)))
        {
            System.out.println("Spielerbereits vorhanden");
            return true;
        }
        else
        {
            System.out.println("Spieler nicht vorhanden");
            return false;
        }
    }
}

Variablen klein schreiben!

 

[guradon]

vielen Dank, jetzt kann ich beruhigt rasten!

 

[Guest]

Du hast soeben den Vertrag von equals/hashCode gebrochen:

http://java.sun.com/javase/6/docs/api/java/lang/Object.html#equals(java.lang.Object)

Note that it is generally necessary to override the hashCode method whenever this method is overridden, so as to maintain the general contract for the hashCode method, which states that equal objects must have equal hash codes.

Hier mein Vorschlag:

class Spieler
{
	public final String Vorname, Nachname, Kontaktdaten;

	public Spieler(String vorname, String nachname, String kontaktdaten)
	{
		Vorname = vorname;
		Nachname = nachname;
		Kontaktdaten = kontaktdaten;
	}

	public boolean equals(Object obj)
	{
		if (!(obj instanceof Spieler)) return false;

		final Spieler spieler = (Spieler) obj;
		return Vorname.equals(spieler.Vorname)
			&& Nachname.equals(spieler.Nachname)
			&& Kontaktdaten.equals(spieler.Kontaktdaten);
	}
	
	public int hashCode()
	{
		return (Vorname.hashCode() * 37 + Nachname.hashCode()) * 37 + Kontaktdaten.hashCode();
	}
}

Fred