Suche Hilfe bei dem Spiel Hangman in C++

[Mirko (mirkomikro)]

Hallo,


if (user != troll) {

ich versuche mein erstes Spiel in C++ zu programmieren - Hangman. Hier 
der Link für Nicht-Kenner: https://en.wikipedia.org/wiki/Hangman_(game)

Im Groben ist mein Spiel spielbar...
Natürlich ist es noch nicht alles, nach meinen Wünschen entsprechend 
umgesetzt, aber das werde ich noch peu à peu integrieren, soweit es 
meine Fähigkeiten zulassen.


Nun zu meinem Problem, wobei ich nicht weiterkomme:

Beim Spiel Hangman wird ein Buchstabe in einem Wort gesucht und wenn es 
was findet, in irgend einer Form dargestellt, dass man weiß auf welcher 
Position es ist, wie bei Glücksrad (für die ältere Generation unter 
euch).

Mein Wunsch wäre z.B. so: Eingabe "e" -> e_e_ (lol die Formatierung 
spielt nicht mit)

Da ich mir nicht anders zu helfen wusste, habe ich das sehr rudimentär 
umgesetzt, da ich nicht weiß, wie ich den Wert von string.find so 
ausgeben kann, dass die Buchstaben so dargestellt werden und beim 
nächsten Erraten noch die neuen Buchstaben hinzugefügt werden.

Hat jemand für mich ein Tip oder ein Beispiel, das mir das aufschlüsseln 
kann?

Wäre sehr dankbar, Ihr würdet mir viele schlaflose Nächte ersparen... :D

/*
 * Hangman - The Game
 */ 


#include <iostream>
#include <string>


bool loop        = true;
int attempt        = 5;
std::string secretWord  = "concealed";
std::string guess;


// Won game
int won () {
  if (secretWord == guess) {
    loop = false;
    std::cout << "\nCongratulations, you have won!\n";
  }
  return 0;
}


// Lost Game
int lost () {
  if (attempt == 0) {
    loop = false;
    std::cout << "\nGame over!\n";
  }
  return 0;
}


// Exit Game
int exit () {
  if (guess == "exit") {
    loop = false;
    std::cout << "\nByebye!\n";
  }
  return 0;
}


// Notification
int notification () {
  std::cout << "Note your result on a paper!\n\n";
  return 0;
}


int main () {

  // Welcome Screen
  std::cout << "Hangman - The Game\n\n";

  while (loop) {

    // Input guess
    std::cout << "Attempt: " << attempt << "\n";
    std::cout << "Hint   : Secret\n";
    std::cout << "Rules  : Only write lower case letters or the secret word.\n";
    std::cout << "Guess  : ";
    std::cin >> guess;

    size_t found = secretWord.find (guess);
    if (found != std::string::npos) {
      // Do stuff if guess is valid
      if (guess == "c") {
        std::cout << "\nc**c*****\n";
        notification();
      }
      if (guess == "o") {
        std::cout << "\n*o*******\n";
        notification();
      }
      if (guess == "n") {
        std::cout << "\n**n******\n";
        notification();
      }
      if (guess == "e") {
        std::cout << "\n****e**e*\n";
        notification();
      }
      if (guess == "a") {
        std::cout << "\n*****a***\n";
        notification();
      }
      if (guess == "l") {
        std::cout << "\n******l**\n";
        notification();
      }
      if (guess == "d") {
        std::cout << "\n********d\n";
        notification();
      }
    }
    else {
      // Do stuff when guess is invalid
      std::cout << "\n";
      attempt--;
    }

    won();
    lost ();
    exit ();
    }
}

}


else {
 exit ()
}

[Micha (nichtgast)]

Hi,

zuerst mal was anderes. Wenn du aus einer Funktion nichts zurückgeben 
willst, dann mach statt dem in ein void als Rückgabetyp. Alles andere 
ist verwirrend.

Code mag ich gar nicht liefern, du willst ja lernen.

du könntest einen weiteren String anlegen, der so viele '*' enthält, wie 
dein Suchwort.

string::find liefert dir die position des gesuchten chars zurück. Du 
kannst dann also hergehen und an der Position in deinem zusätzlichen 
String das '*' durch den passenden Buchstaben ersetzten.

Danach gibts du einfach immer nur deinen zweiten String aus. Der enthält 
dann automatisch mehr und mehr Buchstaben des Lösungswortes.


Grüße

[Roger S. (edge)]

Mirko schrieb:
> Hat jemand für mich ein Tip oder ein Beispiel, das mir das aufschlüsseln
> kann?

Hier ein quick&dirty refactoring das Deiner Anforderung entspricht.

Cheers.

[Mirko (mirkomikro)]

Hallo,


und danke für die zwei unterschiedlichen Ansätze an Antworten.
Den Code habe ich mal überflogen, aber ohne Erklärung.... Mein Programm 
wurde auch geändert... was ich so eigentlich erstmal nicht wollte.

Deswegen tendiere ich zu ersterem...

Micha schrieb:
> Hi,
>
> zuerst mal was anderes. Wenn du aus einer Funktion nichts zurückgeben
> willst, dann mach statt dem in ein void als Rückgabetyp. Alles andere
> ist verwirrend.

