...aparte tienes un error semántico.
Creas la instancia de polígono, pero en ningún momento invocas calcular el polígono, luego al solicitar getangulo, devolverá 0.
Puede ser buena idea que al crearla instancia con NEW, si el parámetro no es 0, invoque la funcion CalcularAngulo
Algo así, mejor:
public class Poligono {
private int numLados;
private int angulo;
public Poligono(int x){
numLados=x;
if (x>0){
CalcularAngulo;
}
}
public void calcularAngulo(){
angulo=(360/numLados);
}
public int getAngulo(){
return angulo;
}
}
Nota finalmente como al método calcularAngulo, se le ha retirado el parámetro... la razón es que el parámetro ya está interno, tenerlo es tolerar errores, pués si se le pasa otro valor distinto a aquel con el que se creó la instancia el valor de ángulo quedará erróneo.
Por otro lado, si el ángulo se calcula cuando se crea la instancia, carece de sentido que dicho método sea público, basta calcularlo una vez, su valor no va a cambiar aunque lo llames 40 veces, luego procede hacerlo privado, o mejor aún quitarlo y pasar su código a la función NEW.
Podria interesar tener una clase estática, con un método que pase el parámetro numLados y devuelva el ángulo sin almacenar nada, así cada instancia al ser creada además de almacenar ambos datos, podrían invocar dicho método para hacer el cálculo a la vez que el usuario externamente podría hacer tales cálculos cuando fuere preciso, sin necesidad de tener que crear explícitamente una instancia para ocasión... pero entiendo que es algún ejercicio...
Autor
En línea
