Warum mal Hangman-Spiel bei jedem Buchstabendruck oder gar nicht mehr?

Hier steht die Lösung:  https://www.mycsharp.de/wbb2/thread.php?threadid=21782

Würde meinen, dass Dein If in der Buchstaben-Prüfung nicht so arbeitet, wie gedacht.

Hätte eher folgendes erwartet:

C#-Code:

if (!items.Tag.Equals("a") || !items.Tag.Equals("A"))

bzw.

C#-Code:

if (!items.Tag.ToLower().Equals("a"))

Aber die Prüfung/den Programmablauf kannst Du einfach via Debugger nach schauen:  [Artikel] Debugger: Wie verwende ich den von Visual Studio?

Zitat:

Vermutung rein vom ansehen deines Codes. Wenn man einen Buchstaben eingibt, prüfst du für JEDE Stelle, ob sie diesen Buchstaben enthält oder nicht. Wenn nicht, zeichnest du den Galgen für JEDE falsche Stelle ein Stück weiter. Das heißt, ein Wort mit 5 Buchstaben von denen einer der gedrückte ist, zeichnet den Galgen trotzdem um 4 Teile weiter. Tatsächlich solltest du zuerst prüfen ob es eine Stelle mit dem Buchstaben gibt. Wenn ja, ALLE passenden Buchstaben aufdecken. Wenn nein, den Galgen um EIN Stück erweitern.

Zitat von chilic:

Was soll "_ = pictureBox1.CreateGraphics();" sein? Das kann m.M. raus. //Danke, für den Tip :) Intellisense spuckte mir das aus, hatte auch das Gefühl, das das doppelt gemoppelt ist.

Danke an Alle, wenn ich aber so mache:

C#-Code:

private void Btn_A_Click(object sender, EventArgs e)         {             galgenzaehler += 1;             foreach (ListViewItem items in listView1.Items)             {                 if (items.Tag.Equals("a") || items.Tag.Equals("A"))                 {                   items.Text = "A";                     CheckAufSieg();                 }             }         }

Dann weis ich nicht wie ich den Galgen weiter malen soll. Jetzt macht die Abfrage nämlich genau was sie soll: Sie prüft ob der Buchstabe richtig ist, für ALLE und weiter ? Ich glaube das bei CheckAufSieg der Galgen rein muss, aber ich weis gerade nicht wie ...

C#-Code:

private void CheckAufSieg()         {              ListViewItem itemx = listView1.FindItemWithText("__");              if (itemx == null)              {                  MessageBox.Show("Gewonnen");              }              else if (itemx.Tag.Equals("__"))              {                  Galgen(galgenzaehler);              }         }

Danke werde ich mal trsten ;)

Ja, war natürlich auch gedacht dass man das in eine eigene Klasse steckt und so die Schichten etwas trennt.

Beispiel:

C#-Code:

interface HangmanGame() {    void CheckLetter(char eingabe);    string HiddenWord; // A _ _ _ ...    int RemainingLetters;    bool IsGameWon;    event EventHandler NoLettersLeft; ... }

// Edit:
und doch, das geht schon, wenn man das denn so machen möchte:
 https://stackoverflow.com/questions/6640...ions-in-c-sharp

Danke Leute für Eure Hilfe :)

Mein Problem ist NICHT, das ich die Objekte in der Listview nicht selektieren und dementsprechen verändern /ansprechen kann sondern, das mir der Übergang vom gedrückten Buchstaben zum Sieg fehlt.

Ich denke so:
User drückt Buchstabe -> Hab ich
Code checkt ob Buchstabe vorhanden ist und deckt diesen auf -> Hab ich und nun kommts: Wie frage ich jetzt, in C# ,was passiert wenn der Buchstabe NICHT dem gedrückten entspricht ??? Wenn dann muss ich jetzt den Balken malen.

C#-Code:

if (item.Tag.Equals("A") /alles toll else if not {   ///male den Balken }

Ich habs mit tausend Varianten probiert )Nach meinen geringen Kenntnisstand) Aber egal wie ich den Code drehe und wende ich komme NIE dahin das er erst dann den Balken malt, wenn der Buchstabe falsch ist...

Ich weis mit Linq könnte man die Sache vereinfachen, aber es geht mir gerade nicht ums vereinfachen, sonder wie ich mit meinem Code eine Lösung hinbekomme

Ich stecke im C# nicht so drin -> leider

Danke @ chillic

ich gehe diesen Ansatz mal durch ... Danke für die Mühe ;)