Discussion :

[jjeuvrey]

Bonjour,
Je cherche à compiler mon code après rajout de ce morceau de code suivant:

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
 
#define SET_SENSOR_PARAMETER(id, type, value)
    {
    type## _TYPE buf = value;
    errorCode = SetSensorParameter(id, type, &buf, sizeof(buf));
    if(errorCode!=BIRD_ERROR_SUCCESS) ;
    }
// DATA_FORMAT_TYPE already defined as an enumerated type
    typedef DOUBLE_ANGLES_RECORD ANGLE_ALIGN_TYPE;
    typedef DOUBLE_ANGLES_RECORD REFERENCE_FRAME_TYPE;
    typedef bool XYZ_REFERENCE_FRAME_TYPE;
    // HEMISPHERE_TYPE already defined as an enumerated type
    typedef bool FILTER_AC_WIDE_NOTCH_TYPE;
    typedef bool FILTER_AC_NARROW_NOTCH_TYPE;
    typedef double FILTER_DC_ADAPTIVE_TYPE;
    typedef ADAPTIVE_PARAMETERS FILTER_ALPHA_PARAMETERS_TYPE;
    typedef bool FILTER_LARGE_CHANGE_TYPE;
    typedef QUALITY_PARAMETERS QUALITY_TYPE;

Je rajoute ce code au tout début de mon main.cpp et avant ma fonction int main () incluse dans main.cpp.

J'obtiens les erreurs suivantes:

||=== Build: Debug in RV1 (compiler: GNU GCC Compiler) ===|
C:\WORK\CPP\SDLprojet\src\main.cpp|15|error: stray '##' in program|
C:\WORK\CPP\SDLprojet\src\main.cpp|14|error: expected unqualified-id before '{' token|

J'ai bien sur tenté de faire des recherches sur les préprocesseurs et macros ainsi que sur le principe de concaténation mais j'avoue que je suis largué!
Un peu d'aide serait la bienvenue.
Merci!

[Bktero]

Bonjour,

Il est intentionnel l'espace après ## ?

Après, on est en C++. Cette macro ne pourrait-elle pas devenir une fonction template ?

[jjeuvrey]

Bonjour,
En fait cet espace ne change rien à l'erreur, j'avais essayé de mettre en espace pour voir si cela créait une différence mais ce n'est pas le cas.
Les templates effectivement c'est une idée.
merci!

[bacelar]

En fait cet espace ne change rien à l'erreur,

Alors, c'est que vous avez merdé ailleurs.
La suppression de l'espace DOIT avoir un impact.

Mais bon, en C++, on fait pas ce genre de MACRO bien cracra.

[jo_link_noir]

Pour l'erreur en elle-même, il manque les \ en fin de ligne pour faire une macro multi-ligne.

Code :

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

1
2
3
#define multiline \
  première ligne \
  seonde ligne

[jjeuvrey]

Effectivement c'est ca!
problème résolu!
merci!

[koala01]

Salut,

J'arrive un peu après la bataille, mais ce genre de macro devrait être banni de toute manière de penser en C++.

Jusqu'à présent, tu n'avais "que" le problème de la syntaxe qui empêchait le compilateur de te comprendre correctement, si bien qu'il pouvait te dire très exactement ce qui clochait.

Mais par la suite, les choses vont clairement se compliquer, ne serait-ce que parce que l'instruction preprocesseur #define fait en sorte que le symbole indiqué est remplacé "physiquement" par le texte équivalent dans le code avant même que le compilateur ne commence son travail.

Du coup, si tu as des erreur dans la macro (et le fait est que j'en vois (*), si le code que tu as donné est le même que celui de ton projet), c'est tout le code par lequel le symbole aura été remplacé qui sera pris en compte. Si bien que l'erreur affichée deviendra totalement incompréhensible. Surtout si (bien que ce ne soit pas le cas ici) les instructions du code font elle-même appel à des macros. Et, bien sur, il en ira de même lorsque tu aura besoin de débuger ton programme.

C'est d'autant plus dommage que tu pourrait créer une fonction template proche de

Code :

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

1
2
3
4
5
6
7
template <typename TYPE>
void setSensorParameter( size_t id, TYPE value){
    TYPE buf= value;
    errorCode = SetSensorParameter(id, type, &buf, sizeof(buf));
    /* (*) je reproduit le code avec l'erreur */
    if(errorCode!=BIRD_ERROR_SUCCESS) ;
}

qui pourrait être appelé sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
int main(){
    size_t idAngle = 158753;
    ANGLE_ALIGN_TYPE angleVal=3.1415926;
    setSensorParameter(idAngle, angleVal);
    size_t id frameId = 987654;
    XYZ_REFERENCE_FRAME_TYPE refFrame = true;
    setSensorParameter(frameId , refFrame );
}

et qui fonctionnerait exactement de la même manière que ta macro (avec les avantages d'avoir un paramètre en moins, et d'être directement écrite en bon C++)

Tu gagne donc en facilité d'écriture (enfin, tu gagnes autant qu'en créant une fonction) et en compréhension du code avec ta macro quand "tout va bien", mais, d'un autre coté, tu perd énormément en compréhension (entre autres des erreurs émises par le compilateur et en débuggage).

(*) Dans ton code, tu fais un test sur la valeur de errorCode, mais le ; qui suit indique que rien ne sera fait s'il n'est pas egal à BIRD_ERROR_SUCCESS, si bien qu'il ne sert strictement à rien.

Dés lors, soit tu dois faire quelque chose si la valeur de errorCode ne correspond pas à ce que tu attendais, soit tu dois envisager de faire quelque chose si elle y correspond, soit tu dois envisager de faire quelque chose dans les deux cas.

Un peu de lecture (n'hésite pas à suivre les liens vers evil#1, evil#2, evil#3 et evil#4). Malheureusement, c'est en anglais