Delete de composant dans un GroupBox

Version imprimable

29/04/2006, 22h22
maw

Delete de composant dans un GroupBox
bonsoir,
voici mon pb.
J'insere dynamique des composants dans un group box et je les supprime, pour l'ajout ca va. Pour la suppression j'utilise le code suivant :
Code:

1 2 3 4 5 6 7 8 TControl *temp; int nbControl = gbCat->ControlCount; for(int i= nbControl ; i>0;i--){ temp = gbCat->Controls[i-1]; gbCat->RemoveControl(temp); }
Ca marche mais c lent. Avez vous une autre solution plus simple ou plus rapide. J'ai pas mal cherché mais pas trouvé de fonction qui remove tout les controles d'un coup.
Merci.
30/04/2006, 17h53
bandit boy

Salut,
T'as essayé le delete ?
30/04/2006, 18h54
DjmSoftware
bonjour,
dans la construction dynamique des composants contenu dans le groupBox,
il te suffit d'affecter la property Owner des composants crés avec le groupBox

lors de la destruction de la GroupBox tous les composants contenu dans ce container seront automatiquement détruit
exemple
Code:

1 2 3 4 5 6 grpBox; // Pointeur sur ta groupeBox crée statiquement TLabel *mlabel= new TLabel(grpBox); // creation d'un object Label TButton* mbuton = new TButton(grpBox);
Cordialement
30/04/2006, 21h57
maw

oui mais non :) car je ne delete pas le groupbox, je delete uniquement les controles que j'ai placé dedans.
01/05/2006, 07h28
Gilles Louïse
En principe, on ne supprime pas les composants, sauf nécessité explicable, ce qui est rare, on joue simplement sur leur visiblité. De plus, la documentation déconseille d'utiliser RemoveControl.

Donc pour rendre invisibles les composants un GroupBox, on peut écrire :
Code:

1 2 int i ; for(i=0;i<GroupBox1->ControlCount;i++) GroupBox1->Controls[i]->Visible=false;
À bientôt
Gilles
01/05/2006, 09h59
Greybird
Citation:

Envoyé par Gilles Louïse

En principe, on ne supprime pas les composants, sauf nécessité explicable, ce qui est rare, on joue simplement sur leur visiblité. De plus, la documentation déconseille d'utiliser RemoveControl.

La nécessité pourrait être que les contrôles sont totalement dynamiques en ce sens qu'ils sont chargés d'un fichier ou sur des évenements très dépendant du périmètre extérieur à l'application de manière générale.

Sinon, effectivement, affecter un Owner à NULL à la création, puis supprimer le contrôle avec delete me semble adéquat.

Au niveau vitesse, il se peut que ce soit le raffraichissement graphique qui ralentisse la chose.

Tu devrais voir si tu ne pourrais pas locker la mise à jour graphique avec quelque chose du genre :
Code:

1 2 3 4 LockWindowUpdate(GroupBox1->Handle); // actions LockWindowUpdate(0);

SAlut !

Dans cet exemple je crée dynamiquement 4 CheckBox pour le GroupBox1 :

Le composant propriétaire des objets qui vont être détruits :

Code:

TComponent *Proprietaire;

Je crée le propriétaire et les CheckBox :
Code:

1 2 3 4 5 6 7 8 9 10 Proprietaire = new TComponent(Form1); //Form1 ou this selon que... TCheckBox *Cb; for(int j = 0; j < 4; j++) { Cb = new TCheckBox(Proprietaire); Cb->Parent = GroupBox1; Cb->Caption = "Essai " + IntToStr(j + 1); Cb->SetBounds(16, 24 + (j* 21), 100, 19); }
Je détruit d'un coup d'un seul les CheckBox en détruisant leur propriétaire :
Code:

1 2 3 4 5 if(Proprietaire != NULL) { delete Proprietaire; Proprietaire = NULL; }
A plus !

01/05/2006, 11h13
maw

Merci pour toutes vos réponses. En effet il est nécessaire que je delete les composants car ils sont crée dynamiquement à partir d'une base de donnée qui peut potentiellenement changer. Donc meme si c déconseillé, je le fait quand meme (un peu comme rouler à 150 sur un nationnale :).
Je ne pense pas que ce soit le jeu non plus de delete le GroupBox.
Je pense donc que la bonne solution c'est de liquider tout simplement les controles inserés, dans le but d'un inserer d'autre juste apres.
C'est pour cela que la rapidité compte, car effacé 15 controles puis en ajouter 15, cela peut poser des problèmes de vitesse, surtout si l'on fait tourner l'appli sur un pc pas vraiment vaillant.
L'exemple que henderson montre n'est pas mal, je vais essayer et l'optimisation du rafraichissement par Greybird me parrait une bonne id, je teste tout cela et je vous dit.
Merci les gars
01/05/2006, 11h25
Gilles Louïse

Il faudrait savoir si ce sont les mêmes composants ou non. Si ce sont les mêmes, il est alors possible de jouer sur leur visibilité seule. On peut même, si par exemple on connaît leur nombre maximum les créer en invisible au départ et ne jouer que sur leur visiblité, rien ne peut être plus rapide que cette technique.

Si vous ne pouvez pas assigner un maximum possible, la visibilté au fil de l'eau convient aussi. Vous en créez e.g. 5 au départ, ensuite il en faut e.g. 7 donc on en rajoute 2, ensuite e.g. 3 seulement donc on en efface 4 et ainsi de suite.

Il est très rare d'avoir à détruire des composants par delete, le fait que vos données viennent d'une base ne change pas, selon moi, cette règle. Il est très rare d'avoir à supprimer un composant pour ensuite avoir à le reconstruire.

Mais je ne connais pas votre projet donc je ne peux pas en dire plus. En tout cas, des séries de delete et de new sont fatalement plus coûteuse en temps que de jouer sur la seule visiblité. Car dans ce cas soit vous faites réapparaître un composant invisible, soit le créez par new, ce qui en rajoute définitivement un.

À bientôt
Gilles
01/05/2006, 11h37
Gilles Louïse

Citation:

Envoyé par Greybird

La nécessité pourrait être que les contrôles sont totalement dynamiques en ce sens qu'ils sont chargés d'un fichier ou sur des évenements très dépendant du périmètre extérieur à l'application de manière générale.

Pour moi, ça ne change rien, on peut toujours créer une algorithmique qui reprenne l'existant et l'adapte en fonction des nouvelles nécessités.

Dans ma réponse, je demandais s'il s'agissait des mêmes composants parce que dans ce cas, c'est assez facile.

Mais je crois que dans tous les cas, c'est possible puisque vous passez situation S1 à une situation S2. Il est toujours possible d'utiliser S1 pour aller à S2 en n'utilisant que new et la visibilité. C'est ainsi que j'essaierais de procéder et pour éviter au maximum le delete. Il n'y a guère que le cas où le nombre d'éléments serait très grand qui poserait problème.

Chacun fait comme il veut mais il existe obligatoirement un algorithme passant de S1 à S2 n'utilisant que new et la visibilté.

À bientôt
Gilles

Il y a une solution simple :

Mettre la GroupBox propriétaire (Owner) des CheckBox.
Un propiétaire peut détruire tous ses composants avec la fonction DestroyComponents.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 void __fastcall TForm1::Button1Click(TObject *Sender) { TCheckBox *Cb; for(int j = 0; j < 10; j++) { Cb = new TCheckBox(GroupBox1); Cb->Parent = GroupBox1; Cb->Caption = "Essai " + IntToStr(j + 1); Cb->SetBounds(16, 24 + (j* 21), 100, 19); } }
Destruction de tous les composants dont la GroupBox est propriétaire
Code:

1 2 3 4 5 void __fastcall TForm1::Button2Click(TObject *Sender) { GroupBox1->DestroyComponents(); }
Ne pas confondre parent et owner (propriétaire)
RemoveControl n'affecte que la propriété Parent elle ne détruit pas le composant.

01/05/2006, 14h49
maw

alors CGi ta solution marche sauf que j'ai un pb, car en fait, je déclenche le DetroyComponents() sur un evenement OnClick() d'un bouton que je supprime, du coup il aime pas et il me plante une erreur ( exeption class EPrivilege with message 'Privilege Instruction' ) ce qui est plus ou moins normal car je suis dans l'evenement onclick. Donc je me demande comment faire ?
01/05/2006, 15h26
pottiez

Salut, pour résoudre ce probleme, tu peut faire un truc tout simple, c'est peut etre un peu bidouillage mais ca marche,
tu met le code de l'evenement OnClick dans l'evenement OnTimer d'un composant Timer, ce Timer sera toujours inactif, et tu l'activera dans l'evenement OnClick du bouton, avec un temps très court du timer, ca passera comme si cété le click sauf que ca viendra juste apres le click, et donc plus dans l'événement OnClick. J'espere que tu as tout compris, sinon demande moi.
PS : a la fin de la destruction, ne pas oublié de re désactiver le timer, sinon il va rééssayé de détruir ce qui est déja detruit.
A plus
01/05/2006, 15h43
maw

Ok, c'est bien car c'est exatement ce que j'ai et cela me parraissais vraiement bidouille, comme koi, il n'y a pas que moi qui bidouille :). Merci.
01/05/2006, 16h05
maw

sauf que cela ne marche toujours pas. Encore cette satané erreur de privilege

Salut !

Dans ce cas, utilise le TComponent Proprietaire pour les objets qui doivent disparaître.

Le TSpeedButton pourra être détenu par le GroupBox.

Dans la OnClick du SpeedButton il suffira d'appliquer la solution apportée par CGi :

Citation:

...->DestroyComponents();

Ce propriétaire ni le SpeedButton n'étant jamais détruit par cette manip pour reconstruire éventuellement le contenu du GroupBox !

Pour info j'ai testé ainsi :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
__fastcall TForm1::TForm1(TComponent* Owner)
    : TForm(Owner)
{
Prop = new TComponent(this);
int j;
TRadioButton *R;
for(j = 0; j < 8; j++)
    {
    R = new TRadioButton(Prop);
    R->Parent = GroupBox1;
    R->SetBounds(16,20 + (j * 20),50,19);
    }
TSpeedButton *S = new TSpeedButton(GroupBox1);
S->Tag = 0; //pour le test
S->Parent = GroupBox1;
S->SetBounds(80,20,50,19);
S->OnClick = SupprimerClick;
}

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
27
28
29
30
void __fastcall TForm1::SupprimerClick(TObject *Sender)
{
TSpeedButton *Spd = (TSpeedButton*)Sender;
//Destruction des objets
Prop->DestroyComponents();
int j;
//Reconstruction d'objets selon le Tag
if(Spd->Tag == 1)
    {
    TRadioButton *R;
    for(j = 0; j < 8; j++)
        {
        R = new TRadioButton(Prop);
        R->Parent = GroupBox1;
        R->SetBounds(16,20 + (j * 20),50,19);
        }
    }
else
    {
    TCheckBox *Cb;
    for(j = 0; j < 8; j++)
        {
        Cb = new TCheckBox(Prop);
        Cb->Parent = GroupBox1;
        Cb->SetBounds(16,20 + (j * 20),50,19);
        }
    }
//Prochaine reconstruction :
Spd->Tag = Spd->Tag ^1;
}

A plus !