Il y a un problème avec l'instruction if dans notre setter

Question

Nous sommes en train de créer un setter avec l'instruction if. Nous essayons de limiter les valeurs approuvées et utilisons une méthode d'impression pour imprimer la valeur qui est définie, ou "faux" si la valeur n'est pas acceptée.

Ceci est la méthode:

Public class Sykkel extends Kjøretøy 
{ 

private int height; 


public Sykkel(String colour) 
{ 
    // initialise instance variables 
    super(1, colour); 
    this.height = height; 
} 

public void setHeight (int height) { 
    if(height > 35 && height < 70){ 
     System.out.println("Sykkelen er " + height + " cm høy."); 
    } else { 
     System.out.println("Wrong"); 
    } 
} 

public int getHeight() { 
    return height; 
} 

Peu importe ce que nous avons la hauteur il imprime « Faux ». Quelqu'un voit-il l'erreur dans le code?

nous avons essayé à la fois this.height et juste la hauteur, mais le résultat est toujours le même.

Merci d'avance.

Answer 1

Mes observations.

Votre constructeur devrait être:

public Sykkel(String colour, int height) 
{ 
    // initialise instance variables 
    super(1, colour); 
    this.height = height; 
} 

---

Dans votre méthode setHeight, vous devez attribuer la hauteur à la variable d'instance.

public void setHeight (int height) { 
    if(height > 35 && height < 70){ 
     System.out.println("Sykkelen er " + height + " cm høy."); 
    } else { 
     System.out.println("Wrong"); 
    } 
    // add the below line: 
    this.height = height; 
} 

---

L'autre méthode semble bien.

Quand j'appelle setHeight(40);, je reçois

Sykkelen er 40 cm høy. 

---

Hope this helps!

Answer 2

Il n'y a pas d'affectation dans la méthode setter, ajoutez juste

this.hauteur = hauteur;

public void setHeight (int height) { 
    if(height > 35 && height < 70){ 
     System.out.println("Sykkelen er " + height + " cm høy."); 
    } else { 
     System.out.println("Wrong"); 
    } 
this.height = height; 
}