[SOLVED]Problem: NullPointerException - ich finde den Fehler nicht

Proffni · 2. Juni 2012

Hi Android-Freunde,

Ich versuche mir derzeit die Grundkenntnisse in Android anzueignen, habe bereits ein bisschen Java Hintergrundwissen und ein Spiel auf Grundlage des Buches "Android-Apps entwickeln" von Galileo Computing nachgebaut.

nun bin ich auf einem Stand, auf dem es eigentlich funktionsfähig sein sollte (in der einfachsten/schlichtesten Version), erhalte jedoch beim Start der Spielrunde eine NullPointerException und weder ich, noch mein aktuell Informatik studierender Bruder können den Fehler im Code finden

Java

long alter = (new Date()).getTime() - geburtsdatum.getTime(); //Zeile mit error














/* zerteilt in: */long alterakt=aktuell.getTime(); 














 		long altergeb= geburtsdatum.getTime();














/* um herauszufinden dass der Fehler beim auslesen des getTime von geburtsdatum ist , geburtsdatum wird hier initiiert: */ 














Date geburtsdatum = (Date) mueckev.getTag(R.id.geburtsdatum);














/* die id von muecke wird hier geschrieben: */ 














ImageView muecke = new ImageView(this);






























		muecke.setImageResource(R.drawable.muecke);






























 	muecke.setTag(R.id.geburtsdatum, new Date());














/* mueckev wird hier initiiert: */ 














ImageView mueckev = (ImageView) spielbereich.getChildAt(nummer);

Alles anzeigen

hat jemand selbst das Spiel nachgeschrieben und das selbe Problem gehabt/ den Fehler gefunden - oder kann mir jemand anderes meinen Fehler sagen?

mit der funktionierenden Spielversion auf der DVD zum Buch habe ich bereits verglichen - konnte jedoch keine unterschiede finden (außer bei variablenbezeichnungen o.ä. - habe inzwischen nahezu 100% angeglichen - der fehler besteht weiterhin)

Informationen zum System:

Ich schreibe mit Eclipse, Compile in 1.6 (hab inzwischen auch gecheckt dass der Text im Buch in 1.7 so nicht funktioniert) - die Basis des Buches ist auf Android 2.2 also habe ich auch dies eingestellt,

getestet habe ich es auf Virtuellen Android 2.1, 2.2 Geräten, die Version von der DVD läuft auf beiden Problemlos

Liebe Grüße,

Proffni

Java

package de.proffni.mueckenmassaker;






import java.util.Date;


import java.util.Random;






import android.app.Activity;


import android.app.Dialog;


import android.os.Bundle;


import android.os.Handler;


import android.view.Gravity;


import android.view.View;


import android.view.View.OnClickListener;


import android.view.ViewGroup.LayoutParams;


import android.widget.FrameLayout;


import android.widget.ImageView;


import android.widget.TextView;






public class GameActivity extends Activity implements OnClickListener,Runnable {


	


	private static final int INTERVALL = 100;


	private static final int ZEITSCHEIBEN = 600;


	private static final long HOECHSTALTER_MS = 2000;


	private float massstab;


	private int zeit;


	private int punkte;


	private int runde;


	private int muecken;


	private int gefangeneMuecken;


	private Random zufallsgenerator = new Random();


	private FrameLayout spielbereich;


	private Handler handler = new Handler();


	


	@Override


    public void onCreate(Bundle savedInstanceState) {


        super.onCreate(savedInstanceState);


        setContentView(R.layout.game);


		massstab = getResources().getDisplayMetrics().density;


		spielbereich = (FrameLayout)findViewById(R.id.spielbereich);


		spielStarten();


    }


	private void spielStarten() {


		runde = 0;


		punkte = 0;


		starteRunde();


	}


	private void starteRunde() {


		runde = runde +1;


		muecken = runde * 10;


		gefangeneMuecken = 0;


		zeit = ZEITSCHEIBEN ;


		bildschirmAktualisieren();		


		handler.postDelayed(this,INTERVALL);


	}


