Question

我是一名高中生，在我的最后一个项目中，我的任务包括功能原型。我将包含下面的代码，但始终显示的错误是“未初始化的本地变量”名称“已使用”。我在一个单独的函数中定义该变量并返回它，但它不会返回到int main。我确信这是显而易见的，但如果有人能帮助我，我会非常感激。感谢

    // Rock, Paper, Scissors Game
#include <iostream>
#include <cstdlib>
#include <ctime>
#include <string>
using namespace std;

// Global constants to represent rock,
// paper, or scissors.

const  int rock = 1;
const int paper = 2;
const int scissors = 3;

int getComputerChoice(int);
int getUserChoice(char);
void determineWinner(int, int);

int main()
{
    int compChoice;
    char uChoice;

    getComputerChoice(compChoice);
    getUserChoice(uChoice);
    if (uChoice == 'r' || uChoice == 'p' || uChoice == 's') {
        determineWinner(compChoice, uChoice);
        getComputerChoice(compChoice);
        getUserChoice(uChoice);
    }

    return 0;
}

// ********************************************************
// The getComputerChoice function returns the computer's  *
// game choice. It returns 1 for rock (via the ROCK       *
// constant), or 2 for paper (via the PAPER constant),    *
// or 3 for scissors (via the SCISSORS constant).         *
// ********************************************************

int getComputerChoice(int compChoice) {


    // Get the system time so we can use it
    // to seed the random number generator.
    unsigned seed = time(0);


    // Use the seed value to seed the random
    // number generator.
    srand(seed);

    // Generate a random number in the range of 1-3.
    compChoice = (1 + rand() % 3);

    return compChoice;
}


// ********************************************************
// The getUserChoice function displays a menu allowing    *
// the user to select rock, paper, or scissors. The       *
// function then returns 1 for rock (via the ROCK         *
// constant), or 2 for paper (via the PAPER constant),    *
// or 3 for scissors (via the SCISSORS constant).         *
// ********************************************************

int getUserChoice(char uChoice) {


    cout << "Welcome to rock, paper, scissors. Choose 'r' for rock, 'p' for paper, or 's' for scissors.\n";

    if (uChoice == 'r' || uChoice == 'p' || uChoice == 's')
        cin >> uChoice;
    else
        cout << "This is not a valid choice.\n";


    return uChoice;

}

// ********************************************************
// The determineWinner function accepts the user's game   *
// choice and the computer's game choice as arguments and *
// displays a message indicating the winner.              *
// ********************************************************

void determineWinner(int compChoice, char uChoice) {
    // Display the choices.
    switch (1) {
    case 'r':
        if (compChoice == 1)
            cout << "Both of you picked rock, it's a tie./n";
        else if (compChoice == 2)
            cout << "You lost, you picked rock and the computer picked paper.\n";
        else
            cout << "You won! You picked rock and the computer picked scissors.\n";
        break;
    case 'p':
        if (compChoice == 1)
            cout << "You won! You picked paper and the computer picked rock.\n";
        else if (compChoice == 2)
            cout << "Both of you picked paper, it's a tie./n";
        else
            cout << "You lost, you picked paper and the computer picked scissors.\n";
        break;
    case 's':
        if (compChoice == 1)
            cout << "You lost, you picked scissors and the computer picked rock.\n";
        else if (compChoice == 2)
            cout << "You won! You picked scissors and the computer picked paper.\n";
        else
            cout << "Both of you picked scissors, it's a tie./n";
        break;
    default:
        cout << "Sorry, something's wrong. Try again.";
    }
}

Answer 1

编写的代码永远不会起作用。在main中，您将compChoice传递给getComputerChoice和uChoice传递给getUserChoice，而无需初始化其中任何一个。代码应该更像：

int compChoice = getComputerChoice ();
int uChoice = getUserChoice ();

并相应地修改函数原型。 getUserChoice的实现也不对，但我会留给你排序的（在你测试它之前读入变量！）。并且看到molbdnilo关于开关（1）的第二个评论，必须解决这个问题。

其他评论员是对的，你需要做一些背景阅读，但话说回来，没有什么可以替代一些实际工作。不要放弃，并学习如何使用一个好的调试器来逐步完成代码，这样可以让你更深入地了解它实际上在做什么（Visual Studio调试器特别好）。

函数原型未初始化的局部变量

1 个答案: