Wie so ein Exception in dieser HashMap?

[Harry1980]

package client.chat;

import java.util.HashMap;

public class RequestIP {
	
	private HashMap<String, String> hm;
	
	public RequestIP(){
		this.hm = new HashMap<String, String>();
		this.hm.put("Berlin","192.168.1.1");
		this.hm.put("Muenchen","192.168.1.2");
		this.hm.put("Hamburg","192.168.1.3");
		this.hm.put("Koeln","192.168.1.4");
		this.hm.put("Frankfurt","192.168.1.5");
		
	}
	
	public String checkIP(String ip){
		String responseCity = "";
		responseCity = hm.get(ip).toString();
		return responseCity;
			
	}

	public static void main(String []args){
		RequestIP g = new RequestIP(); 
		System.out.println(g.checkIP("192.168.1.1"));
	}
	

}

Obwohl ich im Konstruktor die HashMap instanziert habe, warum verlang er weiteres Objekt?
Sch mal dank für euer Bemühen.

 

[André Uhres]

Versuch's mal so:
System.out.println(g.checkIP("Berlin"));

 

[Gelöschtes Mitglied 5909]

poste mal die eception, ich wette aber es ist ne nullpointerexception.

mit get bekommst du den wert für einen key.
Du suchst mit checkip den ort, der aber als key und nicht als value eingetragen ist

-> vertausche bei deinen puts ip und stadt, dann sollte es klappen

 

[Der Müde Joe]

weil eine Map keys auf values mapped und nicht values auf keys..

wenns nichts gefunden wird in der Map kommt null zurück.
toString auf null gibt ne NPE

EDIT:
Btw:
bei get kommt schon ein String --> generics
da wäre kein toString mehr notwendig

 

[Harry1980]

Ja, Stimmt, da war ein NullPointerException, vielen Dank euch beiden, jetzt klappts
:applaus:

Gruß Harry
(noch Anfänger!)

 

[André Uhres]

"toString()" bei "hm.get(ip).toString()" ist eh überflüssig.
Nur so nebenbei: mit nur einer HashMap können wir auch beides machen, cityToIp und ipToCity.
Das folgende Beispiel zeigt eine mögliche Lösung anhand der oben gezeigten HashMap:

package demo;
/*
* RequestIP.java
*/

import java.util.*;
import java.util.Map.*;
import java.util.regex.*;
import javax.swing.*;

public class RequestIP {

    private HashMap<String, String> hm;

    public RequestIP() {
        this.hm = new HashMap<String, String>();
        this.hm.put("BERLIN", "192.168.1.1");
        this.hm.put("MUENCHEN", "192.168.1.2");
        this.hm.put("HAMBURG", "192.168.1.3");
        this.hm.put("KOELN", "192.168.1.4");
        this.hm.put("FRANKFURT", "192.168.1.5");

    }

    public String cityToIp(final String city) {
        return hm.get(city.trim().toUpperCase());
    }

    public String ipToCity(final String ip) {
        Pattern regex = Pattern.compile("\\b" +
                "(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?\\.)" +
                "(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?\\.)" +
                "(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?\\.)" +
                "(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\b");
        Matcher matcher = regex.matcher(ip.trim());
        StringBuilder ipVal = new StringBuilder();
        if (matcher.find()) {
            for (int i = 1; i <= matcher.groupCount(); i++) {
                ipVal.append(matcher.group(i).replaceAll("^[0]+", ""));
            }
            String ipValue = ipVal.toString();
            Set<Entry<String, String>> es = hm.entrySet();
            for (Entry<String, String> entry : es) {
                if (entry.getValue().equals(ipValue)) {
                    return entry.getKey();
                }
            }
            return "(IP missing in city table)";
        }
        return "(invalid IP)";
    }

    public static void main(final String[] args) {
        String city = " Berlin  ";
        String ip = "192.168.01.001 ";
        RequestIP g = new RequestIP();
        JOptionPane.showMessageDialog(null,
                "'" + city + "' to IP = " + g.cityToIp(city) + "\n" +
                "'" + ip + "' to city = " + g.ipToCity(ip));
    }
}

Der regex erlaubt IP's von 0.0.0.0 bis 255.255.255.255.
Wir durchlaufen die 4 Zahlen als 'capturing groups' um eventuell vorhandene führende Nullen zu entfernen.

 

[Landei]

Zwei HashMaps gehen, sind aber auch nicht der Knaller. Aber Apache commons collections oder Google collections haben bessere Lösungen parat.
Natürlich braucht man immer noch die Funktionalität zum splitten, aber wäre da nicht eine Baumstruktur cleverer?

 

[André Uhres]

Zwei HashMaps gehen, sind aber auch nicht der Knaller. Aber Apache commons collections oder Google collections haben bessere Lösungen parat.
Natürlich braucht man immer noch die Funktionalität zum splitten, aber wäre da nicht eine Baumstruktur cleverer?

Ist wohl overkill.