c++ - Functions子手w/函数-编译错误-没有匹配的调用

Question

我一直在尝试使用功能(从Michael Dawson的书中获得)来执行此Hangman，但我有一个我不太了解的错误。我意识到我的代码可能有各种各样的不良做法，但是当我是新手时，请对我轻松一点。我感觉自己快要到了，但是我很难弄清楚这个错误。我正在使用CodeBlocks。错误是:

32 |错误:找不到与调用'(std::__ cxx11::string {aka std::__ cxx11::basic_string})(std::__ cxx11::basic_string::size_type，char)的匹配项'|

//Hangman from Michael Dawson's code
//Uses functions to create the program

#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
#include <ctime>
#include <cctype>

using namespace std;

//FUNCTION DECLARATION
string pickword();
char playerGuess();
void isitinthere();

char guess = 0;
string soFar = "word";
string used = "";
int wrong = 0;

int main()
{

    const int MAX_WRONG = 8;

    string WORD = pickword();

    soFar = WORD;

    soFar(WORD.size(), '-');
    used = "";

    cout << "Welcome to Hangman! Godspeed!" << endl;

    while ((wrong < MAX_WRONG) && (soFar != WORD))
    {
        cout << "\n\nYou have " << (MAX_WRONG - wrong);
        cout << " incorrect guesses left.\n";
        cout << "\nYou've used the following letters:\n" << used << endl;
        cout << "\nSo far, the word is:\n" << soFar << endl;
    }

    playerGuess();

    while (used.find(guess) != string::npos)
    {
        cout << "\nYou've already guessed " << guess << endl;
        cout << "Enter your guess: ";
        cin >> guess;
        guess = toupper(guess);
    }

    used += guess;

    isitinthere();

if (wrong == MAX_WRONG)
{
    cout << "\nYou've been hanged!";
}
else
{
    cout << "\nYou guessed it!";
}

cout << "\nThe word was " << WORD << endl;

return 0;

}


//FUNCTION DEFINITION

string pickword()
{
    srand(static_cast<unsigned int>(time(0)));

    vector<string> words;

    words.push_back("INDUBITABLY");
    words.push_back("UNDENIABLY");
    words.push_back("CRUSTACEAN");
    words.push_back("RESPONSIBILITY");
    words.push_back("MISDEMEANOR");
    words.push_back("FORENSIC");
    words.push_back("BALLISTIC");
    words.push_back("PARADIGM");
    words.push_back("TROUBARDOR");
    words.push_back("SUPERCALIFRAGILISTICEXPIALLADOCIOUS")

    random_shuffle(words.begin(), words.end());

    theword = words[0];

    return theword;

}

char playerGuess()
{
    cout << "\n\nEnter your guess: ";
    cin >> guess;
    guess = toupper(guess);
    return guess;
}

void isitinthere()
{
    if (WORD.find(guess) != string::npos)
    {
        cout << "That's right! " << guess << " is in the word.\n";

        for (int i = 0; i < WORD.length(); ++i)
        {
            if (WORD[i] == guess)
            {
                soFar[i] = guess;
            }
        }
    }
    else
    {
        cout << "Sorry, " << guess << "isn't in the word. \n";
        ++wrong;
    }
}

在此先感谢您的帮助!

Answer 1

这是一个simple program应该可以解决您的问题。

#include <iostream>
#include <string>
#include <vector>
#include <cstdlib>
#include <ctime>
#include <algorithm>
#include <cctype>

// since you must have function here are some
bool removeGuessFromWord(std::string& word, const char guess);
bool isGuessInWord(const std::string& word, const char guess);
bool hasAlreadyGuessed(const std::vector<char>& gussList, const char guess);

// this is a simple program that should solve your question. It is not optimized for speed or efficency.
int main()
{
    std::vector<std::string> wordList = {"dog","cat","rat"}; // vector of words to select from and use as the word in hangman
    std::vector<char> guessList; // empty vector of gusses
    // Note that I assume a MAX_GUESS_COUNT of 0 means no guesses are allowed
    const unsigned int MAX_GUESS_COUNT = 4U; // number of guesses your allowed

    std::srand(time(0)); // use current time as seed for random generator
    std::string word = wordList.at(std::rand()%wordList.size()); // get a random word in the list
    std::string letersLeft = word; // keep track of what letters will still need to remove

    std::cout << "Welcome to Hangman! Godspeed!" << std::endl;

    char guess = 0;
    for(unsigned int numBadGusses=0U; numBadGusses<MAX_GUESS_COUNT && letersLeft.size()>0U; guess = 0)
    {
        std::cin>>guess;

        if(std::isprint(guess) == 0)
        {
            // may want more error checking
            std::cout << "You ented a non-printable charecter" << std::endl;
        }
        else if(isGuessInWord(word, guess))
        {
            // this was a good guess because the charecter is still in the word
            // so remove all the remaining chars of this type from the word
            if( removeGuessFromWord(letersLeft,guess) )
            {
                std::cout << guess << " was a good guess" << std::endl;
            }
            else
            {
                std::cout << guess << " was a good guess, but you already guessed it once" << std::endl;
            }
        }
        else if(hasAlreadyGuessed(guessList, guess))
        {
            std::cout << "You've already guessed " << guess << std::endl;
        }
        else
        {
            // this was a new bad guess
            guessList.push_back(guess);
            numBadGusses++; // Note that this isn't technicly needed and could use size of vector
            std::cout << guess << " was a bad guess" << std::endl;
        }

    }

    if(letersLeft.size() == 0U)
    {
        std::cout<<"You Win"<<std::endl;
    }
    else
    {
        std::cout<<"You Lose"<<std::endl;
    }
    std::cout << "The word was "<< word << std::endl;

    return 0;
}

bool removeGuessFromWord(std::string& word, const char guess)
{
    return word.erase(std::remove(word.begin(), word.end(), guess), word.end()) != word.end() ? true : false;
}

bool isGuessInWord(const std::string& word, const char guess)
{
    return word.find(guess) != std::string::npos ? true: false;
}
bool hasAlreadyGuessed(const std::vector<char>& gussList, const char guess)
{
    return std::find(gussList.begin(), gussList.end(), guess) != gussList.end() ? true: false;
}