Discussion :

[Maf77]

Bonjour à vous!

J'ai juste une erreur de write avec:

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
void    my_error()
{
  printf("%d %s", errno, strerror(errno));
}
 
int     my_strlen(char *str)
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
      i = i + 1;
  return (i);
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}
 
int     main()
{
  my_putstr("toto");
}

"toto" s'affiche bien.
Et je test le retour du write à 0 pour une raison, que j'expliquerais.
ensuite, j'ajoute le code à ma lib:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void    my_error()
{
  printf("%d %s\n", errno, strerror(errno));
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}

Et lorsque j'utilise ma librairie, il me dit que le retour du write dans celui-ci vaut 0!

Code :

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

my_putstr("Salut");

Ca segV!
J'ai comme retour de strerror:

Code :

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

22 Unknown error: 0

Je ne trouve malheureusement pas la raison...
Quelqu'un le sait?

[Emmanuel Delahaye]

J'ai juste une erreur de write avec:

Code EPITECH detecté...

Pas du C standard ni même correct...

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
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c:2: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `my_error':
C:\dev\hello\main.c:3: warning: implicit declaration of function `printf'
C:\dev\hello\main.c:3: error: `errno' undeclared (first use in this function)
C:\dev\hello\main.c:3: error: (Each undeclared identifier is reported only once
C:\dev\hello\main.c:3: error: for each function it appears in.)
C:\dev\hello\main.c:3: warning: implicit declaration of function `strerror'
C:\dev\hello\main.c:3: warning: format argument is not a pointer (arg 3)
C:\dev\hello\main.c: In function `my_putstr':
C:\dev\hello\main.c:20: warning: implicit declaration of function `write'
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:26: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:27: warning: passing arg 1 of `my_putstr' discards qualifiers from pointer target type
Process terminated with status 1 (0 minutes, 0 seconds)
3 errors, 7 warnings

Ceci est correct :

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
31
32
33
34
35
36
37
38
 
#include <stdio.h>
#include <errno.h>
#include <string.h>             /* strerror() (extension) */
 
/* extension */
#include <io.h>                 /* write() */
 
void my_error (void)
{
   printf ("%d %s", errno, strerror (errno));
}
 
int my_strlen (char const *str)
{
   int i = 0;
 
   while (str[i] != 0)
   {
      i++;
   }
   return i;
}
 
void my_putstr (char const *str)
{
   int verif = write (STDOUT_FILENO, str, my_strlen (str));
   if (verif < 0)
   {
      my_error ();
   }
}
 
int main (void)
{
   my_putstr ("toto");
   return 0;
}

Pose des questions si tu ne comprends pas.

[Maf77]

J'avais pas poster les inclusions systemes, ca me semblait implicite désolé.
J'ai toujours le même soucis.

[Emmanuel Delahaye]

J'avais pas poster les inclusions systemes, ca me semblait implicite désolé.

Bah, non; on ne sait pas ce que tu as réellement oublié...

J'ai toujours le même soucis.

Avec quel code exact ?

[Maf77]

Je compile sous FreeBSD.
Avec exactement ce code là:

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
31
32
33
34
#include <sys/types.h>
#include <sys/uio.h>
#include <unistd.h>
#include <errno.h>
#include <string.h>
 
void    my_error()
{
  printf("%d %s", errno, strerror(errno));
}
 
int     my_strlen(char *str)
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
      i = i + 1;
  return (i);
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}
 
int     main()
{
  my_putstr("toto");
}

"toto" s'affiche bien.

Code :

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

1
2
3
4
5
6
7
8
int     main(int ac, char **av, char **envp)
{
  char  **my_env;
  t_env *my_list_env;
 
  my_putchar('l');
  my_putstr("Salut");
}

il me dit que le retour du write dans celui-ci vaut 0 et ca segV!
J'ai comme retour de strerror:

Code :

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

22 Unknown error: 0

Je ne trouve malheureusement pas la raison...
Quelqu'un le sait?

[Emmanuel Delahaye]

Je compile sous FreeBSD.
Avec exactement ce code là:
<...>
"toto" s'affiche bien.

Ce code est incorrect. Avec ceci 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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
#ifdef _WIN32
#include <io.h>
#else
#include <sys/types.h>
#include <sys/uio.h>
#include <unistd.h>
#endif
 
#include <errno.h>
#include <string.h>
 
void    my_error()
{
  printf("%d %s", errno, strerror(errno));
}
 
int     my_strlen(char *str)
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
      i = i + 1;
  return (i);
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}
 
int     main()
{
  my_putstr("toto");
}

j'obtiens :

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
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c:13: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `my_error':
C:\dev\hello\main.c:14: warning: implicit declaration of function `printf'
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:37: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:38: warning: passing arg 1 of `my_putstr' discards qualifiers from pointer target type
C:\dev\hello\main.c:39: warning: control reaches end of non-void function
Output size is 20.78 KB
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 5 warnings

