android java code error

Question

Я создаю клавиатуру, но есть некоторая ошибка в использовании локальной переменной.

private void updateCandidateText(){ 
     try{ 

      ExtractedText r= getCurrentInputConnection().getExtractedText(new ExtractedTextRequest(),InputConnection.GET_EXTRACTED_TEXT_MONITOR); 
      String strbeforeCursor=""; 
      String strafterCursor =""; 
       strbeforeCursor = getCurrentInputConnection().getTextBeforeCursor(1000000000, 0).toString(); 
      strafterCursor = getCurrentInputConnection().getTextAfterCursor(1000000000, 0).toString(); 
      String str = strbeforeCursor + "|"+strafterCursor; 
      if(mTamilPreviewView != null) 
       mTamilPreviewView.update(str, strbeforeCursor.length()); 

      mTamilPreviewView.update(r.text.toString() , 0); 
     } 
     catch (Exception e) { 
      Log.e("t", "errr", e); 
     }  
    } 

Answer 1

Вы проверить, если mTamilPreviewView != null позвонить

mTamilPreviewView.update(str, strbeforeCursor.length()); 

, но даже если это null, вы будете делать

mTamilPreviewView.update(r.text.toString() , 0); 

и вы получите NullPointerException. Это действительно то, что вы хотите сделать? Вы имеете в виду

if (mTamilPreviewView != null) { 
    mTamilPreviewView.update(str, strbeforeCursor.length()); 
    mTamilPreviewView.update(r.text.toString() , 0); 
} 

Кроме того, вы инициализировать strbeforeCursor и strafterCursor с пустой строкой, и вы даете им другие значения в следующих строках. Вы можете просто удалить

String strbeforeCursor=""; 
String strafterCursor =""; 

и сделать

String strbeforeCursor = getCurrentInputConnection().getTextBeforeCursor(1000000000, 0).toString(); 
String strafterCursor = getCurrentInputConnection().getTextAfterCursor(1000000000, 0).toString();