Discussion :

[elraton]

Bonjour,

vu que je commence a comprendre comment marche wxWidget et qu'il faut éviter les declaration de wxWindow* en tant que membre de class (evité les duplications), j'ai essayé de récuperer des valeurs d'une wxCheckBoxList en utilisant FindWindow.

Le probleme c'est que visiblement la wxWindow* que je recupere n'est pas du type wxCheckBoxList, et je ne comprend pas pourquoi...

qqun aurait une idée d'ou vient mon probleme ??

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
 
void MyFrame::initComponent() {
 
wxPanel *panel = new wxPanel(this);
 
...
 
wxPanel *subpanel = new wxPanel(panel);
wxCheckListBox *clb = new wxCheckListBox(subpanel,ID_CTRL_CAM_ON, wxDefaultPosition, wxDefaultSize,strings, wxLB_MULTIPLE);
Connect(ID_CTRL_CAM_ON,wxEVT_COMMAND_CHECKLISTBOX_TOGGLED,wxCommandEventHandler(MyFrame::OnEnableCam));
 
}
 
 
void MyFrame::OnEnableCam(wxCommandEvent& event) {
	wxWindow *wintmp = this->FindWindow(ID_CTRL_CAM_ON);
	if(wintmp->IsKindOf(CLASSINFO(wxCheckListBox))) { ##Always false
		wxCheckListBox* clb = (wxCheckListBox*)wintmp;
                clb->IsChecked(0); ## If we don't check the KindOf we have an error here
        }
}

[Médinoc]

Es-tu sûr au moins que FindWindow n'a pas retourné NULL ?

[Ti-R]

Beaucoup plus simple, il faut simplement tester:

Code :

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

event.IsChecked()

et qu'il faut éviter les declaration de wxWindow* en tant que membre de class (evité les duplications)

Non c'est pas ce qui a été dit, dans ta classe tu peux stocker tes pointeurs sans aucun problème, ils seront toujours valide du début à la fin, sauf si tu détruis ton composant !!!
Ce qui parait très logique, le pointeur pointera un espace invalide si le composant a été détruit !

[elraton]

1. J'ai bien verifier le pointeur wxWindow *wintmp n'est pas NULL.
2. event.IsChecked(), ca marche mais ca me permet pas de savoir lequel des CheckBox est checker dans la checkListBox

[Ti-R]

Le mieux c'est de créer une map.

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
27
28
29
30
31
32
 
//.h
class MyFrame
{
...
std::map<int, wxWindow*> mAllComponents;
...
}
 
//.cpp
// Pour chaque composant on l'ajoute dans une map
// On peut faire des maps séparées, on peut s'organiser comme on veut
void CreateComponents()
{
...
wxWindow *tNewComponent = new wxCheckListBox(...);
mAllComponents[tNewComponent->GetId()] = tNewComponent ;
...
}
 
// dans l'event
void MyFrame::OnEnableCam(wxCommandEvent& event)
{
std::map<int, wxWindow*>::iterator itComponent;
if((itComponent = mAllComponents.find(event.GetId())) != mAllComponents.end())
{
itComponent->GetName();
itComponent->GetId()
...
   event.IsChecked()
}
}

Lorsqu'un composant est détruit, il suffit de mettre à jour la map.

[3DArchi]

Sauf que là tu fais de la redondance avec wxWidget. Et tu risques d'avoir un désynchronisme entre ta map et les composants. C'est pour ça qu'à mon avis, il vaut mieux s'appuyer sur wxWidget pour obtenir les composants.
Pour récupérer le composant:event.GetId()?