Sie sind nicht angemeldet.

DAU

Verteidiger einer zerschmetterten Welt

Beiträge: 2 603

Wohnort: Karlsruhe

Beruf: Software Developer

  • Nachricht senden

21

15.06.2008, 10:20

Laufen würde es, allerdings würde ich noch paar Schönheitstipps geben :). Außerdem wird dein Timer nie gestoppt ;)

[c]timer1.Start();[/c] << brauchste nicht, weil wenn Enabled auf true ist, wird der Timer schon gestartet.

[c]count = count - 1;[/c] wird zu [c]--count;[/c]

[c]if (count == 0)
{
MessageBox.Show("blub");
}[/c]

wird zu

[c]if (count <= 0)
{
MessageBox.Show("blub");
timer1.Enabled = false;
}[/c]

So eine Abbruchsbedingung muss eindeutig sein. Wenn zum Beispiel dein User ne negative Zahl eingibt, erhälst du eine Endlosschleife, weil du nur bei 0 abbrichst. Wenn du mit <= abfragst, deckst du auch so einen Fall ab.

MfG DAU
vergisst man schnell mal, dass die DAUs eigentlich in der überzahl sind.

ich weiß sehr genau, dass AIDS viele (Geschechts-)Krankheiten, darunter HIV, vorbeugt.