Discussion :

[Invité]

les amis enfin j'ai pu réaliser mon programme mais il ne marche pas pouvez vous me trouver le problème ou me correcte le programme svp.
voila le 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
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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
#include <stdio.h>
#include <string.h>
 
void crypter ( char* phrase , int cle );
void decrypter ( char* phrase , int cle );
int caractereValide ( char caractere );
 
int main ()
{
    int cle = 0 , menu = 1;
    char phrase [225];
    printf("\t<<<<<<<<<<<<<<<<<<<<<<BIENVENUE A NOTRE PROGRAMME >>>>>>>>>>>>>>>>>>>>>>>>\n");
    printf("\n\n --Bienvenue dans le programme de cryptage - decryptage\n de phrases selon la methode Cesar\n\n");
    printf("1. Cryptage \n");
    printf("2. Decryptage \n");
    printf("3. EXIT\n");
    printf("Votre choix :  ");
 
    do
    {
        scanf("%d" , &menu);
 
        if (menu == 1)
        {
            printf ("\n\nVous avez choisi de crypter.\nVeuillez entrer       la phrase a crypter : ");
            scanf("%s" , phrase);
            printf("\nVeuillez choisir votre cle de cryptage : ");
            scanf("%d" , &cle );
            crypter ( phrase , cle );
            printf("\nVotre phrase cryptee est : %s" , phrase );
        }
        if(menu==3){
        	return 0;
		}
 
        else if (menu == 2)
        {
            printf ("\n\nVous avez choisi de decrypter.\nVeuillez entrer la phrase a decrypter : ");
            scanf("%s" , phrase);
            printf("\nVeuillez entrer votre cle de cryptage : ");
            scanf("%d" , &cle );
            decrypter ( phrase , cle );
            printf("\nVotre phrase decryptee est : %s" , phrase );
        }
 
        else
        {
            printf("\nVous n'avez pas entre un choix valide veuillez entrer un choix valide");
        }
    }while (menu != 1 && menu != 2);
 }
 
void crypter (char* phrase , int cle)
{
    int i ;
 
    for (i = 0; i<strlen(phrase);  i ++)
    {
        if (caractereValide(phrase [i]) == 1)
        {
            phrase [i] = (((phrase[i]-'a')+cle)%26)+'a';
        }
 
        else if (caractereValide (phrase[i]) == 2)
        {
            phrase [i] = (((phrase[i]-'A')+cle)%26)+'A';
        }
    }
}
 
void decrypter (char* phrase , int cle)
{
    int i ;
 
    for (i = 0; i<strlen(phrase);  i ++)
    {
        if (caractereValide(phrase [i]) == 1)
        {
            phrase [i] = (((phrase[i]-'a')-cle)%26)+'a';
        }
 
        else if (caractereValide (phrase[i] == 2))
        {
            phrase [i] = (((phrase[i]-'A')-cle)%26)+'A';
        }
    }
}
 
int caractereValide (char caractere)
{
    int etat = 0;
 
    if( caractere >= 'a' && caractere<= 'z')
    {
        etat == 1;// etat 1 = minuscule
    }
 
    else if ( caractere>= 'A' && caractere <= 'Z')
    {
        etat == 2;// etat 2 = Majuscule
    }
 
    return etat; //etat 0 = autres
}

et voici le projet

[sambia39]

Bonsoir,
Avant que l'on puisse vous aider pourriez vous faire un tout petit effort en indentant votre code source en vous servant des balises '#' (balise code) qui sont mise à votre disposition ?

À bientôt

[JujuPomme]

Hello,

Petit rajout : peut-être nous dire ce qui fonctionne pas? n_n

[droggo]

Bonjour,

Il faut oser le dire :

"J'ai pu réaliser mon programme ..."

en enchaînant avec "il ne marche pas".

Chapeau bas

[Invité]

Hello,

Petit rajout : peut-être nous dire ce qui fonctionne pas? n_n

merci mon ami
le problème est que le programme me demande de donner la phrase a crypter ou décrypter mais après il cèsse de fonctionneret la console s'échappe.

[Médinoc]

Pour commencer, ne vois-tu pas un truc bizarre aux lignes 64 et 82?

Je ne vois pas d'autre erreur flagrante, et surtout aucune qui soit censée pouvoir provoquer comportement indéfini, tant que la chaîne de caractères saisie ne dépasse pas 224 caractères.
Par contre, ton programme se termine immédiatement après avoir affiché son résultat; sous Windows, si tu l'as lancé en double-cliquant dessus (ou si tu l'as lancé dans le debugger de Visual Studio, etc.), la console se fermera aussitôt. Tu devrais le tester dans un Invite de Commandes à la place.

