JComboBox setSelectedItem(Object) Problem

[casi91]

Servus Community,

habe schon durch das Forum geschaut und auch ähnliche Threads gefunden, leider konnte mir keiner die Erleuchtung bringen.
Vielleicht klappts ja mit einem eigenen

Folgendes:
Ich lade Objekte aus einer DB und zeige diese in einer Combobox an

List<?> domainListe = crit.list();
for (int i=0; i<domainListe.size(); i++)		
{
	Kategorie kategorie = (Kategorie) domainListe.get(i);
	combobox.addItem(kategorie);
}

(Kategorie besitzt eine "toString()" Methode.)


Danach kann der Benutzer entsprechend einen Wert auswählen und speichern.
(Soweit so gut, dass funktioniert)

Nun soll beim nächsten Laden in der Combobox direkt das richtige Element selektiert sein.
Dies wollte ich durch die Methode: "setSelectedItem" lösen.

Da diese Methode intern ja mit "equals" arbeitet, dachte ich auch, dass es kein Problem ist.
Jedoch scheinen die Objekte nicht gleich zu sein.
Hier ein kleiner Test um zu schauen ob die Objekte gleich sind:

Kategorie kategorie = xxx.getMyKategorie();
Kategorie kategorie2 = combobox.getItemAt(1);
if (kategorie.equals(kategorie2))
{
	kategorie = kategorie2;
	combobKategorie.setSelectedItem(kategorie);
}

auch hierbei schlägt das "equals" fehl.
Wenn ich mir die Objekte jedoch im Debugger anschaue, scheinen diese soweit den gleichen Inhalt zu haben.

Was habe ich übersehen?

Danke schonmal

 

[Marco13]

'Kategorie' muss die equals- und hashCode-Methoden richtig überschreiben. Moderne IDEs können das automatisch (Eclipse Rechtsklick auf Klasse -> Source -> Generate equals and hashCode). Ansonsten websuche oder genauer beschreiben wie "Kategorie" aussieht.

 

[casi91]

Hallo,
leider hat das mit "Generate equals..." nichts geholfen.

hier mal der Quellcode meine Klasse "Kategorie" (nachdem ich "Generate..." ausgeführt habe)
(eben vergessen zu erwähnen, sieht man nun aber in der Klasse, ich arbeite mit Hibernate und Annotations)

package my.test.Database;

import java.util.HashSet;
import java.util.Set;

import javax.persistence.*;

@Entity
@Table( name = "KATEGORIE")
public class Kategorie {

	@Id
	@GeneratedValue(strategy = GenerationType.IDENTITY) 
	@Column(name = "ID")	
	private Integer id;
	
	@Column(name="BEZEICHNUNG")
	private String bezeichnung;	
	
	@OneToMany(fetch=FetchType.EAGER, cascade = CascadeType.ALL, mappedBy="myKategorie")
	private Set <Filiale> filiale = new HashSet<Filiale>();	
	
	public Kategorie() {
	}
	
	public Integer getId() {
		return id;
	}

	public void setId(Integer id) {
		this.id = id;
	}

	public String getBezeichnung() {
		return bezeichnung;
	}

	public void setBezeichnung(String bezeichnung) {
		this.bezeichnung = bezeichnung;
	}

	public Set <Filiale> getFiliale() {
		return filiale;
	}

	public void setFiliale(Set <Filiale> filiale) {
		this.filiale = filiale;
	}
	
	public String toString() { 
		return this.getBezeichnung(); 
	}

	@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result
				+ ((bezeichnung == null) ? 0 : bezeichnung.hashCode());
		result = prime * result + ((filiale == null) ? 0 : filiale.hashCode());
		result = prime * result + ((id == null) ? 0 : id.hashCode());
		return result;
	}

	@Override
	public boolean equals(Object obj) {
		if (this == obj)
			return true;
		if (obj == null)
			return false;
		if (getClass() != obj.getClass())
			return false;
		Kategorie other = (Kategorie) obj;
		if (bezeichnung == null) {
			if (other.bezeichnung != null)
				return false;
		} else if (!bezeichnung.equals(other.bezeichnung))
			return false;
		if (filiale == null) {
			if (other.filiale != null)
				return false;
		} else if (!filiale.equals(other.filiale))
			return false;
		if (id == null) {
			if (other.id != null)
				return false;
		} else if (!id.equals(other.id))
			return false;
		return true;
	} 

}

 

[casi91]

Ah mir ist grad selbst was aufgefallen ^.^

Ich habe ja noc ein Attribut einer anderen Klasse (Filiale) in meinr Klasse "Kategorie"
Also muss ich dort ja wahrscheinlich auch das ganze mit Generate... machen?

Das hab ich auch gemacht, nur muss ich jetzt schauen, warum ich einen Fehler bekomme:

"illegal access to loading collection"

Aber das gehört ja erst nicht mehr zu dieser Frage.

Überschreibt man hashCode() und equals(Object obj) am besten immer?
Oder wirklich nur dann, wenn mans auch zu 100% braucht?

 

[casi91]

So funktioniert nun.
Danke für den Tipp, genau das war mein Problem.

Das andere Problem das ich hatte war ein Verständnissproblem mit Hibernate, woran ich noch arbeiten muss.
Aber das, ist eine andere Geschichte :-D

 

[Marco13]

Überschreibt man hashCode() und equals(Object obj) am besten immer?
Oder wirklich nur dann, wenn mans auch zu 100% braucht?

Nicht so einfach. Das Problem ist, dass man u.U. nicht weiß, ob jemand anderes es vielleicht braucht - oder eben "versehentlich" verwendet, weil er solche Objekte in eine List packt oder so. Man kann wohl sagen, dass es bei "Modellobjekten" praktisch IMMER empfehlenswert ist. Bei sowas wie einer GUI-Klasse macht es natürlich keinen Sinn....