eigenes Equal mit IndexOf()

[port45]

Hallo zusammen,

habe hier ein kleines Problem und hoffe jemand kann mal drüber schauen : /


Safeequals soll ein true zurück geben, wenn a und b gleich sind oder beide null sind.

private static boolean safeEquals(Object a, Object b) {
      
    	if(a.equals(b) || a.equals(null) && b.equals(null))
    	{
    		return true;
    	}
    	else
    	{
    		return false;
    	}

IndexOf soll den Index von den gesuchten Object zurückgeben:

 public int indexOf(T gesucht) {
       
    	for(int i = 0;i<data.length;i++)
    	{
    		if(safeEquals(gesucht,data[i])==true)
    		{
    			return i;
    		}
    		
    	}
    	//data[size++] = null;
    	return NOT_FOUND;

 

[JaxxTe]

Und was für ein Problem gibt es hier?

Ich rate mal: falls die Objects null sind wirst du in eine NullPointerException laufen, da du equals auf den Objekten rufst.

Lösung: Teste die beiden Objekte zuerst auf null (nicht mit equals) und überprüfe danach ob sie gleich sind.

 

[port45]

Hab ich gerade versucht, aber hilft nicht :/
hab einmal mit == auf null geprüft und dann mit equals auf gleichheit.
Einmal mit einem if und dann nochmal mit zwei

Also Fehlermeldung bekomme ich nicht.
Habe so eine Testklasse an der ich mich halten muss.
Aber meiner Meinung nach stimmt das alles xD

Das sind noch die Fehler bei mir.
Aber Test ob Equals genutzt werden habe ich doch gemacht : (

public void testObEqualsGenutzt() {
        a.add("xyz");
        a.add("abcde");
        a.add("abd");
    	assertEquals( "== ist verkehrt, equals() benutzen!",
    			1, a.indexOf(new String("abcde")));
    }

 public void testIndexOfNull() {
        fuelleA();
        assertEquals("null ist nicht vorhanden", -1, a.indexOf(null));
        a.add(null);
        assertEquals("jetzt ist null vorhanden", 4, a.indexOf(null));
    }

    public void testIndexOf() {
    	fuelleA();
    	assertEquals("a wird richtig gefunden", 0, a.indexOf("a"));
    	assertEquals("c wird richtig gefunden", 2, a.indexOf("c"));
    	assertEquals("x ist nicht vorhanden", -1, a.indexOf("x"));

 

[Natac]

Hab ich gerade versucht, aber hilft nicht :/

Sicher? Du hast diesen Code geschrieben und willst sagen, dass er bei dir nicht funktioniert?

boolean safeEquals(Object a, Object b){
  if(a == null && b == null){
    return true; // beide null -> TRUE
  } 
  else if(a == null){
    return false; // a ist null, b nicht -> FALSE
  } 
  else {
    return a.equals(b); // a ist nicht null -> Test mit equals
  }
}

Und dir ist bewusst, dass so ein Ausdruck:

a.equals(null)

IMMER false liefert? So steht es in der Dokumentation von equals.:rtfm:

 

[VfL_Freak]

Moin,

Safeequals soll ein true zurück geben, wenn a und b gleich sind oder beide null sind.

if(a.equals(b) || a.equals(null) && b.equals(null))

Ich denke mal, dass Du hier den hinteren Teil klammern musst, da andernfalls IMMER ("b" == null) sein müsste !

if( a.equals(b) || (a.equals(null) && b.equals(null)) )

ergo: lieber einmal zuviel als einmal zuwenig klammern

Gruß
Klaus

 

[port45]

if(a == null)
    	{
    		if(b == null)
    		{
    			return true;
    		}
    		return false;
    	
    	
    		
    	}
    	return a.equals(b);

war die Lösung für das Programm.
Danke für eure Antworten.
Fehler kam hauptsächlich durch einen Folgefehler in einer anderen Methode XD