Discussion :

[Toutankharton]

Bonjour,
J'espere que le sujet n'est pas trop flou : J'ai une liste chainee qui contient une suite de chaine. Je voudrais comparer le premier caractere de chacune de ces chaines de caractere a un char. J'ai tout betement teste

Code :

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

printf("%c", tmp->data[0])

et

Code :

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

printf("%c", tmp->(data[0]));

mais le premier n'affiche rien et le second

error: parse error before '(' token

Cela est-il seulement possible ?

Merci d'avance !

[Franck.H]

Essaye comme ca:

Code :

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

printf("%c", tmp.data[0]);

[Toutankharton]

Bonjour,
Merci de ta reponse rapide
Je travaille dans une sous fonction qui a recu un pointeur sur ma liste chainee. Je ne peut donc pas faire tmp.data, mais seulement tmp->data.

tmp.data ni tmp.data[0] ne marche donc..

Un autre idee ?

[Franck.H]

Ton tableau, c'est un tableau statique ou dynamique ? Sinon montre ton code, réduit au minimum en situant le problème mais compilable !

Normalement:

Code :

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

printf("%c", tmp->data[0]);

devrais fonctionner d'après mes tests rapides donc si ca ne t'affiche rien... montre le code remplissant ce tableau et la fonction qui lit le tableau !

[Toutankharton]

J'ai teste, l'erreur est :

