Iteratoren funktionieren nicht

[Hutmacher]

Habe mir mal, nachdem ich in einem Buch über Vector, Hashtable etc. gelesen habe (auf der Basis vom JDK 1.4), just4try versucht, eine Testdatabase aufzubauen, die ich quasi durchsuchen kann. Diese nimmt hier nur Zahlen auf.
Das ist meine Klasse SimpleDatabase:

class SimpleDatabase
{
	private Vector data; //Quasi die Auflistung
	
    //Konstruktor
	SimpleDatabase(int number)
	{
		data.add(number);
	}
	
    //Hinzufüge-Methode
	public void add(int number)
	{
		data.add(number);
	}

    /**
     * Durchsucht die Database nach einer Nummer. Ist diese vorhanden, 
     * wird true zurückgegeben, ansonsten false.
     */
	public boolean suchen(int number)
	{
		boolean result = false;
		
		Iterator it = data.iterator();
		
		while (it.hasNext())
		{
			if ((Integer)it.next() == number)
			{
				result = true;
			}
		}
		
		return result;
	}
}

[Ob jetzt Iterator oder Enumeration ist ja an dieser Stelle wayne]

Wenn ich jetzt aber in einer main-Methode

SimpleDatabase db = new SimpleDatabase(20);
		db.add(30); db.add(11); db.add(15); db.add(88);
		System.out.print(db.suchen(111)); //=> false

z.B. schreibe, bekomme ich in Zeile 14 & beim Konstruktoraufruf selbst NullPointerExceptions.

Aber wie kann denn da ein Nullverweis sein? :bahnhof:

So wie im Buch beschrieben, sollte der Code doch funktionieren …

 

[André Uhres]

    SimpleDatabase(int number)
    {
        data = new Vector();//<------------------------------------------
        data.add(number);
    }

 

[MarcB]

Dein Vector ist nicht instanziiert. Zuerst musst du im Konstruktor
data = new Vector();

aufrufen.

In deiner suchen Funktion weisst du der Variable "result" nicht das Ergebnis deines Vergleichs zu (Sondern der nirgendwo deklarierten Variable "ret")

 

[Hutmacher]

Ah, danke. Daran lag es also.

Ja, sowas passiert dann, wenn ich meine Variablen kurzfristig ohne STRG+F umbenenne …


Im Übrigen erscheinen in meinem Code in Eclipse Hinweise (z.B. beim Iteratorenbeschaffen in Zeile 25):
"Iterator is a raw type. References to generic type Iterator<E> should be parameterized"

Davon stand in dem Buch auch nichts … was hat das konkret zu bedeuten?

 

[Gelöschtes Mitglied 5909]

dass du doch kein Java 1.4 verwendest sondern 1.5 oder 1.6 und du Generics verwenden solltest.
Vectot<Integer> data = new Vector<Integer>();

-> dann musst du nicht mehr casten beim auslesen

Iterator<Integer> it = data.iterator();

...

Integer integer = it.next(); // stat (Integer) it.next();

Der vorteil ist außerdem Typsicherheit

und Objekte vergleicht man mit equals. Bei den primitiven Wrapper Klassen ist dies eine außname wegen AUtoboxing,
aber it.next().intValue() == number finde ich trotzdem besser