Correction d'un code (exercice)

Correction d'un code (exercice) - C - Programmation

Marsh Posté le 15-06-2009 à 17:12:27    

Bonjour bonjour,
 
C'est encore moi avec des exercices ^^.
Celui ci il faut juste corriger un code où il y a intentionnellement des erreurs.
 
Je pense avoir corrigé la plupart, mais il me reste deux warnings là ou j'ai marqué "erreurs à corriger"
Je voudrais juste votre avis.
 
Voilà le code de départ:
 

Code :
  1. #include <conio.h>
  2. #include <stdio.h>
  4. char codage (char lettre[], int i,int nb)
  5. {
  6.      char result;
  7.      int signe=1;
  8.      if (i%2==0) signe=-1;
  9.      i=(i/2)+1;
  10.      result=lettre+signe*i*nb;
  11.      return(result);
  12. }
  14. void main (void);
  16. {
  17.      char mot[30], code[30];
  18.      int nombre, erreur, i, chks;
  19.      do
  20.      {
  21.          printf ("Bonjour, veuillez saisir le nombre correspondant à votre choix :\n 1\t pour coder un mot\n 2\tpour decoder un mot\n 3\t pour voir le nombre correspondant à un mot\n" );
  22.          scanf ("%f", &nombre);
  23.          erreur = 0;
  24.          switch(nombre)
  25.          {
  26.                        case 1 : printf("Veuillez saisir votre mot\n" );
  27.                        scanf("%s",&mot);
  28.                        chks=0;
  29.                        for (i=0; i<strlen(mot); i++)
  30.                        {
  31.                            code[i]=codage(mot[i],i,1);
  32.                            chks+=mot[i]-65;}
  33.                            code[strlen(mot)]=0;
  34.                            printf ("Le mot %i se code en %i.\nSon code de validation est : %i\n",mot,code,chks);
  35.                        case 2 : printf("Veuillez saisir votre code\n" );
  36.                            scanf("%s",&code);
  37.                            chks=0;
  38.                            for (i=0; i<strlen(code); i++);
  39.                            {
  40.                                mot[i]=codage(code[i],i,-1);
  41.                                chks+=mot[i]-65;
  42.                                }
  43.                            mot[strlen(code)]=0;
  44.                            printf ("Le code %s se decode en %s.\nSon code de validation est : %i\n",code,mot,chks);
  45.                        case 3 : printf("Veuillez saisir votre mot\n" );
  46.                            scanf("%s",mot);
  47.                            chks=0;
  48.                            for (i=0; i<strlen(mot); i++)
  49.                            {
  50.                                chks+=mot[i]-65;}
  51.                                printf ("Son code de validation est : %i\n",chks);
  52.                            default : erreur=1;
  53.                                printf("Vous avez commis une erreur lors de la saisie.\n Pressez une touche pour recommencer\n" );
  54.                                getch();
  55.                                }
  56.          } while (erreur=1)
  57.                            printf("Le programme est maintenant fini. Appuyez sur une touche\n" );
  59.                            getch();
  60. }


 
Ma correction:

