Discussion :

[velociraptor5679]

De quelle manière puis-je optimiser mon programme ? :

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
 
#include <iostream>
 
using namespace std;
 
int main(int argc, char *argv[])
{
    string chaine=argv[1],      cryptee="",     clef=argv[2];    
 
    if (argv[1]==NULL || argv[2]==NULL)
       cout << "Utilisation :" << endl << "CRYPT chaine clef" << endl;
    else
    {
        for (int i(0); i<chaine.length(); ++i)
            cryptee += chaine[i] ^ clef [ i % clef.length() ];
        cout << cryptee;
    }
 
    return 0;
}

[Médinoc]

Avant de l'optimiser, essaie d'abord de le corriger pour que

Code :

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

crypt Frederic 1code

Ne coupe pas la chaîne...

[velociraptor5679]

ben il marche

[Médinoc]

Pas avec le code que tu me montres: Si un caractère de la chaîne de départ est le même que celui correspondant de la clé, c'est un \0 que tu rajoutes...

Edit: Je viens de tester, alors j'ajoute:

1. Que tu as oublié le header <string>,
2. Que je la chaîne cryptée s'arrête effectivement au bout de trois caractères comme je l'avais dit,
3. Que c'est pas prudent d'afficher une chaîne pouvant contenir des caractères <32...
   En testant le programme avec les paramètres déjà mentionnés sur un serveur FreeBSD, le prog m'affiche des caractères de contrôle qui m'effacent l'écran...

[velociraptor5679]

Pas avec le code que tu me montres: Si un caractère de la chaîne de départ est le même que celui correspondant de la clé, c'est un \0 que tu rajoutes...

oui c'est normal

[Médinoc]

oui c'est normal

Ben non, c'est pas normal.
C'est ton algo de cryptage qui est foireux, c'est à corriger.
(je sais que c'est facile, j'ai déjà eu à faire cette correction-là).

[MatRem]

Voilà ce que j'aurais fait, (pour les test):

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
#include <iostream>
#include <cstdlib>
using namespace std;
 
int main(int argc, char *argv[])
{
    if(argc<3){
       cout << "Utilisation :" << endl << "CRYPT chaine clef" << endl;
       return EXIT_FAILURE;
    }
 
    string chaine=argv[1],      cryptee="",     clef=argv[2];    
 
    for (int i(0); i<chaine.length(); ++i)
        cryptee += chaine[i] ^ clef [ i % clef.length() ];
 
    cout << cryptee;
 
    return EXIT_SUCESS;
}

[velociraptor5679]

Voilà ce que j'aurais fait, (pour les test):

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
#include <iostream>
#include <cstdlib>
using namespace std;
 
int main(int argc, char *argv[])
{
    if(argc<3){
       cout << "Utilisation :" << endl << "CRYPT chaine clef" << endl;
       return EXIT_FAILURE;
    }
 
    string chaine=argv[1],      cryptee="",     clef=argv[2];    
 
    for (int i(0); i<chaine.length(); ++i)
        cryptee += chaine[i] ^ clef [ i % clef.length() ];
 
    cout << cryptee;
 
    return EXIT_SUCESS;
}

pas mal...