Discussion :

[ChPr]

Bonjour à toutes et à tous,

Dans une application "Arduino", j'utilise une bibliothèque que j'ai créée (un bon millier de lignes) dont la partie congrue révélant le problème est la suivante :

fichier "BaseComp.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
#ifndef BaseComp_h
#define BaseComp_h
 
#include <Arduino.h>
 
class CompBase {
public:
	CompBase() { // constructeur
		surAppui = NULL;
	}
	void (* surAppui)(); // pointeur sur une fonction d'action en cas d'appui sur le composant
	virtual void select();
};
 
#endif

le fichier "BaseComp.cpp" :

Code :

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

1
2
3
4
5
6
7
#include <Arduino.h>
#include <BaseComp.h>
 
void CompBase::select() { // fonction virtuelle
	if (surAppui != NULL)
		surAppui();
}

Dans l'appli :

Code :

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

1
2
3
4
5
6
7
8
...
 
void maProcedure() {
  ...}
 
CompBase monComposant;
...
  monComposant.surAppui = &maProcedure

Qu'est-ce que "BaseComp ? C'est l'équivalent d'une bibliothèque de composants visuels qui répondent à des évènements. Dans mon cas, c'est un appui sur l'écran tactile qui est détecté via la méthode "select()" et qui, si un méthode a été attachée à la variable "surAppui", va effectuer ce que doit faire cette méthode (ici, le méthode "maProcedure()")


Le problème semble venir de l'affectation de la méthode "maProcedure" à la variable "surAppui" de l'instance "monComposant" de la classe "CompBase".

Pour en revenir au problème, si je n'affecte pas de méthode à la variable "surAppui", le problème disparait ... comme tout l'intérêt de ma construction.

Pour autant je subodore que le problème tourne autour de cette affectation. Vous semble t-elle correcte ? quel pourrait être le "bug" dans tout ça ?

Je vous remercie d'avoir déjà pris la peine de lire tout cela et vous remercierai encore de votre aide.

Cordialement.

Pierre

[skeud]

Salut, quel est le problème rencontré? L'appel ne fonctionne pas? segfault?

L'assignation me semble correct, ensuite tout dépends du problème que tu rencontre .

[Bousk]

salut,

ça s'apelle un pointeur sur fonction, avec ce terme tu retrouveras de très nombreux posts sur ce forum-même à ce sujet.

Je suppose que ça crash au moment d'appeler la fonction pointée ?
Puisque la syntaxe, si j'ai bonne mémoire, devrait être (*surAppui)();

[ChPr]

Salut, quel est le problème rencontré? L'appel ne fonctionne pas? segfault?

L'assignation me semble correct, ensuite tout dépends du problème que tu rencontre .

Je l'ai reporté ici car ce me semble être un problème de pure syntaxe du C++.

... Je suppose que ça crash au moment d'appeler la fonction pointée ?
Puisque la syntaxe, si j'ai bonne mémoire, devrait être (*surAppui)();

Non, ça ne crash pas à l'appel de la fonction, voir le lien donné dans la réponse précédente. Le problème rencontré me semble complètement farfelu !!! Comme si la compilation de ce lien de fonction allait écrire dans des registres dans lequel le programme se perd.

La syntaxe que tu proposes fonctionne comme celle que j'ai emplyée et le problème reste le même.

Cordialement.

Pierre

[Bousk]

Merci de poster le problème complet ici, nous ne devrions pas avoir à aller sur un lien externe à DVP. Surtout pour un "problème" aussi simple.

Si le problème est

PbConst.cpp.o: In function `majJours()':
/Arduino/PbConst.ino:10: undefined reference to `nbZones'
collect2: error: ld returned 1 exit status

Alors il suffit de lire l'erreur pour la comprendre...
Ton linker ne trouve pas ta variable déclarée extern et jamais déclarée réellement dans ton programme.
La syntaxe de foncteur est correcte, ce n'est pas du tout le problème ici.

[ChPr]

Merci de poster le problème complet ici, nous ne devrions pas avoir à aller sur un lien externe à DVP. Surtout pour un "problème" aussi simple. ...

Désolé, vous le trouverez ci-après

... Si le problème est