	private void bildschirmAktualisieren() {


		TextView tvPunkte = (TextView)findViewById(R.id.points);


		tvPunkte.setText(Integer.toString(punkte));


		TextView tvRunde = (TextView)findViewById(R.id.round);


		tvRunde.setText(Integer.toString(runde));


		TextView tvMuecken = (TextView)findViewById(R.id.hits);


		tvMuecken.setText(Integer.toString(gefangeneMuecken));


		TextView tvZeit = (TextView)findViewById(R.id.time);


		tvZeit.setText(Integer.toString(zeit));


		FrameLayout flTreffer = (FrameLayout)findViewById(R.id.bar_hits);


		FrameLayout flZeit = (FrameLayout)findViewById(R.id.bar_time);


		LayoutParams lpTreffer = flTreffer.getLayoutParams();


		lpTreffer.width = Math.round(massstab * 300 * Math.min(gefangeneMuecken,muecken) / muecken );


		LayoutParams lpZeit = flZeit.getLayoutParams();


		lpZeit.width = Math.round( massstab * zeit * 300/ZEITSCHEIBEN );


	}


	private void zeitHerunterzaehlen() {


		zeit = zeit-1;


		float zufallszahl = zufallsgenerator.nextFloat();


		double wahrscheinlichkeit = muecken *1.5 /ZEITSCHEIBEN;


		if (wahrscheinlichkeit &gt;1){


			eineMueckeAnzeigen();


			if ( zufallszahl &lt; wahrscheinlichkeit -1) {


				eineMueckeAnzeigen();


			}


		} else {


			if (zufallszahl &lt; wahrscheinlichkeit ) {


				eineMueckeAnzeigen();


			}


		}


		mueckenVerschwinden();


		bildschirmAktualisieren();


		if(!pruefeSpielende()) {


			if(!pruefeRundenende()){


				handler.postDelayed(this,INTERVALL);


			}


		}






	}


	private boolean pruefeSpielende() {


		if ( zeit == 0 && gefangeneMuecken &lt; muecken ) {


			gameOver();


			return true;


		}


		return false;


	}


	private boolean pruefeRundenende() {


		if (zeit == 0) {


			starteRunde();


			return true;


		}


		return false;


		}


	private void eineMueckeAnzeigen() {


		int breite = spielbereich.getWidth();


		int hoehe = spielbereich.getHeight();


		ImageView muecke = new ImageView(this);


		muecke.setImageResource(R.drawable.muecke);


		muecke.setTag(R.id.geburtsdatum, new Date());


		muecke.setOnClickListener(this);


		int muecke_breite = (int) Math.round(massstab*50);


		int muecke_hoehe = (int) Math.round(massstab*42);


		


		int links = zufallsgenerator.nextInt( breite - muecke_breite );


		int oben = zufallsgenerator.nextInt( hoehe - muecke_hoehe );


		










		FrameLayout.LayoutParams params = new FrameLayout.LayoutParams(muecke_breite,muecke_hoehe);


		params.leftMargin = links;


		params.topMargin = oben;


		params.gravity= Gravity.TOP + Gravity.LEFT;


		spielbereich.addView(muecke,params);


		


				


	}


	


	private void mueckenVerschwinden() {


		spielbereich = (FrameLayout) findViewById(R.id.spielbereich);


		


		int nummer = 0;


		while(nummer &lt; spielbereich.getChildCount() ) {


			ImageView mueckev = (ImageView) spielbereich.getChildAt(nummer);	


			Date geburtsdatum = (Date) mueckev.getTag(R.id.geburtsdatum);


			Date aktuell = new Date();


			long alterakt=aktuell.getTime();


			long altergeb= ((Date)geburtsdatum).getTime();


			long alter = (new Date()).getTime() - geburtsdatum.getTime();


			if(alter &gt; HOECHSTALTER_MS) {


				spielbereich.removeView(mueckev);


			} else {


				nummer++;


			}


		}


	}


