Android Nullpointer in Runnable zum zweiten Mal

Question

Ich bin sicher, dass dieser Fehler ist, weil ich nicht in vollem Umfang Fäden verstehen, aber hier geht es ...

Ich habe einen lauffähiges, dass gestartet wird, wenn onCreate() innerhalb aufgerufen eine Methode:

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    //Set all app specific starting points here 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_avatar); 
    ... 
    soundMeterLoop(); 
} 

public void soundMeterLoop() { 
    Log.d("SpeechKit", "Start Sound Meter"); 
    soundMeterHandler = new Handler(); 
    soundMeterRunnable = new Runnable() { 
     @Override 
     public void run() { 
      if(!soundMeter.SoundMeterRunning) { 
       Log.d("SpeechKit", "Start SoundMeter in the runnable"); 
       startSoundMeter(); 
      } 
      if (soundMeter.mMediaRecorder != null) { 
       amplitude = soundMeter.getAmplitude(); 
       decibelLevelOutput.setText("" + amplitude); 
       if (amplitude > threshold) { 
        decibelLevelOutput.setTextColor(Color.RED); 
        Log.d("SpeechKit", "Interrupt and run startNuance()"); 
        startNuance(); 
       } else { 
        decibelLevelOutput.setTextColor(Color.BLACK); 
        Log.d("SpeechKit", "Running"); 
        soundMeterHandler.postDelayed(this, 100); 
       } 
      } 

     } 
    }; 
    soundMeterHandler.postDelayed(soundMeterRunnable, 100); 
} 

das läuft ganz gut, wenn es in den onCreate erstellt wird. Wie Sie sehen können, wird es selbst beendet (indem die Schleife nicht erneuert wird, wenn die Anweisung fehlschlägt) und startNuance() ausgeführt wird.

public void startNuance() { 
    soundMeterHandler.removeCallbacksAndMessages(soundMeterRunnable); 
    nuance.toggleReco(); 
} 

Ich kille dann die Runnable und starten Sie eine Methode in einer anderen Klasse. Diese Klasse läuft gut, dann, wenn es seine Sache tut fertig ist, nenne ich diese Hauptklasse zurück mit avatar.stopNuance();

Dies ist in der Nuance.java Klasse

@Override 
    public void onFinishedRecording(Transaction transaction) { 
     Log.d("SpeechKit", "onFinishedRecording"); 
     //We have finished recording the users voice. 
     //We should update our state and stop polling their volume. 
     state = State.PROCESSING; 
     stopAudioLevelPoll(); 
     avatar.stopNuance(); // <<<<< 
    } 

Es dann zurück zu meiner Haupttätigkeit zurück (avatar) und führt diese stopNuance() Methode:

public void stopNuance() { 
    Log.d("SpeechKit", "stopNuance(), start loop again"); 
    soundMeterLoop(); 
} 

Dann versucht es, aus, bevor die gleiche Schleife laufen. Nur dieses Mal bekomme ich eine Menge Fehler, die zu Nullpunkterfassungen führen. speziell beginnend mit decibelLevelOutput.setText("" + amplitude);

Ich bin mir nicht sicher, warum diese Dinge null sind oder wie sie zu beheben sind. Ist das so, weil es einen neuen Thread gestartet hat, der nicht beim Erstellen des Runnable gestartet wurde?

Answer 1

Nach dem Gespräch im Chat der aktuellen Ausgabe an anderer Stelle im Code-Basis wurde.

Das Problem war:

public class Nuance { 

     private Activity activity; 
     private Session session; 
     public Avatar avatarActivity = new Avatar(); // DONT DO THIS 

.... 

     @Override 
     public void onFinishedRecording(Transaction transaction) { 
      Log.d("SpeechKit", "onFinishedRecording"); 
      //We have finished recording the users voice. 
       //We should update our state and stop polling their volume. 
       state = State.PROCESSING; 
       stopAudioLevelPoll(); 
      avatarActivity.stopNuance(); 
      } 

Sie sollten nie, nie jemals Ihre eigene Instanz einer Aktivität erstellen. Sie werden vom System erstellt und verwaltet. Das System ruft die Lifecycle-Methoden für die Instanz auf (onCreate usw.), aber wenn Sie eine Instanz erstellen, werden diese Methoden nicht aufgerufen - daher passiert eine Menge merkwürdiges Verhalten.

Das Update hier ist diese:

public class Nuance { 

     private Avatar activity; 
     private Session session; 

.... 

     @Override 
     public void onFinishedRecording(Transaction transaction) { 
      Log.d("SpeechKit", "onFinishedRecording"); 
      //We have finished recording the users voice. 
       //We should update our state and stop polling their volume. 
       state = State.PROCESSING; 
       stopAudioLevelPoll(); 
       activity.stopNuance(); 
      } 

Answer 2

Sie möchten nicht jedes Mal einen neuen Runnable erstellen, wenn soundMeterLoop() aufgerufen wird.

Versuchen Sie folgendes:

private final Handler soundMeterHandler = new Handler(); 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    //Set all app specific starting points here 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_avatar); 
    ... 
    soundMeterLoop(); 
} 

public void soundMeterLoop() { 
    Log.d("SpeechKit", "Start Sound Meter"); 
    soundMeterHandler.postDelayed(soundMeterRunnable, 100); 
} 

private final Runnable soundMeterRunnable = new Runnable() { 
     @Override 
     public void run() { 
      if(!soundMeter.SoundMeterRunning) { 
       Log.d("SpeechKit", "Start SoundMeter in the runnable"); 
       startSoundMeter(); 
      } 
      if (soundMeter.mMediaRecorder != null) { 
       amplitude = soundMeter.getAmplitude(); 
       decibelLevelOutput.setText("" + amplitude); 
       if (amplitude > threshold) { 
        decibelLevelOutput.setTextColor(Color.RED); 
        Log.d("SpeechKit", "Interrupt and run startNuance()"); 
        startNuance(); 
       } else { 
        decibelLevelOutput.setTextColor(Color.BLACK); 
        Log.d("SpeechKit", "Running"); 
        soundMeterHandler.postDelayed(this, 100); 
       } 
      } 

     } 
    };