Erreur de conflit de type

Version imprimable

Bonjour à tous je ne comprend pas pourquoi gcc ne veux pas compiler en me disant que je commet une erreur de conflit de type à l'appel de la fonction rechercherPiece.
Pourtant j'ai lu et relu les types que j'utilise et je ne comprend pas.
LapinCompris
Merci d'avance.
Jhulk
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 struct piece{ float valeur; }; void rendrePiece(float rendu){ int a=rechercherPiece(2.0); } int rechercherPiece(float p){ int i,ret=0; for(i=0;i<100;i++){ if(machine[i].valeur==p) ret++; } return ret; }
Titre édité par Anomaly : merci de mettre des titres clairs pour obtenir de meilleurs réponses !

16/01/2006, 14h00
Trap D

Pense aux balises :tagcode:

Citation:

Bonjour à tous je ne comprend pas pourquoi gcc ne veux pas compiler en me disant que je commet une erreur de conflit de type à l'appel de la fonction rechercherPiece.

Tu ne montres pas cet appel, simplement la fonction.
16/01/2006, 14h03
Jhulk

heu si si je l'ai montré il est à la ligne "int a=rechercherPiece(2.0);" dans la fonction void rendrePiece(float rendu)
LOL t'aurai pû lire les 10 lignes de code :wink:
Merci
16/01/2006, 14h06
hegros

Le bon prototype de recherchePiece devrait être:

Code:

recherchePiece( const float a)

Mais je peux me gourrer
16/01/2006, 14h08
Trap D

Effectivement :oops: :oops:
Peut-être as-tu oublié de déclarer la fonction rechercherPiece(2.0); avant de l'utiliser, par défaut (mais là c'est sans aucune garantie) le compilo voyant 2.0 l'a pris pour un double et ensuite il voit un float dans la déclaration, donc il n'est pas comptant.
16/01/2006, 14h11
hegros

Tu pourrais nous donner l'erreur exacte que renvoi gcc ?

Re: Petit probleme incomprehensible

Citation:

Envoyé par Jhulk

Bonjour à tous je ne comprend pas pourquoi gcc ne veux pas compiler en me disant que je commet une erreur de conflit de type à

Merci de poster du code plus compilable que ça.... Difficile de savoir ce que tu as oublé.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Compiling: main.c
main.c:5: warning: no previous prototype for 'rendrePiece'
main.c: In function `rendrePiece':
main.c:7: error: implicit declaration of function `rechercherPiece'
main.c:7: warning: nested extern declaration of `rechercherPiece'
main.c:7: warning: unused variable `a'
main.c: At top level:
main.c:10: warning: no previous prototype for 'rechercherPiece'
main.c:10: error: conflicting types for 'rechercherPiece'
main.c:10: note: an argument type that has a default promotion can't match an empty parameter name list declaration
main.c:7: error: previous implicit declaration of 'rechercherPiece' was here
main.c: In function `rechercherPiece':
main.c:13: error: `machine' undeclared (first use in this function)
main.c:13: error: (Each undeclared identifier is reported only once
main.c:13: error: for each function it appears in.)
main.c:17:2: warning: no newline at end of file
Process terminated with status 1 (0 minutes, 4 seconds)
7 errors, 5 warnings

16/01/2006, 14h21
Jhulk

Voilà ce que me donne le compilateur

>gcc piece.c
error: conflicting types for 'rechercherPiece'
note: an argument type that has a default promotion can't match an empty parameter name list declaration
error: previous implicit declaration of 'rechercherPiece' was here

L'erreur 1 correspond à la ligne du prototype de la fonction.
La note aussi.
Et la dernière à l'appel de la fonction.

voili voilou.

voici le code entier
(y'as pas de main, utiliser le paramètre -o avec gcc)
system("pause") ne marche que sous windows.

Code:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
#include <stdio.h>
#include <time.h>
#include <stdlib.h>
 
struct piece{
	float valeur;
};
 
struct piece portefeuille[50];
struct piece machine[100];
 
void creerPortefeuilleVide(){
	int i=0;
	for(i=0;i<50;i++)
		portefeuille[i].valeur=-1;
}
 
void creerMachineVide(){
	int i=0;
	for(i=0;i<100;i++)
		machine[i].valeur=-1;
}
 
void creerMachineAleatoire(){
 
	srand(time(NULL));
	int i=0;
	int val;
 
	for(i;i<100;i++){
		val=(int)rand()%100;
		printf("\n%d\n",val);
		system("pause");
		if(val<((1.0/6.0)*100)){
			machine[i].valeur=1;
		}else{
			if(val>((1.0/6.0)*100)&&val<((2.0/6.0)*100)){
				machine[i].valeur=2;
			}else{
				if(val>((2.0/6.0)*100)&&val<((3.0/6.0)*100)){
					machine[i].valeur=0.05;
				}else{
					if(val>((3.0/6.0)*100)&&val<((4.0/6.0)*100)){
						machine[i].valeur=0.1;
					}else{
						if(val>((4.0/6.0)*100)&&val<((5.0/6.0)*100)){
							machine[i].valeur=0.2;
						}else
							machine[i].valeur=0.5;
 
					}
				}
			}
		}
	}
}
 
 
void creerPortefeuilleAleatoire(){
 
	srand(time(NULL));
	int i=0;
	int val;
 
	for(i;i<50;i++){
		val=(int)rand()%100;
		printf("\n%d\n",val);
		system("pause");
		if(val<((1.0/6.0)*100)){
			portefeuille[i].valeur=1;
		}else{
			if(val>((1.0/6.0)*100)&&val<((2.0/6.0)*100)){
				portefeuille[i].valeur=2;
			}else{
				if(val>((2.0/6.0)*100)&&val<((3.0/6.0)*100)){
					portefeuille[i].valeur=0.05;
				}else{
					if(val>((3.0/6.0)*100)&&val<((4.0/6.0)*100)){
						portefeuille[i].valeur=0.1;
					}else{
						if(val>((4.0/6.0)*100)&&val<((5.0/6.0)*100)){
							portefeuille[i].valeur=0.2;
						}else
							portefeuille[i].valeur=0.5;
 
					}
				}
			}
		}
	}
}
 
 
void addPiece(float v){
	int i=0;
	while((portefeuille[i].valeur)!=-1)
		i++;
	portefeuille[i].valeur=v;
}
 
int depensePiece(float v){
	int i=0;
	while((portefeuille[i].valeur)!=v){
		i++;
		if(i==50)
			return 0;
	}
	portefeuille[i-1].valeur=-1;
	return 1;
}
 
void rendrePiece(float rendu){
 
	//while(((rendu%2)>=1)&&(rechercherPiece(2.0))){
	int a=rechercherPiece(2.0);
	/*while(((((int)(rendu*100.0))%200)>=1)&&(rechercherPiece(2.0))){
		//if(retirerPieceMachine(2))
		rendu-=2.0;
	}	*/
}
int rechercherPiece(float p){
	int i,ret=0;
	for(i=0;i<100;i++){
		if(machine[i].valeur==p)
			ret++;
	}
	return ret;
}

Citation:

Envoyé par Jhulk

#include <stdio.h>

Pourquoi ne lis-tu pas les regles du forum ?
Pour ne mets-tu pas les balises code ?
C'est fatiguant...

Voici du code compilable. A toit de faire le code de test...

Code:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
 
#include <stdio.h>
#include <time.h>
#include <stdlib.h>
 
/* -ed- ajouts... */
#include <float.h>
#include <math.h>
 
struct piece
{
   /* -ed- tout en double (plus de precision, format par defaut...) */
   double valeur;
};
 
struct piece portefeuille[50];
struct piece machine[100];
 
/* -ed- ajoute pour la comparaison des flottants */
static int feq(double a, double b)
{
   return fabs(a - b) < DBL_EPSILON;
}
 
 
/* -ed- manque 'void' */
static void creerPortefeuilleVide(void)
{
   int i = 0;
   for (i = 0;i < 50;i++)
      portefeuille[i].valeur = -1;
}
 
/* -ed- manque 'void' */
static void creerMachineVide(void)
{
   int i = 0;
   for (i = 0;i < 100;i++)
      machine[i].valeur = -1;
}
 
/* -ed- manque 'void' */
static void creerMachineAleatoire(void)
{
 
   int i = 0;
   int val;
 
   for (i;i < 100;i++)
   {
      val = (int)rand() % 100;
      printf("\n%d\n", val);
      system("pause");
      if (val < ((1.0 / 6.0)*100))
      {
         machine[i].valeur = 1;
      }
      else
      {
         if (val > ((1.0 / 6.0)*100) && val < ((2.0 / 6.0)*100))
         {
            machine[i].valeur = 2;
         }
         else
         {
            if (val > ((2.0 / 6.0)*100) && val < ((3.0 / 6.0)*100))
            {
               machine[i].valeur = 0.05;
            }
            else
            {
               if (val > ((3.0 / 6.0)*100) && val < ((4.0 / 6.0)*100))
               {
                  machine[i].valeur = 0.1;
               }
               else
               {
                  if (val > ((4.0 / 6.0)*100) && val < ((5.0 / 6.0)*100))
                  {
                     machine[i].valeur = 0.2;
                  }
                  else
                     machine[i].valeur = 0.5;
 
               }
            }
         }
      }
   }
}
 
/* -ed- manque 'void' */
static void creerPortefeuilleAleatoire(void)
{
 
   int i = 0;
   int val;
 
   for (i;i < 50;i++)
   {
      val = (int)rand() % 100;
      printf("\n%d\n", val);
      system("pause");
      if (val < ((1.0 / 6.0)*100))
      {
         portefeuille[i].valeur = 1;
      }
      else
      {
         if (val > ((1.0 / 6.0)*100) && val < ((2.0 / 6.0)*100))
         {
            portefeuille[i].valeur = 2;
         }
         else
         {
            if (val > ((2.0 / 6.0)*100) && val < ((3.0 / 6.0)*100))
            {
               portefeuille[i].valeur = 0.05;
            }
            else
            {
               if (val > ((3.0 / 6.0)*100) && val < ((4.0 / 6.0)*100))
               {
                  portefeuille[i].valeur = 0.1;
               }
               else
               {
                  if (val > ((4.0 / 6.0)*100) && val < ((5.0 / 6.0)*100))
                  {
                     portefeuille[i].valeur = 0.2;
                  }
                  else
                     portefeuille[i].valeur = 0.5;
 
               }
            }
         }
      }
   }
}
 
 
static void addPiece(double v){
   int i = 0;
 
   /* -ed-
   while ((portefeuille[i].valeur) != -1)
   ce n'est pas comme ca que l'on compare de flottants. */
   while (!feq(portefeuille[i].valeur, -1.0))
      i++;
   portefeuille[i].valeur = v;
}
 
static int depensePiece(double v){
   int i = 0;
   while (!feq(portefeuille[i].valeur, v))
   {
      i++;
      if (i == 50)
         return 0;
   }
   portefeuille[i -1].valeur = -1;
   return 1;
}
 
/* -ed- disposition des fonctions selon le principe
 
        "definir avant d'utiliser".
 */
static int rechercherPiece(double p){
   int i, ret = 0;
   for (i = 0;i < 100;i++)
   {
      if (feq (machine[i].valeur, p))
         ret++;
   }
   return ret;
}
 
static void rendrePiece(double rendu){
 
   //while(((rendu%2)>=1)&&(rechercherPiece(2.0))){
   int a = rechercherPiece(2.0);
   /*while(((((int)(rendu*100.0))%200)>=1)&&(rechercherPiece(2.0))){
   //if(retirerPieceMachine(2))
   rendu-=2.0;
   } */
}
 
int main (void)
{
   /* -ed- ne doit etre appele qu'une fois dans l'application.
   Ajoute cast (unsigned) pour la coherence des types
   */
   srand((unsigned) time(NULL));
   return 0;
}

16/01/2006, 14h34
Jhulk

Vôalà j'ai édité, dsl
16/01/2006, 14h40
Trap D

Il y a déclaration implicite de la fonction int rechercherPiece(float p) dans, void rendrePiece(float rendu), ça a l'air de correspondre à ma deuxième remarque.
16/01/2006, 15h24
Jhulk

Citation:

Il y a déclaration implicite de la fonction int rechercherPiece(float p) dans, void rendrePiece(float rendu), ça a l'air de correspondre à ma deuxième remarque.

Heuuu j'peux avoir une tite explication pour quelqu'un de bête comme moi.
Histoire que j'ai une réponse qui m'aide à résoudre mon probleme.
Merki

Tu as oublié de déclarer le prototype de ta fonction avant de l'utiliser, c'est tout, en général, on le fait dans un fichier ou en haut du source, après les includes.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 include <stdio.h> #include <time.h> #include <stdlib.h> /* -ed- ajouts... */ #include <float.h> #include <math.h> struct piece { /* -ed- tout en double (plus de precision, format par defaut...) */ double valeur; }; struct piece portefeuille[50]; struct piece machine[100]; static void rendrePiece(double rendu);

16/01/2006, 15h40
Jhulk

Cay bon ça marche métnant j'ai changer l'ordre des fonctions, et j'ai mis des daubeules.
Merki
16/01/2006, 15h47
Trap D

Now, you can click on :resolu: