TextView gibt Null zurück

Question

Ich habe einen Listener, der einfach den Text einer Schaltfläche ändert und die Zeit für den Benutzer anzeigt. Ich habe auch eine Textansicht, die auf die neue Gesamtanzahl der Zeit geändert werden soll, wenn der Benutzer abmeldet. Ich habe das XML überprüft, um sicherzustellen, dass ich die richtige R.Id packe und der Button im Listener wird gerade nicht gefunden. Hier ist der Code:

public class HoursListener implements OnClickListener{ 

GlobalApp appState; 
Button startStopHoursButton; 
TextView hoursTotalTextView; 
public boolean clockedIn; 

public HoursListener(Context ctx){ 
    appState = ((GlobalApp)ctx.getApplicationContext()); 
} 

public void onClick(View v) { 
    startStopHoursButton = (Button) v.findViewById(R.id.hourstogglebutton); 
    hoursTotalTextView = (TextView) v.findViewById(R.id.totalWorktimeTextView); 
    if(!clockedIn){ 
     startStopHoursButton.setText(R.string.clockout); 
     appState.getCurrentCompany().getCurrentNewWeeklyTimestamp().CreateFinishTimeStamp(); 
     clockedIn = true; 
    }else{ 
     startStopHoursButton.setText(R.string.clockin); 
     appState.getCurrentCompany().getCurrentNewWeeklyTimestamp().CreateFinishTimeStamp(); 
     clockedIn = false; 
     hoursTotalTextView.setText(appState.getCurrentCompany().getCurrentNewWeeklyTimestamp().totalTimeDoneThisWeekToString());     
    } 
} 

}

Heres die xml für die Textview:

<TextView 
     android:id="@+id/totalWorktimeTextView" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_alignParentLeft="true" 
     android:layout_below="@+id/textView2" 
     android:text="" 
     android:textAppearance="?android:attr/textAppearanceSmall"/> 

Der Fehler ist mit dieser Zeile:

hoursTotalTextView.setText(appState.getCurrentCompany().getCurrentNewWeeklyTimestamp().totalTimeDoneThisWeekToString()); 

ich gerade dachte, Den Code in die Aktivität selbst einfügen, aber ich habe das Gefühl, dass ich es so machen kann. Ich möchte etwas, das ich wie einen Listener bezeichnen kann, um die Redundanz in meinem Code zu reduzieren. Ich habe überprüft, dass es die hoursTotalTextView ist, die null ist. Aber der Knopf ist nicht. Irgendwelche Ideen?

Screenshot von Eclipse (in Verbindung mit Full-Size-Version) zeigt, dass die entsprechenden Werte nicht null:

Answer 1

Vermutlich ist die OnClickListener dessen klicken Sie hören ist die Taste? Das onClick übergibt die Ansicht, auf die geklickt wurde - also die Schaltfläche - und die Textbox ist kein untergeordnetes Element der Schaltfläche, sodass Sie sie nicht finden können.

Wenn HoursListener in Ihrer Aktivität erklärt wird, tun nur findViewById statt v.findViewById dh

hoursTotalTextView = (TextView) findViewById(R.id.totalWorktimeTextView); 

Wenn nicht, übergeben Sie den Textview in den HoursListener Konstruktor und setzen hoursTotalTextView dort (wenn man bedenkt das fühlt sich wie es könnte Speicherlecks verursachen).

Answer 2

was GlobalApp AppState; und warum Sie den Kontext für GlobalApp von der unteren Zeile aus eingeben. Ich denke, appstate wird hier null sein.

appState = ((GlobalApp) ctx.getApplicationContext());

Wenn GlobalApp eine Klasse ist, dann erstellen Sie ein Objekt dafür und verwenden Sie dann Getter- und Setter-Methoden.

Answer 3

müssen Sie

startStopHoursButton = (Button) v.findViewById(R.id.hourstogglebutton); 
hoursTotalTextView = (TextView) v.findViewById(R.id.totalWorktimeTextView); 

einige, wo vor verwenden, können Konstruktor verursachen Ihre setText auf null Textview-Objekt arbeitet.

Answer 4

prüfen jedes Objekt in dieser Zeile

if(appState.getCurrentCompany()==null) 
    Log.d("","getCurrentCompany is null"); 
if(appState.getCurrentCompany().getCurrentNewWeeklyTimestamp()==null) 
    Log.d("","getCurrentNewWeeklyTimestamp is null"); 
if(appState.getCurrentCompany().getCurrentNewWeeklyTimestamp().totalTimeDoneThisWeekToString()==null) 
    Log.d("","totalTimeDoneThisWeekToString is null");