Discussion :

[Trap D]

Et qui plante puisqu'il n'y a pas le zéro terminal
Il reste aussi éventuellement le '\n' en fin de chaîne

[Pragmateek]

Et qui plante puisqu'il n'y a pas le zéro terminal

Si fgets s'en occupe.

Il reste aussi éventuellement le '\n' en fin de chaîne

EDIT : en effet.

[gege2061]

Ca j'attend l'avis de Emdel.

Mais laissez faire sa sieste tranquillement, le pauvre

fgets() lit au plus size - 1 caractères depuis stream et les place dans le buffer pointé par s. La lecture s'arrête après EOF ou un retour-chariot. Si un retour-chariot (newline) est lu, il est placé dans le buffer. Un caractère nul '\0' est placé à la fin de la ligne.

[Pragmateek]

En effet, alors une petite purge du '\n' en plus:

Code :

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

1
2
3
4
char s[100];
char *p_s=malloc(strlen(fgets(s,100,stdin)));
*strchr(s,'\n')=0;
strcpy(p_s,s);

[fearyourself]

En effet, alors une petite purge du '\n' en plus.

Et si fgets rend NULL?

Jc

[Pragmateek]

Et si fgets rend NULL?

Jc

Comment serait-ce possible?

[Trap D]

Ton code n'est pas safe.

• fgets peut renvoyer NULL
• le '\n' peut ne pas être dans la saisie et ton *strchr(s, '\n') = 0 fait boum
• et j'insiste, sur ton ancien code

  Code :

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

  1
  2
  char *p_s=malloc(strlen(fgets(s,100,stdin)));
  strcpy(p_s,s);

  il y avait plante ensuite à l'utilisation puisque tu allouais juste la taille (rien ne prouve que '\n' soit dans la saisie).

[gege2061]

Comment serait-ce possible?

Il faut peut être lire la doc, je ne vais pas recopier les man pages à chaque fois :
man fgets

fgets() renvoient le pointeur s si elles réussissent, et NULL en cas d'erreur, ou si la fin de fichier est atteinte avant d'avoir pu lire au moins un caractère.

[Pragmateek]

Il faut peut être lire la doc, je ne vais pas recopier les man pages à chaque fois :
man fgets

Je sais mais dans le cas présent au moin 1 caractère, '\n', est lu; et je ne voie pas de raison d'echec même si dans ce cas ça plante l'application.
Ce code vaut ce qu'il vaut, il est compact c'est tout, ni sécurisé, ni lisible, ni maintenable.

[Trap D]

Je sais mais dans le cas présent au moin 1 caractère, '\n', est lu; et je ne voie pas de raison d'echec même si dans ce cas ça plante l'application.
Ce code vaut ce qu'il vaut, il est compact c'est tout, ni sécurisé, ni lisible, ni maintenable.

C'est bien ça le problème, pourquoi donner un code s'il n'a pas ces qualités (et qu'on le sait !!!)

[Pragmateek]

C'est bien ça le problème, pourquoi donner un code s'il n'a pas ces qualités (et qu'on le sait !!!)

Parce que apparemment le code se devait d'être compact.
Sinon une question: dans le cas de lecture sur "stdin" qu'est-ce qui pourrait provoquer l'échec de "fgets"?

[Emmanuel Delahaye]

Code :

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

char *p_s=malloc(strlen(fgets(s,100,stdin)));

Pourquoi 100 et pas sizeof s ?

[Emmanuel Delahaye]

Et qui plante puisqu'il n'y a pas le zéro terminal

Avec fgets(), il y a toujours le 0 terminal (sauf si il retourne NULL, bien sûr...).

[Emmanuel Delahaye]

Code :

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

*strchr(s,'\n')=0;

Et si il n'y a pas de '\n' ?

Tu crois vraiment vouloir faire le malin et faire mieux que le code habituel, maintes fois publié, qui est le résultat de 30 ans de reflexion sur le C ?

[Emmanuel Delahaye]

Comment serait-ce possible?

Ctrl-D, Ctrl-Z...

[Pragmateek]

Pourquoi 100 et pas sizeof s ?

C'est un exemple.

Tu crois vraiment vouloir faire le malin et faire mieux que le code habituel, maintes fois publié, qui est le résultat de 30 ans de reflexion sur le C ?

Je n'ai pas cette prétention, voir mes réponses plus haut.

Ctrl-D, Ctrl-Z...

Ctrl-D passe... Ctrl-Z .
A quoi ça correspond au fait ces séquences?

[gl]

Avec fgets(), il y a toujours le 0 terminal (sauf si il retourne NULL, bien sûr...).

Oui, mais il n'est pas reserve pour p_s dans le malloc, malloc dont le retour n'est en outre pas teste.

[Trap D]

Avec fgets(), il y a toujours le 0 terminal (sauf si il retourne NULL, bien sûr...).

Emmanuel lit tout

Code :

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

1
2
char *p_s=malloc(strlen(fgets(s,100,stdin)));
strcpy(p_s,s);

on appelle malloc avec juste la longueur de la saisie, sans le zéro terminal, et ensuite on recopie dedans, donc plante.

[Pragmateek]

avec juste la longueur de la saisie

Bon alors il suffit d'ajouter 1.

donc plante.

Non, mais en effet sûrement comportement indéfini, à voir.

[Emmanuel Delahaye]

Emmanuel lit tout

Code :

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

1
2
char *p_s=malloc(strlen(fgets(s,100,stdin)));
strcpy(p_s,s);

on appelle malloc avec juste la longueur de la saisie, sans le zéro terminal, et ensuite on recopie dedans, donc plante.

Oui, je vois. J'évite de passer des fonctions en paramètre de fonction. Trop gore... Alors 3 , c'est du Grand Guignol...