Erreur redéfinition fonction

Version imprimable

Bonjour,
Je travaille actuellement sur un projet en c++ et je souhaite utiliser des fichiers autre que le principal, j'ai donc un fichier .h et un fichier .cpp. Mais le problème c'est quand je compile le tout j'ai un message d'erreur :

Code:

1
2
3
 
error: redefinition of `int clavier()'
error: `int clavier()' previously defined here

or les numéros de lignes sont les mêmes et correspondent à une accolade ????
Le moteur.h

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
 
#pragma once
#ifndef MOTEUR_H
#define MOTEUR_H
 
 
#include <GL/gl.h>
#include <SDL/SDL.h>
 
double anglez = 0;
double anglex = 0;
double posx = 0.5;
double posy = 0.5;
double posz = -1;
 
bool tourner = true;
bool deplac = true;
    SDL_Event event;
int camera=1;
 
int clavier();
 
#include "moteur.cpp"
 
#endif //MOTEUR_H

Le moteur .cpp

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
 
#include "moteur.h"
#include <SDL/SDL.h>
#include <GL/gl.h>
#include <GL/glu.h>
#include <cstdlib>
int clavier()
{
    while (SDL_PollEvent(&event))
        {
 
            switch(event.type)
            {
                case SDL_QUIT:
                break;
                case SDL_KEYDOWN:
                switch (event.key.keysym.sym)
                {
                    case SDLK_ESCAPE:
                    exit(0);
                    break;
                    case SDLK_1:
                    camera=1;
                    break;
                    case SDLK_2:
                    camera=2;
                    break;
                    case SDLK_3:
                    camera=3;
                    break;
                    case SDLK_t:
                    if (!tourner)
                    {
                    tourner = true;
                    break;
                    }
                    if(tourner)
                    {
                    tourner = false;
                    anglez = 0;
                    anglex = 0;
                    break;
                    }
                    case SDLK_UP:
                    if ((event.key.keysym.mod & KMOD_LSHIFT) == KMOD_LSHIFT)
                    {
                        posx--;
                        if (posx<-19)
                        posx=-19;
                    }
                    break;
                    case SDLK_DOWN:
                    if ((event.key.keysym.mod & KMOD_LSHIFT) == KMOD_LSHIFT)
                    {
                        posx++;
                        if (posx>19)
                        posx=19;
                    }
                    break;
                    case SDLK_LEFT:
                    if ((event.key.keysym.mod & KMOD_LSHIFT) == KMOD_LSHIFT)
                    {
                        posy--;
                        if (posy<-19)
                        posy=-19;
                    }
                    else
                    {
                        anglez++;
                    }
                    break;
                    case SDLK_RIGHT:
                    if ((event.key.keysym.mod & KMOD_LSHIFT) == KMOD_LSHIFT)
                    {
                        posy++;
                        if (posy>19)
                        posy=19;
                    }
                    else
                    {
                        anglez--;
                    }
                    break;
                }
                break;
            }
        }
        return 0;
}

et le main.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
#include <SDL/SDL.h>
#include <GL/gl.h>
#include <GL/glu.h>
#include <cstdlib>
#include <string> //Permet de créer des chaines de caracteres
#include <fstream> //Permet la manipulation des fichiers
#include <conio.h>
#include "sdlglutils.h"
#include "moteur.h"
 
void Dessiner();
void importobjmodif(const char * coordfile, const char * ordrefile);
void repere();

tout du moins car le code est trop important

C'est pas la première fois que j'ai cette erreur, donc si vous avez des idées merci de bien vouloir m'aider.

29/08/2010, 14h48
Invité

Tu as sûrement à un autre endroit de ton code déclaré une fonction "int clavier();" En C, ce n'est pas possible, en C++ pour surcharger une fonction il faut qu'elles aient une signature différente.
29/08/2010, 15h11
Invité

Le problème c'est que je l'ai déclaré nul part ailleurs :cry::cry::cry:
De plus la ligne 7 où sont signalés les deux erreurs c'est une accolade :calim2:
Merci pour votre reponse
29/08/2010, 15h21
Invité

Pourquoi y-a-t-il

Code:

#include "moteur.cpp"

dans le fichier d'entête?
D'abord, on ne met pas de fichier .cpp en include, et puis, un include veut dit au compilateur "J'ai la flemme de recopier des tas de déclarations, alors t'as qu'à aller les chercher toi-même".
Alors, regardez bien ce qui se passe et vous comprendrez pourquoi le compilateur n'est pas content.

Merci pour votre réponse
J'ai retiré le :

Code:

#include "moteur.cpp"

