2016-03-03 103 views
-2

问题是罗马整数:link。 给定一个罗马数字,将其转换为整数。 输入保证在1到3999的范围内。 所以我在我的答案中首先使用了toCharArray()方法。当我使用toCharArray()方法时,为什么会得到java.lang.NullPointerException?

public int romanToInt(String s) { 
     HashMap<String, Integer> dict = new HashMap<String, Integer>(); 
     dict.put("I", 1); 
     dict.put("V", 5); 
     dict.put("X", 10); 
     dict.put("L", 50); 
     dict.put("C", 100); 
     dict.put("D", 500); 
     dict.put("M", 100); 

     int sum = 0; 
     char[] arrayS = s.toCharArray(); 
     for(int i = 0 ; i < arrayS.length ; i++){ 
      int Val1 = dict.get(arrayS[i]).intValue(); 
      if(i < arrayS.length -1){ 
      int Val2 = dict.get(arrayS[i+1]).intValue(); 
       if(Val1 < Val2){ 
        sum += Val2 - Val1; 
        i++;} 
       else 
        sum += Val1; 
      } 
      else 
       sum += Val1; 
     } 
     return sum; 

    } 

然后,在位置int Val1 = dict.get(arrayS[i]);,所述编译器显示Line 19: java.lang.NullPointerException。但是,如果我不用toCharArray()重写它,它就可以工作。

int sum = 0; 
     int val1, val2; 
     for (int i = 0; i < s.length(); i++) { 
      val1 = map.get(s.charAt(i)); 
      if (i < s.length()-1) { 
       val2 = map.get(s.charAt(i+1)); 
       if (val1 >= val2) { 
        sum += val1; 
       } else { 
        sum += (val2 - val1); 
        i += 1; 
       } 
      } else { 
       sum += val1; 
      }   
     } 
     return sum; 

为什么我在使用toCharArray()时遇到java.lang.NullPointerException问题?

+1

当你调试这个时,哪个对象是'null'?你期望该物体被分配一个值? – David

+5

可能是因为's'本身是'null'。 – user2004685

+3

为了清楚这一点:你真的认为,当你问一个带'String'的地图作为返回除null之外的其他关键字的地图时,当你问它是否包含某个'char'时?你为什么这么认为? – Tom

回答

0

您的代码存在的问题是您创建了一个将String映射到整数的HashMap,但是您将一个字符作为参数传递给romanToInt方法。 您可以通过两种方式解决这个问题

解决方案1 ​​ -

public int romanToInt(String s) 
{ 
HashMap<String, Integer> dict = new HashMap<String, Integer>(); 
dict.put("I", 1); 
dict.put("V", 5); 
dict.put("X", 10); 
dict.put("L", 50); 
dict.put("C", 100); 
dict.put("D", 500); 
dict.put("M", 100); 
int sum = 0; 
char[] arrayS = s.toCharArray(); 
for(int i = 0 ; i < arrayS.length ; i++) 
{ 
    int Val1 = dict.get(String.valueOf(arrayS[i])); // Convert character to string 
    if(i < arrayS.length -1) 
    { 
    int Val2 = dict.get(String.valueOf(arrayS[i+1])); //Convert character to string 
    if(Val1 < Val2) 
    { 
    sum += Val2 - Val1; 
    i++; 
    } 
    else 
    sum += Val1; 
    } 
    else 
    sum += Val1; 
} 
return sum; 
} 

解决方案2 -

public int romanToInt(String s) 
{ 
HashMap<Character, Integer> dict = new HashMap<Character, Integer>(); 
//Map Character to Integer 
dict.put('I', 1); //Used single quote to represent character 
dict.put('V', 5); 
dict.put('X', 10); 
dict.put('L', 50); 
dict.put('C', 100); 
dict.put('D', 500); 
dict.put('M', 100); 
int sum = 0; 
char[] arrayS = s.toCharArray(); 
for(int i = 0 ; i < arrayS.length ; i++) 
{ 
    int Val1 = dict.get(arrayS[i]); 
    if(i < arrayS.length -1) 
    { 
    int Val2 = dict.get(arrayS[i+1]); 
    if(Val1 < Val2) 
    { 
    sum += Val2 - Val1; 
    i++; 
    } 
    else 
    sum += Val1; 
    } 
    else 
    sum += Val1; 
} 
return sum; 
} 
+0

是的你是对的,我也只是想通了,我会接受你的答案。 – JiPanNYC

0

dict.get(arrayS[i]);如果没有翻译,则返回null。
从Integer到int的空箱拆箱导致NullPointerException

+0

谢谢你的建议。我忘了做改造。然后我编辑它像'int Val1 = dict.get(arrayS [i])。intValue();'。但我仍然遇到'NullPointerException'。 :( – JiPanNYC

+0

OP在替代方法中完全相同 –

+0

是的,我想这可能不是转换问题。在第二种方法中,我没有做任何转换并且仍然有效,但是在拆箱时更好地转换 – JiPanNYC

0

其实我做的数据类型的错误。我使用了元素为Char类型的Char数组。我试图使用dict.get(arrayS[i])阵列[我]应该是一个字符串类型,因为我建立了这个HashMap<String, Integer>。 然后我将HashMap<String, Integer>更改为HashMap<Character, Integer>然后它工作。 感谢Tom @Tom指出我的问题。

相关问题