Bonsoir, alors j'ai ce bout de code que je trouve assez lourd, y'aurait-il un moyen de le rendre plus petit ?

if(Red<32 && Green<32 && Blue<32 || Red == 0 && Blue == 0 && Green == 0 ){
LCD.drawString("Noir", 0, 5);

}else if (Red-Green>32 && Red-Blue>32){
LCD.drawString("rouge", 0, 5);
} else if (Blue-Red>32 && Blue-Green>32){
LCD.drawString("Bleu", 0, 5);
}else if (Green-Red>32 && Green-Blue>32){
LCD.drawString("Vert", 0, 5);
}else if ( Red==Blue && Blue ==Green){
//abs(Red-Green)<33 && abs(Red-Blue)<33 && abs (Blue-Green)<33
LCD.drawString("Gris", 0, 5);
} else if (Red > 223 && Green > 223 && Blue > 223){
LCD.drawString("Blanc", 0, 5);

} else if (Red-Green<33){
LCD.drawString("Jaune", 0, 5);

}/*else if ( abs(Red-Blue) < 33){
LCD.drawString("Rose", 0, 5);

}*/

Merci : D

Y a pas de façon évidente de raccourcir le code non.
Quelques petites remarques tout de même :

• plutôt que d'appeler LCD.drawString("nom de la couleur", 0, 5); dans chaque branche, tu devrais plutôt stocker le nom de la couleur dans une variable, et n'appeler LCD.drawString qu'à la fin. Ca te permettrait d'extraire toute la suite de if dans une méthode à part (rgbToColorName() par exemple) et d'appeler juste LCD.drawString(rgbToColorName(r, g, b), 0, 5)
• tu peux virer le || Red == 0 && Blue == 0 && Green == 0 de ta première condition, c'est redondant avec ce qu'il y a juste avant
• tu peux grouper les logiques similaires ensemble, par exemple en mettant le noir/blanc/gris en premier, puis les calculs plus "bizarres"
• tu peux donner des noms aux constantes 32 et 223 pour que ce le code soit compréhensible plus facilement ("MIN_SHADE"/"MAX_SHADE" par exemple...)

Merci beaucoup pour ta réponse, elle m'a été bien utile !

Tu fais une fonction externe private que tu utilises pour tes conditions ? // un switch ? à voir si tu peux l'utiliser