	private void gameOver() {


		Dialog dialog = new Dialog(this, android.R.style.Theme_Translucent_NoTitleBar_Fullscreen);


		dialog.setContentView(R.layout.gameover);


		dialog.show();


	}


	@Override


	public void onClick(View muecke) {


		gefangeneMuecken++;


		punkte += 100;


		bildschirmAktualisieren();


		spielbereich.removeView(muecke);


	}


	


	


	@Override


	public void run() {


		zeitHerunterzaehlen();


	}


	






	 


}

Alles anzeigen

titus · 2. Juni 2012

Hi Proffni,

Was sagt den die LogCAt?
Wo tritt den dein Fehler auf?

Mfg Titus

Proffni · 3. Juni 2012

Zitat

Code

06-01 18:14:56.025: W/dalvikvm(1002): threadid=1: thread exiting with uncaught exception (group=0x4001d800)
06-01 18:14:56.035: E/AndroidRuntime(1002): FATAL EXCEPTION: main
06-01 18:14:56.035: E/AndroidRuntime(1002): java.lang.NullPointerException
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at de.proffni.mueckenmassaker.GameActivity.mueckenVerschwinden(GameActivity.java:140)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at de.proffni.mueckenmassaker.GameActivity.zeitHerunterzaehlen(GameActivity.java:84)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at de.proffni.mueckenmassaker.GameActivity.run(GameActivity.java:165)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at android.os.Handler.handleCallback(Handler.java:587)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at android.os.Handler.dispatchMessage(Handler.java:92)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at android.os.Looper.loop(Looper.java:123)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at android.app.ActivityThread.main(ActivityThread.java:4627)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at java.lang.reflect.Method.invokeNative(Native Method)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at java.lang.reflect.Method.invoke(Method.java:521)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:626)
06-01 18:14:56.035: E/AndroidRuntime(1002): 	at dalvik.system.NativeStart.main(Native Method)

Alles anzeigen

und in dieser Zeile 140 von gameActivity steht eben des long altergeb =((Date)geburtsdatum).getTime(), welches ich ja schon aus der kombinierten Formel herausgeholt hab um den Fehlerbereich einzuschränken. (Auch ohne das (Date) funktionierts nicht, welches ich nur nochmal extra dazu gemacht hab weil ich befürchtet hab dass es beim initialisieren vllt ned funktioniert hat aber daran liegts auch ned)

titus · 3. Juni 2012

Hi Proffni,

Java

long altergeb =((Date)geburtsdatum).getTime();

Im allgemeinen besagt die NullPointerException das du auf etwas zugreifst was nicht da ist bzw das Object ist Null.
Da hilft auch kein Cast nach Date weil wenn "geburtsdatum" null ist wäre es auch nach dem Cast null.

So da die Exception beim Aufruf der getTime() ausgelöst wird schauen wir uns mal die Erzeugung von "Geburtsdatum" an.
Ich vermutet das dein Problem hier liegt:

Java

Date geburtsdatum = (Date) mueckev.getTag(R.id.geburtsdatum);

Was ist denn R.id.geburtsdatum bzw wie sieht es aus? Ist das orginal so oder hast du das verändert?

Mfg Titus

Proffni · 3. Juni 2012

is im Original so, und wie du im Quelltext unter mueckeAnzeigen finden kannst, wird dieser Tag in R.id.geburtsdatum geschrieben beim erstellen der Datei, auch als (Date) eigentlich.

die id ist in einer ids.xml festgelegt die ich über den xml wizard in eclipse erstellt hab, wird auch in der (gen) java datei angezeigt, wo die ids festgehalten werden, müsste also eigentlich alles da sein .__.

titus · 3. Juni 2012

Hi Proffni,

