Discussion :

[didier17062006]

Bonjour à tous,


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

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
 
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¸"

[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...

[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...

[Médinoc]

Tu penses à l'ajouter en fin de chaîne, au moins, le zéro terminal?

[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

[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 :

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

1
2
 
char* secret = new char[strlen(chaine)]+1;

et apres ta boucle :

Code :

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

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..

[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

[jobherzt]

De rien

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"..

[didier17062006]

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