Discussion :

[Pitou5464]

salut a tous,

Je souhaite ajouter une chaine de caractere dans data de la structure de ma liste chainee, mais quand jessaie de limprimer a l ecran il me met null donc je pense qu'il n'ajoute pas d'elements

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
t_listoperandes         *my_operandes_in_list(char *str)
{
  t_listoperandes       *begin;
  int                   i;
  int                   l;
  int                   a;
  char                  *res;
 
  res = xmalloc(sizeof(*res));
  begin = xmalloc(sizeof(*begin));
  begin->data = 0;
  begin->next = 0;
  i = 0;
  l = 0;
  a = 0;
 
  while (str[i] != '\0')
    {
      if (str[i] > 47 && str[i] < 58)
        {
          res[a] = str[i];
          a++;
        }
      begin = add_to_list_operandes(res, begin);
      begin = begin->next;
      i++;
    }
  while (begin)
    {
      printf("%s", begin->data);
      begin = begin->next;
    }
}

[Pitou5464]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
t_listoperandes         *add_to_list_operandes(char *data, t_listoperandes *begin)
{
  t_listoperandes        *temp;
 
  temp = xmalloc(sizeof(*temp));
  temp->data = data;
  temp->next = begin;
  begin = temp;
 
  return (begin);
}

[Rits]

ca a l'air bien crade ton code
explique ce que tu fais la pour commencer

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
  while (str[i] != '\0')
    {
      if (str[i] > 47 && str[i] < 58)
        {
          res[a] = str[i];
          a++;
        }
      begin = add_to_list_operandes(res, begin);
      begin = begin->next;
      i++;
    }

c pas plutot

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  while (str[i] != '\0')
    {
      if (str[i] > 47 && str[i] < 58)
        {
          res[a] = str[i];
          a++;
        }
      i++;
     }
 
      begin = add_to_list_operandes(res, begin);
      begin = begin->next;

ensuite ici

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
t_listoperandes         *add_to_list_operandes(char *data, t_listoperandes *begin)
{
  t_listoperandes        *temp;
 
  temp = xmalloc(sizeof(*temp));
  temp->data = data;  // c pas plutot une copie que tu devrais faire ici
  temp->next = begin;
  begin = temp;
  return (begin);
}

[Thierry Chappuis]

Salut,

Je ne pense pas que tu peux utiliser ta variable res sous la forme d'un tableau:

Code :

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

1
2
3
4
5
6
7
8
9
 
    while (str[i] != '\0')
    {
          if (str[i] > 47 && str[i] < 58)
             {
                  res[a] = str[i];
                  a++;
              }
               /*...*/

Si je ne trompe pas, res pointe sur un espace de 1 octet, non?

Code :

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

1
2
3
4
 
  char                  *res;
 
  res = xmalloc(sizeof(*res));

Je ne comprend pas bien ce que tu veux faire. Sinon, pour une utilisation propre de l'allocation dynamique, ce tutoriel est excellent:
http://rperrot.developpez.com/articles/c/allocationC/

[deck_bsd]

Ben son res peut être considéré comme un tableau car il alloue ici :

res = xmalloc(sizeof(*res));

Mais effectivement il serai bien de nous dire ce que fait ton code. et aussi, xmalloc ? késako ? pourquoi pas malloc ?

[deck_bsd]

De plus si data dans ta structure est une chaine de caractère,

temp->data = data;

ne saurai pas fonctionner. Pour mettre une chaine de caractère dans une variable prévue à cet effet, il faut utiliser strcpy (ds string.h) :

strcpy(temp->data,data);


Voila

[diogene]

Oui, mais un tableau de 1 char. C'est pas beaucoup pour se permettre

Code :

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

1
2
3
4
5
6
7
8
  while (str[i] != '\0')
    {
      if (str[i] > 47 && str[i] < 58)
        {
          res[a] = str[i];
          a++;
        }
....

dès que a dépasse 0, ca plante
deck_bsd

De plus si data dans ta structure est une chaine de caractère,

temp->data = data;

ne saurai pas fonctionner.

Ce qu'elle n'est pas, sinon il n'aurait pas pu écrire temp->data = data; le compilateur l'aurait interdit