Code :
  1. #include <conio.h>
  2. #include <stdio.h>
  3. /*Erreur corrigée ci-dessous */
  4. #include <stdlib.h>
  6. char codage (char lettre[], int i,int nb)
  7. {
  8.      char result;
  9.      int signe=1;
  10.      if (i%2==0) signe=-1;
  11.      i=(i/2)+1;
  12.      /*Erreur à corrigée  */
  13.      result=lettre+signe*i*nb;
  14.      return(result);
  15. }
  16. /*Erreur corrigée ci-dessous */
  17. /*Erreur corrigée ci-dessous */
  18. int main (void)
  19. {
  20.         char mot[30], code[30];
  21.         int nombre, erreur, i, chks;
  22.         do
  23.         {
  24.             printf ("Bonjour, veuillez saisir le nombre correspondant à votre choix :\n 1\t pour coder un mot\n 2\t pour decoder un mot\n 3\t pour voir le nombre correspondant à un mot\n" );
  25.             /*Erreur corrigée ci-dessous */
  26.             scanf ("%i", &nombre);
  27.              /*Erreur corrigée ci-dessous */
  28.             erreur == 0;
  29.             switch(nombre)
  30.             {
  31.                           case 1 : printf("Veuillez saisir votre mot\n" );
  32.                           scanf("%s",&mot);
  33.                           chks=0;
  34.                           for (i=0; i<strlen(mot); i++)
  35.                           {
  36.                               /*Erreur à corrigée  */
  37.                               code[i]=codage(mot[i],i,1);
  38.                               chks+=mot[i]-65;
  39.                           }
  40.                           code[strlen(mot)]=0;
  41.                           /*Erreur corrigée ci-dessous */
  42.                           printf ("Le mot %i se code en %i.\nSon code de validation est : %i\n",mot,code,chks);break;
  43.                           case 2 : printf("Veuillez saisir votre code\n" );
  44.                           scanf("%s",&code);
  45.                           chks=0;
  46.                           /*Erreur corrigée ci-dessous */
  47.                           for (i=0; i<strlen(code); i++)
  48.                           {
  49.                               /*Erreur à corrigée  */
  50.                               mot[i]=codage(code[i],i,-1);
  51.                               chks+=mot[i]-65;
  52.                           }
  53.                           mot[strlen(code)]=0;
  54.                           /*Erreur corrigée ci-dessous */
  55.                           printf ("Le code %s se decode en %s.\nSon code de validation est : %i\n",code,mot,chks);break;
  56.                           case 3 : printf("Veuillez saisir votre mot\n" );
  57.                           /*Erreur corrigée ci-dessous */
  58.                           scanf("%s",&mot);
  59.                           chks=0;
  60.                           for (i=0; i<strlen(mot); i++)
  61.                           {
  62.                               chks+=mot[i]-65;
  63.                           }
  64.                           /*Erreur corrigée ci-dessous */
  65.                           printf ("Son code de validation est : %i\n",chks);break;
  66.                           default : erreur = 1;
  67.                           printf("Vous avez commis une erreur lors de la saisie.\n Pressez une touche pour recommencer\n" );
  68.                           /*Erreur corrigée ci-dessous */
  69.                           getch();break;
  70.             }
  71.              /*Erreur corrigée ci-dessous */
  72.         } while (erreur == 1);
  73. printf("Le programme est maintenant fini. Appuyez sur une touche\n" );
  75. getch();
  76. }


 
Merci beaucoup!

Reply

Marsh Posté le 15-06-2009 à 17:12:27   

Reply

Marsh Posté le 15-06-2009 à 17:24:18    

(et les warnings, on doit les deviner ?)
Ouh putain, la ligne 13 [:ciler]
Et si jamais ton utilisateur saisit des mots de plus de 30 caractères, tu penseras à lui dire de redémarrer son PC ensuite, pour anticiper le plantage qui ne manquera pas d'arriver

Message cité 2 fois

---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 15-06-2009 à 17:31:46    

Harkonnen a écrit :

(et les warnings, on doit les deviner ?)
Ouh putain, la ligne 13 [:ciler]
Et si jamais ton utilisateur saisit des mots de plus de 30 caractères, tu penseras à lui dire de redémarrer son PC ensuite, pour anticiper le plantage qui ne manquera pas d'arriver


 
Nan mais c'est terminé depuis presque 10 ans Win9x et ses écrans bleus sur un scanf(), hein. [:dawa]

Message cité 1 fois
Reply

Marsh Posté le 15-06-2009 à 17:35:45    

Elmoricq a écrit :


 
Nan mais c'est terminé depuis presque 10 ans Win9x et ses écrans bleus sur un scanf(), hein. [:dawa]


Ca correspond justement à la période où j'ai du écrire ma dernière ligne de C [:dawa]


---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 15-06-2009 à 18:03:12    

Harkonnen a écrit :

(et les warnings, on doit les deviner ?)


 
Oups !
 
