Discussion :

[chroufa87]

bonjour
j'ai un probleme en ce programme.le probleme est que l'operation de cryptage n'est pas realiser.et voila mon code

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
 #include<stdio.h>
#include<string.h>
#include<math.h>
#include<ctype.h>
void main()
{
 
	char *alphabet="abcdefghijklmnopqrstuvwxyz";
	char texte[200];//texte à coder
	char *ptexte=texte;
	char clef[20];//la clé
	char *pclef=clef;
	char crypte[200];
	char *p=crypte;//le texte crypte
	int lgtexte;//longueure du texte à crypter
	int lgclef;//longueure de la clef
	//int nbocc,rbocc;//nombre d'occurence de clef
	/* cryptage */
	puts("+++taper le texte à coder+++  \n");
	gets(texte);
	lgtexte=strlen(texte);
	printf("+++taper la clef+++ \n");
	gets(clef);
	lgclef=strlen(clef);
	//rbocc=lgtexte%lgclef;
	//nbocc=(lgtexte-rbocc)/lgclef;
 
	int i=0;
	do
	{
		int j=0;
 
		do
		{
			if(isalpha(*(ptexte+i)!=0))
			{			
				int k=0;
				*(p+i)==(*(ptexte+i)+*(pclef+j)-2*'a')%26+'a';
				//printf("%d",k);
				//*ptexte=alphabet[k];
				i++;
				j++;
			}
			else
				i++;
 
		}
		while(*(pclef+j)<lgclef);
 
	}
	while(*(ptexte+i)<lgtexte);
	puts("/*message crypte*/");
	 puts(crypte);
}

[zehle]

Code :

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

1
2
 
*(p+i)==(*(ptexte+i)+*(pclef+j)-2*'a')%26+'a';

t'es sure que c'est un "==" et non pas un simple "="???!!

PS : Pense aux balises [ CODE ]

[crashtib]

salut,

déjà, évite les déclarations en plein milieu de code (i et j), surtout dans une boucle.

ensuite, je ne sais pas si les parenthèses de

if(isalpha(*(ptexte+i)!=0))

sont bien agencées. Je ne sais pas ce que fais isalpha mais je pense que ça renvoie un int. dans ce cas-là c'est plutôt

if(isalpha(*(ptexte+i))!=0)

ou encore mieux

if(isalpha(*(ptexte+i)))


D'autre part,

je ne sais pas si tes boucles font exactement ce que tu voudrais. Je n'ai pas envie de tester ni de me mettre à fond dedans (désolé ) mais si j'étais toi je vérifierait le comportement de mon code en faisant des printf("\n valeur de i : %d valeur de j : %d valeur de texte %s", i, j, texte); fin tu vois le principe.

et le message de zehle est pertinent à mon sens.

derniere chose, lgtexte et lgclef sont inutiles. strlen est fait pour ça.

bon courage

[Invité(e)]

Bonjour,
Il y a aussi un problème sur les tests de fin de boucle while :

Code :

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

1
2
3
4
        }
    while(*(pclef+j)<lgclef);
    }
while(*(ptexte+i)<lgtexte);

Code :

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

1
2
3
4
        }
    while(j<lgclef);
    }
while(i<lgtexte);

En tenant compte des deux messages précédant, on peut écrire :

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
#include<stdio.h>
#include<string.h>
#include<math.h>
#include<ctype.h>
int main(void)
{
 
    char *alphabet = "abcdefghijklmnopqrstuvwxyz";
    char texte[200];            //texte à coder
    char *ptexte = texte;
    char clef[20];              //la clé
    char *pclef = clef;
    char crypte[200];
    char *p = crypte;           //le texte crypte
    int lgtexte;                //longueure du texte à crypter
    int lgclef;                 //longueure de la clef
/* cryptage */
    puts("+++taper le texte à coder+++ \n");
    gets(texte);
    lgtexte = strlen(texte);
    printf("+++taper la clef+++ \n");
    gets(clef);
    lgclef = strlen(clef);
 
    int i = 0;
    do {
        int j = 0;
 
        do {
            /* attention aux parenthèses */
            if (isalpha(*(ptexte + i))!= 0) {
                /* = à la place de == */
                *(p + i) =
                    (*(ptexte + i) + *(pclef + j) - 2 * 'a') % 26 + 'a';
                i++;
                j++;
            } else {
                i++;
            }
        }
        /* attention aux condition de sortie */
        while (j < lgclef);
    }
    while (i < lgtexte);
    puts("/*message crypte*/");
    puts(crypte);
}

donne

Code :

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

1
2
3
4
5
6
7
+++taper le texte à coder+++
 
hello
+++taper la clef+++
foobar
/*message crypte*/
mszmo