Problem mit ArithmeticException

[Luk10]

Hallo,

Ich bin gerade total verwirrt weil ich hier einen

java.lang.ArithmeticException

bekomme.
Das seltsame ist, dass ich den Code eigentlich nicht angerührt habe und der gestern noch funktioniert hat. :noe:

Hier die Methode, bei welcher die Exception auftritt:

	private void setTime () {
		delta = System.currentTimeMillis() - last;
		last = System.currentTimeMillis();
		fps = ((long)1e9) / (delta*1000000);
	}

Hier die Exception selbst:
Exception in thread "main" java.lang.ArithmeticException: / by zero
at cells_sim.Gamepanel.setTime(Gamepanel.java:50)
at cells_sim.Gamepanel.doSetup(Gamepanel.java:39)
at cells_sim.Gamepanel.<init>(Gamepanel.java:31)
at cells_sim.Gamepanel.main(Gamepanel.java:17)

Also ich finde meinen mathematischen Fehler einfach nicht ... ich teile ja nicht durch 0 oder sowas in der Art.

Danke, Luk

 

[eRaaaa]

Das sagt die Exception aber
Schau dir mal dein delta an....

 

[Luk10]

Hm also

last

weise ich schon in

	private void doSetup() {
		Admin.getAdmin().getGamepanel(this);
		
		last = System.currentTimeMillis();

Einen Wert zu.
Also ist der wert von last sagen wir 100
-> 10 ms vergehen
110 - 100 = 10

Wo soll da ein mathematischer Fehler sein?

 

[bifidus]

Mit was hast du "last" initialisiert? delta ist ja die aktuelle Zeit (in Millisekunden) minus last. Aber last hat hier noch den Wert, den du initialisiert hast.

Den neuen Wert bekommt last erst, nachdem delta berechnet wurde!

 

[Ruzmanz]

Da dürfte keine 0 drinnen sein. Zeige mal bitte deine doSetup().

 

[Gast2]

Und was ist los wenn nur 0,1ms vergehen?

Außerdem kannst du dich auf Werte <20ms (je nach Betriebssystem) nicht wirklich verlassen.

Versuchs mal irgendwie so:

delta = Math.max(1, System.currentTimeMillis() - last);

 

[eRaaaa]

Keine Ahnung, das musst ja du wissen..vllt wird halt setTime vor doSetup aufgerufen oder so...
last = 0 ... daher solltest du dir die Werte in setTime ja mal anschauen..Debugger/System.out..oder wie auch immer..solche Fehler sind doch schnell gefunden !

 

[bifidus]

currentTimeMillis

public static long currentTimeMillis()

Returns the current time in milliseconds. Note that while the unit of time of the return value is a millisecond, the granularity of the value depends on the underlying operating system and may be larger. For example, many operating systems measure time in units of tens of milliseconds.

See the description of the class Date for a discussion of slight discrepancies that may arise between "computer time" and coordinated universal time (UTC).

Returns:
the difference, measured in milliseconds, between the current time and midnight, January 1, 1970 UTC.
See Also:
Date

 

[Luk10]

Mit was hast du "last" initialisiert? delta ist ja die aktuelle Zeit (in Millisekunden) minus last. Aber last hat hier noch den Wert, den du initialisiert hast.

Den neuen Wert bekommt last erst, nachdem delta berechnet wurde!

Was ja nichts zur Sache tut, wann last einen anderen Wert bekommt.
Ich meine ich initialisiere last, dann vergeht Zeit, dann ziehe ich last von der aktuelle Zeit (größer als last) ab.

Folglich einen positive Zahl, folglich kein Fehler. :noe:

 

[bifidus]

Das besagt, dass du je nach Betriebssystem eine gewisse Spannweite berücksichtigen musst. Das Programm scheint hier also eindeutig zu schnell zu sein, um unterschiedliche Werte zu generieren.

 

[Ruzmanz]

public class Test {
    private long delta;
    private long last;
    private long fps;

    public Test() {
        while(true)
        setTime();     
    }

    private void setTime () {
        delta = System.currentTimeMillis() - last;
        last = System.currentTimeMillis();
        fps = ((long)1e9) / (delta*1000000);
    }
}

So wird das nicht funktionieren. Du brauchst mindestens einen sleep oder mehr Anweisungen, die zwischen dem last und dem delta ausgeführt werden. Ich nehme mal an, dass du eine Spielschleife öffnest?

 

[Luk10]

Okay, der Fehler war, dass zu wenige Zeit zwischen Initialisieren von last und berechnung von delta vergeht.

Mein PC ist wohl heute leistungsstärker als gestern

Danke, für die schnellen Antworten.

Luk

 

[Luk10]

	private long delta, last, fps;
	
	public static void main (String[] args) {
		new Gamepanel(300, 300);
	}
	
	public Gamepanel (int h, int w) {
		game_running = true;
		
		JFrame frame = new JFrame("Cell's Sim");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		this.setPreferredSize(new Dimension(w, h));
		
		frame.add(this);
		frame.pack();
		
		doSetup();
		
		frame.setVisible(true);
	}
	private void doSetup() {
		Admin.getAdmin().getGamepanel(this);
		
		last = System.currentTimeMillis();
		
		Admin.getAdmin().createCell("yeast");
		
		Thread gameloop = new Thread(this);
		gameloop.start();
		
	}
	private void setTime () {
		System.out.println(last);
		delta = System.currentTimeMillis() - last;
		System.out.println(delta);
		last = System.currentTimeMillis();
		fps = ((long)1e9) / (delta*1000000);
	}
	public void run () {
		while (game_running) {
			
			setTime();

Ja, da muss ich wohl erstmal schlafen bis ich berechnen kann