Comparaison sur structure

Bonjour à tous, voilà j'obtiens 2 erreurs dans ce code que je ne comprends pas :oops:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

void parse(char *ligne, char *FS, instruction *lst_pattern){   char *ptr1; instruction temp, temp2; temp = *lst_pattern;   ptr1 = malloc(100*sizeof(char)); ptr1 = strtok(ligne, FS); while(ptr1 != NULL){ while(temp != NULL){ if(strcmp(ptr1, temp.pattern)==0){ printf("FIND\n"); } temp2 = temp->next; temp = temp2; } ptr1 = strtok(NULL, FS); } }

---

La structure instruction ressemble à ceci:

Code:

---

1 2 3 4 5 6

typedef struct instruction_{ char *pattern; int nb_procedure; char **procedure; struct instruction_ *next; }instruction;

---

J'obtient celà comme erreur :
error: used struct type value where scalar is required
error: invalid type argument of ‘->’

C'est surtout la 1ère que je comprends pas, le problème est sur :

Code:

---

while(temp != NULL){

---

apparement, mais je vois pas pourquoi j'ai pas le droit de le faire 8O
Et la 2ème sur :

Code:

---

temp2 = temp->next;

---

Et là aussi pour moi c'est bon :oops:

Alors je suis vraiment nul ?

Merci à tous

Salut

Tu tentes simplement de comparer une structure à une constante qui n'est pas du meme type, qui n'est pas du meme type assimilable.
Si ta variable etait un pointeur, tu pourrais la comparer à NULL.

ah oui en effet, quand je fais

Code:

---

while(&temp != NULL){

---

ça larche mieux :oops:

Merci beaucoup :)

Et la 2ème erreur c'est aussi une histoire de pointeur ?

edit : j'ai rien dis j'ai trouvé la soluce :D

Bonjour,

Citation:

---

Envoyé par HaTnuX

ah oui en effet, quand je fais

Code:

---

while(&temp != NULL){

---

ça larche mieux :oops:

Merci beaucoup :)

Et la 2ème erreur c'est aussi une histoire de pointeur ?

edit : j'ai rien dis j'ai trouvé la soluce :D

---

Tu es sûr de ce que tu dis ?

Avec ta nouvelle écriture, tu ne fais que contrôler que ton temp a une adresse différente de NULL !!

De plus, plus loin, tu écris

Code:

---

temp2 = temp->next;

---

alors que temp n'est pas un pointeur.

Je doute que ton code compile s'il correspond à ce que tu nous montres.

Il faut faire attention aux types que tu utilises. Là, compte tenu du reste du code, tu as certainement besoin de changer la déclaration

Code:

---

instruction temp, temp2;

---

en

Code:

---

instruction *temp, *temp2;

---

et d'adapter le code en fonction.

Ton code me donne l'impression d'une écriture dans le genre
"Si j'essayais comme ça ? Après tout, on verra bien si ça marche."

Tu ne fais pas attention à ce que tu écris :

Code:

---

1 2 3 4

... if(strcmp(ptr1, temp.pattern)==0) printf("FIND\n"); temp2 = temp->next; ...

---

La première ligne indique que temp est un struct (ou une union) et la deuxième que temp est un pointeur sur un struct (ou une union). Ce code est incohérent.
Un peu plus haut :

Code:

---

1 2	ptr1 = malloc(100*sizeof(char)); ptr1 = strtok(ligne, FS);

---

Comment espères-tu pouvoir récupérer la mémoire allouée par malloc puisque tu écrases la valeur retournée ? A quoi sert ce malloc ?
Il est à peu près sur que, comme le dit thewho, on devrait avoir qq chose du genre

Code:

---

1 2 3 4 5 6 7 8 9

char *ptr1; instruction *temp = lst_pattern; .... while(temp != NULL) { if(strcmp(ptr1, temp->pattern)==0) printf("FIND\n"); temp = temp->next; } .....

---