2015-04-30 54 views
-1

我正在使用android應用程序,在這個應用程序中我使用邏輯從協調中找到一個人的方向。一切工作正常,但我在控制檯錯誤::「死代碼」。我的代碼如下,請解釋我這件事。使用android eclipse的死代碼

private void direction() { 

     String userLocation = mLatitude + "," 
       + mLongitude ; 

     if(userLocation!=null){ 
      String distLocation = Constants.sMY_LOCATION.getLatitude() + "," 
        + Constants.sMY_LOCATION.getLongitude(); 
      String url = "https://maps.google.com/maps?f=d&saddr=" + userLocation 
        + "&daddr=" + distLocation; 
      Intent i = new Intent(Intent.ACTION_VIEW, Uri.parse(url)); 
      startActivity(i); 
     }else{ // Getting Dead Code Yellow error here 
      finish(); 
      Toast.makeText(getBaseContext(), "Please check your internet and try again!", Toast.LENGTH_SHORT).show(); 
     } 


    } 
+0

'死Code'是代碼,您不要再使用,如它永遠不會被調用方法。這是**不是錯誤**。 –

回答

6

這是因爲你的人能不能達到

String userLocation = mLatitude + "," 
      + mLongitude ; 

    if(userLocation!=null){ 
     ... 
    }else{ 
     ... 
    } 

用戶位置永遠不會爲空

2

中的代碼您的其他{}將永遠不會達到的,因爲你的字符串用戶位置之前初始化if語句,意味着它永遠不會爲空。

所以你的其他代碼實際上是死代碼。

1

您應該檢查mLatitudemLongitude爲空而不是整個字符串。

例子:

if (mLatitude != null && mLongitude != null) { 
    // String userLocation = ... 
} 
else { 
    // Your else code 
}