Edit: Je constate aussi que tu ne vides pas le buffer d'entrée après tes appels à scanf(). Les scanf("%d", ) laissent le \n dans le buffer d'entrée, perturbant la saisie de la phrase.
Tu ne vérifies pas non-plus le retour de scanf(), au passage.

[Invité]

merci mon ami
le problème est que le programme me demande de donner la phrase a crypter ou décrypter mais après il cèsse de fonctionneret la console s'échappe.

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
#include <stdio.h>
#include <string.h>
 
void crypter ( char* phrase , int cle );
 
int main ()
{
    int cle = 0 , menu = 1;
    char phrase [225];
    puts("<<<<<<<<<<<<<<<<<<<<<<BIENVENUE A NOTRE PROGRAMME >>>>>>>>>>>>>>>>>>>>>>>>");
    puts(" --Bienvenue dans le programme de cryptage - decryptage de phrases selon la methode de Cesar");
 
    while(menu!=3)
    {
        puts("*** MENU ***");
        puts("1. Cryptage ");
        puts("2. Decryptage");
        puts("3. EXIT");
        puts("Votre choix :");
        scanf("%d" , &menu);
 
		switch(menu)
		{
			case 1:
				printf ("\n\nVous avez choisi de crypter.\nVeuillez entrer       la phrase a crypter : ");
				scanf("%s" , phrase);
				printf("\nVeuillez choisir votre cle de cryptage : ");
				scanf("%d" , &cle );
				crypter ( phrase , cle );
				printf("\nVotre phrase cryptee est : %s\n\n" , phrase );
				break;
			case 2:
				printf ("\n\nVous avez choisi de decrypter.\nVeuillez entrer la phrase a decrypter : ");
				scanf("%s" , phrase);
				printf("\nVeuillez entrer votre cle de cryptage : ");
				scanf("%d" , &cle );
				crypter ( phrase , cle * -1 );
				printf("\nVotre phrase decryptee est : %s\n\n" , phrase );
				break;
			default:
				printf("\nVous n'avez pas entre un choix valide veuillez entrer un choix valide");
				break;
        }
    }
 }
 
void crypter (char* phrase , int cle)
{
    int i ;
	int c;
	char caractere;
 
    for (i = 0; i<strlen(phrase);  i ++)
    {
        	caractere=phrase[i];
	        c=0;
            if( (caractere >= 'a') && (caractere<= 'z') )
		        c='a';
            else
		        if ( (caractere>= 'A' && caractere <= 'Z') )
			        c='A';
            phrase [i] = (((caractere-c)+cle)%26)+c;
    }
}

voici le code fonctionnel et simplifié! A noté que j'ai choisi d'utiliser la même fonction pour l'encryption et le décryptage! Pour transformer le + en -, je transforme la valeur de l'argument en nombre négatif avant l'appel.

J'ai tranformé la boucle while qui n'était pas claire. Enlever une méthode inutile.

[sambia39]

Bonsoir
Êtes-vous sûr que le programme soit fonctionnel ?.
Je suis bien curieux de savoir comment se comporte le programme si je saisissais un caractère au moment du choix proposer par le menu. Et en admettant que je me trompe non pas au menu, mais en deuxième partie, c'est-à-dire au moment où l'on doit fournir la clef de chiffrement (qui est censé être le nombre de décalages) et que je saisis un caractère à la place ; comment se comporterait le programme à ce moment-là ?
Et surtout avez-vous pris en compte les remarques de @Médinoc dans le correctifs de l'applications simplifié ?
À bientôt

[Invité]

Salutations!

Bonsoir
Êtes-vous sûr que le programme soit fonctionnel ?.

je l'ai testé, tu peux en faire autant:
https://www.onlinegdb.com/

Bonsoir
Et surtout avez-vous pris en compte les remarques de @Médinoc dans le correctifs de l'applications simplifié ?
À bientôt

pas besoin, le scanf insère un char \0 à la fin de la séquence lue, que tu vides ou non le buffer, la séquence tapée sera quand même null-terminated. Scanf ne vide pas le buffer, mais il insére quand même un NULL à la fin de la séquence lu.


Pour être plus rigoureux, il faudrait limiter ce qui peut être lu en ligne de commande et faire :

Code :

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

1
2
3
4
 
phrase[224]=0; //AU DEBUT, pas obliger de le mettre dans la boucle
 
scanf ("%224s",str);        //limite la taille lu et garde 1 octet supplémentaire pour le 0 de fin.

Bonsoir
Je suis bien curieux de savoir comment se comporte le programme si je saisissais un caractère au moment du choix proposer par le menu. Et en admettant que je me trompe non pas au menu, mais en deuxième partie, c'est-à-dire au moment où l'on doit fournir la clef de chiffrement (qui est censé être le nombre de décalages) et que je saisis un caractère à la place ; comment se comporterait le programme à ce moment-là ?

