Problem iterating using Map.Entry for HashMap

[d0nmin0]

Hi, I would like advice on alternative solutions to using the 'Map.Entry' for iterating of HashMap
As normally done, this is the coding for iterating through a HashMap:

Iterator iterator = places.entrySet().iterator();
		while (iterator.hasNext()) {
			Map.Entry pi = (Map.Entry)places.next();
			System.out.println(pi.getKey() + " "  + pi.getValues());
		}

As such, I have to import this class as well -> import java.util.Map

The problem I face at this moment is because I have a Map interface created on my own and is this is clashing with the util.Map class. Is there another way around for this?

A few examples of the errors I get upon implementation of the util.Map class:

MapPanel.java:56: cannot find symbol
symbol : method getPlaces()
location: interface java.util.Map
Set<Place> actualPlaces = map.getPlaces();
^
MapPanel.java:86: cannot find symbol
symbol : method getRoads()
location: interface java.util.Map
Set<Road> actualRoads = map.getRoads();

[Iron Lion]

If you don't want to rename your Map class, then you can avoid importing java.util.Map by fully qualifying the name, as such:

java.util.Map m = new java.util.HashMap();

Alternatively, you can import java.util.Map and fully qualify references to your own Map class.

[sunde887]

You should avoid giving your classes the same names as classes in the Java library. It makes code more difficult to work with and requires you to use fully qualified names when using some class.

java.util.Map

You can also iterate through a map much easier than creating an iterator

for(E type : map.keySet()){
  System.out.println(type + ": " + map.get(type));
}

[Tolls]

You can also iterate through a map much easier than creating an iterator

for(E type : map.keySet()){
  System.out.println(type + ": " + map.get(type));
}

You'll find code analysis tools recommending you iterate over the entrySet and not the keySet. It means you don't have to go back to the map to pull out the value for a key.

for (Map.Entry entry : map.entrySet()) {
    System.out.println(entry.getKey()+ ": " + entry.getValue);
}

[sunde887]

Thanks for pointing that out. Ignorance on my part; thanks Tolls.

[Tolls]

I only really remember it because Findbugs used to always point it out at one place I worked.
Annoying thing that it can be sometimes...