PbConst.cpp.o: In function `majJours()':
/Arduino/PbConst.ino:10: undefined reference to `nbZones'
collect2: error: ld returned 1 exit status

Alors il suffit de lire l'erreur pour la comprendre...
Ton linker ne trouve pas ta variable déclarée extern et jamais déclarée réellement dans ton programme.
La syntaxe de foncteur est correcte, ce n'est pas du tout le problème ici.

Effectivement, l'erreur dit bien ce que ça veut dire ... à ceci près, et je l'ai dit dans mon explication, que si je ne lie pas la méthode majCours à la variable surAppui, ou que je n'appelle pas la méthode majZN_CY, et bien ça compile sans problème. Ma déclaration extern est donc correcte. Il semble donc que ce soit l'une des deux actions citées qui viennent provoquer le problème. C'est peut-être un problème du compilateur Arduino. Je voudrais m'assurer que ce n'est pas dû à un mauvais codage de ma part.

Cordialement.

Pierre

L'application suivante est destinée à des modules "Arduino"

Dans une application constituée d'un fichier "PbConst.ino" et d'un fichier "LesPages.cpp" je souhaite pouvoir utiliser la constante "NbZones" dans chacun des deux fichiers. Pour cela, je la crée dans le fichier "LesPages.cpp" et la déclare comme "extern" dans le fichier "PbConst.ino". Cette construction fonctionne normalement. Si dans le fichier "PbConst.ino".vous mettez en commentaire soit la ligne : majZn_Cy(zN); ou la ligne : leBouton.surAppui = &majJours; alors, ça compile, sinon, j’obtiens l'erreur suivante :

Code :

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

1
2
3
PbConst.cpp.o: In function `majJours()':
/Arduino/PbConst.ino:10: undefined reference to `nbZones'
collect2: error: ld returned 1 exit status

le fichier "PbConst.ino :

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
#include <BaseComp.h>
 
extern byte nbZones;
extern void majZn_Cy(byte);
 
CompBase leBouton;
 
void majJours() {
  byte zN;
  for (byte i = 0; i < nbZones; i++)
      zN = i;
  majZn_Cy(zN);
}
 
void setup() {
  leBouton.surAppui = &majJours;
}
 
void loop() {
}

le fichier "LesPages.cpp" :

Code :

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

1
2
3
4
5
6
#include <Arduino.h>
 
const byte nbZones = 6; // Nombre de zones
 
void majZn_Cy(byte Z) {
}

Le problème semble venir de l'affectation de la méthode "majJours" à la variable "surAppui" de l'instance "leBouton" de la classe "CompBase".

"CompBase" est déclaré dans une bibliothèque que j'ai créée : "BaseComp.h" et BaseComp.cpp" qui, au complet représentent un bon millier de lignes. J'ai réussi à en extraire la partie congrue (quelques lignes seulement) qui occasionne le problème. La voici :

fichier "BaseComp.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
 
#ifndef BaseComp_h
#define BaseComp_h
 
#include <Arduino.h>
 
class CompBase {
public:
	CompBase() { // constructeur
		surAppui = NULL;
	}
	void (* surAppui)(); // pointeur sur une fonction d'action en cas d'appui sur le composant
	virtual void select();
};
 
#endif

le fichier "BaseComp.cpp" :

Code :

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

1
2
3
4
5
6
7
#include <Arduino.h>
#include <BaseComp.h>
 
void CompBase::select() { // fonction virtuelle
	if (surAppui != NULL)
		surAppui();
}

Qu'est-ce que "BaseComp ? C'est l'équivalent d'une bibliothèque de composants visuels qui répondent à des évènements. Dans mon cas, c'est un appui sur l'écran tactile qui est détecté via la méthode "select()" et qui, si un méthode a été attachée à la variable "surAppui", va effectuer ce que doit faire cette méthode (ici, le méthode "majJours()")

Pour en revenir au problème, si je n'affecte pas de méthode à la variable "surAppui", le problème disparait ... comme tout l'intérêt de ma construction.

A savoir : si je n'utilise pas la constante "nbZones" dans le fichier "PbComp.ino", tout fonctionne correctement.

Pour autant je subodore que le problème tourne autour de cette affectation. Vous semble t-elle correcte ? quel pourrait être le "bug" dans tout ça ?

