Sie sind nicht angemeldet.

Lieber Besucher, herzlich willkommen bei: Silkroad Online Forum. Falls dies Ihr erster Besuch auf dieser Seite ist, lesen Sie sich bitte die Hilfe durch. Dort wird Ihnen die Bedienung dieser Seite näher erläutert. Darüber hinaus sollten Sie sich registrieren, um alle Funktionen dieser Seite nutzen zu können. Benutzen Sie das Registrierungsformular, um sich zu registrieren oder informieren Sie sich ausführlich über den Registrierungsvorgang. Falls Sie sich bereits zu einem früheren Zeitpunkt registriert haben, können Sie sich hier anmelden.

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.