CS2016 CS2016 - 1 year ago 66
Java Question

NullPointerException when indexing HashMap with character

I'm writing an English to Morse translator and every time I run it an error appears:

Exception in thread "main" java.lang.NullPointerException
at java.lang.String.concat(Unknown Source)
at Code.translate(Code.java:49)
at Code.<init>(Code.java:34)
at Morse.main(Morse.java:8)

I've tried debugging. Here's the function with the problem:

String translate(String phrase, HashMap map) { // line 37
//String to contain the morse code
String morse = "";
//Char array to contain the English phrase
char[] phraseList = new char[(int)phrase.length()];
//Fill the char array
phraseList = phrase.toCharArray();
//Loop through the array and concat the morse string with the value returned from the key
for (int x =0; x < phrase.length(); x++) {
if(phraseList[x] == ' ') {
morse.concat(" ");
} else {
//Here's where the error is produced
morse.concat((String) map.get(phraseList[x]));
return morse;

The error is produced when the code reaches the "else" condition. Here is the constructor that sets up my HashMap that's passed to this function:

HashMap<String,String> codeMap = new HashMap<>();
//Gets passed a string phrase from the main class
public Code(String phrase) throws FileNotFoundException { //line 14
String filePath;
String letter;
String code;
//Creates the object that reads the file location
Scanner fileLocate = new Scanner(System.in);
System.out.println("Enter file path of morse code");
//Object reads file location
filePath = fileLocate.nextLine();
//Create file object
File morse = new File(filePath);
//Create scanner object that reads the file
Scanner fileRead = new Scanner(morse);
//Loop to read the file and store the info as a key/value pair in a hashmap
while (fileRead.hasNext()) {
letter = fileRead.next().toLowerCase();
code = fileRead.next();
codeMap.put(letter, code);

is full of the correct, lowercase values and the
array is full of the lowercase chars of the phrase, but for some reason it still produces the error. Any advice on what I'm doing wrong would be greatly appreciated.

cxw cxw
Answer Source


morse.concat((String) map.get( Character.toString(phraseList[x]) ));
                          //  ^^^^^^^^^^^^^^^^^^^^ added this

Your map is indexed by String but translate doesn't know that due to type erasure. You are indexing with char instead of String. That means you are getting an autoboxed Character, which will not be the String you put in. As a result, the map won't contain the entry you expect, and will return null — whence the NullPointerException.

Explanation: Per this answer, hashCode is a first check in the HashMap. After that, Object.equals() is used to look for the key in the map. Both String.equals(Character) and Character.equals(String) will return false. Therefore, even though the hashCode values may be the same for String("a") and Character('a'), those two are not considered equal, so they cannot be used interchangeably as keys.

Recommended from our users: Dynamic Network Monitoring from WhatsUp Gold from IPSwitch. Free Download