Attention à utiliser les bonnes options de ton compilateur, pour qu'il voit bien les problèmes de codage.

Nota : j'ai déjà montré pourquoi ce code était incorrect. Si tu ne lis pas les réponses apportées, tu perds ton temps à poster ici et nous, on perd le notre à te répondre... J'efface tout ?

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Code :

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

1
2
3
4
5
6
7
8
int     main(int ac, char **av, char **envp)
{
  char  **my_env;
  t_env *my_list_env;
 
  my_putchar('l');
  my_putstr("Salut");
}

Ce code est incomplet et ne compile pas.

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
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c: In function `main':
C:\dev\hello\main.c:4: error: `t_env' undeclared (first use in this function)
C:\dev\hello\main.c:4: error: (Each undeclared identifier is reported only once
C:\dev\hello\main.c:4: error: for each function it appears in.)
C:\dev\hello\main.c:4: error: `my_list_env' undeclared (first use in this function)
C:\dev\hello\main.c:6: warning: implicit declaration of function `my_putchar'
C:\dev\hello\main.c:7: warning: implicit declaration of function `my_putstr'
C:\dev\hello\main.c:3: warning: unused variable `my_env'
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:1: warning: unused parameter 'ac'
C:\dev\hello\main.c:1: warning: unused parameter 'av'
C:\dev\hello\main.c:1: warning: unused parameter 'envp'
Process terminated with status 1 (0 minutes, 0 seconds)
4 errors, 6 warnings

[Médinoc]

Et tu persistes à utiliser printf() sans inclure <stdio.h>.
Je rappelle que si tu n'as pas le droit d'inclure <stdio.h>, c'est que tu n'as pas le droit d'utiliser printf().

[Maf77]

C'est vraiment très dur de ce faire comprendre par des gens...
Oh my god.
Voici ton main tout jolie Emmanuel:

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
31
32
33
34
35
36
37
38
39
40
41
#ifdef _WIN32
#include <io.h>
#else
#include <sys/types.h>
#include <sys/uio.h>
#include <unistd.h>
#endif
 
#include <stdio.h>
#include <errno.h>
#include <string.h>
 
void    my_error()
{
  printf("%d %s", errno, strerror(errno));
}
 
int     my_strlen(char *str)
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
      i = i + 1;
  return (i);
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}
 
int     main()
{
  my_putstr("toto");
  return (0);
}

Bref. J'arrive vraiment pas à vous expliquer mon probleme! Génial: perte + perte = perte * 2 =D

[Emmanuel Delahaye]

J'arrive vraiment pas à vous expliquer mon probleme!

C'est pourtant simple. Poste le code exact qui pose problème. Si c'est celui qui tu as déjà posté, corrigé le parce qu'il est très incomplet.

[Maf77]

bon..
exactement à la virgule près:
my_putstr.h:

Code :

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

1
2
3
4
5
6
7
8
#ifndef __MY_PUTSTR_H__
# define __MY_PUTSTR_H__
 
void    my_error();
int     my_strlen(char *str);
void    my_putstr(char *str);
 
#endif

my_putstr.c:

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
#include <sys/types.h>
#include <io.h> /*ou #include <sys/uio.h> pour ce qui sont freebsd*/
#include <unistd.h>
#include <errno.h>
#include <string.h>
#include <stdio.h>
 
void    my_error()
{
  printf("%d %s", errno, strerror(errno));
}
 
int     my_strlen(char *str)
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
      i = i + 1;
  return (i);
}
 
void    my_putstr(char *str)
{
  int   verif;
 
  verif = write(1, str, my_strlen(str));
  if (verif == 0)
    my_error();
}

Et dans mon main.c:

Code :

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

1
2
3
4
5
6
7
#include "my_putstr.h"
 
int     main(int ac, char **av, char **envp)
{
  my_putstr("Salut");
  return (0);
}

Ce code segV sous FreeBSD.

[Médinoc]

errno n'est pas définie si write() a retourné zéro, car ce n'est pas à proprement parler une condition d'erreur. Il faut que write() ait retourné un résultat négatif pour qu'errno soit définie.

De plus, tes fonctions manquent de const, et tu devrais utiliser STDOUT_FILENO plutôt que 1.

[Maf77]

C'est bien pour ca que ca me semble louche!
errno me renvoi des choses bizard...
edit:
Quand il n'y pas de caractère:

Code :

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

Num error: 22 description: Unknown error: 0

Quand il y une chaine de plusieurs caracteres dans my_putstr j'obtiens:

Code :

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

Num error: 22 description: Invalid argument

grace à un meilleur printf dans my_error:

Code :

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

