Bonjour,

Sa fait maintenant 3h que je planche sur un truc, il me manque que sa pour finir mon devoir mais je trouve vraiment pas...
Voila mon code:

public class LedCouleur
{

private String couleur;
private static String[] tableauCouleur={"rouge","jaune","vert","bleu"};

public static boolean isCouleurValide(String couleur)
{
for(int i=0 ; i < LedCouleur.tableauCouleur.length ; i++)
{

if(couleur==LedCouleur.tableauCouleur[i])

return true;
else
return false;
}

}

La méthode test si la couleur rentrée fait partie des 4 couleurs possible et retourne true ou false selon sa validité.

A la compilation le problème qui se pose serait qu'il manquerait une accolade alors que non tout est bien délimité, je me suis dis que le problème viendrait du fait qu'on ne sort pas de la boucle for avant d'avoir parcouru tout le tableau... en fait je sais pas NEED HELP

Merci de la réponse, mais l'accolade fermante y est c'est juste que je vous ai c/c le début du programme (la totalité fait à peu près 150 lignes) et même avec la méthode equals sa ne change rien j'ai pensé que c'était du au fait que la fonction ne retourné finalement rien a la fin mais je sais pas en fait comment faire...

En fait j'avais fait comme ça au départ mais le soucis est qu'il me retourne toujours false...

(dans me devoir il me demande d'écrire la méthode isCouleurValide()
je m'en tiens à ce qui est donnée même si c'est moyens xD)

Ah nan j'ai rien dis sa marche parfaitement ! Mille merci !

Qu'est-ce que le compilateur t'envoie PRÉCISÉMENT comme message d'erreur?

:~/Documents/Java/luminaire$ javac LedCouleur.java
LedCouleur.java:22: missing return statement
}
^
1 error

C'était ça mais la y a plus de problème merci encore !

J'en profite pour vous posez ma dernière question tout d'abord voila la suite du code:

public class LedCouleur extends Led
{
private String couleur;
private static String[] tableauCouleur={"rouge","jaune","vert","bleu"};

public static boolean isCouleurValide(String couleur)
{
for(int i=0 ; i < LedCouleur.tableauCouleur.length ; i++)
{

if(couleur.equals(LedCouleur.tableauCouleur[i])
)

return true;
}

return false;

}

public LedCouleur()
{
super();
this.couleur="rouge";
}

public LedCouleur(int reference , boolean etat , String couleur)
{
super(reference,etat);
this.couleur=couleur;
}

Je dois ensuite faire un constructeur par copie mais je sais pas trop comment faire en fait j'ai fais ça comme ça:

public LedCouleur(Led Couleur l)
{
super(l.code,l.etat);
this.couleur=l.couleur;
}

mais sa marche pas...

Need help again (c'est le dernier truc qu'il me manque)

petit problème again...

~/Documents/Java/luminaire$ javac LedCouleur.java
LedCouleur.java:40: code has private access in luminaire.Led
super(l.code,l.etat);
^
LedCouleur.java:40: etat has private access in luminaire.Led
super(l.code,l.etat);
^
2 errors

C'est bon j'ai trouvé mon erreures j'ai juste mis des get et ça a marché

En plus l'appel à super() est implicite, donc t'avais de toute façon pas à le mettre