also wenn ich deinen Code richtig verstehe dann wird in "R.id.geburtsdatum" NUR in der Funktion "eineMueckeAnzeigen()" ein Datum rein geschrieben.
Die Funktion "eineMueckeAnzeigen()" wird auch NUR in der Funktion "zeitHerunterzaehlen()" aufgerufen.

Hoffe du kannst mir soweit folgen.

Sollte jetzt dies IF Abfrage in "zeitHerunterzaehlen()" scheitern:

Java

if (wahrscheinlichkeit &gt;1){
			eineMueckeAnzeigen();
			if ( zufallszahl &lt; wahrscheinlichkeit -1) {
				eineMueckeAnzeigen();
			}
		} else {
			if (zufallszahl &lt; wahrscheinlichkeit ) {
				eineMueckeAnzeigen();
			}
		}


mueckenVerschwinden();

Alles anzeigen

Das heißt ist die erste If False und die If innerhalb des Else Zweiges auch False, wird kein Datum in "R.id.geburtsdatum" geschrieben.
Nach dieser If Abfrage wird jetzt aber die Funktion "mueckenVerschwinden();" aufgerufen und in ihr fragst du das Datum ab.
Steht kein Datum in "R.id.geburtsdatum" kommt es zur NullPointerException.

So soweit die Theorie du müsstest jetzt deinen Code entweder durch den Debugger jagen und mittels Haltepunkte sehen ob die If-Bedingungen durchlaufen werden oder du machst dir mittels Log.d() ein paar Kontrollausgaben innerhalb der If Abfrage.

Sollten die Kontrollausgaben vor dem Stacktrace der Execption in der LogCat stehen ist meine THeorie hinfällig und wir müssen uns irgendetwas anderes überlegen.

mfg Titus

P.s. habs eben mal durch gerechnet.

Java

double wahrscheinlichkeit = muecken *1.5 /ZEITSCHEIBEN;

0,025 = 10 * 1.5/600;
so damit ist die erste If Abfrage false.

die zweite wird wahrscheinlich auch false sein da diese simple Test:

Java

Random zufallsgenerator = new Random();
System.out.println(zufallsgenerator.nextFloat());

fast immer eine Zahl geliefert hat die größer als 0,025 war.

Um sicher zugehen mach das mit dem Debugger bzw Kontrollausgeben.

Proffni · 3. Juni 2012

du liegst wohl richtig, habe in allen 3 bereichen der if-konstellation eine Lognachricht eingestellt, keine wurde vor der exception angezeigt, nur verstehe ich dann nicht was ich falsch gemacht hab bzw wie ich das behebe, da die if-konstellation im original genau gleich ist... und dort auch zu funktionieren scheint

Proffni · 3. Juni 2012

Zitat von Proffni

du liegst wohl richtig, habe in allen 3 bereichen der if-konstellation eine Lognachricht eingestellt, keine wurde vor der exception angezeigt, nur verstehe ich dann nicht was ich falsch gemacht hab bzw wie ich das behebe, da die if-konstellation im original genau gleich ist... und dort auch zu funktionieren scheint

habe gerade nochmal durch den code geschaut... dieser fehler wird ja eigentlich durch die while schleife in mueckenVerschwinden() ausgeschlossen... die Nummer wird auf 0 gesetzt und nur wenn die nummer kleiner ist als die anzahl der muecken welche ja in "spielbereich.getChildCound()" ausgegeben werden (im spielbereich sind keine anderen childs), sollte überhaupt die whileschleife ausgeführt werden

EDIT: ok vergiss es, ich hab mir ja gerade selber erklärt wo der fehler liegt im design hatte ich noch einen imageview in "spielbereich" standardmäßig drin der nur ausgeblendet ist... in der anleitung stand dass man den testweise erstellen soll um sich die attribute anzuschauen, und ich depp hab vergessen die wieder rauszulöschen - danke.. problem solved mein programm läuft - hätte nur die scheiß mücke ausm design löschen müssen xD

[SOLVED]Problem: NullPointerException - ich finde den Fehler nicht

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema