[Erreur] : définitions multiples de

**lefou123456** · 10/11/2017, 19h58

Bonjour !

Je suis en train de créer une bibliothèque pour la programmation Arduino.
Cependant, je n'arrive pas à la compiler et ça fait quelques jours que je recherche cette erreur...

led.h

Code :

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

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
 
#ifndef __LED_H__
#define __LED_H__
 
#include "Arduino.h"
 
class Led {
    public:
        Led(char pin, bool etat);
        ~Led();
 
        void changerEtat();
        void allumer();
        void eteindre();
 
        char getPin() const;
        void setPin(char pin);
 
        bool getEtat() const;
        void setEtat(bool etat);
 
    private:
        void mettreAJour();
        void preparerPin();
        void appliquerEtat();
 
        char m_pin;
        bool m_etat;
};
 
#endif

led.cpp

Code :

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

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
 
#include "led.h"
 
Led::Led(char pin, bool etat) {
    setPin(pin);
    setEtat(etat);
    mettreAJour();
}
 
void Led::changerEtat() {
    setEtat(1 - getEtat());
}
 
void Led::allumer() {
    setEtat(HIGH);
}
 
void Led::eteindre() {
    setEtat(LOW);
}
 
char Led::getPin() const {
    return m_pin;
}
 
void Led::setPin(char pin) {
    m_pin = pin;
    mettreAJour();
}
 
bool Led::getEtat() const {
    return m_etat;
}
 
void Led::setEtat(bool etat) {
    m_etat = etat;
    mettreAJour();
}
 
/* ######### PRIVATE ######### */
 
void Led::mettreAJour() {
    preparerPin();
    appliquerEtat();
}
 
void Led::preparerPin() {
    pinMode(m_pin, OUTPUT);
}
 
void Led::appliquerEtat() {
    digitalWrite(m_pin, m_etat);
}

LIFINO.ino

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include <led.h>
 
Led led(13, LOW); //LOW = 1
 
void setup() {}
 
void loop() {
    led.changerEtat();
    delay(500);
}

Erreurs :

LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::getPin() const*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::getEtat() const*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::preparerPin()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::appliquerEtat()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::mettreAJour()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::setPin(char)*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::setEtat(bool)*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::Led(char, bool)*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::Led(char, bool)*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::changerEtat()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::allumer()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*:
/home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::eteindre()*»
led.cpp.o:led.cpp:49*: défini pour la première fois ici
LIFINO.cpp.o*: Dans la fonction «*_GLOBAL__sub_D_led*»*:
/usr/share/arduino/LIFINO.ino:3*: référence indéfinie vers «*Led::~Led()*»
collect2: error: ld returned 1 exit status

J'ai d'abord pensé à une erreur de directive préprocesseur, mais je n'ai rien trouvé de ce côté-ci...
Merci beaucoup de l'attention !

**oodini** · 13/11/2017, 09h46

C'est comme si quelque part, tu incluais le .cpp au lieu du .h.

**ternel** · 13/11/2017, 16h29

éventuellement, ca peut arriver si le même .cpp est présent deux fois dans la liste des .cpp à compiler.
Ca peut arriver avec un makefile tordu, par exemple

**lefou123456** · 15/11/2017, 09h52

Je viens de trouver la raison; en fait, j'ai marqué dans LIFINO.ino :

Code :

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

#include <led.h>

Donc ça va chercher le fichier dans les librairies de Arduino.
Mais il n'a pas l'air d'aimer car led.h est dans le même fichier que LIFINO.ino...
Il me suffit alors d'écrire à la place :

Code :

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

#include "led.h"

Cela dit, je ne comprend pas pourquoi il relit les prototype; il y a quand même la condition

:

Code :

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

1
2
3
4
5
 
#ifndef __LED_H__
#define __LED_H__
[...]
#endif

Mais ça marche maintenant, merci de vos réponses !

**bacelar** · 15/11/2017, 12h47

Le header-guard fonctionne très bien.
Il ne sert qu'à éviter d'inclure plusieurs fois le même header lors de la COMPILATION d'un fichier source .cpp.
Si vous compilez plusieurs .cpp, chacun inclura son "propre fichier header".
C'est pour cela qu'on ne met que des déclarations dans un fichier d'en-tête et pas de définition.

**Bousk** · 15/11/2017, 13h08

Ta conclusion colle difficilement à ton problème. Et ta solution encore moins.

Le .h que tu montres plus haut ne contient que des déclarations et a un header guard. Ce fichier tel que présenté ici est tout à fait valide et ne génèrera aucune erreur.
L'erreur de définition multiple est une erreur de link. Tout indique que le cpp est compilé plusieurs fois.
L'erreur classique du débutant étant un include du fichier cpp.
Ou une mauvaise condifuration qui entraîne la compilation du même fichier plusieurs fois.

Et si tu regardes un peu l'erreur, tout indique que le problème se trouve dans LED::eteindre() LIFINO/led.cpp.o*: Dans la fonction «*Led::eteindre()*»*: /home/louis/sketchbook/libraries/LIFINO/led.cpp:49*: définitions multiples de «*Led::getPin() const*» led.cpp.o:led.cpp:49*: défini pour la première fois ici et la ligne 49 revient toujours.
Sans compter la référence indéfinie vers «*Led::~Led()*».

**lefou123456** · 15/11/2017, 13h45

Je n'avais pas très bien compris la méthode de l'IDE Arduino, c'est pour cela que j'ai fais une erreur.
Cela dit, l'erreur concernant le constructeur n'a pas l'air fondée, et j'ai pensé à une indication erronée du compilateur...

**ternel** · 15/11/2017, 14h09

Non, ce n'est pas le constructeur dont il est question, mais le destructeur.
Par contre, ton compilateur a toujours raison. S'il rapporte une erreur, c'est que ce que tu lui demandes est faux.
Ca peut venir de la manière de lui demander (les options de compilations) ou du code à compiler.

En l'occurence, il doit manquer la définition de ~Led()