Discussion :

[Elstak]

bonjour,

voilà j'ai quelques petits soucis avec des déclarations de fonctions ()

en fait j'ai un fichier PlateformeJeu.h, un fichier PlateformeJeu.c et mon main.c

dans mon fichier PlateformeJeu.h j'ai déclaré mon entete de fonction (j'ai ajouté un extern que j'avais pas au départ mais ca n'a rien changé)

Code :

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

extern int AffichageTerrain(SDL_Surface *Terrain,int Ligne,int Colonne,int **Carte);

ensuite j'ai ca dans PlateformeJeu.c (j'ai aussi rajouté le extern au cas ou mais ca n'a rien fait )

Code :

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

1
2
3
4
5
6
7
#include "PlateformeJeu.h"
 
 
extern int AffichageTerrain(SDL_Surface *Terrain,int Ligne,int Colonne,int **Carte)
{
blabla..
}

et mon main :

Code :

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

1
2
3
4
5
 
#include "PlateformeJeu.h"
 
 
tempo = AffichageTerrain(Terrain,NbLigneCarte,NbColCarte,Carte);

warning lors de l'appel :

main.c(71) : warning C4013: 'AffichageTerrain' undefined; assuming extern returning int

je comprend pas...

[kromartien]

dans mon fichier PlateformeJeu.h j'ai déclaré mon entete de fonction (j'ai ajouté un extern que j'avais pas au départ mais ca n'a rien changé)

Code :

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

1
2
3
4
5
6
 
/*
en-tete PlateformeJeu.h
par Estak
*/
#include "PlateformeJeu.h"

Vous avez donc inclus le fichier d'en tête dans le fichier d'en-tête, ça s'appelle un inclusion multiple.
Au début d'un en-tête, il faut toujours se protéger des inclusions multiples.

par exemple pour votre header, il devrait commencer par :

Code :

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

1
2
3
4
5
6
7
8
#ifndef PLATEFORMEJEU
#define PLATEFORMEJEU
 
/*
Definitions de fonctions
*/
 
#endif

De cette façon, vous êtes protégé contre les inclusions multiple, et c'est primordial quand on écrit des grands programmes de savoir se servir des instructions du préprocesseur.

Je ne comprends pas votre fonctionnement. Est-ce que la fonction 'AffichageTerrain' fait appel à d'autres fonctions définies dans votre en-tête "PlateformeJeu.h" ? Si non, il est inutile d'inclure ce header dans le fichier de la fonction. Pouvez vous expliquer la structure de votre programme ?

[gl]

Je ne comprends pas votre fonctionnement. Est-ce que la fonction 'AffichageTerrain' fait appel à d'autres fonctions définies dans votre en-tête "PlateformeJeu.h" ? Si non, il est inutile d'inclure ce header dans le fichier de la fonction. Pouvez vous expliquer la structure de votre programme ?

Si tu parles de l'inclusion de PlateformeJeu.h dans PlateformeJeu.c. Si c'est utile deja car ca permet de s'assurer que la declaration et la definition de la fonction sont bien identique ce qui peut eviter des compretements indefinis.
D'ailleurs gcc avec l'option adequate emet un warning si le fichier d'en-tete n'est pas inclus. Il attend que la fonction soit statique ou que le prototype soit visible.

En outre il est fort probable que le header contienne egalement les definitions de type, structure, enum, etc. donc autant l'inclure de maniere systematique.

[souviron34]

mais comme l'a dit gl dans PlateformeJeu.c il ne faut pas d'"extern" dans la définition de la fonction.

[Elstak]

ok,

merci de vos réponse j'ai pu modifier un peu...
en fait j'avais fait le ifndef mais j'avais fait un copier coller d'un autre fichier.h donc c'etait le meme nom apres le ifndef ()

j'ai donc corrigé ca mais il me reste toujours un problème :

plateforme.c

Code :

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

1
2
3
#include "PlateformeJeu.h"
 
int AffichageJoueur(Joueur *Joueur,int Ligne,int Colonne,int **Carte,SDL_EventType Mouvement)

plateforme.h (j'avais deja inclus la librairie SDL mais je l'avais pas repris de mon code pour mettre ici donc pas de problème à ce niveau)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef PlateformeJeu_H
#define PlateformeJeu_H
 
#include <SDL/SDL.h>
#include "Structures.h"
 
//déclaration des entetes
void pause();
extern int AffichageTerrain(SDL_Surface *Terrain,int Ligne,int Colonne,int **Carte); // VOIR CE QUE CA RENVOIT SI non succes !
extern int AffichageJoueur(Joueur *Joueur,int Ligne,int Colonne,int **Carte,SDL_EventType Mouvement);
 
#endif

dans mon main :

Code :

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

achanger=AffichageJoueur(&Player,NbLigneCarte,NbColCarte,Carte,NULL);

j'ai 2 warning sur cette ligne :

main.c(72) : warning C4047: 'function' : 'int ' differs in levels of indirection from 'void *'
main.c(72) : warning C4024: 'AffichageJoueur' : different types for formal and actual parameter 5

est ce que c'est à cause de mon NULL ? car je pense qu'on peut ecrire comme ca...(pour le second warning)

mais le premier je comprend pas pourquoi il me parle d'un void * car j'ai ca nulle part

[gl]

est ce que c'est à cause de mon NULL ? car je pense qu'on peut ecrire comme ca...(pour le second warning)

Probablement, NULL ne doit etre utilise que pour les pointeurs, pas pour autre chose

mais le premier je comprend pas pourquoi il me parle d'un void * car j'ai ca nulle part

Ben si, le NULL qui dit etre definit dans ton cas par (void*)0

[gl]

dans mon fichier PlateformeJeu.h j'ai déclaré mon entete de fonction (j'ai ajouté un extern que j'avais pas au départ mais ca n'a rien changé)

Code :

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

extern int AffichageTerrain(SDL_Surface *Terrain,int Ligne,int Colonne,int **Carte);

extern n'est pas vraiment utile, mais c'est tout a fait correct.

Par contre:
* Il n'y a pas de protection contre les inclusions multiples.
* Il manque au moins l'inclusion d'un autre fichier d'en-tete car SDL_Surface n'est pas connu.

ensuite j'ai ca dans PlateformeJeu.c (j'ai aussi rajouté le extern au cas ou mais ca n'a rien fait )

Code :

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

1
2
3
4
5
6
7
#include "PlateformeJeu.h"
 
 
extern int AffichageTerrain(SDL_Surface *Terrain,int Ligne,int Colonne,int **Carte)
{
blabla..
}

Ici il ne faut pas d'extern.

et mon main :

Code :

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

1
2
3
4
 
#include "PlateformeJeu.h"
 
tempo = AffichageTerrain(Terrain,NbLigneCarte,NbColCarte,Carte);

warning lors de l'appel :

main.c(71) : warning C4013: 'AffichageTerrain' undefined; assuming extern returning int

je comprend pas...

Dans le peu de code que tu donnes, je ne vois pas de probleme.
As-tu d'autres erreurs ou warning lors de la compilation ?
N'aurais tu pas un ifdef ou ifndef mal place qui masque le prototype ?