Arithmetic Exception, obwohl nichts 0 ist

[corvus]

Hallo,

ich bräuchte mal Hilfe bei einer sehr speziellen Frage, auf die ich bisher keinerlei Antwort finden konnte.
Mein Code sieht folgendermaßen aus:

int requiredWidth = 320;
int scale_start=2;
		if (Runtime.getRuntime().maxMemory() > 64*1024*1024)
			scale_start=1;
int scale = requiredWidth <= 480 ? 2 : scale_start;
		debug("scale: " + scale);
		try{
			while(o.outWidth / scale / 2 >= requiredWidth)
				scale *= 2;
		}catch(ArithmeticException e){
			warn("Cannot calculate scale. Setting it to 2...", e);
			scale = 2;
		}

Jetzt kommt es manchmal dazu, dass eine ArithmeticException geworfen wird:

java.lang.ArithmeticException: divide by zero

. Soweit ja noch alles klar, dachte ich mir, ist "scale" halt eben komischerweise 0. Deshalb hab ich die debug-Nachricht davor eingebaut. Ergebnis: "scale" ist in allen Fällen 2! Und ab der Stelle hab ich keine Ahnung mehr :noe:
Meine beste Idee dazu wäre noch, dass Java - warum auch immer - in speziellen Fällen Variablen auf einen Standardwert (in diesem Fall wahrscheinlich 0) setzt, aber zu so etwas konnte ich nichts im Internet finden.
Ich weiß nicht, ob ihr den Fehler schonmal hattet, aber falls ja oder ihr auch sonst eine Idee habt, woran das liegen könnte, wäre ich sehr dankbar über eure Mithilfe Vielen Dank im Voraus für alle hilfreichen Antworten!
Viele Grüße
corvus

 

[Joose]

Vor der Schleife ist es zwar 2, aber da der Wert in der Schleife geändert wird solltest du überprüfen welche Werte scale innerhalb der Schleife annimmt.

 

[JCODA]

Es gibt den Fall, dass scale_start=1; ...(wegen der Memory)

dann kann scale 1 sein, wegen der requiredWidth.

In der Schleife scale mit Ganzzahlarithmetik durch 2 => 1/2 = 0 .
und dann teilst du outWidth durch 0.

 

[Joose]

In der Schleife scale mit Ganzzahlarithmetik durch 2 => 1/2 = 0 .
und dann teilst du outWidth durch 0.

Hätte ich mir auch gedacht, bin mir da aber nicht ganz sicher, da meiner Meinung nach Java auch von links nach rechts rechnet.
Somit wird die Division o.outWidth / scale zuerst berechnet und dann erst durch 2 dividiert.

 

[corvus]

In diesem Fall liegt Joose richtig. Denn ich hab überhaupt nicht daran gedacht, dass sich scale ja auch in der Schleife ändern kann -.- Ich hab die Schleife also erstmal folgendermaßen umgebaut:

while(o.outWidth / scale / 2 >= requiredWidth){
	scale *= 2;
	debug("scale loop: " + scale + ", o.outWidth: " + o.outWidth + ", requiredWidth: " + requiredWidth);
}

Die Ausgabe sah dann ungefähr so aus:

12-18 09:26:54.630: D/GB::ImageHelper(546): scale loop: 4, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.631: D/GB::ImageHelper(546): scale loop: 8, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.632: D/GB::ImageHelper(546): scale loop: 16, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.635: D/GB::ImageHelper(546): scale loop: 32, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.636: D/GB::ImageHelper(546): scale loop: 64, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.637: D/GB::ImageHelper(546): scale loop: 128, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.639: D/GB::ImageHelper(546): scale loop: 256, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.641: D/GB::ImageHelper(546): scale loop: 512, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.643: D/GB::ImageHelper(546): scale loop: 1024, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.646: D/GB::ImageHelper(546): scale loop: 2048, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.651: D/GB::ImageHelper(546): scale loop: 4096, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.653: D/GB::ImageHelper(546): scale loop: 8192, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.661: D/GB::ImageHelper(546): scale loop: 16384, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.663: D/GB::ImageHelper(546): scale loop: 32768, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.666: D/GB::ImageHelper(546): scale loop: 65536, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.668: D/GB::ImageHelper(546): scale loop: 131072, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.683: D/GB::ImageHelper(546): scale loop: 262144, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.684: D/GB::ImageHelper(546): scale loop: 524288, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.686: D/GB::ImageHelper(546): scale loop: 1048576, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.687: D/GB::ImageHelper(546): scale loop: 2097152, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.689: D/GB::ImageHelper(546): scale loop: 4194304, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.690: D/GB::ImageHelper(546): scale loop: 8388608, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.696: D/GB::ImageHelper(546): scale loop: 16777216, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.697: D/GB::ImageHelper(546): scale loop: 33554432, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.698: D/GB::ImageHelper(546): scale loop: 67108864, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.700: D/GB::ImageHelper(546): scale loop: 134217728, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.701: D/GB::ImageHelper(546): scale loop: 268435456, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.703: D/GB::ImageHelper(546): scale loop: 536870912, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.704: D/GB::ImageHelper(546): scale loop: 1073741824, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.705: D/GB::ImageHelper(546): scale loop: -2147483648, o.outWidth: 1378, requiredWidth: 0
12-18 09:26:54.707: D/GB::ImageHelper(546): scale loop: 0, o.outWidth: 1378, requiredWidth: 0

Was ich nicht bedacht hatte: "requiredWidth" wird unter anderem anhand von zwei lokalen statischen Variablen berechnet, die zwar standardmäßig in der Klasse mit 0 initialisiert werden, aber eigentlich von einer anderen Klasse, die den genauen Wert dafür berechnet, folglich auch den richtigen Wert gesetzt werden. Bloß gab es dabei wohl manchmal einen Fehler, weshalb diese Variablen 0 blieben und requiredWidth auf 0 setzten, was zu diesem Überlauf führte. Ich hab das Problem jetzt erstmal damit gelöst, dass die zwei Variablen Standardwerte ungleich 0 haben. Bisher gab es damit noch keine Probleme.
Also, nochmal vielen Dank, Joose
Falls wir uns hier nicht mehr sehen, frohe Weihnachten und einen guten Rutsch an alle
Viele Grüße
corvus

P.S.: Ich find den Spoiler hier nie...

 

[Joose]

Ich dachte schon an ein Problem mit dem Überlauf, nur war ich mir da nicht sicher ob er auf -214..... bleibt oder dann doch noch auf 0 springt