allocation mémoire d'une chaine de caractères

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

Citation:

---

Envoyé par Trap D

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

---

Si fgets s'en occupe.

Citation:

---

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

---

EDIT : en effet.

Citation:

---

Envoyé par seriousme

Ca j'attend l'avis de Emdel.

---

Mais laissez faire sa sieste tranquillement, le pauvre :calin:

Citation:

---

Envoyé par man fgets

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.

---

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

Code:

---

1 2 3 4

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

---

Citation:

---

Envoyé par seriousme

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

---

Et si fgets rend NULL? :aie:

Jc

Citation:

---

Envoyé par fearyourself

Et si fgets rend NULL? :aie:

Jc

---

Comment serait-ce possible?

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:

  ---

  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).

Citation:

---

Envoyé par seriousme

Comment serait-ce possible?

---

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

Citation:

---

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.

---

Citation:

---

Envoyé par gege2061

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.

Citation:

---

Envoyé par seriousme

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 !!!) 8O

Citation:

---

Envoyé par Trap D

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

---

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"?

Citation:

---

Envoyé par seriousme

Code:

---

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

---

---

Pourquoi 100 et pas sizeof s ?

Citation:

---

Envoyé par Trap D

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...).

Citation:

---

Envoyé par seriousme

Code:

---

*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 ?

Citation:

---

Envoyé par seriousme

Comment serait-ce possible?

---

Ctrl-D, Ctrl-Z...

Citation:

---

Pourquoi 100 et pas sizeof s ?

---

C'est un exemple.

Citation:

---

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.

Citation:

---

Ctrl-D, Ctrl-Z...

---

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

Citation:

---

Envoyé par Emmanuel Delahaye

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.

Citation:

---

Envoyé par Emmanuel Delahaye

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

---

Emmanuel lit tout

Code:

---

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.

Citation:

---

avec juste la longueur de la saisie

---

Bon alors il suffit d'ajouter 1.

Citation:

---

donc plante.

---

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

Citation:

---

Envoyé par Trap D

Emmanuel lit tout

Code:

---

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...