Discussion :

[clio671]

Bonjour,

Donc voila je developpe en ce moment l'algo de vigenére en C sous dev C++

Code :

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

1
2
3
4
5
6
7
8
9
10
char Vigenere(char chaine_A_cryptee, char cle_message,int taille_de_la_chaine)
{
       char *chaine_cryptee[256] ;       
       int i ;
       
       for (i = 0 ; i <= taille_de_la_chaine ; i++)
           chaine_cryptee[i] = chaine_A_cryptee[i] + cle_message[i] - 1 ; //pour le A
       
       return(chaine_cryptee) ;         
}

Bon mon algo est bon (j'suis sûr) mais j'ai une erreur à la compil ( ligne en gras ds mon code) :

subscripted value is neither array nor pointer

Je ne comprends pas trop mon erreur à priori j'ai bien déclaré mes tableaux
J'ai cherché sur le forum mais les erreurs ne s'appliquent pas trop à mon cas.

Je pourrais aussi tt coder dans le main et ça pourrait marcher mais je préfere que ça soit propre
avec des fonctions.

Si vous avez des conseils à me donner j'suis preneur !! bye

[diogene]

Code :

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

1
2
3
char Vigenere(char chaine_A_cryptee, char cle_message,int taille_de_la_chaine)
....
           chaine_cryptee[i] = chaine_A_cryptee[i] + cle_message[i] - 1 ; //pour le A

chaine_A_cryptee et cle_message ne sont pas des tableaux de char mais des char. peut être un erreur dans l'en-tête

Code :

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

char *Vigenere(char  *chaine_A_cryptee, char *cle_message,int taille_de_la_chaine)

[souviron34]

Code :

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

1
2
3
4
5
6
{
       char *chaine_cryptee[256] ;       

       for (i = 0 ; i <= taille_de_la_chaine ; i++)
         [B]  chaine_cryptee[i] = chaine_A_cryptee[i] + cle_message[i] - 1

et aussi la tu declares un tableau de pointeurs de char, et non un tableau de char ou un poiinteur...

Si tu veux dupliquer la chaine, tu dois faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
{
       char *chaine_cryptee = strdup(chaine_A_cryptee);       
 
       if ( chaine_cryptee != NULL )
         {
          for (i = 0 ; i <= taille_de_la_chaine ; i++)
            chaine_cryptee[i] = chaine_A_cryptee[i] + cle_message[i] -1;
         }
   return chaine_cryptee ;
}

ou si tu veux la modifier sur place, tu te sers juste d'un pointeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
{
       char *p = chaine_A_cryptee ;       
 
          for (i = 0 ; i < taille_de_la_chaine ; i++)
           {
            *p = *p + cle_message[i] -1;
             p = p + 1 ;
            }
}

et la ta fonction peut etre void, car tu retournes dans la meme chaine...

De plus, je te signale que tu as mis <= taille_de_la_chaine, alors que c'est < .