Si le passage à l'écriture est un endroit critique pour vous, essayez d'utiliser un crayon une gomme et du papier (si, si, ça existe encore et c'est bien pratique, là, je ne plaisante pas).

Pour les trois premiers champs il suffit de tester si il y a un nombre et s'il est compris entre 0 et 255, par contre pour le code, c'est un peu plus compliqué, il faut tester qu'il y ait bien 7 caractères, que le premier soit "#", que les autres soient des chiffres ou des lettres de "A-F" et les passer en majuscules au besoin, rien de compliqué.

Pour le champ code, pas besoin de se casser la tête, il est unique, donc le test se fera à l'intérieur de la fonction, j'ai dit, Ugh !

Pour tester ensuite si le nombre est plus petit que zéro ou plus grand que 255 (pour retourner une erreur) on utilisera tout bêtement les opérateurs "<" et ">", et "||" (OU), c'est plus simple que de tester si il est égal à zéro, puis à un, puis à deux, etc... et de retourner "true" si tous ces cas sont vrais.

Ensuite pour le code, on va réutiliser la méthode .charAt() pour savoir si le "#" est bien en position 1, une propriété pas encore vue (je crois) pour connaître la longueur de la chaîne de caractères .length, puis la solution la plus simple me semble-t-il c'est de faire une boucle pour parcourir et tester les caractères 2 à 7, dans cette boucle (je choisi une boucle for) je vais simplement prendre le caractère avec .charAt et contrôler qu'il soit compris entre 0 et 9 (comme ci-dessus), ou A à F, voire a-f (et le passer en majuscule, mais là ça demande encore réflexion), et voilà.

Comment vais-je donc passer un caractère en majuscule ? Hum, il existe bien une méthode .toUpperCase, mais elle n'aime pas beaucoup les chiffres et on ne peut pas modifier le contenu initial de la variable avec, la seule solution que je vois c'est de reconstruire dans une autre variable, caractère par caractère, le nouveau code en profitant de la boucle de test, en transférant sans modification les nombres et caractères valides et en ne les changeant en majuscule que quand c'est nécessaire.

Bon, je crois que c'est à peu prêt tout ce dont j'ai besoin, je vais donc pouvoir passer à l'étape suivante.

Pour les trois premières, je n'ai pas besoin de variable, il faut juste faire les tests, mais pour la dernière, il me faut une variable pour la boucle for (y fera l'affaire), et une autre pour le "nouveau" code, je vais prendre z, comme elles sont déjà déclarées et initialisées pour le reste du programme et que l'on peut les modifier sans rien perturber car elles n'ont qu'un rôle temporaire partout, c'est tout bénef (car j'ai un poil dans la main).

Il faut aussi penser à saupoudrer des quelques fonctions alert() pour indiquer le dépassement des limites et autres erreurs, mais je ne vais pas m'embêter à remettre le champ fautif en état et à le resélectionner, j'ai déjà donné dans la validation de formulaire.

function frouge(x) {rr=x.value; dechexa(rr,vv,bb); rouge=rr; refresh();}

et je la modifie comme ça:

function frouge(x)
{
  if (parseInt(x.value)!=x.value) {return false}   // Pas un nombre
  if (x.value<0 || x.value>255)                    // Erreur si hors limites
  {
    alert('Erreur:\n
           Le nombre doit être compris entre 0 (noir) et 255 (rouge maximum)');
    return false;
  }
  rr=x.value;        // Pas de changement
  dechexa(rr,vv,bb); // Pas de changement
  rouge=rr;          // Pas de changement
  refresh();         // Pas de changement
  return true;                                     // OK on l'accepte
}

On modifie fvert et fbleu de la même façon sans oublier de changer les couleurs.

Et on attaque avec fcode qui s'écrivait précédemment:

function fcode(x) {cd=x.value; hexadec(cd); rr=rouge; vv=vert; bb=bleu; refresh();}

et qui devient:

function fcode(x)
{
  if (x.value.charAt(0)!="#")                 // Manque "#" en premier
  {
    alert('Le premier caractère du code doit être: \"#\"');
    return false;
  }
  if (x.value.length!=7)                      // Pas sept caractères
  {
    alert('Erreur:\n
           Le format du code est: \"#rrvvbb\" rr=rouge, vv=vert, bb=bleu.\n
           La valeur est à donner en hexadécimal (00 à FF).');
    return false;
  }
  z='#';                                      // Initialise le nouveau code
  for (y=1;y<7;y++)                           // Boucle de 1 à 6
  {
    if (parseInt(x.value.charAt(y))==x.value.charAt(y))     // Nombre ?
    {
      z=z+x.value.charAt(y);                  // Oui, on concatène au code
    }else                                     // Pas un nombre
    {
      if (x.value.charAt(y)>='A' && x.value.charAt(y)<='F')  // Lettre A à F ?
      {
        z=z+x.value.charAt(y);                // Oui, on concatène au code
      }else                                   // Pas lettre A-F
      {
        if (x.value.charAt(y)>='a' && x.value.charAt(y)<='f')// Lettre a à f ?
        {
          z=z+hexa.charAt(10+"abcdef".indexOf(x.value.charAt(y))); // voir note
        }else                                 // Alors c'est une erreur
        {
          break;                              // Pas la peine de continuer
        }
      }
    }
  }
  if (y<7)                                    // Boucle pas finie ?
  {
    alert('Erreur:\n
           Le format du code est: \"#rrvvbb\" rr=rouge, vv=vert, bb=bleu.\n
           La valeur est à donner en hexadécimal (00 à FF).');
    return false;
  }
  x.value=z;                                  // Nouveau code dans ancien
  cd=x.value;  // Pas de changement
  hexadec(cd); // Pas de changement
  rr=rouge;    // Pas de changement
  vv=vert;     // Pas de changement
  bb=bleu;     // Pas de changement
  refresh();   // Pas de changement
  return true;                                // Fin normale de la routine
}

Note: Comme j'ai utilisé quelques finesses de pro dans cette routine, je me sens obligé de vous donner encore quelques explications.

Les valeurs de la boucle for: on part du deuxième caractère du code avec y=1 (le premier est en position 0) et on boucle TANT QUE l'on a pas atteint la 8ième position avec y<7 (0 à 7 ça fait 8), pour les détails retournez voir les boucles.

On teste d'abord si c'est un nombre, puis si c'est une majuscule, et enfin si c'est une minuscule, et là, problème, la méthode .toUpperCase() me fait un caca nerveux, impossible de l'utiliser sur trois navigateurs différents. Crotte alors, je m'en passe et je fabrique moi même ma propre fonction miracle qui marche comme ça:

Et enfin l'astuce, en cas d'erreur la méthode habituelle consiste à mettre dans une variable une indication d'erreur que l'on teste plus loin, ici on casse la baraque avec un break qui sert deux buts, premièrement et surtout arrêter la boucle (qui aurait pu être très longue), et deuxièmenent à limiter la valeur de la variable de comptage, en effet, si la boucle se termine, la variable y contient 7, dans le cas d'une erreur et même sur le dernier caractère, cette valeur sera plus petite que 7 et on utilise cette propriété dans la suite du programme.

Il ne faut pas oublier non plus de remettre le nouveau code en place dans la bonne variable et c'est fini.

Quand à ceux qui ne sont même pas venus ici, pas la peine que je les engueule, ils ne le sauront de toutes façons pas (regardez le compteur et soyez fier, il n'y a pas beaucoup de courageux comme vous)...