probleme implémentation code crypto chiffre de césar

Version imprimable

Bonsoir à tous ,

j'essaie de mettre un code crytpo méthode césar
Code:

1 2 3 4 5 6 7 8 9 10 11 12 void decalage (char *chaine, int cle){ if (chaine){ int i,j; const char alphabet[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j','k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't','u', 'v', 'w', 'x', 'y', 'z'}; for (i=0;i<strlen(chaine);i++){ for(j=0; j<strlen(alphabet);j++) if(chaine[i]==alphabet[j]) chaine[i] = alphabet[(j+cle)%26]; } } else {exit(1);} }
quand je l'applique sur mon texte par exemple en utilisant 2 comme clé ça m'affiche un texte complets avec que des a et b , si je choisi 3 j'aurai un texte avec a et b et c seulement , alors que ce n'est pas à ce que je m'attends .
pourriez vous m'indiquer qu'est ce qui cloche SVP , merci :)

salut,

basiquement il te manque un break sur ta boucle j, en effet si tu essayes de chiffrer disons la lettre 'h' avec la clé 13, celle ci va devenir un 'u', mais comme la boucle continue de tourner sur l'alphabet, quand on va rencontrer le 'u' on va à nouveau le chiffrer et ça va redonner un 'h'

Code:

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
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
char * decalage (char *chaine, int cle) {
    if (!chaine) return NULL;
 
    int i,j;
    const char alphabet[] = "abcdefghijklmnopqrstuvwxyz";
 
    for (i = 0; i < strlen(chaine); i++) {
        for(j = 0; j < strlen(alphabet); j++) {
            if(chaine[i]==alphabet[j]) {
                chaine[i] = alphabet[(j+cle)%strlen(alphabet)];
                break;
            }
        }
    }
    return chaine;
}
 
int main (void) {
    char chaine[] = "hello world !";
    printf ("%s\n", chaine);
    printf ("%s\n", decalage (chaine, 13));
    printf ("%s\n", decalage (chaine, 13));
    return 0;
}

Citation:

Envoyé par hakim lyonnais

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void decalage (char *chaine, int cle){
		if (chaine){
		int i,j;
				const char alphabet[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j','k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't','u', 'v', 'w', 'x', 'y', 'z'};
				for (i=0;i<strlen(chaine);i++){
					for(j=0; j<strlen(alphabet);j++)
						if(chaine[i]==alphabet[j]) chaine[i] = alphabet[(j+cle)%26];
 
																} 
			}
			else {exit(1);}
	}

Bonjour
Tu n'as pas le droit d'appeler strlen() pour "alphabet" car ce n'est pas un chaine (il n'y a pas de '\0' qui termine ce tableau). Et on ne sort pas d'une fonction par exit().

Accessoirement vaut mieux essayer de ne pas utiliser strlen() de façon aussi systématique. N'oublies pas qu'il s'agit d'une fonction qui effectue un travail de recherche. Si tu l'inclus comme condition de fin du for, elle est appelée à chaque tour de boucle...

Code:

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
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
char * decalage (char *chaine, int cle) {
    if (!chaine) return NULL;
 
    int i,j;
    const char alphabet[] = "abcdefghijklmnopqrstuvwxyz";
    size_t sz_alpha;
 
    sz_alpha=strlen(alphabet);
    for (i = 0; chaine[i] != '\0'; i++) {
        for(j = 0; alphabet[j] != '\0'; j++) {
            if(chaine[i]==alphabet[j]) {
                chaine[i] = alphabet[(j+cle)%sz_alpha];
                break;
            }
        }
    }
    return chaine;
}
 
int main (void) {
    char chaine[] = "hello world !";
    printf ("%s\n", chaine);
    printf ("%s\n", decalage (chaine, 13));
    printf ("%s\n", decalage (chaine, 13));
    return 0;
}

Et tu peux en plus éviter ces appels répétés à [i] en passant par des pointeurs de chaines, ça optimise encore un peu plus...

14/04/2015, 11h22
hakim lyonnais

Bonjour , merci a vous de m'avoir aidé :)

excusez moi du dérangement , mais en faite quand j’essaie de faire le décodage maintenant celà ne marche pas bien , à partir d'un certain chiffre de clé ça déchiffre pas

voici le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
	char* decalagedecode (char *chaine, int cle){
		if (!chaine) return NULL;
 	    int i,j;
    	const char alphabet[] = "abcdefghijklmnopqrstuvwxyz";
    	size_t sz_alpha;
    	sz_alpha=strlen(alphabet);
    	for (i = 0; chaine[i] != '\0'; i++) {
        	for(j = 0; alphabet[j] != '\0'; j++) {
            	if(chaine[i]==alphabet[j]) {
            		chaine[i] = alphabet[(j-cle)%sz_alpha];
                	break;
           		 }
        	}
   		 }
    	return chaine;
	}

pourriez vous m'indiquer à nouveau où est ce qu'il se situe le problème ?

16/04/2015, 17h04
Sve@r

Citation:

Envoyé par hakim lyonnais

Code:

chaine[i] = alphabet[(j-cle)%sz_alpha];

pourriez vous m'indiquer à nouveau où est ce qu'il se situe le problème ?

Pfff... Ce n'est pas parce que tu as commencé des études d'informatique qu'il faut arrêter de réfléchir. Tu t'es demandé un peu ce que pourrait bien donner "j-cle" quand "j" vaut 2 et que "cle" vaut 5 ???!!!??? :moinsser:

PS: c'est quand-même malheureux: on te donne une fonction super bien écrite, bien indentée proprement, agréable à lire et t'es même pas capable de la recopier avec une indentation correcte !!! Ben tu vois, la façon d'écrire représente souvent la façon de travailler...