fonction de cryptage

Version imprimable

Bonjour à tous,

j'ai créé la fonction de cryptage suivante:

Code:

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
 
char* crypter(char* chaine)
    {
        char* alphabet_normal = new char[30];
        alphabet_normal = "abcdefghijklmnopqrstuvwxyz ,;.";
        char* alphabet_secret = new char[30];
        alphabet_secret = ",;. abcdefghijklmnopqrstuvwxyz";
        //
        char* secret = new char[strlen(chaine)];
        char tmp;
        int tmp2 = 0;
        int unsigned i,j;
 
        for(i=0; i<strlen(chaine); i++)
        {
            tmp2 = -1;
            tmp = chaine[i];
            for(j=0; j<strlen(alphabet_normal); j++)
               if(alphabet_normal[j] == tmp)
                  tmp2 = j;
            if(tmp2 >=0)
               secret[i] =  alphabet_secret[tmp2];
            else
               secret[i] = chaine[i];
        }
        return secret;
    };

le problème que je rencontre se situe au niveau des fins de chaines, il m'ajoute des caractères bizares, par exemple si je veut crypter la chaine "test", je devrait avoir "paop", mais j'obtiens "paop¸"

09/10/2006, 15h08
Médinoc

Tu as oublié le zéro terminal en allouant ta chaîne cryptée (il faut strlen() +1).

PS: On n'utilise plus trop des char* en C++ : On passe généralement par des strings...
09/10/2006, 15h19
didier17062006

non, ça ne change rien, les caratères que je vois ajoutés en fin de chaînes sont assez alléatoires sont différents même si je crypte deux fois de suite la même chaine...
09/10/2006, 15h22
Médinoc

Tu penses à l'ajouter en fin de chaîne, au moins, le zéro terminal?
09/10/2006, 15h27
didier17062006

le texte à crypter est récupéré d'une zone de saisie, je suppose que le zéro terminal y est, j'ai testé en le concaténant dans le programme, j'obtiens toujours le même résultat
09/10/2006, 16h32
jobherzt
on ne parle pas de la chaine que tu as en entrée, mais de la chaine que tu as en sortie, ajoute genre :
Code:

1 2 char* secret = new char[strlen(chaine)]+1;
et apres ta boucle :
Code:

1 2 3 4 } secret[strlen(chaine)]='\0'; return secret;
et en effet, l'usage des string est plus que recommandé :-)

PS : j'espere que cette fonction de cryptage est réalisé a titre d'exercice et pas pour proteger des données reellement confidentielle..
09/10/2006, 16h39
didier17062006

Oui oui, c'est rien qu'un exercice, pour crypter des données je pense qu'il faut passer par le code binaire.
Le cryptage ici est bidon, c'est juste pour tester, je reprends le c++ après 2 années de non-pratique, et y'a pas à dire, c'est un sacré bon langage.

Je comprends mieux maintenant, ça fonctionne, je vais la tenter en utilisant les strings

Merci des conseils
09/10/2006, 16h44
jobherzt

De rien

Citation:

Envoyé par didier17062006

pour crypter des données je pense qu'il faut passer par le code binaire.

en fait il faut surtout passer par une fonction de cryptage eprouvée, et avec une clé !! si je t'ai posé cette question, c'est que beaucoup de gens sont assez "naifs" en ce qui concerne le cryptage, et pense que leur fonction maison "meme si elle est pas solide repoussera quand meme la plupart des petits curieux".. donc j'ai toujours le reflexe de demander quand je vois un algo de ce genre, s'il est destiné a etre utilisé en pratique, et j'essaie de repandre la "bonne parole"..
09/10/2006, 16h52
didier17062006

héhé, merci du conseil,
j'avais déjà réalisé un cryptage RSA en C pour un tchat, ça fonctionnait bien