Discussion :

[remitbo]

Salut à tous

J'ai un petit souci avec des templates que je voudrais hériter tout en spécialisant le type. Je vais essayer d'être clair.

Pour le contexte je suis en train de développer un petit accordeur numérique sous forme de AudioUnit (format de plugin audio Apple), et pour la vue (autrement dit la Gui), je vais mettre en place deux indicateurs : un qui affiche la note jouée et un autre qui affiche l'écart avec la note exacte (le pitch).

Ce qu'affichent les indicateurs dépend de données de types différents :

Code :

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

1
2
3
4
5
6
7
typedef double Pitch_t;
typedef	enum {
	kNote_La = 0,
	kNote_Sib = 1,
	...
	kNote_None = 12
} NoteID_t;

J'ai donc créer un template indicateur de la façon suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
template <class T>
class ViewIndicator
{
public:
	ViewIndicator (...);
	virtual ~ViewIndicator ();
 
	void getData();       // va chercher les données dans la partie audio du plugin
	virtual void draw();
protected:
	T mData;
};

et des classes héritées-spécialisées de la manière suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class PitchIndicator : public ViewIndicator<Pitch_t>
{
public:
	PitchIndicator	(...);
	~PitchIndicator();
 
	void draw ();
};
 
// idem pour NoteIndicator

Les fonctions draw() des classes héritées appellent d'abord ViewIndicator::draw() pour dessiner l'arrière-plan de l'afficheur puis utilise mData pour afficher ce qu'il faut.

Là ou ça se gate, c'est dans l'implémentation, notamment l'initialisation par les super-constructeurs : j'ai fais (entre autres) deux essais :

Code :

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

1
2
3
4
5
6
PitchIndicator::PitchIndicator(...)
	:	ViewIndicator(...)
 
PitchIndicator::PitchIndicator(...)
	:	ViewIndicator<Pitch_t>::ViewIndicator(...)
{}

Le premier génère une error de compilation :
error: no matching function for call to 'ViewIndicator<Pitch_t>::ViewIndicator()'
error: class 'PitchIndicator' does not have any field named 'ViewIndicator'

Le second compile mais génère des erreurs de linking comme quoi il lui manque ViewIndicateur<double> et ViewIndicateur<NoteID_t>


Je suis prêt à envisager d'autres conceptions, seulement j'aimerais garder si possible le mécanisme d'héritage qui me permet d'automatiser un certain nombre de choses (d'autant plus que j'ai l'intention d'inclure la classe ViewIndicator dans API, il faudrait donc que les utilisateurs aient le moins possible à écrire, par exemple ne surcharger que la fonction draw()).

Merci d'avance
Rémi

[Mat007]

Salut,

Il faut faire ça je dirais :

Code :

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

1
2
3
PitchIndicator::PitchIndicator(...)
	:	ViewIndicator<Pitch_t>(...)
{}

MAT.

[remitbo]

Salut,

Il faut faire ça je dirais :

Code :

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

1
2
3
PitchIndicator::PitchIndicator(...)
	:	ViewIndicator<Pitch_t>(...)
{}

MAT.

Merci c'est super, ça marche

J'ai quand même une dernière question : dans ViewIndicator, j'ai défini une fonction (publique) à laquelle j'aimerais accéder à travers une instance d'un PitchIndicator :

Code :

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

1
2
3
PitchIndicator* monPitch;
...
(ViewIndicator<Pitch_t>*) monPitch->maFonction();

là, gcc me dit : error: 'ViewIndicator<Pitch_t>' is not an accessible base of 'PitchIndicator'

pourtant j'ai bien déclaré
class PitchIndicator : public ViewIndicator<Pitch_t>

Si je ne voulais pas me prendre la tête, j'aurais pu surcharger la fonction dans PitchIndicator, mais dans l'esprit du programme elle n'a vraiment pas lieu de l'être.

Quelqu'un à une idée là dessus ?
Rémi

[Mat007]