Dadurch, dass ich etwas ändern musste, habe ich die Funktionen einfach 
rausgenommen, somit sollten die Verwirrungen entworren sein, 
hoffentlich.


Micha schrieb:
> du könntest einen weiteren String anlegen, der so viele '*' enthält, wie
> dein Suchwort.
>
> string::find liefert dir die position des gesuchten chars zurück. Du
> kannst dann also hergehen und an der Position in deinem zusätzlichen
> String das '*' durch den passenden Buchstaben ersetzten.
>
> Danach gibts du einfach immer nur deinen zweiten String aus. Der enthält
> dann automatisch mehr und mehr Buchstaben des Lösungswortes.

Ich habe den String hiddenWord angelegt und musste dann erstmal 
recherchieren wie der Befehl aussehen muss. Hab dann gemerkt, dass ich 
nur mit einem Char arbeiten konnte und musste somit meinen "Exit" 
rausschmeißen und hier und da etwas anpassen.
Nach einem kurzen Test bin ich auch zufrieden, aber jetzt habe ich das 
Problem, dass z.B. zwei gleiche Werte nicht erkannt werden, sondern nur 
der erste. Wie kann ich den ganzen String Buchstabe für Buchstabe 
überprüfen?

/*
 * Hangman - The Game
 */ 


#include <iostream>
#include <string>


char guess;
bool loop        = true;
int attempt        = 5;
std::string secretWord  = "concealed";
std::string hiddenWord  = "*********";


int main () {

  // Welcome Screen
  std::cout << "Hangman - The Game\n\n";

  while (loop) {

    // Input guess
    std::cout << "Attempts: " << attempt << "\n";
    std::cout << "Hint    : Secret\n";
    std::cout << "Rules   : Only write lower case letters.\n";
    std::cout << "Guess   : ";
    std::cin >> guess;

    size_t found = secretWord.find (guess);
    if (found != std::string::npos) {
      //Do stuff if guess is valid
      hiddenWord[found] = guess;
      std::cout << "\n" << hiddenWord << "\n\n";
    }
    else {
      //Do stuff when guess is invalid
      std::cout << "\n";
      attempt--;
    }

    if (secretWord == hiddenWord) {
      std::cout << "\nCongratulations, you have won!\n";

    }

    if (attempt == 0) {
      loop = false;
      std::cout << "\nGame over!\n";
    }
    }
}

[Rolf M. (rmagnus)]

Mirko schrieb:
> Nach einem kurzen Test bin ich auch zufrieden, aber jetzt habe ich das
> Problem, dass z.B. zwei gleiche Werte nicht erkannt werden, sondern nur
> der erste.

Das ist ja auch nicht verwunderlich. Dein Aufruf von 
std::string:::find() gibt dir das erste Vorkommen des gesuchten 
Buchstaben. Die Funktion weiß ja nicht, dass der schon mal gefunden 
wurde und du diesmal das zweite Vorkommen suchst.

> Wie kann ich den ganzen String Buchstabe für Buchstabe überprüfen?

Eine Möglichkeit wäre, mit dem secretWord das umgekehrte zu machen wie 
mit dem hiddenWord, also alle gefundenen Buchstaben durch Sternchen 
ersetzen. Besteht das hiddenWord nur noch aus Sternchen, hat man 
gewonnen.

[Roger S. (edge)]

Mirko schrieb:
> Nach einem kurzen Test bin ich auch zufrieden, aber jetzt habe ich das
> Problem, dass z.B. zwei gleiche Werte nicht erkannt werden, sondern nur
> der erste. Wie kann ich den ganzen String Buchstabe für Buchstabe
> überprüfen?

Mit einem simplen for-loop anstatt dem std::string::find()

for (size_t i = 0; i < secretWord.size(); i++) {
  if (secretWord[i] == guess) {
    // your sophisticated handling goes here (instead of printing)
    std::cout << "found at position: " << i << "\n";
  }
}

[Mirko (mirkomikro)]

Roger S. schrieb:
> Mirko schrieb:
>> Nach einem kurzen Test bin ich auch zufrieden, aber jetzt habe ich das
>> Problem, dass z.B. zwei gleiche Werte nicht erkannt werden, sondern nur
>> der erste. Wie kann ich den ganzen String Buchstabe für Buchstabe
>> überprüfen?
>
> Mit einem simplen for-loop anstatt dem std::string::find()

Ha, genau das hab ich gesucht! Ich hab schon mit einer for Schleife 
experimentiert, aber es hat nicht so geklappt. Jetzt muss ich nur noch 
das Konstrukt darum ändern, aber ich denke das bekomme ich schon hin. 
Danke vielmals.

[Daniel A. (daniel-a)]

Als nächstes, versuch es mal z.B. mit dem Wort "Bär".

[Tim S. (Firma: tsx89) (freak_ts)]

Mirko schrieb:
> if (user != troll) {
> }
> else {
> exit ()
> }

Undefined variable 'user', Undefined variable 'troll',
Syntaxerror at exit (), no-whitespace, need semicolon.
EOF.

Schade - Ich kann's nicht richtig ausführen.