Je vous remercie d'avoir déjà pris la peine de lire tout cela et vous remercierai encore de votre aide.

Cordialement.

Pierre

[foetus]

Compiles-tu le fichier LesPages.cpp et comment est-il linké?


PS: tu as loupé l'insertion de ton code: le fichier PbConst.ino: il est en 3 morceaux plus l'introduction dedans

[ChPr]

... PS: tu as loupé l'insertion de ton code: le fichier PbConst.ino: il est en 3 morceaux plus l'introduction dedans

Je me suis mélangé les crayons en faisant la copie ; c'est réparé.

Compiles-tu le fichier LesPages.cpp et comment est-il linké? ...

Les deux fichiers "PbConst.ino" et "LesPages.cpp" sont dans un même répertoire et pris en charge pas le compilateur Arduino. Je ne saurais en dire plus la-dessus. Mais je le répète, tout fonctionne bien si il n'y a pas le lien avec surAppui ou pas d'appel à la méthode majZN_CY. Le compilateur prend donc normalement en compte ces deux fichiers.

Cordialement.

Pierre

[Bousk]

Mais je le répète, tout fonctionne bien si il n'y a pas le lien avec surAppui ou pas d'appel à la méthode majZN_CY. Le compilateur prend donc normalement en compte ces deux fichiers.

Et je répète que la syntaxe du foncteur est correcte. Y'a rien de surprenant que si tu utilises pas la fonction l'erreur "disparaisse", le compilateur peut faire des optimisations, et supprimer le code inutile en fait partie.
Il n'en reste pas moins, je le répète, que le problème est cette variable.

Code :

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

1
2
extern byte nbZones;
const byte nbZones = 6;

[ChPr]

Et je répète que la syntaxe du foncteur est correcte. Y'a rien de surprenant que si tu utilises pas la fonction l'erreur "disparaisse", le compilateur peut faire des optimisations, et supprimer le code inutile en fait partie.
Il n'en reste pas moins, je le répète, que le problème est cette variable.

Code :

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

1
2
extern byte nbZones;
const byte nbZones = 6;

Je dois être vraiment bouché

Si tu dis que le problème est cette variable, alors c'est quoi le problème ?

Que dois-je écrire pour que le problème disparaisse ?

Cordialement.

Pierre

[skeud]

L'erreur est pourtant explicite:
cannot find symbol = il ne trouve pas la source de ta variable, autrement dis tu ne compile pas la source contenant la variable.
Le externe n'est la que pour signifier de ne pas recompiler l'instanciation de la variable mais pour faire comprendre au compilo qu'une variable portant ce nom est instancier dans un autre fichier.

[ChPr]

L'erreur est pourtant explicite:
cannot find symbol = il ne trouve pas la source de ta variable, autrement dis tu ne compile pas la source contenant la variable. ...

Soit, mais alors, pourquoi la trouve-t-il si je ne mets pas les lignes qui provoquent le problème ? Pour cette variable, je ne fais rien de plus, rien de moins dans ce cas ... et ça compile !

Pour moi, ces deux lignes à problème n'ont strictement rien à voir avec cette variable et pourtant, il y a interaction. C'est ce que je n'arrive pas à comprendre.

Formulé autrement : OK, le compilateur ne voit pas cette variable. Mais alors, pourquoi la voit-il dans un cas et pas dans l'autre alors que je n'ai rien modifié en ce qui concerne cette variable ?

C'est peut-être clair pour vous, mais pas pour moi.

Cordialement.

Pierre

[Bousk]

pourquoi la voit-il dans un cas et pas dans l'autre alors que je n'ai rien modifié en ce qui concerne cette variable ?

le compilateur peut faire des optimisations, et supprimer le code inutile en fait partie.

une fonction non utilisée, c'est la définition de code inutile..

[ChPr]

une fonction non utilisée, c'est la définition de code inutile..

Ben oui, ça supprime le code de la fonction non utilisée, mais je ne vois pas pourquoi ça supprimerait celui attaché à la variable à laquelle je n'ai pas touché.

J'ai transféré cette variable dans une bibliothèque et tout fonctionne, comme il se doit, normalement.

Merci pour vos suggestions.

Cordialement.

Pierre