Discussion :

[Death83]

Salut à tous,

pour débuter et me faire la main en C++ je suis en train de faire un petit programme de cryptage (sans aucune autres prétentions que de me faire la main).

Donc la je travail sur le code de césar mais j'ai un petit soucis. En effet pour ce code il faut décaler toute les lettres. Donc lorsque qu'on fait le tour (on depasse z ) il faut revenir sur a. Sur mon code ca marche pour les Majuscule mais pas pour les minuscules alors que c'est exactement le même. Comment ca ce fait?

Voici le code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
  if(TextaCrypter[i]>=65 && TextaCrypter[i]<=90)
  {
             TextaCrypter[i]=(TextaCrypter[i]+Cle_Cesar);
              if(TextaCrypter[i]>90)
              {
 
                    TextaCrypter[i]=(TextaCrypter[i]-26);
              }
 
   }
   else if(TextaCrypter[i]>=97 && TextaCrypter[i]<=122)
   {
         TextaCrypter[i]=(TextaCrypter[i]+Cle_Cesar);
          if(TextaCrypter[i]>122)
          {
                 TextaCrypter[i]=(TextaCrypter[i]-26);
          }
    }

où TextaCrypter est une chaine de caractères et Cle_Cesar un chiffre compris entre 1 et 26.
Par exemple si je met T j'obtiens bien F mais si je met t j'obtient le sigle Euro :/
.

[JolyLoic]

Ton problème est probablement du au fait que sur ta plateforme, le type char est signé et codé sur 8 bits, ce qui fait que 127+1 = -128.

Mais il y a d'autres problèmes dans ce code :
- Il suppose que les lettres sont contigues, ce qui n'a rien d'obligatoire, et même si ça peut marcher pour des cas simples, c'est le genre de mauvaises habitudes qu'il vaut mieux éviter de prendre,
- Il utilise des valeurs magiques. Si vraiment tu veux garder la première supposition, alors écrit le code : if(TextaCrypter[i]>='A' && TextaCrypter[i]<='Z')

[Death83]

Ca ne marche toujours pas en mettant les caractères au lieu du code ascii.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
 if(TextaCrypter[i]>='A' && TextaCrypter[i]<='Z')
                {
                        //Le caractère à coder est une majuscule
                        TextaCrypter[i]=(TextaCrypter[i]+Cle_Cesar);
                        if(TextaCrypter[i]>'Z')
                        {
                                TextaCrypter[i]=(TextaCrypter[i]-26);
                        }
 
                }
                else if(TextaCrypter[i]>='a' && TextaCrypter[i]<='z')
                {
                        //Le caractère à coder est une minuscule
                       TextaCrypter[i]=(TextaCrypter[i]+Cle_Cesar);
                        if(TextaCrypter[i]>'z')
                        {
                                TextaCrypter[i]=(TextaCrypter[i]-26);
                        }
                }

Comment dois-je m'y prendre?

[JolyLoic]

Je t'ai expliqué dans mon premier message pourquoi ça ne marche pas.

Si tu veux que ça "marche", le plus simle est de t'assurer que tu ne sortira jamais, même temporairement, de la plage des caractères. Un truc genre ce qui suit devrait marcher sur certaines plate-formes, tant que Cle_Cesar n'est pas trop grand :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

TextaCrypter[i] = 'a' + (TextaCrypter[i] - 'a' + Cle_Cesar)%26;

[Death83]

Je t'ai expliqué dans mon premier message pourquoi ça ne marche pas.

Si tu veux que ça "marche", le plus simle est de t'assurer que tu ne sortira jamais, même temporairement, de la plage des caractères. Un truc genre ce qui suit devrait marcher sur certaines plate-formes, tant que Cle_Cesar n'est pas trop grand :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

TextaCrypter[i] = 'a' + (TextaCrypter[i] - 'a' + Cle_Cesar)%26;

Je vais essayer ca. Pour le chiffre de césar dans mon cas il est compris entre 1 et 26.