Discussion :

[Kerod]

Bonsoir,

J'utilise des pointeurs dans une fonction mais à la compilation il me dit que le code n'est pas correct.

Alors cette fonction doit mettre à jour un élément dans une pile que j'ai crée.
Elle recoit donc en parametre 2 données à mettre à jour dans une structure interne à la pile.Mais tout d'abord il doit rechercher l'élément à mettre à jour.
Pour l'instant j'utilise un entier passé en parametre lors de l'appel de la fonction.

Alors j'ai deux questions :
1- Il y aurait pas un autre moyen autre que mettre un entier pour savoir quel élément de la pile qui doit etre mis à jour ?
2- Pourquoi ai-je des erreurs de compilation pour les pointeurs ?

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
void modifier (tab_robot *p,int ligne,int col,int nb){ 
     maillon *q;
     q=*p;int j=1;
     while(j++<nb)
        q= q->suiv;
     q->ligne=ligne; 
     q->col=col;
}

Et merci d'avance d'y porter attention

[Vodlich]

Bonsoir,
2- Pourquoi ai-je des erreurs de compilation pour les pointeurs ?

A tu bien mis

Code :

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

#include <stdlib.h>

pour malloc,free,.. ?

[diogene]

2- Pourquoi ai-je des erreurs de compilation pour les pointeurs ?[

Code :

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

1
2
3
void modifier (tab_robot *p,int ligne,int col,int nb){ 
     maillon *q; 
     q=*p;....

*p est un tab_robot et q un pointeur sur maillon. Les types sont très probablement incompatibles. Il faut connaître la déclaration de maillon et de tab_robot pour trancher

1- Il y aurait pas un autre moyen autre que mettre un entier pour savoir quel élément de la pile qui doit etre mis à jour ?

Il faudrait avoir plus de détails. Mais repérer un élément d'une pile par sa position d'empilement semble une idée bizarre

[Kerod]

ah oui désolé j'ai oublié le plus important :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
typedef struct {
	char nom;
	int energie;
}cases;
 
typedef struct {
	int ligne;
	int col;
	struct maillon *suiv;
}maillon;
 
typedef maillon *tab_robot;

[fearyourself]

Code :

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

1
2
maillon *q;
     q=*p;int j=1;

En C standard, tu n'as pas le droit de declarer les variables locales apres une instruction... Il faut commencer avec les declarations...

Les compilateurs sont plus ou moins embetants pour ca...

[Médinoc]

Le C99 reste du C standard, et en C99, on peut.

Par contre, trop de compilos ne le supportent pas encore...

[Kerod]

Oui ce n'est pas de là que vient mon problème mais plutot du coté pointeur

[diogene]

Code :

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

1
2
3
4
5
typedef struct { 
   int ligne; 
   int col; 
   struct maillon *suiv; 
}maillon;

Certains compilateus n'aiment pas ce genre de choses et n'identifient pas struct maillon avec maillon. ils préfèrent

Code :

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

1
2
3
4
5
typedef struct mm{ 
   int ligne; 
   int col; 
   struct mm *suiv; 
}maillon;

Peut être est-ce la cause de tes problèmes. Le message d'erreur à la compilation pourrait aider à le savoir

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
typedef struct { 
   int ligne; 
   int col; 
   struct maillon *suiv; 
}maillon;

Certains compilateus n'aiment pas ce genre de choses et n'identifient pas struct maillon avec maillon.

C'est pire que ça. C'est carrément du C incorrect.

[Kerod]

Alors j'ai modifié et voici le code que ca me donne :

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
/*Définition de la pile robot*/
 
typedef struct ssmaillon{
	int ligne;
	int col;
	struct ssmaillon *suiv;
}maillon;
 
typedef maillon *tab_robot; 
/*Fonction qui modifie un élément de la pile*/
 
void modifier (tab_robot *p,int ligne,int col,int nb){ 
 
     maillon *q;
     q=*p;int j=0;
     while(j++<nb)
        q= q->suiv;
     q->ligne=ligne; 
     q->col=col;
 
}

[Kerod]

Bon j'ai à nouveau modifier et là ca fonctionne...Mais il reste un petit problème, mes valeurs ne sont pas modifiés comme elles devraient l'etre pourquoi ?!

[diogene]

Mais il reste un petit problème, mes valeurs ne sont pas modifiés comme elles devraient l'etre pourquoi ?!

Peux tu préciser ce qui se passe

[Kerod]

en fait c'est bon j'ai trouvé la solution, j'ai finalement fais comme ceci et ca fonctionne pour le moment :

Code :

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

1
2
3
4
5
6
7
8
9
10
void modifier (tab_robot *p,int old_ligne,int old_col,int new_ligne,int new_col){ 
 
     maillon *q;
     q=*p;
     while((q)->ligne!=old_ligne || (q)->col!=old_col){
        printf("%d %d\n",q->ligne,q->col);q= q->suiv;}
     q->ligne=new_ligne; 
     q->col=new_col;
 
}

Mais je me demande une chose...Je recherche dans ma pile la ligne et la col mais quand je mets dans le while le && il fait nimporte koi...
par exemple j'ai dans ma pile :
3 7
9 3
2 3

je veux remplacer le 2 3 par 3 2 et bien il s'arrete à 9 3 et remplace ces valeurs au lieu du 2 3