Discussion :

[monster07]

Bonjour,

quand je compile mon code voici les messages d'erreurs que le compilateur m'affiche :

gcc -std=gnu99 -Wall  -o Vie main.o io.o grille.o jeu.o
io.o*: Dans la fonction «*affiche_grille_bis*»*:
io.c:(.text+0x149)*: définitions multiples de «*affiche_grille_bis*»
main.o:main.c:(.text+0x5b2)*: défini pour la première fois ici
io.o*: Dans la fonction «*efface_grille*»*:
io.c:(.text+0x1d0)*: définitions multiples de «*efface_grille*»
main.o:main.c:(.text+0x639)*: défini pour la première fois ici
io.o*: Dans la fonction «*debut_jeu*»*:
io.c:(.text+0x1fd)*: définitions multiples de «*debut_jeu*»
main.o:main.c:(.text+0x666)*: défini pour la première fois ici
grille.o*: Dans la fonction «*set_vivante*»*:
grille.c:(.text+0x0)*: définitions multiples de «*set_vivante*»
main.o:main.c:(.text+0x0)*: défini pour la première fois ici
grille.o*: Dans la fonction «*set_morte*»*:
grille.c:(.text+0x3c)*: définitions multiples de «*set_morte*»
main.o:main.c:(.text+0x3c)*: défini pour la première fois ici
grille.o*: Dans la fonction «*set_nonviable*»*:
grille.c:(.text+0x78)*: définitions multiples de «*set_nonviable*»
main.o:main.c:(.text+0x78)*: défini pour la première fois ici
grille.o*: Dans la fonction «*est_vivante*»*:
grille.c:(.text+0xb4)*: définitions multiples de «*est_vivante*»
main.o:main.c:(.text+0xb4)*: défini pour la première fois ici
grille.o*: Dans la fonction «*alloue_grille*»*:
grille.c:(.text+0xf0)*: définitions multiples de «*alloue_grille*»
main.o:main.c:(.text+0x909)*: défini pour la première fois ici
grille.o*: Dans la fonction «*libere_grille*»*:
grille.c:(.text+0x1e3)*: définitions multiples de «*libere_grille*»
main.o:main.c:(.text+0x9fc)*: défini pour la première fois ici
grille.o*: Dans la fonction «*init_grille_from_file*»*:
grille.c:(.text+0x23a)*: définitions multiples de «*init_grille_from_file*»
main.o:main.c:(.text+0xa53)*: défini pour la première fois ici
grille.o*: Dans la fonction «*copie_grille*»*:
grille.c:(.text+0x449)*: définitions multiples de «*copie_grille*»
main.o:main.c:(.text+0xc62)*: défini pour la première fois ici
jeu.o*: Dans la fonction «*valeur_cyclique*»*:
jeu.c:(.text+0x0)*: définitions multiples de «*valeur_cyclique*»
main.o:main.c:(.text+0xf0)*: défini pour la première fois ici
jeu.o*: Dans la fonction «*compte_voisins_vivants_c*»*:
jeu.c:(.text+0x37)*: définitions multiples de «*compte_voisins_vivants_c*»
main.o:main.c:(.text+0x127)*: défini pour la première fois ici
jeu.o*: Dans la fonction «*compte_voisins_vivants_nc*»*:
jeu.c:(.text+0xf6)*: définitions multiples de «*compte_voisins_vivants_nc*»
main.o:main.c:(.text+0x1e6)*: défini pour la première fois ici
jeu.o*: Dans la fonction «*evolue*»*:
jeu.c:(.text+0x1b7)*: définitions multiples de «*evolue*»
main.o:main.c:(.text+0x2a7)*: défini pour la première fois ici
main.o*: Dans la fonction «*main*»*:
main.c:(.text+0xda7)*: référence indéfinie vers «*affiche_grille*»
collect2: error: ld returned 1 exit status

J'ai mis en joint le code. J'ai pourtant inclus toutes les fonction.c dans l'include, je ne comprends pas d'où vienne ces erreurs ?

[-Eks-]

Salut monster07,

Les erreurs de définitions multiples sont dues au fait que dans main.c, tu inclues des fichiers .c (en plus de fichiers .h qui leurs correspondent).

Tu dois supprimer les lignes suivantes de ton main.c :

Code :

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

1
2
3
#include "jeu.c"
#include "io.c"
#include "grille.c"

Cela réglera toutes les erreurs de définitions multiples.

En principe, tu ne dois jamais faire de include d'un fichier .c, seulement les .h

Cela ne réglera pas la dernière erreur

référence indéfinie vers «*affiche_grille*»

dans laquelle gcc se plaint de ce qu'il ne trouve pas l'implémentation de la fonction affiche_grille(), qui est appelée dans main.c

En observant le code, on voit que le prototype de cette fonction se trouve bien dans io.h, cependant l'implémentation de affiche_grille() est absente de io.c (et n'est pas ailleurs non plus).

Tu dois l'implémenter conformément à son prototype pour que ce code puisse produire un exécutable.

Il y a aussi une fonction affiche_grille_bis() dont le prototype est dans io.h et qui est implémentée dans io.c. Elle a un prototype différent en revanche. Si elle correspond à ton besoin, ton code compilerait en remplaçant dans main.c l'appel de affiche_grille() par un appel de affiche_grille_bis() et en passant les paramètres que cette fonction attend.

en utilisant :

Code :

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

void affiche_grille_bis(const grille *g, unsigned int evol,int vieillissement);

au lieu de :

Code :

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

void affiche_grille (const grille *g);

Note que j'ignore si tout cela a du sens ... tu dois savoir mieux que moi d'où vient ce code et ce qu'il fait ;-)

Je n'ai pas analysé le code plus de cela.. sinon ton Makefile est un peu à côté de ses pompes aussi, mais ce n'est pas le plus grave :-)

[monster07]

Bonsoir! Merci beaucoup pour ta réponse c'est beaucoup plus clair maintenant! Mais je n'ai pas compris pourquoi affiche_grille() doit être implémenté dans io.c ?

[Sve@r]

Bonjour

Mais je n'ai pas compris pourquoi affiche_grille() doit être implémenté dans io.c ?

Si tu appelles une fonction, il faut alors que la fonction soit implémentée quelque part. Or son prototype se trouve dans "io.h". Par équivalence, son implémentation se fera donc dans...

[monster07]

Ok! Vous êtes sur que c'est ça l'erreur ? Parce que je l'ai implémenté dans io.c mais il m'affiche toujours la même erreur...

[-Eks-]

Ok! Vous êtes sur que c'est ça l'erreur ? Parce que je l'ai implémenté dans io.c mais il m'affiche toujours la même erreur...

pas dans le io.c que tu as attaché à ton message d'origine en tout cas :

Code :

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

1
2
3
4
5
6
$ grep -n "affiche_grille" io.c 
51:void affiche_grille_bis(const grille *g, unsigned int evol,int vieillissement) {
87:            affiche_grille_bis(gc,evol,vieillissement);
113:        affiche_grille_bis(&osci,temps,vieillissement);
125:              affiche_grille_bis(&osci,temps,vieillissement);
$