Methoden Equals Methode für Integer und Objekte überschreiben?

[JavaExpInc]

Hallo zusammen,

da ich leider noch keine Lösung für mein Problem so gefunden habe, stelle ich hier mal mein Problem. Ich habe ein Entitiy Event, welches eine Map<Integer, Post>Postlist sowie Set<User>Userlist beinhaltet.

Public Class Event
    @OneToMany(cascade = CascadeType.ALL, fetch = FetchType.EAGER, mappedBy = "eventPost")
    @MapKey
    private Map<Integer, Post> postList;
    @ManyToMany
    @MapKey
    private Set<User> userList;

und einen Post

Public Class Post
 
    private int postId;
    private String content;
    private String header;
    private long creationDate;

Leider weiß ich nicht wie ich im Hinblick auf die postList und userList die Equals Methode überschreiben kann um diese später in Testzwecken vergleichen zu können. Meine momentane Equals Methode sieht so aus:

    @Override
    public boolean equals(Object o) {
        if (o == null) {
            return false;
        }
        if (o == this) {
            return true;
        }
        if (!o.getClass().equals(getClass())) {
            return false;
        }
        Event that = (Event) o;
        return this.getEventId() == that.getEventId() && this.getCategory() == that.getCategory()
                && this.getDescription() == that.getDescription() && this.getEventName() == that.getEventName()
                && this.getCreationDate() == that.getCreationDate() && this.getAuthor().equals(that.getAuthor())
                && this.getEventDate() == that.getEventDate();
    }

nur diese berücksichtigt momentan noch keine postList und userList.

 

[mrBrown]

Hat Event eine ID?

Ansonsten: Map und Set implementieren beide auch equals, du kannst einfach an deren delegieren.

 

[JavaExpInc]

Hat Event eine ID?

Ansonsten: Map und Set implementieren beide auch equals, du kannst einfach an deren delegieren.

Ja event hat eine ID, ändert sich deshalb etwas? Wie kann ich denn weiter an Map und Set delegieren? Also ich habe das bei mir schon mit dem Author gemacht (der ja der User ist und dort habe ich equals überschrieben) nur ich habe ja keine Klasse Map und Set sondern hole mir die aus den Bibliotheken...

 

[mrBrown]

Ja event hat eine ID, ändert sich deshalb etwas?

Es kann was ändern

Bei Entitäten mit ID hängt IDentität üblicherweise nur von der ID ab. Man würde also nur die ID vergleichen, und alle anderen nicht.
Wenn du jetzt ein Event hast, ist es ja immer noch das gleiche Event, wenn ein Nutzer hinzugefügt wird. (oder analog: auch wenn dir drei Haare mehr wachsen, bist du weiterhin die gleiche Person)

Also ich habe das bei mir schon mit dem Author gemacht (der ja der User ist und dort habe ich equals überschrieben) nur ich habe ja keine Klasse Map und Set sondern hole mir die aus den Bibliotheken...

Map und Set haben alle passende equals, die musst du nur aufrufen
(btw solltest du equals für alle Vergleiche nutzen, die keine primitiven Datentypen sind)

 

[JavaExpInc]

Es kann was ändern

Bei Entitäten mit ID hängt IDentität üblicherweise nur von der ID ab. Man würde also nur die ID vergleichen, und alle anderen nicht.
Wenn du jetzt ein Event hast, ist es ja immer noch das gleiche Event, wenn ein Nutzer hinzugefügt wird. (oder analog: auch wenn dir drei Haare mehr wachsen, bist du weiterhin die gleiche Person)

Das macht Sinn, also müsste ich in meiner bisherige Equals Methode nur den PK überprüfen, bzw. generell müsste man somit nur den PK bei der equals Methode überprüfen oder? Also sähe es bei den Variablen so aus das ich nur this.getEventId() == that.getEventId() machen müsste oder?

Map und Set haben alle passende equals, die musst du nur aufrufen
(btw solltest du equals für alle Vergleiche nutzen, die keine primitiven Datentypen sind)

Das Map und Set auch equals Methoden haben ist mir bekannt, jedoch weiß ich nicht wie das in meinem Fall hier jetzt aussehen soll. Ich kann sie ja nicht überschreiben da ich keine Klasse Map / Set ausprogrammiert habe. Oder kann ich das auch in der Equals Methode von Event machen die ich überschreibe? Falls ja, wie sähe so etwas nun aus?

 

[mrBrown]

Das macht Sinn, also müsste ich in meiner bisherige Equals Methode nur den PK überprüfen, bzw. generell müsste man somit nur den PK bei der equals Methode überprüfen oder? Also sähe es bei den Variablen so aus das ich nur this.getEventId() == that.getEventId() machen müsste oder?

Ja, genau. Ist die ID denn ein primitiver Datentyp?

Das Map und Set auch equals Methoden haben ist mir bekannt, jedoch weiß ich nicht wie das in meinem Fall hier jetzt aussehen soll. Ich kann sie ja nicht überschreiben da ich keine Klasse Map / Set ausprogrammiert habe. Oder kann ich das auch in der Equals Methode von Event machen die ich überschreibe? Falls ja, wie sähe so etwas nun aus?

Was willst du denn auch überschrieben? Du musst doch nur wissen, ob die beiden gleich sind:

this.postList.equals(that.postList)

 

[JavaExpInc]

Ja die ID ist ein Integer also ein primitiver Datentyp. Dann kann ich die restlichen Variablenvergleiche mit == ja wegmachen.

Achso ich habe gedacht das equals was von postList aufgerufen wird muss ich noch überschreiben. Also reicht das generell wenn ich bei Listen mit this.postList.equals(that.postList) überschreibe? Auch wenn dort Objekte drin sind wie in diesem Fall ein Post?

 

[mrBrown]

Achso ich habe gedacht das equals was von postList aufgerufen wird muss ich noch überschreiben. Also reicht das generell wenn ich bei Listen mit this.postList.equals(that.postList) überschreibe? Auch wenn dort Objekte drin sind wie in diesem Fall ein Post?

Ja, solange Post equals passend überschreibt.

 

[JavaExpInc]

Ja, solange Post equals passend überschreibt.

Meine Equals Methode sieht bei Post momentan so aus:

    @Override
    public boolean equals(Object o) {
        if (o == null) {
            return false;
        }
        if (o == this) {
            return true;
        }
        if (!o.getClass().equals(getClass())) {
            return false;
        }
        Post that = (Post) o;
        return this.getAuthor() == that.getAuthor() && this.getContent() == that.getContent()
                && this.getCreationDate() == that.getCreationDate() && this.getEventPost() == that.getEventPost()
                && this.getHeader() == that.getHeader() && this.getPostId() == that.getPostId();
    }

müsste daher ja passen oder? Und auch hier könnte ich da PostId wieder ein PK Integer ist nur diese überprüfen.

 

[mrBrown]

Bis auf die Vergleiche mit == sieht das passend aus.

Wenn es aber wieder eine Entität mit ID ist, sollte es reichen, diese zu vergleichen