Frage zur Implementierung von equals()

[xhi2018]

Hallo,

ich hab hier diese Klasse bei der ich die equals Methode implementiert habe. Ich hab das eine oder andere darüber gelesen und wollte mal wissen, wie Ihr - mit mehr Praxiserfahrung das so hand habt.

public class MyMitarbeiter extends Object implements Comparable<MyMitarbeiter> {
	private final Long persID;
	private String firstName;
	private String lastName;
	
	public MyMitarbeiter(Long id, String fName, String lName) throws IllegalArgumentException {
		if ( id == null ) {
			throw new IllegalArgumentException("fehlerhafter Konstruktoraufruf - Personal-ID ist [ " + id);
		}
		if ( fName == null ) {
			throw new IllegalArgumentException("fehlerhafter Konstruktoraufruf - Personal-ID ist [ " + fName);
		}
		if ( lName == null ) {
			throw new IllegalArgumentException("fehlerhafter Konstruktoraufruf - Personal-ID ist [ " + lName);
		}
		this.persID    = id;
		this.firstName = fName;
		this.lastName  = lName;
	}
	
	@Override
	public boolean equals(final Object other) {
		if ( super.equals(other) )                  return true;
//		if ( other == null )                      return false; 
		if ( ! (other instanceof MyMitarbeiter) )   return false;
		return isEqual( (MyMitarbeiter) other );
	}

	public boolean equals2(final Object other) {
		if ( this == other )                        return true;
		if ( other == null )                        return false;
		if ( this.getClass() != other.getClass() )  return false;
		return isEqual( (MyMitarbeiter) other );
	}
	
	private boolean isEqual(final MyMitarbeiter otherMa) {
		if ( ! (this.getLastName().equalsIgnoreCase(otherMa.getLastName())) ) {
			return false;
		}
		if ( ! (this.getFirstName().equalsIgnoreCase(otherMa.getFirstName())) ) {
			return false;
		}
		if ( ! (this.getPersID().equals(otherMa.getPersID())) ) {
			return false;
		}
		return true;
	}

Die

getter

die ich verwende sind natürlich auch vorhanden - hab ich mir hier aber gespart. Ich hab die

.equals(Object o)

Methode einmal überschrieben und dann nochmals etwas anders als

.equals2(Object o)

implementiert.

Hier hab ich gelesen, das bei Objekten welche direkt von

Object

ableiten nicht

super.equals(Object o)

verwendet werden sollte, weil dies nicht sicher ist, zumindest verstehe ich den Artikel so. Aber wenn ich mir die Implementierung von

Object.equals(Object o)

ansehe, dann steht da

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

also doch nix anderes wie in meiner

 equals2(Object o)

Methode?

Und noch eine Frage zur Typ-Prüfung lieber

if ( other == null )                        return false;
if ( this.getClass() != other.getClass() )  return false;

oder so

if ( ! (other instanceof MyMitarbeiter) )   return false;

Wie würdet Ihr das in dem konkreten Fall machen?

Hier noch der JUnit Test dazu...

public class MyMitarbeiterTest {

	MyMitarbeiter testMaOne;
	MyMitarbeiter testMaTwo;
	MyMitarbeiter testMaThree;
	
	@Before
	public void setUp() throws Exception {
		testMaOne   = new MyMitarbeiter(new Long(100), "VorEins", "NachEins");
		testMaTwo   = new MyMitarbeiter(new Long(200), "VorZwei", "NachZwei");
		testMaThree = new MyMitarbeiter(new Long(300), "VorDrei", "NachDrei");
	}

	@Test
	public void testEquals() {
		MyMitarbeiter tmpMaOne = new MyMitarbeiter(new Long(300), "VorDrei", "NachDrei");
		
		assertFalse("warum ist das gleich...?",	testMaOne.equals(null));
		assertFalse("warum ist das gleich...?",	testMaOne.equals(testMaTwo));
		assertTrue("warum nicht gleich...?", testMaOne.equals(testMaOne));
		assertTrue("warum nicht gleich...?", testMaThree.equals(tmpMaOne));
	}

	@Test
	public void testEquals2() {
		MyMitarbeiter tmpMaOne = new MyMitarbeiter(new Long(300), "VorDrei", "NachDrei");
		
		assertFalse("warum ist das gleich...?",	testMaOne.equals2(null));
		assertFalse("warum ist das gleich...?",	testMaOne.equals2(testMaTwo));
		assertTrue("warum nicht gleich...?", testMaOne.equals2(testMaOne));
		assertTrue("warum nicht gleich...?", testMaThree.equals2(tmpMaOne));
	}
}

viele Grüße und schon mal danke für Eure Antworten!

 

[Niki]

Lass dir einfach von einer kleinen Klasse im Eclipse eine equals Methode generieren. Die sieht dann so aus:

@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result + ((kuerzel == null) ? 0 : kuerzel.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;
		Person other = (Person) obj;
		if (kuerzel == null) {
			if (other.kuerzel != null)
				return false;
		} else if (!kuerzel.equals(other.kuerzel))
			return false;
		return true;
	}

Aja, wenn du equals überschreibst solltest/musst du auch hashCode überschreiben, und umgekehrt. Ist sozusagen eine Art Vertrag. Da gibts auch bestimmte Regeln:
Sind zwei Objekte "equal", müssen Sie auch den selben Hashcode zurück liefern. Umgekehrt aber nicht. Es können also auch zwei Objekte den selben Hashcode haben die aber nicht "equal" sind.

instanceof liefert auch true zurück wenn es sich um eine Instanz einer Subklasse handelt, daher ist getClass zu bevorzugen.

 

[xhi2018]

Hallo,

Lass dir einfach von einer kleinen Klasse im Eclipse eine equals Methode generieren.

über

Source

=>

Generate hashCode() and equals()...

ja klar!! :applaus: War mir jetzt nicht bekannt dass es sooo einfach geht...

Ich hab's immer über

Source

=>

Override/Implement Methods...

gemacht, da muss man(n) dann selber implementieren...

vielen Dank nochmals und schöne Grüße!