printf("Num error: %d description: %s\n", errno, strerror(errno));

Toujours aucune idée...

[Emmanuel Delahaye]

bon..
exactement à la virgule près:

Inutile d'ironiser. L'informatique, c'est précis, c'est pas un jeu de devinettes.

Ce code segV sous FreeBSD.

Ce code est correct et fonctionne bien sous Windows :

Code :

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

1
2
3
4
 
Salut
Process returned 0 (0x0)   execution time : 0.193 s
Press any key to continue.

Le problème est ailleurs.

Je recommande quand même d'améliorer le code comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include "my_putstr.h"
 
int main (void)
{
   my_putstr ("");
   my_putstr ("Salut\n");
 
   return 0;
}

Code :

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

1
2
3
4
5
6
7
8
9
 
#ifndef H_MY_PUTSTR
#define H_MY_PUTSTR
 
void my_error (void);
int my_strlen (char const *str);
void my_putstr (char const *str);
 
#endif

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
31
32
33
 
#include "my_putstr.h"
 
#include <sys/types.h>
#include <io.h>                 /*ou #include <sys/uio.h> pour ce qui sont freebsd */
#include <unistd.h>
#include <errno.h>
#include <string.h>
#include <stdio.h>
 
void my_error (void)
{
   printf ("%d %s", errno, strerror (errno));
}
 
int my_strlen (char const *str)
{
   int i;
 
   i = 0;
   while (str[i] != '\0')
      i = i + 1;
   return i;
}
 
void my_putstr (char const *str)
{
   int verif;
 
   verif = write (STDOUT_FILENO, str, my_strlen (str));
   if (verif < 0)
      my_error ();
}

[Emmanuel Delahaye]

C'est bien pour ca que ca me semble louche!
errno me renvoi des choses bizard...
edit:
Quand il n'y pas de caractère:

Code :

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

Num error: 22 description: Unknown error: 0

C'est normal. si il y a 0 caractères à émettre, write() renvoie 0, ce n'est pas une erreur...(lire la doc...)

L'erreur c'est si write() renvoie une valeur négative (< 0). On se tue à te le répéter, mais tu n'écoutes rien. C'est fatigant...

[Maf77]

Oui, fatiguant, comme se faire comprendre :-)
Super la différence au niveau de la norme pour protéger de l'inclusion multiple, mais j'ai pas le droit à ça et c'est hors sujet.

Oui, l'erreur de write, c'est que ca valeur de retour vaut -1! Je sais bien.
Mais quand, j'essaye d'imprimer à l'écran "toto" avec mypustr et rien ne ce passe... Je fais quoi? J'attends sagement qu'un cochon se mette à voler?
Donc pour ça, j'ai mis un test de l'erreur si celle-ci vaut 0 car, write renvoit le nombre d'octet écrit.
0?! Pourquoi 0? "toto" ne fais pas zéro caractere pourtant...

[Emmanuel Delahaye]

Mais quand, j'essaye d'imprimer à l'écran "toto" avec mypustr et rien ne ce passe...

Je te l'ai déjà dit, le code est correct. Le problème est ailleurs.

Essaye avec "toto\n" comme je l'ai fait (problème de bufferisation)...

[Médinoc]

J'ignore ce que fait ton implémentation, mais essaie en ajoutant un \n à la fin de ta chaîne (je ne sais plus à quel niveau est supposée être gérée l'histoire de la bufferisation et du \n), ou bien un appel à sync() ou fsync().

Et pourquoi zéro? Ben, vérifie le retour de mystrlen() pour commencer (même si le code me parait correct), puis vérifie que l'implémentation de write() que tu utilises respecte la bonne norme...
Si ton FreeBSD est vieux, peut-être que write() ne retourne jamais une valeur positive dessus: Juste 0 ou <0 pour dire si l'écriture a marché ou non...

[Médinoc]

Tiens, aussi:

write renvoie le nombre d'octets écrits (0 signifiant aucune écriture), ou -1 s'il échoue, auquel cas errno contient le code d'erreur. Si count vaut zéro, et si le descripteur est associé à un fichier normal, 0 sera renvoyé sans effets de bord. Pour un fichier spécial, les résultats ne sont pas portables.

Un terminal n'est pas un fichier normal, donc rien ne garantit en fait, que write() retourne autre chose que zéro en cas de succès.

[matafan]

Puisque tu as un coredump, tu pourrais commencer par regarder ce qui a merdé avec un debugger.

En tout cas ton code marche bien chez moi. Est-il possible que le code soit correct, mais que tu ne le compile pas comme il faut ? Tu parles de librarie mais dans ce que tu nous donne je ne vois pas de librairie... Juste deux .o qu'on link entre eux.

[Maf77]

C'est fou... strlen renvoit 0!
Je suis largué là