kleine get-Methode anhand einer Aufgabenstellung

[Natron123]

Hallöchen liebe Javagemeinde!

Da ich das letzte mal schon so schnelle und hervorragende Hilfe bekommen habe kommt schon meine nächste frage


Die Teilaufgabe lautet wiefolgt:

Der Panzer braucht einige Attribute. Da er von der Klasse Sprite erbt, besitzt er bereits eine x- und
y-Koordinate, eine Breite und eine Höhe. Ergänzen Sie ein ganzzahliges Attribut angle in dem der Winkel
des Panzerrohres gespeichert wird. Der Winkel wird im Gradma angegeben und bewegt sich im Bereich
von 0-180. Schreiben Sie für dieses Attribut eine get-Methode.

Mein Ansatz ist folgender (nur auf den Winken bezogen):

public int getAngle (){
		if (this.angle < 0){
			angle = 0;
		}
		if (this.angle > 180){
			angle = 180;
		}
		
		this.angle = angle;
		return angle;
	}

bei "this.angle = angle;" bekomme ich folgende Fehlermeldung: "The assignment to variale angle has no effect"

Wo liegt das Problem? Habe ich überhaupt den richtigen Ansatz?

Vielen Dank im Voraus für konstruktive Beiträge .

LG Natron123

 

[bone2]

lass alle this weg, das ist ja alles die selbe klassenvariable.

 

[Timothy Truckle]

Ganz klar: Du hast keine lokale Variable

angle

deklariert, so dass

angle

und

this.angle

identisch sind.

[EDIT]BTW: änderungen an den Member-Variablem macht man wenn diese gesetzt werden, keines falls im Getter. Grund: die Änderung wird sonst bei

jedem

Lesen abgearbeitet.

Und die Funktion, die Du hier nachbildest heist "Modulo". Dafür hat Java einen Operator:

angle = angle%180; // damit ist angle immer kleiner 180, weil 180/180= 1 Rest 0

[/EDIT]

bye
TT

 

[Natron123]

[EDIT]
Und die Funktion, die Du hier nachbildest heist "Modulo". Dafür hat Java einen Operator:

angle = angle%180; // damit ist angle immer kleiner 180, weil 180/180= 1 Rest 0

[/EDIT]

bye
TT

was ist wenn mein winkel unter 0 geht? funzt das dann immernoch?

also bsp -20%180 = ?

 

[TKausL]

Probiers aus :/

Probiers aus :/

class a{
public static void main(String[] args){
System.out.println(-20%180);
}
}

 

[Natron123]

is doch klar was raus kommt, aber ich muss vermeiden, dass werte über 180 und unter 0 raus kommen...

und das ist eben die frage. wie bekomme ich das hin? weil der modulo hier fehl am platz ist, da -20%180=-20

 

[TKausL]

int grad = -20;
grad = (180+grad)%180;

 

[bone2]

Deine Methode geht schon so in Ordnung, auch wenn ich das mit 0<angle<180 eher im setter, konstruktor oder einer etwaigen logischen methode prüfen würde.
Du musst nur die this weglassen udn sie funktioniert erstmal.

Toll TKausL, was ist bei

int grad = -190

?

 

[TKausL]

Toll TKausL, was ist bei

int grad = -190

?

Bin jetzt mal davon ausgegangen, dass das Objekt sich innerhalb eines Ticks nicht um >180 Grad drehen kann

 

[Timothy Truckle]

Toll TKausL, was ist bei

int grad = -190

?

wenn das im Setter gemacht wird kann dass ja eigendlich nicht passieren, aber:

Math.abs(grad%180);

bye
TT

 

[TKausL]

Math.abs kannst du nicht nutzen, da aus -20 +20 wird, -20 ist aber effektiv +160.

 

[bone2]

Nein -20 is effektiv 0.
Was habt ihr gegen das einfache, saubere if?

Die Antwort auf die Frage des TE ist ein:

this

entfernen.
Zusatztipp: Diese <> modifikationen in Logik, Setter, Konstruktor erledigen, nicht im Getter

Alles andere was gesagt wurde schwankt zwischen falsch und off-topic

 

[faetzminator]

[c]Math.max(0, Math.min(180, grad))[/c]

 

[Crian]

Das ist vielleicht eine philosophische Frage, aber ich würde sowas niemals im Getter machen. Bei Veränderung des Winkels muss auf den Gültigkeitsbereich geachtet werden, also im Setter und ggf. in irgendwelchen drehDasRohrUmGrad() Methoden.