Discussion :

[monalisa_dulouvre]

salut a tous,
svp qq1 peut m'aider a trouver pour quoi ce programme ne marche pas ? je l'ai analyser plusieurs fois mais je trouve pas la faille
le programme je l'ai mis en pièce jointe
en faite ce programme est fait pour crypter des fichier avec l'algorithme IDEA
merci

[Lunixinclar]

Compile pas sur Linux GCC.

Code :

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

IDEA.C:9: error: le C++ ISO interdit la déclaration de « t1 » sans type

Il sort d'ou ce code, sans blagues??

[zooro]

Salut,

uint16 n'est pas défini.

[Thierry Chappuis]

Essaie de tester chacune de tes fonctions de façon isolée afin de localiser un peu mieux le problème et de pouvoir poster un code moins conséquent (c'est plus facile pour nous à debugger).

Par ailleurs, pourrais-tu être plus précis que "mon code ne marche pas"? Quelle messages d'erreurs obtiens-tu de ton compilateur? Erreurs à l'exécution? Tout information supplémentaire nous permettra de t'aider plus efficacement.

Thierry

[zooro]

Compile pas sur Linux GCC.

Code :

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

IDEA.C:9: error: le C++ ISO interdit la déclaration de « t1 » sans type

Il sort d'ou ce code, sans blagues??

Ceci dit, tu compiles du C avec un compilateur C++...
Elles sortent d'où tes options de compilation, sans blagues ??
http://emmanuel-delahaye.developpez....tm#cfg_compilo

[gege2061]

Elles sortent d'où tes options de compilation, sans blagues ??

du .C en majuscule

[zooro]

du .C en majuscule

Mouais...

[monalisa_dulouvre]

chez moi le programme s'excute sauf qu'il m'affiche le message :
votre clé ne doit pas dépasser 8 caractéres
alors que moi j'ai rien taper

[millie]

Compile pas sur Linux GCC.

Code :

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

IDEA.C:9: error: le C++ ISO interdit la déclaration de « t1 » sans type

Il sort d'ou ce code, sans blagues??

Ni sous windows :

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
\test\IDEA.C: In function `uint16 inv(uint16)':
\test\IDEA.C:4: error: `uint16 inv(uint16)' was declared `extern' and later `static'
\test\/idea.h:33: error: previous declaration of `uint16 inv(uint16)'
\test\IDEA.C: In function `void en_key_idea(unsigned int*, unsigned int*)':
\test\IDEA.C:29: error: `void en_key_idea(unsigned int*, unsigned int*)' was declared `extern' and later `static'
\test\/idea.h:30: error: previous declaration of `void en_key_idea(unsigned int*, unsigned int*)'
\test\IDEA.C: In function `void de_key_idea(unsigned int*, unsigned int*)':
\test\IDEA.C:44: error: `void de_key_idea(unsigned int*, unsigned int*)' was declared `extern' and later `static'
\test\/idea.h:31: error: previous declaration of `void de_key_idea(unsigned int*, unsigned int*)'
\test\IDEA.C: In function `void cipher_idea(unsigned int*, unsigned int*, unsigned int*)':
\test\IDEA.C:116: error: `void cipher_idea(unsigned int*, unsigned int*, unsigned int*)' was declared `extern' and later `static'
\test\/idea.h:32: error: previous declaration of `void cipher_idea(unsigned int*, unsigned int*, unsigned int*)'
\test\IDEA.C: In function `void write_char_to_file(char, FILE*)':
\test\IDEA.C:171: error: `exit' undeclared (first use this function)
\test\IDEA.C:171: error: (Each undeclared identifier is reported only once for each function it appears in.)
\test\IDEA.C: In function `void write_word16_to_file(unsigned int, FILE*)':
\test\IDEA.C:180: error: `exit' undeclared (first use this function)
\test\IDEA.C: In function `void swap_files_and_clean_up(char*)':
\test\IDEA.C:295: error: `exit' undeclared (first use this function)
\test\IDEA.C: In function `void getuserkeyfromargv(unsigned int*, char*)':
\test\IDEA.C:326: warning: comparison between signed and unsigned integer expressions
\test\IDEA.C: At global scope:
\test\IDEA.C:338: error: ISO C++ forbids declaration of `main' with no type
\test\IDEA.C: In function `int main(int, char**)':
\test\IDEA.C:353: error: `exit' undeclared (first use this function)

chez moi le programme s'excute sauf qu'il m'affiche le message :

C'est étonnant qu'il compile


Es-tu sûr de bien avoir implémenté ton algorithme ? Est-ce un pur problème algorithmique que tu as ou un problème vraiment lié au langage C ?

du .C en majuscule

Bah oui, ça s'appelle le langage C, pas le langage c

[monalisa_dulouvre]

voila le message qu'il m'affiche sur la console
[IMG]C:\Documents and Settings\perso\Bureau\ecran.jpg[/IMG]

[monalisa_dulouvre]

le probleme n'est pas un probleme d'algo , mais du C

[zooro]

voila le message qu'il m'affiche sur la console
[IMG]C:\Documents and Settings\perso\Bureau\ecran.jpg[/IMG]

Il semble que la clé doive être passée en paramètre lors de l'exécution...

[Emmanuel Delahaye]

voila le message qu'il m'affiche sur la console
[IMG]C:\Documents and Settings\perso\Bureau\ecran.jpg[/IMG]

Visiblement, il faut passer des choses en paramètre sur la ligne de commande...

C'est pas un problème de C, mais d'utilisation de ton application. Vu que tu as conçu le code, tu sais donc ce qu'il faut passer comme paramètres...

[monalisa_dulouvre]

justement c'est pas moi qui a conçu le programme (lire suite )

[zooro]

Bah oui, ça s'appelle le langage C, pas le langage c

Exactement !

[millie]

je l'ai analyser plusieurs fois mais je trouve pas la faille

Moi, je l'ai trouvé :

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
   t1=inv(*Z++);
   t2=-*Z++;
   t3=-*Z++;
   *--p=inv(*Z++);
   *--p=t3;
   *--p=t2;
   *--p=t1;
   for (j=1;j<ROUNDS;j++)
   {
      t1=*Z++;
      *--p=*Z++;
      *--p=t1;
      t1=inv(*Z++);
      t2=-*Z++;
      t3=-*Z++;
      *--p=inv(*Z++);
      *--p=t2;
      *--p=t3;
      *--p=t1;
   }
   t1=*Z++;
   *--p=*Z++;
   *--p=t1;
   t1=inv(*Z++);
   t2=-*Z++;
   t3=-*Z++;
   *--p=inv(*Z++);
   *--p=t3;
   *--p=t2;
   *--p=t1;

Ecrire des choses comme ça, c'est pas possible !!!!! Personne ne pourra t'aider en regardant un code comme ça. Si je voyais ça dans une vrai application, pouff poubelle et je recommence. C'est absolument non maintenable (déjà que la qualité du code est catastrophique, il manque plein de commentaires pour les autres) !

Voici les régles de bases en programmation :

• Lisibilité
• Maintenabilité
• Portabilité
• Réutilisabilité
• Extensibilité

[monalisa_dulouvre]

oui je sait
je programme mais en java, en C je ne connait pas grand chose, si ce n'ai qu'il ressemble a java ,
mais pour repondre a ta question ce programme aetais codé par mon binom qui est partie en vaccance et moi je doit compiler ce programme pour le soumettre au prof,
d'apres mon binome que je ne peut pas joindre, il ma dit qu'il faut passer le nom du fichier en paramettre avec la clé
sinon au niveau de l'lago c'est moi qu'il la ecrit il est correcte parcque je l'ai tester avec des valeurs passer en argument

[Thierry Chappuis]

oui je sait
je programme mais en java, en C je ne connait pas grand chose, si ce n'ai qu'il ressemble a java ,
mais pour repondre a ta question ce programme aetais codé par mon binom qui est partie en vaccance et moi je doit compiler ce programme pour le soumettre au prof,
d'apres mon binome que je ne peut pas joindre, il ma dit qu'il faut passer le nom du fichier en paramettre avec la clé
sinon au niveau de l'lago c'est moi qu'il la ecrit il est correcte parcque je l'ai tester avec des valeurs passer en argument

Alors ton binome t'as filé un code qui ne compile pas et pas du tout commenté. Documenter son code lorsqu'on travaille en équipe, c'est très important! Comme l'a fait remarquer millie, l'algorithme sous-jacent est sans doutes correct, mais le code est tellement illisible qu'on est pas beaucoup à avoir envie de s'y plonger, d'où ma proposition de tester chaque fonction de manière isolée afin de localiser le problème.

Quant aux similitudes entre le langage C et Java, cela se limite à la syntaxe des structures for, while, switch et if et des opérateurs d'auto-incrémentation. Pour le reste, ces langages n'ont pas grand chose en commun.

Thierry

[monalisa_dulouvre]

effectivement sur la similitudes entre le langage C et Java ta raison
mais moi, ce soir je vais finir le compte rendu du programme avec bien évidament les commentaires sur le programme je vais vous l'envoyer
mais en tous cas sur le programme le probleme se pose au niveau de la lecture de fichier qu'il ne trouve pas, mais moi je ne sais pas comment faire pour changer cette fonction afin qu'elle puisse lire un fichier txt (par exemple ) et convertir les lettres en code ASCII, pour appliquer l'algorithme de cryptage
si qq1 peut proposer un code pour la lecture d'un fichier !!

[Thierry Chappuis]

Donc voici les deux fonctions responsable de lecture dans un fichier:

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
char read_char_from_file(FILE *fp)
{
   char temp=0;
 
   if ((fread(&temp, sizeof(char), 1, fp)) != 1)
      end_of_file=1;
 
   return (temp);
 
}
 
 
 
word16 read_word16_from_file(FILE *fp)
{
   word16 temp=0;
 
   if ((fread(&temp, sizeof(word16), 1, fp)) != 1)
      end_of_file=1;
 
   return temp;
 
}

end_of_file en global, je comprends que tu n'arrives pas à debugger ce programme! C'est du code spaghetti et illisible. Je ne m'avancerais pas à évaluer ce code sur l'échelle du goret, mais tu peux te faire une idée par toi-même: http://emmanuel-delahaye.developpez.com/goret.htm

Si c'était moi, (et cela n'engage que moi) cela me prendrai certainement moins de temps et d'effort de repartir de l'algo de départ et de tout re-coder depuis zéro.

Thierry