problème de fonctions globales

bonjour,

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

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:

---

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:

---

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:

---

1 2 3 4 5

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

---

warning lors de l'appel :

Citation:

---

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

---

je comprend pas... :roll:

Citation:

---

Envoyé par Estak

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:

---

1 2 3 4 5 6

/* en-tete PlateformeJeu.h par Estak */ #include "PlateformeJeu.h"

---

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

---

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 ?

Citation:

---

Envoyé par Elstak

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:

---

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.

Citation:

---

Envoyé par Elstak

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

Code:

---

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.

Citation:

---

Envoyé par Elstak

et mon main :

Code:

---

1 2 3 4

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

---

warning lors de l'appel :

Citation:

---

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

---

je comprend pas... :roll:

---

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 ?

Citation:

---

Envoyé par kromartien

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.

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

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 (:oops:)

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

plateforme.c

Code:

---

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:

---

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:

---

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

---

j'ai 2 warning sur cette ligne :

Citation:

---

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

Citation:

---

Envoyé par Elstak

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

Citation:

---

Envoyé par Elstak

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

---

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

ok,

mais ce n'est donc pas possible de passer un parametre en moins dans un fonction en c ?

car là je voudrait ne rien lui donner au niveau de ce parametre pour cet appel (mais je passerai un parametre de type SDL_Eventtype lors de prochains appels)
comment faire?:roll:

Citation:

---

Envoyé par Elstak

mais ce n'est donc pas possible de passer un parametre en moins dans un fonction en c ?

---

Non ce n'est pas possible en C

Citation:

---

Envoyé par Elstak

car là je voudrait ne rien lui donner au niveau de ce parametre pour cet appel (mais je passerai un parametre de type SDL_Eventtype lors de prochains appels)
comment faire?:roll:

---

Il suffit d'utiliser une valeur particuliere de ce parametre signifiant que ce parametre est absent et non significatif, puis de traiter ce cas particulier dans la fonction.
C'est grosso modo ce que fait NULL dans le cadre des pointeurs.

ok merci !