Code :

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

1
2
3
PitchIndicator* monPitch;
...
monPitch->maFonction();

Devrait fonctionner normalement.

Par contre là ton transtypage est fait sur le retour de maFonction et non sur monPitch, il faudrait faire :

Code :

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

1
2
3
4
 
PitchIndicator* monPitch;
...
((ViewIndicator<Pitch_t>*) monPitch)->maFonction();

Mais pas besoin a priori.
C'est super moche les transtypages C-style, si vraiment tu tiens à en faire un il vaut mieux :

Code :

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

1
2
3
4
 
PitchIndicator* monPitch;
...
dynamic_cast< ViewIndicator<Pitch_t>* >( monPitch )->maFonction();

MAT.

[remitbo]

Salut,

Il faut faire ça je dirais :

Code :

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

1
2
3
PitchIndicator::PitchIndicator(...)
	:	ViewIndicator<Pitch_t>(...)
{}

MAT.

merci MAT

En fait, j'ai crié victoire un peu trop vite : ça compile bien mais j'ai les mêmes erreurs de linking que précedemment :

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
Undefined symbols:
  "ViewIndicator<double>::~ViewIndicator()", referenced from:
      PitchIndicator::~PitchIndicator()in PitchIndicator.o
      PitchIndicator::~PitchIndicator()in PitchIndicator.o
  "ViewIndicator<NoteID_t>::~ViewIndicator()", referenced from:
      NoteIndicator::NoteIndicator( ... )in NoteIndicator.o
      NoteIndicator::~NoteIndicator()in NoteIndicator.o
      NoteIndicator::~NoteIndicator()in NoteIndicator.o
      NoteIndicator::~NoteIndicator()in NoteIndicator.o
      NoteIndicator::~NoteIndicator()in NoteIndicator.o
  "ViewIndicator<double>::draw()", referenced from:
      PitchIndicator::draw()      in PitchIndicator.o
  "ViewIndicator<NoteID_t>::draw()", referenced from:
      NoteIndicator::draw()      in NoteIndicator.o
  "ViewIndicator<double>::ViewIndicator( ... )", referenced from:
      PitchIndicator::PitchIndicator( ... )in PitchIndicator.o
  "ViewIndicator<NoteID_t>::ViewIndicator( ... )", referenced from:
      NoteIndicator::NoteIndicator( ... )in NoteIndicator.o

Si je les comprend bien, les classe NoteIndicator et PitchIndicator sont incapables d'acceder aux fonctions de leur classe (ou l'éditeur de lien ne les trouve pas). c'est à dire qu'il n'a pas compilé l'implémentation de ViewIndicator<NoteID_t> et ViewIndicator<Pitch_t> ?

hesitez pas à me demander plus de code au besoin.

[Mat007]

Est-ce que par hasard ton problème ne serait pas exactement le même que celui décrit dans cette entrée de la FAQ ?

MAT.

[remitbo]

Est-ce que par hasard ton problème ne serait pas exactement le même que celui décrit dans cette entrée de la FAQ ?

MAT.

C'est exactement ça.
Merci beaucoup MAT.

[NiamorH]

Code :

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

1
2
3
4
5
6
typedef	enum {
	kNote_La = 0,
	kNote_Sib = 1,
	...
	kNote_None = 12
} NoteID_t;

En C++, on écrit les enums comme ça :

Code :

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

1
2
3
4
5
6
enum NoteID_t {
	kNote_La = 0,
	kNote_Sib = 1,
	...
	kNote_None = 12
};

[remitbo]

En C++, on écrit les enums comme ça :

Code :

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

1
2
3
4
5
6
enum NoteID_t {
	kNote_La = 0,
	kNote_Sib = 1,
	...
	kNote_None = 12
};

ok je prend note.

[poukill]

De plus, si mes souvenirs sont bons, il suffit de spécifier la première valeur de l'enum, et les autres valeurs de l'énumération ajouteront automatiquement 1.