Objet en argument

Version imprimable

Objet en argument

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:

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:

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:

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:

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.

20/01/2014, 15h38
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:

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) { }
20/01/2014, 15h39
Invité
Code:

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 ? :mrgreen:
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:

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:

1 2 Knob(int posX = VALEUR_PAR_DEFAUT, int posY = VALEUR_PAR_DEFAUT, int size = VALEUR_PAR_DEFAUT);
EDIT: grilled :aie:
20/01/2014, 15h49
Ynostel

Merci ;)

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:

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:

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:

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:

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 ;).

20/01/2014, 19h25
Ynostel

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

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

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

Code:

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:

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