Discussion :

[HaTnuX]

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

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

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

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 :

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

while(temp != NULL){

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

Code :

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

temp2 = temp->next;

Et là aussi pour moi c'est bon

Alors je suis vraiment nul ?

Merci à tous

[Gruik]

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.

[HaTnuX]

ah oui en effet, quand je fais

Code :

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

while(&temp != NULL){

ça larche mieux

Merci beaucoup

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

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

[thewho]

Bonjour,

ah oui en effet, quand je fais

Code :

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

while(&temp != NULL){

ça larche mieux

Merci beaucoup

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

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

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 :

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

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 :

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

instruction temp, temp2;

en

Code :

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

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

[diogene]

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

Code :

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

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 :

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

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 :

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

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