Mais maintenant quand je compile je recoit un message d'erreur :
It seems that this project has not been buit yet. Do you want to built it now ?
Je répond oui, et il ne se passe rien à part cela dans le build log :

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
 
-------------- Build: default in SDL GL Intro ---------------
 
Linking executable: sdlgl.exe
.objs\main.o:main.cpp:(.bss+0x0): multiple definition of `_anglez'
.objs\moteur.o:moteur.cpp:(.bss+0x0): first defined here
.objs\main.o:main.cpp:(.bss+0x8): multiple definition of `_anglex'
.objs\moteur.o:moteur.cpp:(.bss+0x8): first defined here
.objs\main.o:main.cpp:(.data+0x0): multiple definition of `_posx'
.objs\moteur.o:moteur.cpp:(.data+0x0): first defined here
.objs\main.o:main.cpp:(.data+0x8): multiple definition of `_posy'
.objs\moteur.o:moteur.cpp:(.data+0x8): first defined here
.objs\main.o:main.cpp:(.data+0x10): multiple definition of `_posz'
.objs\moteur.o:moteur.cpp:(.data+0x10): first defined here
.objs\main.o:main.cpp:(.data+0x18): multiple definition of `_tourner'
.objs\moteur.o:moteur.cpp:(.data+0x18): first defined here
.objs\main.o:main.cpp:(.data+0x19): multiple definition of `_deplac'
.objs\moteur.o:moteur.cpp:(.data+0x19): first defined here
.objs\main.o:main.cpp:(.bss+0x10): multiple definition of `_event'
.objs\moteur.o:moteur.cpp:(.bss+0x10): first defined here
.objs\main.o:main.cpp:(.data+0x1c): multiple definition of `_camera'
.objs\moteur.o:moteur.cpp:(.data+0x1c): first defined here
collect2: ld returned 1 exit status
Process terminated with status 1 (0 minutes, 0 seconds)
0 errors, 0 warnings

Je me trouve sous code blocks

A première vue, les variables indiquées sont des variables globales (présence du suffixe '_').
Si j'ai bien vu, cela correspond à ces déclarations.
Code:

1 2 3 4 5 6 7 8 9 10 double anglez = 0; double anglex = 0; double posx = 0.5; double posy = 0.5; double posz = -1; bool tourner = true; bool deplac = true; SDL_Event event; int camera=1;
De la même façon qu'on ne doit pas mettre d'include de .cpp, on ne doit pas initialise des variables dans un .h.
Si on les déclare hors d'une structure ou d'une classe, elle sont globales. Dans ce cas, elle doivent être déclarées dans la main, et dans le .cpp où elles sont utilisées, elles sont re-déclatées, mais précédées du mot-clé extern.
Il ne faut pas confondre déclaration et initialisation.
Une déclaration de variable doit être faite une fois et une seule, l'initialisation peut être faite n'importe où.
Mais naturellement rien n'interdit de faire les 2 choses sur la même ligne.

Un fichier d'entête comporte normalement :
- des include de fichiers entête
- des déclarations de structures
- des déclarations de classes
- des prototypes de fonction
On y trouve aussi des défine, et d'autres paramètres de compilation.
Code:

1 2 3 4 5 if (posy<-19) posy=-19; // ce genre de chose s'écrit habituellement posy = max(-19, posy); // min et max sont des macro de cette façon c'est plus clair à relire.

Merci
J'ai suivi vos indications mais je rencontre toujours la même erreur
moteur.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
 
extern double anglez = 0;
extern double anglex = 0;
extern double posx = 0.5;
extern double posy = 0.5;
extern double posz = -1;
 
extern bool tourner = true;
extern bool deplac = true;
    SDL_Event event;
extern int camera=1;

et main.cpp

Code:

1
2
3
4
5
6
7
8
9
10
double anglez = 0;
double anglex = 0;
double posx = 0.5;
double posy = 0.5;
double posz = -1;
 
bool tourner = true;
bool deplac = true;
SDL_Event event;
int camera=1;

29/08/2010, 19h17
Invité

Oui, mais si je peux pas voir le reste du code, j'aurais du mal à trouver l'erreur.
De toute façon, si vous initialisez ces valeurs dans le main, c'est une erreur de logique de les initialiser aussi ailleurs.
Ca n'empèche pas de les modifier, mais après le mot clé extern, ça chagrine.
Il est possible et même probable que ces valeurs soient déclarées ailleurs, sans mot clé extern.
Pourquoi pas la variable event aussi en extern?
les variables posx et posy sont des doubles ou des int ? Quand vous faites poxy++, on s'attend généralement à ce qu'il s'agisse d'un int. Il est vrai que vous les initialisez en flottant.

Citation:
Envoyé par Lord Nazdar
Merci pour votre réponse
J'ai retiré le :

Code:

#include "moteur.cpp"

Mais maintenant quand je compile je recoit un message d'erreur :
It seems that this project has not been buit yet. Do you want to built it now ?
Je répond oui, et il ne se passe rien à part cela dans le build log :
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 -------------- Build: default in SDL GL Intro --------------- Linking executable: sdlgl.exe .objs\main.o:main.cpp:(.bss+0x0): multiple definition of `_anglez' .objs\moteur.o:moteur.cpp:(.bss+0x0): first defined here .objs\main.o:main.cpp:(.bss+0x8): multiple definition of `_anglex' .objs\moteur.o:moteur.cpp:(.bss+0x8): first defined here .objs\main.o:main.cpp:(.data+0x0): multiple definition of `_posx' .objs\moteur.o:moteur.cpp:(.data+0x0): first defined here .objs\main.o:main.cpp:(.data+0x8): multiple definition of `_posy' .objs\moteur.o:moteur.cpp:(.data+0x8): first defined here .objs\main.o:main.cpp:(.data+0x10): multiple definition of `_posz' .objs\moteur.o:moteur.cpp:(.data+0x10): first defined here .objs\main.o:main.cpp:(.data+0x18): multiple definition of `_tourner' .objs\moteur.o:moteur.cpp:(.data+0x18): first defined here .objs\main.o:main.cpp:(.data+0x19): multiple definition of `_deplac' .objs\moteur.o:moteur.cpp:(.data+0x19): first defined here .objs\main.o:main.cpp:(.bss+0x10): multiple definition of `_event' .objs\moteur.o:moteur.cpp:(.bss+0x10): first defined here .objs\main.o:main.cpp:(.data+0x1c): multiple definition of `_camera' .objs\moteur.o:moteur.cpp:(.data+0x1c): first defined here collect2: ld returned 1 exit status Process terminated with status 1 (0 minutes, 0 seconds) 0 errors, 0 warnings
Je me trouve sous code blocks
Et pour cause...

Quand on lit le code de moteur.h, on se rend compte que tu défini les variables en question aux lignes
Code:

1 2 3 4 5 6 7 8 9 10 double anglez = 0; double anglex = 0; double posx = 0.5; double posy = 0.5; double posz = -1; bool tourner = true; bool deplac = true; SDL_Event event; int camera=1;
Le problème, c'est que le préprocesseur est quelque chose de "bête et méchant" qui fait exactement ce que tu lui demande, et que tous les fichiers d'implémentations sont évalués un à un.

Entre le moment où ton compilateur s'attaque à (par exemple) moteur.cpp et celui où il s'attaque à main.cpp, il a "oublié" tout ce qu'il a fait pour moteur.cpp, et tous les symboles qu'il y a rencontrés.

Les gardes anti inclusion multiple (#ifndef MACHIN_CHOSE #define MACHIN_CHOSE ... #endif) ne peuvent fonctionner que lorsque tu essaye d'inclure, par le jeu des inclusions en cascade, plusieurs fois le même fichier dans un seul et unique fichier d'implémentation ;)

Pour créer des variables globales, il faut:
1- déclarer (sans les définir) les variables en les spécifiant extern (ou static selon le cas)
2- les définir (leur donner la valeur qui t'intéresse) dans un seul fichier d'implémentatin (*.cpp).

Au final, le code correct serait donc proche de
moteur.hpp
Code:

1 2 3 4 5 6 7 8 9 10 extern double anglez; extern double anglex; extern double posx; extern double posy; extern double posz ; extern bool tourner; extern bool deplac ; extern SDL_Event event; extern int camera;
moteur.cpp
Code:

1 2 3 4 5 6 7 8 9 10 double anglez = 0; double anglex = 0; double posx = 0.5; double posy = 0.5; double posz = -1; bool tourner = true; bool deplac = true; SDL_Event event; int camera=1;

31/08/2010, 08h10
Invité

Merci
Sans vouloir crier au loup je pense que votre réponse à résolu mon problème, puis je garder ce topic encore une journée que je m'assure que ce n'est pas une illusion ?
31/08/2010, 14h01
koala01

Citation:

Envoyé par Lord Nazdar

Merci
Sans vouloir crier au loup je pense que votre réponse à résolu mon problème, puis je garder ce topic encore une journée que je m'assure que ce n'est pas une illusion ?

Il n'y a pas de raison que ce soit une illusion... je ne suis pas Copperfield :D

Mais, si tu le souhaites, tu peux laisser la discussion ouverte encore quelques heures :D