equals-Methode

[Guest]

Hallo!

Kann mir jemand erklären warum ich hier eine NullPointerException kriege?

public class Test {
	private int id = 0;
	private String name = null;
	private String descr = null;
	
	public Test() {}
	
	public String getName() {
		return name;
	}
	
	public String getDescr() {
		return descr;
	}
	
	public boolean equals(Object obj) {
		if (obj instanceof Test) {
			Test test = (Test) obj;
			return (this.getName().equals(test.getName()) &&
						this.getDescr().equals(test.getDescr()));
		}
		return false;
	}
	
	public static void main(String[] args) {
		Test test = new Test();
		Test test2 = new Test();
		
		System.out.println(test.equals(null));
		System.out.println(test2.equals(new Test()));
		
	}
	
}

Ausgabe:

false
Exception in thread "main" java.lang.NullPointerException
	at eufo.Test.equals(Test.java:20)
	at eufo.Test.main(Test.java:31)

 

[bygones]

oh mann....

du vergleichst die variable namen & descr und hast die oben noch explizit auf null gesetzt....

nun überleg - warum kommt eine NullPointerException !

 

[Guest]

hmpf, ok, sorry, da hätte ich selber auch nocht draufkommen müssen...

und was passiert jetzt wenn ich nur

 private String name;

hinschreibe? Was ist das dann? Gar nichts, nicht mal null, oder?

 

[Guest]

Hmmm, aber was ist denn wenn name und descr bei beiden Objekten null sind? Dann will ich haben, dass (daß) mir die Methode true zurückliefert.

Geht das denn nicht einfacher als so?

public boolean equals(Object obj) {
	if (obj instanceof Test) {
		Test test = (Test) obj;
		if ((this.getName() == null) && (test.getName() == null) ||
				((this.getDescr() == null) && (test.getDescr() == null))) {
			return true;
		}
		return (this.getName().equals(test.getName()) &&
					this.getDescr().equals(test.getDescr()));
	}
	return false;
}

 

[bygones]

wenn du gar nichts schreibst wird ein Objekt immer mit null initialisiert...

wenn du auch den null Fall einbinden willst wirst du nicht drumrumkommen eine Abfrage zu machen wie du sie geschrieben hast....