#define pas pris en compte dans le header

Bonjour.

Je programme sur un arduino (qu'importe)

dans mon main.cpp je fais ça :

Code:

---

1 2 3 4 5 6 7 8

#define MA_CONST 5 #include "mauditheader.h"   void main() { print(MA_CONST); // Je m'attends à lire 5 (lire la suite) }

---

et dans mon mauditheader.h je met ça:

Code:

---

1 2 3 4

#ifndef MA_CONST #define MA_CONST 10 #endif

---

Ce que je ne comprends pas, c'est que je fais le #define dans mon fichier source AVANT de faire le include.
Et pourtant mon header lui pense que je n'ai jamais fais de #define MA_CONST; je ne comprends vraiment pas pourquoi...

du coup : print(MA_CONST); m'affiche 10 au lieu de m'afficher 5

pourriez vous s'il vous plait m'expliquer pourquoi et comment faire?

Cela ne devrait pas se produire. Que se passe-t-il si tu copies le code de l'en-tête directement dans le fichier source ?

Tu n'aurais pas un autre #include avant le #define MA_CONST 5, qui lui incluerait ton mauditheader.h?

Hello

Remplace tous les define de MA_CONST par:

Code:

---

1 2 3 4

#ifndef MA_CONST_GUARD #define MA_CONST_GUARD static constexpr int MA_CONST = 5; #endif

---

Et

Code:

---

1 2 3 4

#ifndef MA_CONST_GUARD #define MA_CONST_GUARD static constexpr int MA_CONST = 10; #endif

---

Puis compile en C++11 : tu verras immédiatement d'où vient le conflit s'il persiste, car une variable ne peut pas être définie deux fois, là ou un #define peut l'être plusieurs fois.

+1 jblecanard : Les constantes en #define sont à proscrire en c++. Mais je ne pense pas qu'arduino supporte le c++... si tel est le cas, ce thread n'est pas dans le bon forum ;)

Ce n'est pas très grave, il peut compiler sur une plateforme "standard" pour identifier le conflit dans le code et reporter la correction sur le code arduino.

Soit, mais s'il génère du code c++ avec des constexpr et compagnie il ne pourra peut-être pas le compiler avec le compilo de l'arduino. D'ailleurs, le fait qu'il s'agisse du compilo arduino explique peut-être les comportements zarbi. Comme tu l'as dit, il serait bien de tester ça sur un compilo standard mais en restant en C de préférence.

Merci à tous pour votre aide, j'ai eu un soucis hier soir, je ne pouvais pas revenir sur le forum...

De ce que j'ai entendu, l'arduino se code en c mais supporte le c++... sujet à approfondir... quoi qu'il en soit, étant habitué à la programmation objet j'ai choisi de travailler en c++.

Il semble que le compilateur utilisé soit gcc, mais c'est un domaine où je suis encore peu à l'aise.
J'utilise visual studio avec l'extension visualmicro

J'utilise des define car les ressources sont ultra limitées sur l'arduino (32k de ROM ; 2k de RAM ; 16MHz). Sans bien connaitre l'ATMEGA328P, je devine que l'alléger de variables au maximum ne pourra que lui faire du bien...

@jblecanard : je vais essayer de faire ce que tu dis puis reviendrai vers vous pour partager mes découvertes

Avec GCC, tu peux utilisez l'option -E qui exécute uniquement le préprocesseur.

Par exemple, chez moi :

mauditheader.h :

Code:

---

1 2 3

#ifndef MA_CONST #define MA_CONST 10 #endif

---

main.cpp :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#include <iostream>   #define MA_CONST 5 #include "mauditheader.h"   int main() { std::cout << MA_CONST << '\n'; system("pause"); return 0; }

---

En ligne de commande, je tape :

Code:

---

g++ -E main.cpp > precompilation.txt

---

Cela me crée un fichier "precompilation.txt" qui commence par le contenu de "iostream" et qui se termine par :

Code:

---

1 2 3 4 5 6

int main() { std::cout << 5 << '\n'; system("pause"); return 0; }

---

Remarque : J'ai Windows 7 et GCC 6.3.0 via MinGW.