C++ Programmierung Button/ Panel

[Florian (Gast)]

Hallo,

ich muss ein kleines Programm schreiben, das wie folgt ablaufen soll:

Es gibt 2 Buttons, deren Zustand OFF oder On sind.

Nun soll, wenn beide Buttons ON sind die Farbe des Panels Grün werden. 
Wenn die Zustände ungleich sind Schwarz.

Habe die Variablen global vergeben. Das Panel muss ich ja auch global 
reinschreiben, aber es kommt immer eine Fehlermeldung.

Bitte um Hilfe.

SG Florian

__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{

}
//-------------------------------------------------------------------------
static bool x = false;
static bool a = false;
if  ((a = true)&&(x=true))
      Panel1->Color=clGreen;

void __fastcall TForm1::Button3Click(TObject *Sender)
{

if (a == true)
{
    Button3->Caption = "On";
}
 else
{
   Button3->Caption = "Off";
   Panel1->Color=clBlack;

}
a = !a;

void __fastcall TForm1::Button4Click(TObject *Sender)
{

if (x == true)
{
    Button4->Caption = "On";

             //   if  ((a = true)&&(x=true))
            //    Panel1->Color=clGreen;
}
 else
{
   Button4->Caption = "Off";
   Panel1->Color=clBlack;

}
x = !x;
}

[Karl H. (kbuchegg)]

saubere und konsequente Code Einrückung ist nicht einfach nur 
Selbstzweck!

__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{

}
//-------------------------------------------------------------------------
static bool x = false;
static bool a = false;

if  ((a = true)&&(x=true))
      Panel1->Color=clGreen;

Huch, was macht der Code da?
In C++ muss Code innerhalb einer Funktion stehen!

[Karl H. (kbuchegg)]

__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//-------------------------------------------------------------------------

static bool x = false;
static bool a = false;

if  ((a = true)&&(x=true))
      Panel1->Color=clGreen;

void __fastcall TForm1::Button3Click(TObject *Sender)
{
  if (a == true)
  {
    Button3->Caption = "On";
  }
  else
  {
    Button3->Caption = "Off";
    Panel1->Color=clBlack;
  }

  a = !a;


  void __fastcall TForm1::Button4Click(TObject *Sender)
  {
    if (x == true)
    {
      Button4->Caption = "On";

      //   if  ((a = true)&&(x=true))
      //    Panel1->Color=clGreen;
    } 
    else
    {
      Button4->Caption = "Off";
      Panel1->Color=clBlack;
    }
    x = !x;
  }

Sieh dir das ganze nochmal an.
da fehlt recht offensichtlich eine schliessende } in der Funktion 
Button3Click. Nämlich die, die die Funtion beendet. Durch dein 
chaotisches Einrücken siehst du das natürlich nicht. Aber wenn man mal 
sauber einrückt, dann sieht man es! In jeder Funktion muss die letzte } 
wieder am linken Rand sein, sonst stimmt was nicht und man muss sich auf 
die Suche begeben, wo man eine { oder eine } verbummelt hat.
Grundvoraussetzung ist aber saubere Einrückung. Bei jeder { geht es um 2 
Leerzeichen weiter nach rechts rein, bei jeder } geht es wieder um 2 
Leerzeichen weiter nach links raus. Dann sieht man auch was.

Exzessize unmitivierte Leerzeilen kannst du dir hingegen sparen. Die 
ziehen den Code nur in die Länge und sonst nichts. Eine Leerzeile kommt 
rein, wenn ein neuer Gedankengang beginnt. Das kann innerhalb einer 
Funktion sein, kann aber auch beim Beginn einer neuen Funktion sein. Wie 
auch immer: Achte auf gut lesbaren Code, bei dem man die Struktur auch 
aus 2 Meter Entfernung noch sieht, dann siehst du derartige Fehler 
gleich viel besser.

[Sven B. (scummos)]

Überhaupt, was ist den "a = true && x = true" für ein Murks? Das setzt a 
auf true und x auf true. Was du willst ist wohl eher "a && x".

[Karl H. (kbuchegg)]

