Discussion :

[Clover]

J'aimerais faire une programmation modulaire du ROT 13. J'ai code mais ça ne compile pas j'aimerais connaître l'erreur.
Voici mon main .c

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "essai.h"
 
int main()
{
 
char text[5], crypter_txt[5],decrypter_txt[5];
 
printf("Enter le texte que l�on veut coder : \n");
scanf("%s",&text);
 
strcpy(crypter_txt, crypte(text));
printf("\n Le texte code est : \n");
scanf("%s",crypter_txt);
 
strcpy(decrypter_txt, decrypte(crypter_txt));
printf("\n\n Le texte decode est : \n");
scanf("%s",decrypter_txt);
 
    return 0;
}

Voici mon code essai.c

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "essai.h"
 
 
char* crypter(char crypter_txt[]){
int i,n;
char ch;
for(i=0;i<strlen(crypter_txt);i++){
    n=crypter_txt[i];
    n=n+13;
    if(n>90){
        n=n-90+64;
        ch=n;
        crypter_txt[i]=ch;
 
    }else{
    ch=n;
    crypter_txt[i]=ch;
    }
 
}
return crypter_txt;
}
 
char* decrypter(char decrypter_txt[]){
int i,n;
char ch;
for(i=0;i<strlen(decrypter_txt);i++){
    n=decrypter_txt[i];
    n=n-13;
    if(n<65){
        n=91-(65-n);
        ch=n;
        decrypter_txt[i]=ch;
 
    }else{
    ch=n;
    decrypter_txt[i]=ch;
    }
 
}
return decrypter_txt;
}

Et voici mon essai.h

Code :

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

1
2
3
4
5
6
7
8
#ifndef ESSAI_H
 
#define ESSAI_H
 
 
char* decrypte(char decrypter_txt[]);
char* crypte(char crypter_txt[]);
#endif // ESSAI_H

[Xelland]

déjà quels sont les messages d'erreur retournés par ton compilateur ?
normalement, il te donne des pistes

dans ton main, tu appelles crypte et decrypte , elles n'existent pas (erreur de frappe/copier coller ?)

à quoi servent les 2 derniers scanf ?

ensuite il y a un problème dans la fonction decrypter (elle compile, mais elle ne fait pas ce que tu veux)


pour des questions de vitesse :
tu appelles strlen dans le for
=> si tu ne modifies pas la taille de la chaine dans la bouche, tu peux sortir le strlen de ta boucle for
tu évalues strlen avant le for et dans le for tu testes avec le résultat
car à chaque boucle le strlen est évalué, donc perte de performance

[Clover]

Je viens de constater que j'avais fait une erreur, merci pour la remarque , j'ai enlevé les scanf , ajouter mes fonctions dans le essai.h et mtn ça compile mais ça ne differenci pas les caractères majuscules et minuscules lors du codage.

[Sve@r]

Bonjour
ligne 12: scanf("%s", &text).
ligne 16: scanf("%s", crypter_txt).
Deux instructions similaires qui ne sont pas écrites de la même façon => au-moins l'une des deux est donc mal écrite

ligne 18: strcpy(decrypter_txt, decrypte(crypter_txt)).
ligne 20: scanf("%s", decrypter_txt).
Tu remplis deux fois la même variable sans la lire entre temps => au-moins l'une des deux instructions est inutile

ligne 10: for(i=0;i<strlen(crypter_txt);i++).
Pas de strlen() dans une boucle (la fonction est appelée à chaque tour) => for(i=0; crypter_txt[i] != '\0'; i++) (et si tu pouvais passer par un pointeur ce n'en serait que mieux)

ligne 19:

Code c :

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

1
2
ch=n;
crypter_txt[i]=ch;

Mouais, pourquoi pas. Mais moi j'aurais plutôt écrit

Code c :

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

1
2
3
4
5
6
7
ch=n;
x=ch;
y=x;
z=y;
ppp=z;
toto=ppp;
crypter_txt[i]=toto;

Tant qu'à écrire des instructions inutiles, autant y aller à fond

mais ça ne differenci pas les caractères majuscules et minuscules lors du codage.

Et comment tu programmes cette différenciation ?

PS: pas de nombre magique (65, 97, etc). Utilise ce qu'ils représentent ('A', 'a', etc)