Liste - Observer/Observeable - HashMap Hilfe

[Obli]

Guten Tag Java Community

Ich stehe gerade vor einem Problem und komme einfach nicht drauf wie ich das angehen soll:

Visual Class:

	public void actionPerformed(ActionEvent arg0) {
		// TODO Auto-generated method stub
		Button b = (Button)arg0.getSource();
		try{
			if(b == anlegen){
				int id = Integer.parseInt(getID.getText());
				String v = new String(getVorname.getText());
				String n = new String(getZuname.getText());
				Boolean g = checkM.getState();
				c.addPerson(id, v, n, g);
				getVorname.setText("");
				getZuname.setText("");
				getID.setText(""+c.ganz());
			}
			if(b == del)
			{
				c.delPerson(list.getSelectedIndex());
			}
		}catch(Exception e)
		{
			MessageLine.setText("Es ist ein Fehler aufgetreten!");
		}
	}
	
	public void update(Observable arg0, Object arg1) {
		// TODO Auto-generated method stub
		HashMap hm  = (HashMap)arg1;
		list.removeAll();
		
	    for (Iterator it = hm.entrySet().iterator();it.hasNext();) {
	    	Map.Entry me = (Map.Entry)it.next();
	    	list.add(me.getValue() + " " + me.getKey().toString());
	    }
	}

Klasse Controller, Löschfunktion:

public class Controller extends Observable{
	HashMap hm  = new HashMap();
	int id;
	char G;
	
	void addPerson(int i, String v, String n, Boolean g)
	{
		Person p = new Person(i,v,n,g);
		if(g){ G='M'; }else{ G='W'; }
		hm.put(p, p.gid());
		update();
	}
	
	void delPerson(int pos)
	{
		try
		{
			hm.remove(pos);
			update();
		}
		catch(Exception e)
		{
			System.out.println("Es ist ein Fehler aufgetreten");
		}
	}
	
	void update()
	{
		setChanged();
		notifyObservers(hm);
		clearChanged();
	}
}

Und zwar: Ich gebe Personendaten ein, dies soll in eine HashMap gespeichert werden und in einer Liste ausgegeben werden. Um die Daten zu Löschen, wähle ich in der Liste aus und drücke auf den Button löschen. Jetzt übergebe ich die Position des selectierten Datensatzes (welches in der Liste ja als String da steht) und weiß nicht mehr weiter
Ich weiß zwar die Position in der Liste, aber nicht die Position der Map.

Vielen dank für eure Hilfe!

 

[Andi_CH]

HashMaps sollten Typisiert werden - ich habe nämlich den Verdacht, dass du etwas verdrehst. Du verwendest die Person als Key und die gid als value ...

Auf die Daten in einer HashMap greifst du besser über den key zu, als über die Position.

Ich würde vorschlagen

HasMap<int, Person>

zu verwenden.

Dann kannst du die id des zu löschenden Datensatzes übergeben und löschen.

Falls ich falsch liege habe ich vielleicht dein Problem falsch erfasst ...

 

[sunny-boy]

Dein Model hat doch eine einduetige ID oder?

Dann kannst du deine Person so in die Map ablegegen...

HashMap<Integer, Person> personMap = new HashMap<Integer, Person>

personMap.put(Person.getID, Person)

Da nun der Key eindeutig ist, fragst du ihn im Listenelement ab und suchst in der Liste.

 

[Obli]

Oh mein Gott, was für ein billiger Fehler
Da habe ich wohl was verwechselt, hehe.

Danke Andi_CH und sunny-boy!

 

[Andi_CH]

Genau aus diesem Grund sollte die Map eben typisiert werden ;-)

 

[Obli]

"was für ein billiger Fehler"
Parameter vertauscht

 

[Andi_CH]

Der Kompiler würde dich ja dabei unterstützen, wenn du ihm die Möglichkeit dazu geben würdest ...

 

[Obli]

Ouh, gerade ist mir aufgefallen dass er ab "new" bis zum Semicolon alles rot unterwellt.

HashMap<Integer, Person> hm = new HashMap<Integer, Person>;

Fehlermeldung: Syntax error on token(s), misplaced token constructor(s)

Als neue Löschfunktion hab ich die hier, die aber doch nicht so prickelnd funktioniert.

void delPerson(int pos)
	{
		try
		{
			int i = 0;
			for (Iterator it = hm.entrySet().iterator();it.hasNext();i=i+1) {
		    	Map.Entry me = (Map.Entry)it.next();
		    	if(i==pos)
		    		hm.remove(me.getKey());
		           }
			update();
		}
		catch(Exception e)
		{
			System.out.println("Es ist ein Fehler aufgetreten");
		}
	}

Bin erst jetzt darauf gekommen und hab andere ID's vorgegeben (nicht von 0,1,2,3, ... sondern durchgemischt 6453,1234, ...)

Danke!

 

[Marco13]

HashMap<Integer, Person> hm = new HashMap<Integer, Person>(); // <--- Klammern....

 

[Marco13]

