2014-01-07 112 views
0

我的MainActivity代码空指针访问警告

private void updateConnectionState() 
     { 
       Device localDevice = this.Controller.getConnectedDevice(); 
       if (localDevice == null) 

       updateModelSpinner(localDevice.getType()); //warning at this line   
       str2 = localDevice.getHostName(); 
       if (!TextUtils.isEmpty(str2)) 

      } 




private void updateSpinner(Device.Type paramType) 
     { 
     boolean bool = Device.Type.UNKNOWN.equals(paramType); 
     int i = 0; 
     if (!bool) 
      i = 1 + paramTvType.ordinal(); 
     this.ModelSpinner.setSelection(i); 
     } 

private void ModelSpinner(Device.Type paramType) 
     { 
     boolean bool = Device.Type.UNKNOWN.equals(paramType); 
     int i = 0; 
     if (!bool) 
      i = 1 + paramType.ordinal(); 
     this.ModelSpinner.setSelection(i); 
     } 

我的枚举类是

public class Device { 

    private Type type = Type.A_LOGIC; 

    public static Type getTypeForId(int paramInt) 
     { 
     switch (paramInt) 
     { 
     default: 
      return Type.A_LOGIC;  
     case 0: 
      return TvType.B_LOGIC; 
     case 1: 
      return TvType.A_LOGIC; 
     case 2: 
      return TvType.D_LOGIC; 
     case 3: 
      return TvType.E_LOGIC; 
     case 4: 
     } 
     return TvType.F_LOGIC;   
     } 

    public void setType(Type paramType) 
     { 
     this.Type = paramType; 
     } 

    public enum Type 
    {  
     A_LOGIC("A_LOGIC"),     
     B_LOGIC ("B_LOGIC"),  
     C_LOGIC ("C_LOGIC"), 
     D_LOGIC("D_LOGIC"), 
     E_LOGIC ("E_LOGIC"), 
     UNKNOWN("UNKNOWN"); 

     private String object; 

     TvType(String localobj) 
     { 
      this.object; = localobj; 
     }  
     public String getLetter() 
     { 
      return this.object;; 
     }  
    } 

public Type getType() 
     { 
     return this.type; 
     } 

,并在我的方法中的主要活动正在呼叫的类型

updateSpinner(localDevice.getType());

但这里它显示的警告为

Null pointer access: The variable localTVDevice can only be null at this location 

并在该行抛出一个零点错误。 对于枚举的概念我不熟悉,所以请告诉我为什么抛出这个错误。我提到了堆栈溢出但找不到答案。

+0

您应该共享您的updateSpinner方法。 –

+1

和所有的调用代码。您发布的代码和您的错误消息引用了不同的变量名称,我们需要查看您实际编写的内容。 – chrylis

+0

您可能希望使用带有数字字段的枚举,并在使用'object'的地方使用'name()'。同时检查你的开关'情况1'这里可能有一个错字('C_LOGIC') – 2014-01-07 06:26:19

回答

1

问题是这样的代码

if (localDevice == null) 
    updateModelSpinner(localDevice.getType()); 

如果LocalDevice的是空, 然后调用updateModelSpinner(null.getType())

你可能想验证码:

if (localDevice != null) 
    updateModelSpinner(localDevice.getType()); 

你”必须决定当localDevice为空时会发生什么..

+0

让我试试这个让你知道。谢谢你的回答 – AndroidOptimist

+0

@AndroidOptimist这与RC的答案基本相同 - 你说因为某种原因没有帮助。 –

+0

感谢Rolf我解决了这个问题。这是我没有注意到的愚蠢问题。 – AndroidOptimist

2

从代码:

Device localDevice = this.Controller.getConnectedDevice(); 
if (localDevice == null) 

updateModelSpinner(localDevice.getType()); 

因为如果没有{},这个代码是类似于:

Device localDevice = this.Controller.getConnectedDevice(); 
if (localDevice == null) { 
    updateModelSpinner(localDevice.getType()); 
} 

因此警告

FIX:

Device localDevice = this.Controller.getConnectedDevice(); 
if (localDevice != null) { 
    updateModelSpinner(localDevice.getType()); 
    // more code using localDevice 
} 
+0

我试过这个但仍面临相同的错误 – AndroidOptimist

+0

感谢您指出错误 – AndroidOptimist

0

此编译器警告表明一个变量(在这种情况下,localTVDevice)未分配一个值,并且通过此变量调用方法无疑会在运行时产生NullPointerException

还有一件事我看到你的代码?您可以直接将枚举值与'=='进行比较,而不是像在此行中那样等于:

boolean bool = Device.Type.UNKNOWN.equals(paramType); 

boolean bool = paramType == Device.Type.UNKNOWN;