Discussion :

[Ynostel]

Bonjour, je ne trouve pas mon erreur.

L'erreur apparait dans "ThreadKnob.cpp".

J'ai comme message d'erreur "no default constructor exists for class Knob".
Je ne comprends pas :0

ThreadKnob.cpp:

Code :

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

1
2
3
4
5
6
7
8
9
#include "Knob.h"
#include "ThreadKnob.h"
 
using namespace std;
 
ThreadKnob::ThreadKnob(Knob knob) : Thread(){
 
	t_knob = knob;
}

ThreadKnob.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
#ifndef DEF_TKNOB
#define DEF_TKNOB
 
#include <iostream>
#include <string>
 
class ThreadKnob : public Thread{
public:
 
	ThreadKnob(Knob knob);
 
private:
 
	Knob t_knob;
};
 
#endif

Knob.cpp:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#include "Knob.h"
 
using namespace std;
 
Knob::Knob(int posX, int posY, int size) : ImageComponent() {
	k_posX = posX;
	k_posY = posY;
	k_size = size;
 
}

Knob.h:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class Knob : public ImageComponent{
public:
 
	Knob(int posX, int posY, int size);
 
private:
 
	int k_posX;
	int k_posY;
	int k_size;
};
 
#endif

C'est surement très bête mais bon...
Merci d'avance.

[Médinoc]

Knob n'a pas de constructeur par défaut.
Puisque ta classe ThreadKnob contient un Knob, celui-ci doit être initialisé dans la liste d'initialisation de chacun de ses constructeurs.

Code C++ :

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

1
2
3
4
5
6
7
8
#include "Knob.h"
#include "ThreadKnob.h"
 
using namespace std;
 
ThreadKnob::ThreadKnob(Knob knob) : Thread(), t_knob(knob) {
 
}

[Ynostel]

Merci

[Ynostel]

J'ai une autre erreur qui en découle par contre et je bloque depuis 1h :

error C2248: 'juce::ImageComponent::ImageComponent' : cannot access private member declared in class 'juce::ImageComponent' (ThreadKnob.cpp)

Je repost mon code :

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
#include "Knob.h"
#include "ThreadKnob.h"
 
using namespace std;
 
ThreadKnob::ThreadKnob(Knob knob) : t_knob(knob), Thread("test") {
 
	t_posX = 0;
	t_posY = 0;
}
 
void ThreadKnob::run(){
 
	this->sleep(500);
 
	t_knob.moveMark(t_posX, t_posY);
 
 
}
 
void ThreadKnob::setPoint(float x, float y){
 
	t_posX = x;
	t_posY = y;
}

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
#ifndef DEF_THREADKNOB
#define DEF_THREADKNOB
 
#include <iostream>
#include <string>
 
class ThreadKnob : public Thread{
public:
 
	Knob t_knob;
	ThreadKnob(Knob knob);
	void setPoint(float x, float y);
	void run();
 
private:
	int t_posX;
	int t_posY;
 
};
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
10
#include "Knob.h"
 
using namespace std;
 
Knob::Knob(int posX, int posY, int size) : ImageComponent() {
	k_posX = posX;
	k_posY = posY;
	k_size = size;
 
}

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
#ifndef DEF_KNOB
#define DEF_KNOB
 
#include <iostream>
#include <string>
#include "../JuceLibraryCode/JuceHeader.h"
 
class Knob : public ImageComponent{
public:
 
	Knob(int posX, int posY, int size);
	void paint(Graphics &g);
	void moveMark(float x, float y);
	void setStartPoint(float x, float y);
 
private:
	int k_posX;
	int k_posY;
	int k_size;
};
 
#endif

Merci encore .

[Ynostel]

Ce serait pas un problème d'héritage avec juce::ImageComponent ??

Pourtant je l'ai déclaré en public (enfin je crois)...

[Ynostel]

J'ai trouvé c'était un problème de pointeur :

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
#include "Knob.h"
#include "ThreadKnob.h"
 
using namespace std;
 
ThreadKnob::ThreadKnob(Knob *knob) : Thread("test"){
 
	t_knob = knob;
	t_posX = 0;
	t_posY = 0;
}
 
void ThreadKnob::run(){
 
this->sleep(500);
 
t_knob->moveMark(t_posX, t_posY);
 
 
}
 
void ThreadKnob::setPoint(float x, float y){
 
t_posX = x;
t_posY = y;
}

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
#ifndef DEF_THREADKNOB
#define DEF_THREADKNOB
 
#include <iostream>
#include <string>
#include "../JuceLibraryCode/JuceHeader.h"
 
class ThreadKnob : public Thread{
public:
 
	ThreadKnob(Knob *knob);
	void setPoint(float x, float y);
	void run();
 
 
private:
 
	Knob *t_knob;
	int t_posX;
	int t_posY;
 
};
 
#endif

[Invité]

Code :

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

1
2
3
4
ThreadKnob::ThreadKnob(Knob knob) : Thread(){
 
	t_knob = knob;
}

ton t_knob, avant de faire l'opération "=", il est construit comment selon toi ?
Et ben le compilateur ne le sait pas. C'est pour cela qu'il te demande gentillement d'implémenter un constructeur par défaut pour ton objet type Knob.

Personnellement je vois trois solutions:

1) si tu veux à tout prix éviter de modifier ta classe Knob, il te suffit de remplacer ce qu'il y a avant par:

Code :

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

1
2
ThreadKnob::ThreadKnob(Knob knob) : Thread() , t_knob(knob) {
}

mais je suis pas sur de mon coup, à tester.

2) Si tu peux modifier ta classe Knob, tu peux implémenter le constructeur par défaut "Knob();"

3) la méthode que je préfère (même si ça dépend de l'évolution de ton programme): déclarer les inputs "int posX, int posY, int size" comme facultatives en mettant dans ton header:

Code :

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

1
2
 
Knob(int posX = VALEUR_PAR_DEFAUT, int posY = VALEUR_PAR_DEFAUT, int size = VALEUR_PAR_DEFAUT);

EDIT: grilled