Pour ce qui est du reste, le programme original ne faisait pas de détection de ce qui était tapé, je n'ai rien ajouté en ce sens. Par contre, si tu tapes du texte dans un scanf avec %d, si tu entres du texte bidon au lieu d'un nombre, la chaine bidon sera convertie à 0.

Si on voudrait être propre, ce code devrait quand même avoir des détections de null/non-null au niveau de la fonction appelée. Le code aurait pu être encore optimisé, mais cela peut nuire à la lisibilité.

Tout marche mais en plus simple.

[sambia39]

D'accord autant pour moi je me suis donc trompé ?
je pensais qu'il était nécessaire de vider le buffer Éventuellement avec un while( '\n' != (ret = getchar()) && EOF != ret);. Et sur la variable phrase faire un memsetne serait pas de refus également.

Variable phrase sans initialisation préalable ( même phrase[224]=0)

Code GDB :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<<<<<<<<<<<<<<<<<<<<<<BIENVENUE A NOTRE PROGRAMME >>>>>>>>>>>>>>>>>>>>>>>>
 --Bienvenue dans le programme de cryptage - decryptage de phrases selon la methode de Cesar
*** MENU ***
1. Cryptage 
2. Decryptage
3. EXIT
Votre choix :
1

Vous avez choisi de crypter.
Veuillez entrer       la phrase a crypter : AAAAA
(gdb) p phrase
$1 = "AAAAA", '\000' <repeats 59 times>, "\320\000\376QT\216\067{`\370\277\357\376\177\000\000\030\367\277\357\376\177\000\000\001\000\000\000\000\000\000\000@\367\277\357\376\177\000\000\060\367\277\357\376\177\000\000\000\367\277\357\376\177\000\000\324C\000\000\001\000\000\000\030\367\277\357\376\177\000\000\000\000\000\000\001\000\000\000\060\367\277\357\376\177\000\000\030\367\277\357\376\177\000\000 5\000\000\001\000\000\000\000\000\000\000\001\000\000\000H2\000\000\001", '\000' <repeats 43 times>...
(gdb)

Variable phrase initialiser (memset)

Code GDB :

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
 
<<<<<<<<<<<<<<<<<<<<<<BIENVENUE A NOTRE PROGRAMME >>>>>>>>>>>>>>>>>>>>>>>>
 --Bienvenue dans le programme de cryptage - decryptage de phrases selon la methode de Cesar
*** MENU ***
1. Cryptage 
2. Decryptage
3. EXIT
Votre choix :
1
 
Vous avez choisi de crypter.
Veuillez entrer       la phrase a crypter : AAAAA
 
Veuillez choisir votre cle de cryptage : 1
(gdb) p phrase
$1 = "AAAAA", '\000' <repeats 219 times>
(gdb)

Mais bon tant que le programme fonctionne ......

[Invité]

Par convention, il n'est pas nécessaire de vider le buffer, car toutes les opérations de chaines (printf, strlen) voit une fin de ligne au premier NULL. Le danger serait de remplir le buffer au complet sans avoir de place pour y insérer un ZERO de terminaison. C'est pour ca que c'est toujours recommandé de faire des allocations de N+1 pour contenir le 0 de terminaison.
En mettant un zero à la fin du buffer lors de l'initialisation, ca permet de s'assurer de ne pas dépasser. il faut juste ensuite contrôler le scanf pour ne pas écraser ce zéro final. Scanf mettra lui même un zero après l'entrée de la chaine. Donc pas de soucis.

Par contre, ça dépend toujours du contexte. Idéalement même si cela n'affecte pas le déroulement du programme, c'est pas mauvais de mettre à zero (genre memset). Dans le cas, où tu sauverais le contenu des variables sur disque. Imagine un jeu qui te permet de charger/sauver des parties. Si tu ne met pas le buffer à zero, le programme pourrait sauvegarder des infos qui étaient stockés en mémoire par d'autres programme dans la partie de sauvegarde. Il y a possibilité de data leak!

Idéalement il serait une bonne pratique de vider le buffer lors de l'initialisation et avant la destruction (free), de cette façon, le prochain programme qui utilise cette zone ram, sera vide. La raison pourquoi ce n'est pas fait par défaut, c'est pour la rapidité.

a+

[chrtophe]

Idéalement, j'utiliserais fgets au lieu de scanf.

[sambia39]

Idéalement, j'utiliserais fgets au lieu de scanf.

Oui, je suis d'avis avec vous. scanf manque de souplesse il très difficile à utiliser.
Là où j'attire l'attention, c'est au comportement du programme en lui-même la partie non-vidage du buffer.
Certains appels de scanf échouent systématiquement, car ces appels attendent des séquences de chiffre pour être valide sauf si on ne joue pas le jeu et que l'on injecte des caractères ou des séquences de caractères, ils seront réinjectés dans la variable phrase donc ne pas vider le buffer me parais un peu délicat.