Probleme mit Collection

[SuperJavaMan]

hallo leute

ich habe eine gui in der name, nummer im textfeld eingelesen werden und 2 radiobuttons ob der eintrag dienstlich oder private ist. dazu hab ich eine daten klasse eintrag die einfach die eingelesenen daten übergibt an die model klasse mit einer linkedlist in der alle einträge gespeichert werden.
und jetzt das problem wenn ich einen eintrag einlese und dann wieder den gleichen einlese soll eine fehlermeldung erscheinen tut es aber nicht.

hier der code

GUI/MAIN

private void onSpeichern(java.awt.event.ActionEvent evt)                             
  {                                 
    String name = tfName.getText();
    String nummer = tfNummer.getText();
    boolean privat = rbPrivat.isSelected();
    Eintrag eintrag = new Eintrag(name, nummer, privat);
            try {
            mtm.addEintrag(eintrag);
        } catch (Exception ex) {
            JOptionPane.showMessageDialog(null, "Dieser Eintrag ist schon in der Liste");
        }
  }

MODELKLASSE

public class MiniTelModel extends AbstractListModel {

    private LinkedList<Eintrag> list = new LinkedList<>();

    @Override
    public int getSize() {
        return list.size();
    }

    @Override
    public Object getElementAt(int index) {
        return list.get(index);
    }

    public void addEintrag(Eintrag eintrag) throws Exception {
        if (!list.contains(eintrag)) {
            list.add(eintrag);
            fireContentsChanged(this, list.size() - 1, list.size() - 1);
            System.out.println(list);
        } else {
            throw new Exception("doppeltes einfügen verboten!");
        }
    }

    public void clearEintrag(Eintrag eintrag) throws Exception {
        if (list.contains(eintrag)) {
            list.remove(eintrag);
            fireContentsChanged(this, list.size() - 1, list.size() - 1);
            System.out.println(list);
        } else {
            throw new Exception("eintrag wurde nicht gefunden");
        }
    }

}

löschen funktioniert auch nicht wenn ihr mir bitte dabei helfen könntet
danke wie immer im voraus
danke
lg superjavaman

 

[Harry Kane]

Wie sieht die Eintrag-Klasse aus?

 

[Network]

Das kommt daher das du abfrägst per List.contains(Object o) ob sich das Objekt bereits in der Liste befindet, das tut es aber nicht, denn

Eintrag a = new Eintrag( "Holger P", "01805 69", false );
Eintrag b = new Eintrag("Holger P", 01805 69", false );
if( a == b ); // returns false
else;

Die List-Klasse weiß nicht wann für dich zwei Objekte gleich sind, sie kann dir aber sagen wann zwei Objekte ein und dasselbe Objekt sind und das tut die Methode .contains() - sie überprüft das was ich oben geschrieben habe.

Dir bleibt wohl nichts anderes übrig als die Liste manuell Eintrag für Eintrag zu durchforsten und die einzelnen Variablen miteinander zu vergleichen.


Mehr zu dem Thema:
http://www.java-blog-buch.de/0302-strings-vergleichen/
>> In dem Artikel wird die Methode .equals() erwähnt, wenn du diese auch für deine eigenen Objekte verwenden möchtest musst du die Methode "unter Umständen" vorher aber überschreiben

 

[Harry Kane]

Die List-Klasse weiß nicht wann für dich zwei Objekte gleich sind,...

Doch, natürlich. Ein Objekt o1 is gleich einem zweiten, wenn o1.equals(o2) true ergibt.

..., sie kann dir aber sagen wann zwei Objekte ein und dasselbe Objekt sind und das tut die Methode .contains() - sie überprüft das was ich oben geschrieben habe.

Bitte? Glaubst du wirklich, dass "contains" die Objektidentität verwendet und NICHT die inhaltlichge Gleichheit? Wo hast du das denn her?

>> In dem Artikel wird die Methode .equals() erwähnt, wenn du diese auch für deine eigenen Objekte verwenden möchtest musst du die Methode "unter Umständen" vorher aber überschreiben

Das "unter Umständen" kann ersetzt werden durch "in den meisten Fällen".
Die Klasse Object hat bereits eine equals-Methode, die allerdings nur die Objektidentität prüft. Wenn eine Klasse noch Zustandsinformationen hat (und die Klasse Eintrag hat solche ja), müssen für eine gescheite Überprüfung auf inhaltliche Gleichheit aller relevanten Zustandsinformationen ebenfalls auch Gleichheit überprüft werden, und das erfolgt halt in einer überschriebenen equals-Methode.
Mit meiner Frage wollte ich den TE dazu motivieren, den Quellcode von Eintrag zu zeigen. Ich wette, dass die Klasse Eintrag die Methode equals(Object other) nicht oder falsch überschreibt.

 

[ottens]

kannst du nicht einfach an die Datenbank eine SQL-Abfrage schicken und abfragen ob das Object schon existiert ?

 

[Network]

@Harry Kane
Naja die Standardimplementation von .equals(Object o) lautet

public boolean equals(Object obj) {
return (this == obj);
}

Entsprechen überprüft list.containts(Object o) ob das Objekt das selbe ist und nicht auf den Inhalt des Objektes wie du beschrieben hast.

Daher meine Vorschläge die Liste manuell abzuarbeiten oder wie in dem Artikel beschrieben die .equals(Object o) Methode zu überschreiben.

Die Worte "unter Umständen" waren hier wohl falsch gewählt.
Man kann eigentlich sagen das jede Klasse die .equals() Methode überschreiben und eine eigene Überprüfungslogik verwenden sollte. Das wäre natürlich ein großer Aufwand weshalb sich das für Ein-Mann-Projekte nur lohnt wenn die Klassen überhaupt auf Gleichheit aufeinander überprüft werden die über "==" hinausgeht.
Aber "in den meisten Fällen"? Naja wenn du meinst.

 

[kaoZ]

Die List-Klasse weiß nicht wann für dich zwei Objekte gleich sind, sie kann dir aber sagen wann zwei Objekte ein und dasselbe Objekt sind und das tut die Methode .contains() - sie überprüft das was ich oben geschrieben habe.

Wenn equals(Object o) und hashcode() nicht überschrieben wurden, wird durch die Standardimpementation nur geprüft ob die übergebene Referenz auf den gleichen reservierten Bereich im Speicher / Stack zeigt oder nicht, sprich ob das inhaltlich was identisch ist ist hier vollkommen irrelevant, dazu müsstest du dann selbst eine Implementation liefern wann du zwei Einträge als gleich deutest.

Bei eigens erstellten Klassen , sollte equals(Object o) und hashcode() im Standardfall IMMER überschrieben werden.

Identisch und gleich sind halt immer noch zwei paar Schuhe.

 

[SuperJavaMan]

Ja genau
der fehler lag bei der equals methode in der eintrag klasse
danke leute