Das Löschen... in einer HashMap... mit einem Index? Kann nicht funktionieren (muss aber erstmal kapieren, was du vorhast, bevor ich da nen Tipp geben kann...)

 

[Obli]

Was ich vorhabe ist folgendes:
Ich habe 2 Fenster. In einem der beiden Fenster gebe ich Personendaten ein und gebe dem eine ID (ist gewollt), Vornamen, Nachnamen und Geschlecht.
Wenn ich auf Anlegen drücke, speichere ich die Daten in die HashMap. Anschließend wird die Update Methode aufgerufen um das 2. Fenster zu verständigen und dort gebe ich alle Daten in die Liste aus.
Nun wäre mein nächster Schritt das Löschen des gewählten Datensatzes in der Liste.

mfg

 

[sunny-boy]

Den Wert aus der markierten Liste in person speichern

Person person = meineListe.getSelectedValue();

Jetzt kannst du löschen

delPerson(person.getID());

Die Schleife in delPerson(int pos) kannst du dir sparen, wenn du den Key kennst und nur einen Wert löschen willst.

public void delPerson(int pos) {
  if (pos == 0 || !hashmap.containsValue(pos)) {
    return;
  } else { 
    hashmap.remove(pos);
}

 

[Andi_CH]

Also wenn man den key erst aus dem Objekt holen muss um danach das Objekt löschen zu können, frage ich mich schon ob die HashMap das richtige ist ??? Wenn du mit Inizes arbeiten willst währe ein Vector geeigneter.

Du köntest genau so gut HashMap<Person, Person> machen, denn geht hm.remove(me) (Nein, nicht erschlagen bitte - das ist nicht ernst gemeint)

 

[Obli]

Ich habe so ein ähnliches Bsp. schon mit Vektoren machen müssen. Das geht natürlich einfacher.
Mein Lehrer hat aber gesagt, damit wir den Umgang mit den Maps lernen, sollten wir das hier mit Map machen.

@sunny-boy
Das kann doch gar net gehen?
1. Gibts bei mir keine getSelectedValue() Methode
2. Wie willst du die ID von einem String entnehmen die in der Liste steht?

Irgendwie scheint mir die Sache grad etwas Komplex

 

[sunny-boy]

Also hier mal ein kleiner Versuch, vll hilft es ja.
Erstmal die Liste

final JList list = new JList();
        list.setModel(new AbstractListModel() {
            Person p1 = new Person("Dennis", 23);
            Person p2 = new Person("Chris", 25);
            Person[] values = new Person[] {p1,p2};

Hier ein Testbutton...

JButton btnOk = new JButton("OK");
        btnOk.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                Person person = (Person) list.getSelectedValue();
                System.out.println(person.getName());
                
            }
        });

Jetzt noch toString() überschreiben z.B.:

public String toString() {
            return "" + name;
        }

 

[Andi_CH]

Ich habe so ein ähnliches Bsp. schon mit Vektoren machen müssen. Das geht natürlich einfacher.
Mein Lehrer hat aber gesagt, damit wir den Umgang mit den Maps lernen, sollten wir das hier mit Map machen.

Du hast duch ein Gui (Wie gesagt, bin kein GUI-Guru) aber die Personen werden irgendwie im GUI dargestellt und dann - Klick - diese löschen.

Also schau doch, dass die ID der Person vom GUI in deine Businesslogik gegeben wird - damit ist das Problem gelöst ....

Ansonsten - meine Glaskugel ist immer noch in Reparatur - wie wärs mit dem ganzen Code - oder ist der zu riesig? (Geht auch als zip-File)

 

[Obli]

sunny-boy. Swing verwende ich nicht

@Andi
Ja habe Visuelle Klassen und ich wag es zu bezweifeln dass du eine Glaskugel hast
Hier das ZIP Archiev: File-Upload.net - person.zip

 

[Andi_CH]

Das mit der Glaskugel ist ein stehender Begriff um auszudrücken dass etwas unklar ist ;-)

Wenn du ein Elemnt löscht versuchst du das mit dem Index der GUI - Liste (z.B. das zweite Element auf dem GUI hat den Index 1)
Aus der Map löschen musst du das aber mit dem key und der war in meinem Fall 222.

Da du ja sagst dass die Personen in einer Map bleiben müssen, muss es eine andere Möglichkeit geben vom Index auf den Key zu kommen.

1. Vorschlag generier dir einen Vector<int> in dem du die Keys in der Reihenfolge ablegst wie sie auf dem GUI erscheinen (aber beim löschen immer schön aktuell halten.)

2. Vorschlag (nachdenken was ich da genau mache kannst du selbst

	void delPerson(int pos)
	{
		try
		{
			Integer key = (Integer)hm.keySet().toArray()[pos];
			hm.remove(key);
			update();
		}
		catch(Exception e)
		{
			System.out.println("Es ist ein Fehler aufgetreten");
		}
	}

 

[Obli]

Jetzt klappts, super!

Nachdenken ist gut, nur draufgekommen bin ich bis jetzt noch nicht

 

[Obli]

Hey Andi!
Könntest du mir das bitte erklären? "Abschreiben" kann ja jeder, davon habe ich aber nichts