PS:
Sieh von solchen Dingen ab

  if( a == true )

das bringt nichts. A ist sowieso bei dir ein bool. Den musst du in C++ 
nicht auf true vergleichen. Nenn lieber die Variable besser.

static bool button1Pressed = false;

...


   if( button1Pressed )
      ....


da haben alle mehr davon, als wie wenn du sinnlose Vergleiche machst. 
Das if interessiert sich nur dafür, ob der Ausdruck in der Klammer true 
oder false ergibt. Ob das jetzt so ist, weil die boolsche Variable 
selbst schon true oder false ist, oder ob du da noch einen Vergleich 
machst, das ist dem if völlig wurscht. Das if interessiert nur, welches 
Ergebnis der Ausdruck insgesamt hat, true oder false. Ob dazu ein 
Vergleiche notwendig ist oder nicht, ist dem if Powidel.

Und noch ein PS:
Du sollst nicht selbst über den Status des Buttons Buch führen, sondern 
den Button selbst befragen, ob er gedrückt ist oder nicht! Dann kann 
auch nie irgdnwas durcheinander kommen. Wenn du dieselbe 'Information' 
wie der Button selbst noch einmal speicherst (denn der Button führt ja 
selber auch darüber Buch ob er gedrückt ist oder nicht, schliesslich 
muss er sich ja anders hinmalen), dann läufst potentiell in das 
Datenbank Dlemma: Speichere dieselbe Information 5 mal ab, und alle 5 
Kopien werden irgendwann nicht mehr übereinstimmen. Man speichert ein 
und dieselbe Information nur ein einziges mal, dann kann da auch nichts 
inkonsistent sein.

[Florian (Gast)]

Danke,

habe die fehlende Klammer gefunden, jedoch kommt der Fehler wieder.

Wie müsste ich das umschreiben, dass ich das Panel1 nur dann grün ist, 
wenn beide auf ON sind?

Die Deklaration müsste ja stimmen? Jedoch weiß ich nicht wo ich das 
hineinschreibe, weil im Button selber hat es meiner Meinung nach auch 
nicht verloren? Des wegen müsste ja wenn beide On den Wert Global 
weitergeben?

if  ((a = true)&&(x=true))
      Panel1->Color=clGreen;

[Sven B. (scummos)]

Schrieb ich doch schon: Du willst (a && x).

[Karl H. (kbuchegg)]

Florian schrieb:

> Die Deklaration müsste ja stimmen? Jedoch weiß ich nicht wo ich das
> hineinschreibe, weil im Button selber hat es meiner Meinung nach auch
> nicht verloren?

Der Button weiß selber, ob er gedrückt ist oder nicht.
Du brauchst ihn nur zu 'befragen'. Welche Funktion des Buttons dir diese 
Information liefert steht in der Doku.

[Karl H. (kbuchegg)]

Karl Heinz Buchegger schrieb:
> Florian schrieb:
>
>> Die Deklaration müsste ja stimmen? Jedoch weiß ich nicht wo ich das
>> hineinschreibe, weil im Button selber hat es meiner Meinung nach auch
>> nicht verloren?
>
> Der Button weiß selber, ob er gedrückt ist oder nicht.
> Du brauchst ihn nur zu 'befragen'. Welche Funktion des Buttons dir diese
> Information liefert steht in der Doku.

Bzw. hat der sicher ein 'Property' (so wie er auch ein Caption-Property 
hat), welches dir mitteilt, ob er gedrückt ist oder nicht.

[Karoly (Gast)]

Florian schrieb:
> if  ((a = true)&&(x=true))
>
>       Panel1->Color=clGreen;

Hallo Florian,

definiere eine Funktion z.B. Check()

void __fastcall TForm1::Check()
{
 if  ((a == true)&&(x==true))

       Panel1->Color=clGreen;
}
oder
void __fastcall TForm1::Check()
{
 if  (a && x)

       Panel1->Color=clGreen;
}

und rufe sie auf von beiden Button_click Ereignissen als letzter Befehl.
Nicht vergessen die Funktion im Header zu deklarieren.
private void __fastcall Check(void);