(re)codage de la fonction printf()

codage de la fonction printf() (re) - C - Programmation

Marsh Posté le 23-01-2012 à 14:08:27    

Bonjour à tous !

 

Dans un but didactique, j'essaie de recoder la fonction printf().
Lors de l’exécution de mon programme (cf ci dessous) j'ai l'erreur suivante : "Bad Format".

 

Le programme s'organise comme suit :

 

myprintf.c < myprintf.h
main.c < main.h

 

Ci-dessous les sources :

 

myprintf.h :

 
Code :
  1. #ifndef __MYPRINTF_H__
  2. #define __MYPRINTF_H__
  4. #include <stdio.h>
  5. #include <unistd.h> //pour write()
  6. #include <stdarg.h>
  7. #include <stdlib.h>
  9. int myprintf(const char* format, ...);
  10. int myStrlen(const char* s);
  11. void myWrite(int fildes, const void *buf, size_t nbytes);
  13. #endif


myprintf.c

 
Code :
  1. #include "myprintf.h"
  3. int myprintf(const char* format, ...)
  4. {
  5.     int i=0;
  6.     //char* s=NULL;
  7.     va_list ap;
  8.     va_start(ap, format);
  9.    
  10.     while(format[i] != 0)
  11.     {
  12.         switch(format[i])
  13.         {
  14.             case '%':
  15.                 switch(format[++i])
  16.                 {
  17.                     /*case 's':
  18.                         myWrite(1, s=va_arg(ap, char*), (size_t)myStrlen(s));
  19.                         break;*/
  20.                     case 'd':
  21.                         myWrite(1, va_arg(ap, int), sizeof(int));
  22.                         break;
  23.                     case 'c':
  24.                         myWrite(1, va_arg(ap, char), sizeof(char));
  25.                         break;
  26.                     default:
  27.                         myWrite(1, "%", sizeof(char));
  28.                         myWrite(1, &format[i], sizeof(char));
  29.                 }
  30.                 break;
  31.             default :
  32.                 myWrite(1, &format[i], sizeof(char));
  33.         }
  34.         i++;
  35.     }
  36.    
  37.     va_end(ap);
  38.     return 0;
  39. }
  41. int myStrlen(const char* s)
  42. {
  43.     int i=0;
  44.     while(s[i] != 0)
  45.         i++;
  46.     return i;
  47. }
  49. /* mywrite() :
  50. * Fait exactement comme write, mais en s'assurant
  51. * que l'écriture est réussie.
  52. */
  53. void myWrite(int fildes, const void *buf, size_t nbytes)
  54. {
  55.     if(buf == NULL)
  56.         exit(1);
  57.    
  58.     int k;
  59.     int n = (int)nbytes;
  60.     int i = 0;
  61.    
  62.     while(i < n && (k=write(fildes, buf, nbytes)) != -1)
  63.     {
  64.         i = i+k;
  65.     }
  66.     if(k == -1)
  67.     {
  68.         perror("write";);
  69.         printf("i=%d, k=%d, n=%d, filedes=%dn", i, k, n, fildes);     //Bon là, je sais que je triche :P
  70.         exit(1);
  71.     }   
  72. }


main.h :

 
Code :
  1. #ifndef __MAIN_H__
  2. #define __MAIN_H__
  4. #include <stdio.h>
  5. #include <stdlib.h>
  7. #endif


main.c :

 
Code :
  1. #include "main.h"
  2. #include "myprintf.h"
  4. int main(int argc, char* argv[])
  5. {
  6.     int a=1;
  7.     myprintf("test %d\n", a);
  8.     return 0;
  9. }


Une exécution donne :

 

--------------------------------------------------
$ ./myprintf
test write: Bad address
i=0, k=-1, n=4, filedes=1
--------------------------------------------------

  

Je ne suis pas du tout familier avec les fonctions va_*() c'est pour ça que je fais des essais avec ce myprintf().

 

Auriez-vous une idée de l'origine de l'erreur ?
En vous remerciant d'avance.


Message édité par cybkiller le 23-01-2012 à 14:12:57
Reply

Marsh Posté le 23-01-2012 à 14:08:27   

Reply

Marsh Posté le 23-01-2012 à 20:15:16    

Les paramètres que tu passes à myWrite ne sont pas bons, ton compilateur aurait du t'afficher un avertissement.
 
va_arg() te renvoie une valeur et non une adresse, du coup tu fais un déréférencement invalide dans myWrite().
 
Qui plus est, va_arg(ap, char) est une erreur à cause de la promotion implicite des entiers de taille inférieur à "int". Pour récupérer un entier de type char, il faut écrire: va_arg(ap, int)
 
Edit: cette ligne, bien que commentée, est aussi foireuse: myWrite(1, s=va_arg(ap, char*), (size_t)myStrlen(s));
L'ordre des arguments passé à une fonction n'est pas normalisé, dont dépend du compilateur. Tu n'as aucune garantie qu'au moment d'appeler "myStrlen(s)", s à été initialisé par l'affectation juste avant.


Message édité par tpierron le 23-01-2012 à 20:17:58
Reply

Sujets relatifs:

Leave a Replay

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