error: request for member `data' in something not a structure or union

Voila mon code (reduit et commente) :

Fonction appelant :

Code :

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

1
2
3
4
5
6
7
8
char            *fonction(...)
{
  t_list        *list;  // declaration d'un pointeur sur ma liste chainee
 
  list = 0;             // initialisation de la liste
  appel(&list, ...);  // appel a ma sous fonction, avec l'adresse du pointeur en parametre
  aff_list(list);
}

Fonction appellee :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void            appel(t_list **list, ...)
{
  t_list        *tmp; // un pointeur sur la liste chainee
 
  tmp = *list; // on donne la valeur de l'adresse de la liste chainee initialise dans la fonction precedente
  while (tmp != 0)
    {
      printf("%c", tmp.data[0]);  // voila mon test
      tmp = tmp->next; // on passe a la chaine suivante dans la liste
    }
}

Pour info ma liste chainee ressemble a sa :

Code :

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

1
2
3
4
5
typedef struct          s_list
{
  char                  *data;
  struct s_list         *next;
}                       t_list;

Un exemple de ce que je voudrais :

Un liste chaine contenant ces chaines :

Albert
Alexandre
Bernard
Guillaume

je voudrais enregistrer chaque chaine (chaque adresse plutot) dont le data[0] est egal a A par exemple

[Franck.H]

J'ai teste, l'erreur est ...

Ouais désolé, j'avais pas tout lu dans ton message, j'ai rectifié le tir par la suite

Code :

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

1
2
3
4
5
6
7
8
char            *fonction(...)
{
  t_list        *list;  // declaration d'un pointeur sur ma liste chainee
 
  list = 0;             // initialisation de la liste
  appel(&list, ...);  // appel a ma sous fonction, avec l'adresse du pointeur en parametre
  aff_list(list);
}

Je ne vois pas d'allocation de ta liste et de ton tableau de char, est-ce normal ? Si tu déclares un pointeur que t'initialise à 0 (déjà à NULL ce serait peut être mieux pour un pointeur) puis tu passes son adresse (ici null) à une autre fonction ... tu risque pas d'aller très loin là ! A cet instant précis, ta liste n'existe pas encore et dans la fonction appelée tu veut déjà la parcourrir ... y'a une mouche dans le lait là

[hiko-seijuro]

Code :

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

1
2
3
4
t_list        *list;  // declaration d'un pointeur sur ma liste chainee
 
  list = 0;             // initialisation de la liste
  appel(&list, ...);

euh quel est l'interet d'un pointeur sur pointeur ?????????????????????????????

[Franck.H]

Code :

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

1
2
3
4
t_list        *list;  // declaration d'un pointeur sur ma liste chainee
 
  list = 0;             // initialisation de la liste
  appel(&list, ...);

euh quel est l'interet d'un pointeur sur pointeur ?????????????????????????????

C'est vrai, j'avais pas fait attention à ca je dirais que dans son cas, aucun intérêt vu qu'il ne modifie pas son pointeur dans la fonction ...

[Trap D]

On pourrait voir le code d'initialisation de la liste ?

[Toutankharton]

Je ne vois pas d'allocation de ta liste et de ton tableau de char, est-ce normal ?

Dans ma fonction (ici appellée fonction) j'appelle une sous fonction qui gere l'insertion dans la liste des données avec une boucle de cette façon :

Code :

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

list = my_put_in_list(list, "mes données malocées");

le point t_liste *list sert donc a enregistrer l'adresse de la chaine courante pour pouvoir inserer la suivante à sa suite.

quel est l'interet d'un pointeur sur pointeur ?

Comme je travaille par sous fonctions pour ne pas surcharger ma fonctions principales, je suis obligé d'envoyer des pointeurs. Ici il s'agit d'un pointeur sur une chaine de caractere d'une liste chainée. Donc j'envoit son adresse en faisany t_list **list.
Est-ce clair ?

aucun intérêt vu qu'il ne modifie pas son pointeur dans la fonction ...

Je veut bien envoyer mes fichiers mais tout poster sur le forum serais lourd. Je modifie mon pointeur dans ma fonction, mais je ne vous montre la que les bouts necessaire a la comprehension de mon probleme : afficher le caractere n°1 d'une chaine de caractere donnée dans une liste chainée

On pourrait voir le code d'initialisation de la liste ?

Le code d'initialisation ? Qand je met en commentaire initialisation de la liste, c'est juste que je met à 0 la premiere adresse (il n'y a pas d'adresse pour une chaine suivante). la variable list changera au fur et à mesure que l'on inserera les données. Voici mon my_put_in_list qui insere les données *data a la suite de la chaine pointée par list :

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
t_list          *my_put_in_list(t_list *list, char *data)
{
  t_list        *new_elem;
  t_list        *temp;
 
  new_elem = malloc(sizeof(*new_elem));
  new_elem->data = data;
  new_elem->next = 0;
  if(list == 0)
    return new_elem;
  else
    {
      temp = list;
      while(temp->next != 0)
        {
          temp = temp->next;
        }
      temp->next = new_elem;
      return list;
    }
}

(desolé pour les tabulations, elles partent au copié-collé dans putty apparemment...)

Bref, je veux bien expliquer et publier tout mon code, mais ça ne servirais pas a grand chose, mon probleme est relativement isolé. Je peut faire un petit programme presentant mon probleme si vous voulez. Si temp.data[0] marche en theorie, y a-t-il un equivalent quand on travaille sur des pointeurs de structures (temp->data) ?

Merci d'avance !

[Emmanuel Delahaye]

Si temp.data[0] marche en theorie, y a-t-il un equivalent quand on travaille sur des pointeurs de structures (temp->data) ?

temp->data[0];

[Toutankharton]

Comme je le disais dans mon premier message, ça n'affiche rien, et temp->(data[0]) :

error: parse error before '(' token

[Trap D]

Ça ne devrait pas passer à la compilation ou tout au moins te faire un warning car la fonction ne retourne rien.
Il faut de plus que tu testes le retour de malloc :

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
t_list *my_put_in_list(t_list *list, char *data)
{
  t_list *new_elem;
  t_list *temp;
 
  new_elem = malloc(sizeof(*new_elem));
 
 
  // toujours tester le retour de malloc
  if (new_elem == NULL)
    return NULL;
 
  // Pas très secure comme méthode, c'est une source de bug.
  // il faudrait mieux recopier les données avec un strdup par exemple
  new_elem->data = data;
  new_elem->next = 0;
 
  if(list == 0)
    return new_elem;
 
  temp = list;
  while(temp->next != 0)
  {
    temp = temp->next;
  }
  temp->next = new_elem;
  return list;
}

Ceci dit, ce n'est pas ça qui fait mal fonctionner ton programme.

[Emmanuel Delahaye]

Comme je le disais dans mon premier message, ça affiche

Gné ?
Quand on a un doute, on fait un 'snippet', c'est à dire un code complet mais réduit au minimum qui montre le problème :

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
#include <stdio.h>
 
struct node
{
   char *s;
};
 
int main (void)
{
   struct node n = {"hello"};
   struct node *p = &n;
 
   printf ("%c\n", p->s[0]) ;
 
   return 0;
}

Et là, ben je ne vois pas de problème.

C'est donc dans ton code qu'il y a un lézard, ou alors tu n'as pas expliqué comme il faut...

[Toutankharton]

oops ! J'avais oublié de faire l'appel à cette sous fonction dans la fonction principale ! Je suis désolé, des le debut c'etait bon, je verifiais par acquis de conscience apres mon precedent message...

temp->data[0] marche donc..