Schleife funktioniert nicht richtig

[F1NISH]

Hallo liebes Java Forum,

und zwar funktioniert meine Schleife nicht so wie ich es haben will und ich habe nach meinen Kenntnissen schon alles ausprobiert, aber finde keine Lösung. Ich erzeuge aus einer XML Datei eine ArrayListe mit Objekten, welche ich dann auswerte und je nach Attributen in zwei weitere Listen (regelnAktiv, regelnInaktiv) abspeicher. Nun möchte ich prüfen ob das Objekt aus der neu erzeugten ArrayListe schon in den beiden anderen Listen (aktiv,Inaktiv) vorhanden ist bzw. existiert und wenn NICHT dann soll es in die regelnAktiv Liste. Hier meine Schleife dazu:

            for (Regel r : neueListe) {
                for (Regel t : regelnInaktiv) {
                    for (Regel x : regelnAktiv) {
                        if (regelnInaktiv.contains(r) == false && regelnAktiv.contains(r) == false) {
                            used.add(r);
                        }

                    }
                }
            }

Ich hoffe ihr habt weitere Ideen !

Viele Grüße

 

[krgewb]

for (Regel r : neueListe) {
    if (regelnInaktiv.contains(r) || regelnAktiv.contains(r)) {
        used.add(r);
    }
}

 

[F1NISH]

for (Regel r : neueListe) {
    if (regelnInaktiv.contains(r) || regelnAktiv.contains(r)) {
        used.add(r);
    }
}

Hier wird das Objekt ja hinzugefügt wenn in beiden Listen es schon vorhanden ist, aber es soll genau andersrum sein, dass es hinzugefügt wird wenn es in BEIDEN NICHT vorhanden ist. Habe auch true gegen false ausgetauscht, funktioniert immer noch nicht und jedes Objekt wird einfach hinzugefügt.

 

[Robat]

Wie ist denn die equals Methode deines Objektes überschrieben?

 

[F1NISH]

Wie ist denn die equals Methode deines Objektes überschrieben?

Garnicht, habe kein Override vorgenommen.

 

[mihe7]

Wenn die Gleichheit der Objekte sich nicht über die Referenz definiert, musst Du natürlich equals überschreiben. Unabhängig davon hat Dein Code ein Laufzeit-Problem, weil Du mehrere Schleifen überflüssigerweise verschachtelst.

for (Regel r : neueListe) {
    if (!regelnInaktiv.contains(r) && !regelnAktiv.contains(r)) {
        used.add(r);
    }
}

Der Name "used" ist hier auch nicht ganz klar, liegt aber evtl. daran, dass es sich nur um einen Ausschnitt handelt.

 

[F1NISH]

Wenn die Gleichheit der Objekte sich nicht über die Referenz definiert, musst Du natürlich equals überschreiben. Unabhängig davon hat Dein Code ein Laufzeit-Problem, weil Du mehrere Schleifen überflüssigerweise verschachtelst.

for (Regel r : neueListe) {
    if (!regelnInaktiv.contains(r) && !regelnAktiv.contains(r)) {
        used.add(r);
    }
}

Der Name "used" ist hier auch nicht ganz klar, liegt aber evtl. daran, dass es sich nur um einen Ausschnitt handelt.

Aber ich nutze die equals Methode doch garnicht, sondern nur die contains Methode. Das mit dem Laufzeit Problem hab ich mir schon gedacht, nur so dachte ich ich durchlaufe die drei Listen gleichzeitig und es funktioniert. Wenn ich deine Schleife laufen lasse, dann trifft irgendwie immer noch alles zu und alle Objekte werden der "used" ArrayListe hinzugefügt.

 

[Robat]

Ja und die contains Methode prüft intern ob dein übergebenes Objekt schon in der Liste ist, in dem es die Objekt via equals Methode vergleicht

 

[mihe7]

Aber ich nutze die equals Methode doch garnicht, sondern nur die contains Methode.

contains verwendet equals

Die Sache ist ganz einfach: um festzustellen, ob ein Objekt in einer Liste ist, müssen die Objekte der Liste mit dem gesuchten Objekt verglichen werden. Für den Vergleich von Objekten ist die equals-Methode vorgesehen.

Wenn ich deine Schleife laufen lasse, dann trifft irgendwie immer noch alles zu und alle Objekte werden der "used" ArrayListe hinzugefügt.

Wie gesagt: Du musst equals überschreiben, sofern die Gleichheit nicht über die Referenz definiert ist.

 

[F1NISH]

Achso okay das war mir nicht bewusst. Wie überschreibe ich sie denn, dass sie alle 4 Attribute auf Gleichheit prüft ?
So sieht meine Klasse aus:

public class Regel implements java.io.Serializable {
    public String lasthit;

    public String firsthit;

    public String hitcounter;

    public String name;

    public Regel(String a, String b, String c, String d) {
        setlh(a);
        setfh(b);
        sethc(c);
        setName(d);
    }

 

[mihe7]

import java.util.Objects;
...
@Override
public boolean equals(Object o) {
    if (o == null || o == this || !(o instanceof Regel)) {
        return o == this;
    }

    Regel r = (Regel) o;

    return Objects.equals(lasthit, r.lasthit) && 
            Objects.equals(firsthit, r.firsthit) &&
            Objects.equals(hitcounter, r.hitcounter) &&
            Objects.equals(name, r.name);
}

@Override
int hashCode() {
    return Objects.hash(lasthit, firsthit, hitcounter, name);
}

 

[F1NISH]

import java.util.Objects;
...
@Override
public boolean equals(Object o) {
    if (o == null || o == this || !(o instanceof Regel)) {
        return o == this;
    }

    Regel r = (Regel) o;

    return Objects.equals(lasthit, r.lasthit) &&
            Objects.equals(firsthit, r.firsthit) &&
            Objects.equals(hitcounter, r.hitcounter) &&
            Objects.equals(name, r.name);
}

@Override
int hashCode() {
    return Objects.hash(lasthit, firsthit, hitcounter, name);
}

Danke für deine Hilfe nun bekomme ich diese Fehlermeldung.

java.io.InvalidClassException: Regel; local class incompatible: stream classdesc serialVersionUID = -5231430306337615413, local class serialVersionUID = -8227139035629819211
    at java.io.ObjectStreamClass.initNonProxy(ObjectStreamClass.java:699)
    at java.io.ObjectInputStream.readNonProxyDesc(ObjectInputStream.java:1885)
    at java.io.ObjectInputStream.readClassDesc(ObjectInputStream.java:1751)
    at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:2042)
    at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1573)
    at java.io.ObjectInputStream.readObject(ObjectInputStream.java:431)
    at java.util.ArrayList.readObject(ArrayList.java:797)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:498)
    at java.io.ObjectStreamClass.invokeReadObject(ObjectStreamClass.java:1170)
    at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:2178)
    at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:2069)
    at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1573)
    at java.io.ObjectInputStream.readObject(ObjectInputStream.java:431)
    at Main.main(Main.java:80)

 

[mrBrown]

Du nutzt wohl irgendwo Java-Serialisierung, und versucht Klassen in einer "alten Version" zu laden.

Wenn du keine Felder der Klasse geändert hast, kannst du die serialVersionUID fest setzen (auf den alten Wert der in der Exception steht), das sollte den Fehler beheben.
Solltest dich dann aber noch mal damit beschäftigen, warum der Fehler kommt, und was es mit der serialVersionUID auf sich hat

 

[F1NISH]

Ja danke für die Antwort musste die serialisierten Files löschen.