Discussion :

[Valochxx]

Bonjour, à la suite d'un projet, je souhaite crypter une chaine de caractères. Mais à chaque fois il ne me convertie que les majuscules. Help plz

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
#include <stdio.h>
#include <iostream>
#include <string>
using namespace std;
 
//PRORORYPAGE
void messCryptCesar(const char *in, char *out, int key);
 
int main()
{
  char messageIn[106] = "AZ hello az";
  char messageOut[106];
  messCryptCesar(messageIn,messageOut, 1);
  cout << messageOut << endl; 
 
  return 0;
}
 
void messCryptCesar(const char *in, char *out, int key)
{
  for(int i=0; i < 106; i++)
  {
    //Minuscules
    if((in[i] >= 97) && (in[i] <= 122))
    {
      if(in[i] + key > 122)
      {
        out[i] = 96 + key;
      }
      else{
        out[i] = in[i] + key;
      }
    }
    //Majuscules
    if((in[i] >= 65) && (in[i] <= 90))
    {
      if(in[i] + key > 90)
      {
        out[i] = 64 + key;
      }
      else{
        out[i] = in[i] + key;
      }
    }
    else{
      out[i] = in[i];
    }
  }
}

[Valochxx]

Après avoir chercher de mon côté, je n'ai pas trouvé la raison du pourquoi ça ne voulait pas marcher...
Je suis passé par des string finalement.
Voilà le résultat avec un return de ma fct.

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <stdio.h>
#include <iostream>
#include <string>
using namespace std;
 
string Cesar(const string in, int key);
 
int main()
{
    string r("hello World!");
    string R;
    R = Cesar(r, 3);
    cout << R << endl;
    return 0;
}
 
string Cesar(const string in, int key)
{
    string out;
    out = in;
    for(int i=0;i < in.length();i++)
    {
        //MAJUSCULES
        if(in[i] >= 65 && in[i] <= 90)
        {
            if(in[i] + key > 90)
            {
                out[i] = 64 + key;
            }
            else{
                out[i] = in[i] + key;
            }
        }
        //MINUSCULE
        if(in[i] >= 97 && in[i] <= 122)
        {
            if(in[i] + key <=122)
            {
                out[i] = in[i] + key;
            }
            else{
                out[i] = key + 96;
            }
        }
    }
    return out;
}

[Sve@r]

Bonjour

if(in[i] >= 65 && in[i] <= 90)

Evite les "nombres magiques" => if (in[i] >= 'a' && in[i] <= 'z').
Ou mieux, utilise les outils déjà tout faits => if (islower(in[i]))...

Code c :

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

1
2
3
4
5
6
7
8
9
        {
            if(in[i] + key > 90)
            {
                out[i] = 64 + key;
            }
            else{
                out[i] = in[i] + key;
            }
        }

Tant qu'à faire, essaye aussi de factoriser un peu... => out[i]=(in[i] + key > 'z' ?'a'-1 :in[i]) + key...