Discussion :

[ChatPo]

Bonjour,
j'essaie d'écrire un code qui récupère une ligne dans un flot (ici stdin mais cela peut être différent) SANS connaitre la taille de cette ligne. Ce que j'ai fait n'est pas terrible (je reviendrai vous demander si je ne trouve pas mieux), mais je ne veux pas la réponse à ce problème. Ma question est pourquoi le code suivant ne fonctionne pas (avec la ligne en commentaire, cela marche pourtant).
Merci d'avance.

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *lireLigne2(char *dst,int tailleDep,FILE *flot)
{
  int i=0;
  char *chaine=NULL;
  char *temp=NULL;
  //char *verif=NULL;    
  char lettre=fgetc(flot);
  chaine=calloc((tailleDep+1),sizeof(*chaine));
  if (chaine==NULL)
    {
      printf("pb de calloc.\n");
      exit(0);
    }
  if ((lettre=='\n'))
    return NULL;
  while((lettre!='\n'))
  {
 
    if (i<tailleDep)
    {           
      chaine[i]=lettre;
      chaine[i+1]='\0';
    }
    else
    {
      temp=chaine;
      chaine=realloc(chaine,(strlen(temp)+2)*sizeof(*temp));
 
      if (chaine==NULL)
        printf("Pb de realloc1");
      strcpy(chaine,temp);
      chaine[strlen(chaine)]=lettre;
      chaine[strlen(chaine)+1]='\0';
    }
    lettre=fgetc(flot);
    i++;
  };
  dst=chaine;
  printf("%s %p\n",dst,dst);
  printf("%s %p\n",chaine,chaine);
  return dst;
}
 
int main(void)
{
  char *nom=NULL;
  printf("Tapez votre nom :\n");
  lireLigne2(nom,4,stdin);
  //nom=lireLigne2(nom,4,stdin);
  printf("%s %p\n",nom,nom);
  if (nom==NULL)
  {
    printf("GROS PB\n");
    exit(0);
  }
  printf("Vous êtes %s de taille %d.\n",nom,strlen(nom));
  free(nom);
 
  return EXIT_SUCCESS;
}

[gerald3d]

ton problème est surtout lié à l'adressage de la fonction lireLigne2();. Il y a un post qui pourrait t'intéresser sur le sujet, surtout la réponse qui traite du double adressage : http://www.developpez.net/forums/d12...n/#post6608060

En gros tu ne récupères pas le pointeur nom alloué dans la fonction lireLigne2();.

[fred1599]

Je vois déjà quelquechose qui doit bugger

Code :

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

printf("Vous êtes %s de taille %d.\n",nom,strlen(nom));

Je ferais un cast

Code :

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

printf("Vous êtes %s de taille %d.\n",nom, (int) strlen(nom));

[kwariz]

Salut,

En plus du classique "si une fonction doit modifier un paramètre et si cette modification doit être visible par l'appelant, alors il faut passer par un pointeur dans le prototype et l'appelant donnera l'adresse de la variable dont le contenu sera modifié" pointé par Gerald3d, je me permets de rajouter :

* Dans la fonction lireLigne2, tu alloues de la mémoire à chaine en l12, mais si le caractère lu est un \n en l18 alors tu sors de la fonction en retournant un NULL en l19 sans libérer la mémoire allouée en l12.

* Tu fais des fgetc sans tester si le retour vaut EOF.

* l30-34: bien ! tu testes le retour de realloc (bon réflexe ) mais mal ...
tu pourrais simplifier en :

Code :

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

1
2
3
4
5
6
7
8
 
temp=realloc(chaine, (strlen(chaine)+2,)*sizeof(*chaine));
if (temp==NULL)
  le realloc a échoué
  message + éventuelle sortie soit un exit soit return ce que tu as déjà lu
else 
  le realloc a réussi et chaine (déplacé ou non d'ailleurs) est un pointeur valide
  avec le même contenu qu'avant : pas la peine de recopier le contenu précédent.

De plus dans l'état de ton code, quoiqu'il arrive (realloc réussi ou pas) tu écris quand même ce qui est lu (pas bon si realloc a échoué : tu écris dans un zone non réservée).

* remarque à mettre dans la catégorie bon à savoir : calloc initialise la mémoire allouée en mettant tous les octets à 0, donc pas la peine de les mettre encore une fois à 0 en l26, mais pas de problème ça ne mange pas de pain.
Par contre malloc a le comportement opposé ! la mémoire allouée n'est absolument pas initialisée et peut continir n'importe quoi.
Realloc adopte la même démarche que malloc : si de la mémoire est rajoutée (car realloc permet aussi de réduire la mémoire allouée) celle-ci n'est pas initialisée, seule la partie anciennement utilisée est totalement copiée (si nécessaire), ce qui rend ton affectation en l37 obligatoire (enfin si le realloc réussi).

[Bktero]

Je vois déjà quelquechose qui doit bugger

Code :

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

printf("Vous êtes %s de taille %d.\n",nom,strlen(nom));

Je ferais un cast

Code :

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

printf("Vous êtes %s de taille %d.\n",nom, (int) strlen(nom));

Ce n'est pas utile. Strlen() renvoie une variable de type size_t. Size_t est un entier non signé en règle générale. Sauf si le compilateur met un message d'erreur, pas besoin du cast.

Exemple (qui compile sans warning ni erreur) :

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
#include <stdio.h>
#include <string.h>
 
int main(void)
{
    size_t a=5;
    int b=strlen("helloitsvalentine");
    char c=12;
    char d=strlen("Ca c'est pas bon du tout car la chaine est tres longue "
                  "et donc elle fait plus de 128 caracteres. "
                  "Un char (qui est signe normalement) n'est pas donc assez grand pour en contenir la taille");
    printf("a = %d\n", a);
    printf("b = %d\n", b);
    printf("c = %d\n", c);
    printf("d = %d\n", d);
    printf("direct = %d\n", strlen("Ici ca va le faire meme si la chaine est tres longue "
                  "et qu'elle fait plus de 128 caracteres. "
                  "En effet, ici on renvoit un size_t qui ne sera pas caster en char (qui est signe normalement)."));
    return 0;
}

[ChatPo]

Merci à tous. Les modifs faites fonctionnent parfaitement.
Je vais regarder les remarques de Kwariz