warum double nicht gleich double ist???

[lumo]

hallo

ich hab grad eine kleine zwischenaufgabe zu lösen und steh gerade an.

ich habe ein bild und muss es in der größe verändern.
nun muss ich ermitteln obs landscape ist oder nicht (breitbild)
dazu überprüfe ich obs breiter ist als es hoch ist...

jetzt muss ich noch die aspekt ratio einhalten (damits nicht verzerrt wird)
alles easy. ABER

imgWidth = image.getWidth();
		imgHeight = image.getHeight();
		isLandscape = imgWidth > imgHeight;
		
		
		aspektRatio = new Double(imgWidth) / new Double(imgWidth);
		newHeight = new Double(getSize().width / aspektRatio).intValue();
		newWidth = new Double(getSize().height / aspektRatio).intValue();

		System.out.println("imgWidth:"+imgWidth);
		System.out.println("imgHeight:"+imgHeight);
		System.out.println("aspektRatio:"+aspektRatio);
		System.out.println("newWidth:"+newWidth);
		System.out.println("newHeight:"+newHeight);

so sollte das doch stimmen nur... kommt bei mir folgende ausgabe:

AR: 1.0
imgWidth:566
imgHeight:407
aspektRatio:1.0
newWidth:0
newHeight:0

dachte mir ... kann doch nicht sein... darum eine kleine MAIN funktion zum testen...

int width = 566;
		int height = 407;
		Double ar = new Double(width) / new Double(height);
		System.out.println(ar);
		int newHeight = new Double(800 / ar).intValue();
		System.out.println("as width is scaled to 800, the new height is: "
				+ newHeight);

ratet mal was hier rauskommt?

1.3906633906633907
as width is scaled to 800, the new height is: 575

und das ist eigentlich das, was ich mir auch oben erwartet hätte... kann im code nichts finden was mich auf das 1.0 schiebt...

:bahnhof:

 

[Geeeee]

[c]aspektRatio = new Double(imgWidth) / new Double(imgWidth);[/c]
mehr braucht man wohl nicht sagen

 

[Landei]

aspektRatio = new Double(imgWidth) / new Double(imgWidth);

Wenn man eine Zahl durch sich selber dividiert, kommt bekanntlich 1 raus. Du meinst sicher

aspektRatio = new Double(imgWidth) / new Double(imgHeight);

Davon abgesehen ist dein Code mit den unnötigen Double-Wrappern wirklich grausam, und dann noch ungenau (weil intValue immer abrundet, auch bei 0.9999).

aspektRatio = 1.0 * imgWidth / imgHeight;
newHeight =  (int) Math.round(getSize().width / aspektRatio);
newWidth =  (int) Math.round(getSize().height / aspektRatio);

 

[lumo]

:lol:
mensch, peinlicher gehts gar nicht mehr...
ab und zu sieht man den wald vor lauter bäumen nicht mehr...

DANKE

naja, um ein bild auf ein panel zu skalieren, ists egal ob 15,9999 oder 16 pixel...

 

[SlaterB]

> ists egal ob 15,9999 oder 16 pixel...

tja, aber durchs Abrunden werden es 15 und nicht 16, das ist schon eine etwas höhere Abweichung

 

[Landei]

Trotzdem ist der bevorzugte Weg, aus einem int ein double zu machen (wenn man keine Rundung braucht)

(double) intValue

oder

1.0 * intValue

und nicht

new Double(intValue)