Les voici:
13 C:\Documents and Settings\Administrator\Desktop\correction.c [Warning] assignment makes integer from pointer without a cast  
37 C:\Documents and Settings\Administrator\Desktop\correction.c [Warning] passing arg 1 of `codage' makes pointer from integer without a cast  
50 C:\Documents and Settings\Administrator\Desktop\correction.c [Warning] passing arg 1 of `codage' makes pointer from integer without a cast  

Reply

Marsh Posté le 15-06-2009 à 18:20:37    

Il ne faut pas mettre un & ou un * qq part à la ligne 13?

Reply

Marsh Posté le 15-06-2009 à 23:21:11    

Ligne 13 : faut que tu castes tes int en char
result=lettre+(char)signe*(char)i*(char)nb;

Reply

Marsh Posté le 15-06-2009 à 23:23:04    

Et je pense que pour les 2 autres erreurs, le probleme vient du prototype de la fonction codage, lettre est juste un char, pas un tableau de char
 
char codage (char lettre[], int i,int nb) --> char codage (char lettre, int i,int nb)

Reply

Marsh Posté le 15-06-2009 à 23:46:22    

non tout faux, lire le code ne suffit pas, faut le comprendre aussi
et arreter d epenser qu'un warning de cast se resout en castant comme un porc
l13 :

 

result = lettre[signe*i*nb];

 

Message cité 2 fois
Message édité par Joel F le 15-06-2009 à 23:46:53
Reply

Marsh Posté le 16-06-2009 à 07:43:24    

Joel F a écrit :

non tout faux, lire le code ne suffit pas, faut le comprendre aussi
et arreter d epenser qu'un warning de cast se resout en castant comme un porc
l13 :
 
result = lettre[signe*i*nb];
 


 
J'aurais tendance à être plutôt de l'avis de boblenain, pour moi il ne faut envoyer qu'un char en premier paramètre de la fonction "codage", pas un tableau de char (sinon je vois pas bien l'intérêt de passer l'intégralité du tableau à chaque appel de la fonction, i.e à chaque itération de la boucle for). Du coup, "lettre" est de type char et n'accepte pas "lettre[xxx]". Par contre, je ne comprends pas bien la méthode de codage, cette ligne 13 reste mystérieuse.
 
Autre (petite) erreur spotted : ligne 34, les "%i" pour formater les string "mot" et "code" (d'ailleurs, je suis toujours mal à l'aise avec l'emploi de %i pour les int, j'ai toujours utilisé le %d, je me demandais si tous les compilateurs acceptaient le %i, j'imagine que oui)

Message cité 1 fois

---------------
If you think it could look good, then I guess it should
Reply

Marsh Posté le 16-06-2009 à 07:43:24   

Reply

Marsh Posté le 16-06-2009 à 08:22:01    

Turkleton a écrit :


J'aurais tendance à être plutôt de l'avis de boblenain, pour moi il ne faut envoyer qu'un char en premier paramètre de la fonction "codage", pas un tableau de char (sinon je vois pas bien l'intérêt de passer l'intégralité du tableau à chaque appel de la fonction, i.e à chaque itération de la boucle for). Du coup, "lettre" est de type char et n'accepte pas "lettre[xxx]". Par contre, je ne comprends pas bien la méthode de codage, cette ligne 13 reste mystérieuse.


 
J'ai pas lu le bon bout [:prozac]
 
Ouais char lettre oui mais n'empeche que le transtypage foireux à totu va = niet

Message cité 1 fois
Reply

Marsh Posté le 16-06-2009 à 13:31:58    

Joel F a écrit :

non tout faux, lire le code ne suffit pas, faut le comprendre aussi


 

Joel F a écrit :


J'ai pas lu le bon bout [:prozac]


 :o  
 
Transtypage int <-> char, c'est pas du transtypage foireux à tout va amha. Sinon faut modifier bien plus de choses dans son code ..
 
 

Reply

Marsh Posté le 16-06-2009 à 13:42:48    

Dans l'optique que j'en avais (pointer<->integer) si. La ofc non :o


Message édité par Joel F le 16-06-2009 à 13:42:54
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed