Discussion :

[agentchico]

Bonjour,

Je cherche à faire une boucle pour modifier la couleur de plusieurs TShape, mais je ne sais pas comment on peux faire en utilisant une variable.

Mon bout de code sera plus parlant :

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
 
rep_trame[0]=0xAA;
 
char j=0x01;
if (rep_trame[0]&j) i0->Brush->Color=clLime;
else i0->Brush->Color=clMaroon;
j*=2;
if (rep_trame[0]&j) i1->Brush->Color=clLime;
else i1->Brush->Color=clMaroon;
j*=2;
if (rep_trame[0]&j) i2->Brush->Color=clLime;
else i2->Brush->Color=clMaroon;
.
.
.

voila c'est très simple:
i0,i1,i2,....in sont le nom de petites leds en Tshape et il y en a un nombre n.
Et rep_trame[] constitue l'état des leds, donc si le bit 0 est à 1 alors la led i0 sera verte, etc ...

Comment je peux remplacer les i0,i1,i2 .... par des variables ?

merci

[pottiez]

Tu peut mettre tes TShape dans un tableau, ensuite il ne te suffira plus qu'a faire une boucle for sur ton tableau

[agentchico]

mettre les tshapes dans un tableau ?
Comment on fait ca ?
Je suis désolé, je viens de commencer un stage sous c++ builder alors que je l'ai jamais utilisé et que j'ai fait que de la console à l'école

merci

[pottiez]

Et bien je suppose que tu as un nombre fixe de TShape,
tu peut donc faire un tableau de TSahpe * et mettre tes TShape dedans :

Code :

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

1
2
3
4
5
6
TShape * tabShape[n];
tabShape[0] = i0;
tabShape[1] = i1;
tabShape[2] = i2;
...
tabShape[n-1] = in-1;

ensuite pour les parcourir :

Code :

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

1
2
3
for(int i=0;i<n;i++){
  tabShape[i]->Brush->Color = clLime;
}

[agentchico]

Ok merci je vois.
Et oui il y a un nombre fixe de Tshape, mais c'est de l'ordre de la cinquantaine.
Donc je devrai faire 50 fois :

Code :

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

1
2
3
4
5
 
tabShape[0] = i0;
tabShape[1] = i1;
tabShape[2] = i2;
...

Mais tans pis ça m'allège déjà bien le travail si après je bosse dans un tableau.
J'aurais bien aimé qu'on puisse faire un truc genre :

Code :

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

1
2
3
 
for(int j=0;j<n;j++)
'i'+j->Brush->Color=clLime;

Mais la je rêve peut être trop ^^

[pottiez]

Oui, ça ce n'est pas possible en c++, mais sinon tu peut tout gérer en dynamique plutôt que de reprendre des TShape déjà créé sur la form, au lieu de les rangé dans le tableau tu fait un new pour les créer au démarrage, le problème c'est que tu doit redéfinir dans le code leurs placement sur la form, avantage, tout est géré facilement par des boucle et il y as donc moins de code

[agentchico]

hum ok
Ca m'a l'air un peu trop compliqué pour moi.
Je vais me tenir à la première solution.
Merci

[pottiez]

Ce n'est pas extrêmement compliqué, juste un peut de c++ de base pour créer le Shape (new et delete) et après renseigner les propriétés utile au bon endroit tout comme pour la couleur dans mon premier exemple
Mais je peut comprendre que tu ne veuille pas tout casser (ou une bonne partie) de ce que tu as fait.

[agentchico]

Ah, je vais essayer de regarder cette méthode alors.

Je vais rien casser puisque cette partie je viens de la commencer et j'ai pas eu envie de coder les 50 sans savoir si il y avait pas une autre solution ^^

[henderson]

Salut !

Si on déclare en global :

Code :

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

1
2
3
 
TColor LedColors[2] = {clGreen, clLime};
TShape *Leds[50];

On doit pouvoir y arriver assez facilement en initialisant ce tableau Leds dans le constructeur de la form !

Code :

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

1
2
3
4
5
6
7
8
9
10
 
Leds[0] = Shape1; //Shape50
Leds[1] = Shape2; //..
Leds[2] = Shape3; //..
Leds[3] = Shape4; //..
Leds[4] = Shape5; //..
Leds[5] = Shape6; //..
Leds[6] = Shape7; //..
....
Leds[49] = Shape50; //Shape1

D'autant plus que ça va s'exécuter plus vite qu'avec une boucle...

Ensuite :

Code :

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

1
2
3
4
5
6
7
 
int z = 0xAA; // z = rep_trame[0]
for(int j = 0; j < 8; j++)
    {
    Leds[j]->Brush->Color = LedColors[(z & 1)];
    z = z / 2;
    }

Tu auras donc accès à tes leds depuis le tableau !
Et il n'y aura rien d'étrange à faire :

Code :

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

1
2
 
Leds[8 + j]->Brush->Color = LedColors[(z & 1)];

Sinon, dans l'optique du "i" + j il faut raisonner avec FindComponent.

Code :

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

1
2
3
4
 
AnsiString N = "i" + IntToStr(j+1) // si on part de j=0
TComponent *Led = FindComponent(N);
((TShape*)Led)->Brush->Color ...

A plus !

[agentchico]

Merci beaucoup!
Je vais le coder comme ça.
Ça sera bien plus propre que ce que j'avais commencé de faire