Przypisywanie tej samej wartości

[Michal-S2]

Witam,

Czym można zastąpić kod np.

Kod: Zaznacz cały

      FirstBlack->Checked = false;
      FirstBrown->Checked = false;
      FirstRed->Checked = false;
      FirstOrange->Checked = false;
      FirstYellow->Checked = false;
      FirstGreen->Checked = false;
      FirstBlue->Checked = false;
      FirstViollet->Checked = false;
      FirstGray->Checked = false;
      FirstWhite->Checked = false;

?
Myślałem że może dać by to w tablię, a potem przez pętle przypisywać Checked = false, ale nie wiem czy to dobry pomysł....

[banita]

tablica to jedeno rozwiazanie ale zwazywszy na to iz bcb udostepnia funkcje findcomponent to kiepskie rozwaizanie.
zainteresuj sie owa funkcja, opis znajdziesz w serwisie CB.

[Witold]

iz bcb udostepnia funkcje findcomponent to kiepskie rozwaizanie.

W jaki sposób można przypisać .Checked = false to komponentów o różnych nazwach przy pomocy findcomponent ?
Czemu wersja z findcomponent ma być lepsza od tej z tablicą, wydajność?, kod będzie lepszej jakości ?

[Cyfrowy Baron]

To da się zrobić i nie potrzeba do tego tablicy, ani funkcji FindComponent. Wystarczy posłużyć się funkcją ClassNameIs, która sprawdza nazwę klasy komponentu i zwraca wartość typu true lub false. Trzeba również skorzystać z klasy TComponent do tymczasowego przechowywania nazwy komponentu wyliczanego na formularzu. Do wyliczania komponentów należy posłużyć się właściwościami samego formularza: ComponentCount - zwraca liczę komponentów na formularzu, oraz Components - zwraca uchwyt do komponentu. Potem wystarczy posłużyć się rzutowaniem polimorficznym do zmiany dowolnej wartości wybranego komponentu w oparciu o jego klasę.

W przykładzie znajduje się funkcja, która pobiera jeden argument typu bool i w zależności od jego wartości odpowiednio ustawia właściwość Checked wszystkich komponentów typu TCheckBox na formularzu.

plik nagłówkowy np. Unit1.h

Kod: Zaznacz cały

private:
        void CheckBoxState(bool state);


plik źródłowy np. Unit1.cpp

Kod: Zaznacz cały

//---------------------------------------------------------------------------
void TForm1::CheckBoxState(bool state)
{
  TComponent *Temp;
  int count = this->ComponentCount - 1;
  for(int i = count; i > 0; i--)
  {
    Temp = this->Components[i];

    if(Temp->ClassNameIs("TCheckBox"))
    {
     reinterpret_cast<TCheckBox *>(Temp)->Checked = state;
    }
  }
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button1Click(TObject *Sender)
{
  CheckBoxState(true);
}
//---------------------------------------------------------------------------
void __fastcall TForm1::Button2Click(TObject *Sender)
{
  CheckBoxState(false);
}
//---------------------------------------------------------------------------


[Michal-S2]

Cyfrowy Baronie, mała poprawka do Twojego kodu. Zamiast:

Kod: Zaznacz cały

  TComponent *Temp;
  int count = this->ComponentCount - 1;
  for(int i = count; i > 0; i--)
  {
   Temp = this->Components[i];

   if(Temp->ClassNameIs("TCheckBox"))
   {
    reinterpret_cast<TCheckBox *>(Temp)->Checked = state;
   }
  }

(testowane na Turbo C++)
powinno być:

Kod: Zaznacz cały

  TComponent *Temp;
  int count = this->ComponentCount - 1;
  for(int i = count; i > -1; i--)
  {
   Temp = this->Components[i];

   if(Temp->ClassNameIs("TCheckBox"))
   {
    reinterpret_cast<TCheckBox *>(Temp)->Checked = state;
   }
  }

Ponieważ gdy jest tak jak Ty napisałeś to w pierwszym komponencie (CheckBox1) nie można tym kodem zmienić wartości na true lub false.
Czy takie coś się stało dlatego, bo komponenty są numerowane od 0?

[Cyfrowy Baron]

Zgadza się. Drobne przeoczenie. Można też tak:

Kod: Zaznacz cały

  for(int i = count; i >= 0; i--)