parametrierung von HashMap / Map

[Andi_CH]

Ich bin daran Code zu säubern. (und nebenbei Java zu lernen ;-) ) und habe das Gefül des öfteren schlechte Beispiele zu finden ;-)

Auf Zeile 7, 9 und 11 gibt es logischerweise eine Warnung, ... raw Type .... should be parametrized
und auf Zeile 11 zusätzlich unchecked conversion
(Die braucht ihr mir nicht zu erklären ;-) die verstehe ich)

Braucht es Map.Entry überhaupt? Kann ich nicht direkt über den Iterator zugreifen?

Map.Entry<String,ABeton> entry = null;

gibt mal keine Warnung aus, scheint also richtig zu sein, aber wie muss Iterator definiert sein?

Wie gesagt, es geht mir eigentlich ums Konzept und darum etwas über HasMaps zu lernen
Was wollte wohl der Programmierer dieser Zeilen?
Das muss doch ohne Warnungen und vielleicht sogar einfacher gehen?

public void loadBetonTable (JComboBox pComboBoxBQ, AActionListener pActionListener)  {
	mBetonTable.clear();
	HashMap<String,ABeton> theBetonDB = null;
	//fill a table with beton types from the beton database
	ABeton betonConst = null;
	if (mDatabase != null && (theBetonDB = mDatabase.getBetonTable()) != null) {
		Map.Entry entry = null;
		//get the numbers of valid entries
		Iterator iter = theBetonDB.entrySet().iterator();
		while (iter.hasNext()) {
			entry = (Map.Entry)iter.next();
			betonConst = (ABeton)entry.getValue();
			if (ATextResource.isKeyValid(betonConst.getKey())) {
				mBetonTable.add(betonConst.getKey());
			}
		}
		Collections.sort(mBetonTable);
	}
}

public class ABeton implements Serializable, Comparable<ABeton> {
  // logisch steht da noch was drin, aber das dürfte hier irrelevant sein
}

 

[SlaterB]

allein zu den Generics als hübsche Klammer-Ansammlung:

    public void loadBetonTable()  {
        HashMap<String, Integer> theBetonDB = new HashMap<String, Integer>();
        Integer betonConst = null;
        Entry<String, Integer> entry = null;
        Iterator<Entry<String, Integer>> iter = theBetonDB.entrySet().iterator();
        while (iter.hasNext())  {
            entry = iter.next();
            betonConst = entry.getValue();
        }
    }

 

[Andi_CH]

Hm - Entry can not be resolved to a type ?

import java.util.map -> quick fix schlägt aber noch x andere vor, ich hoffe das ist das passende

und nachdem ich jetzt wirklich alle Integer durch ABeton ersetzt habe ist auch der Kompiler zufrieden ;-)
(Naja, ist noch früh am Morgen, da hab ich noch nicht alles gesehen)

Danke für die Hilfe

 

[bygones]

ist Map.Entry... anscheinend von Slater statisch importiert worden

ansonsten würd ich dann auch gleich auf die neue for schleife gehen

    public void loadBetonTable2() {
        Map<String, Integer> theBetonDB = new HashMap<String, Integer>();
        Integer betonConst = null;
        for (Map.Entry<String, Integer> mapEntry : theBetonDB.entrySet()) {
            
        }
    }

 

[maki]

Da die Keys nicht verwendet werden, könnte man sich die values() auch direkt holen anstatt das EntrySet.