带窗户的石头剪刀布游戏

Question

我正在为学校作业制作摇滚、纸、剪刀游戏，但是当我尝试运行此脚本时，游戏无法正常运行。例如，当我单击 Rock 时，计算机只会选择相同或选择纸张。但玩家永远不会赢。我不知道如何解决这个问题，我已经尝试了几个小时。这是我的代码：public partial class MainWindow : Window{    string Computer;    string[] computer = { "Rock", "Paper", "Scissors" };    Random random = new Random();    int RandomType;    string PlayerPicks;    public MainWindow()    {        InitializeComponent();    }    private void StoneButton_Click(object sender, RoutedEventArgs e)    {        PlayerPicks = "Rock";        RandomType = random.Next(0, 2);        Computer = computer[RandomType];        Game();    }    private void PaperButton_Click(object sender, RoutedEventArgs e)    {        PlayerPicks = "Paper";        RandomType = random.Next(0, 2);        Computer = computer[RandomType];        Game();    }    private void ScissorsButton_Click(object sender, RoutedEventArgs e)    {        PlayerPicks = "Scissors";        RandomType = random.Next(0, 2);        Computer = computer[RandomType];        Game();    }    void Game()    {        string message = "The winner is: ";        string computerWins = "Computer!";        string playerWins = "Player!";        string draw = "N-Nobody?";        if (PlayerPicks == "Rock" && Computer == "Paper") // Player: Rock, Computer: paper = computer wins        {            MessageBox.Show(message + computerWins);        }        else if (PlayerPicks == "Rock" && Computer == "Scissors") // Player: Rock, Computer: Scissors = Player wins        {            MessageBox.Show(message + playerWins);        }   

Answer 1

问题 1

RandomType = random.Next(0, 2)只生成一个介于 0 和 1 之间的数字，您需要这样做RandomType = random.Next(0, 3)这将生成一个介于 0 和 2 之间的数字

问题 2

if (PlayerPicks == "Scissors" && Computer == "Scissor")检查计算机是否选择Scissor而不是Scissors只需将其更改为if (PlayerPicks == "Scissors" && Computer == "Scissors")

Answer 2

这是另一种方法。它已从您的代码中进行了极大的重构。首先，我在表单类中添加了一个私有System.Random成员字段：

private Random _rand = new Random();

然后，我设置了两个enums，一个用于“手势”（石头、纸、剪刀），另一个用于结果：

public enum HandSign

{

    Rock,           // rock beats scissors

    Paper,          // paper beats rock

    Scissors,       // scissors beats paper

}

public enum Result

{

    Draw,

    ComputerWins,

    PlayerWins

}

然后我创建了一些辅助函数，首先确定计算机的下一步动作：

private HandSign GetRandomPlay()

{

    var result = _rand.Next(0, Enum.GetValues(typeof(HandSign)).Length);

    return (HandSign) result;

}

接下来是确定获胜者的规则引擎：

private Result DetermineWinner(HandSign player, HandSign computer)

{

    if (player == computer)

    {

        return Result.Draw;

    }

    if (player == HandSign.Scissors && computer == HandSign.Rock)

    {

        return Result.ComputerWins;

    }

    if (player == HandSign.Rock && computer == HandSign.Scissors)

    {

        return Result.PlayerWins;

    }

    if (player > computer)

    {

        return Result.PlayerWins;

    }

    //finally, otherwise

    return Result.ComputerWins;

}

最后来一张“玩游戏”，在表格上的一把标签上展示了玩的结果：

 private void PlayGame(HandSign playerPicks)

 {

     PlayerPicksLbl.Text = playerPicks.ToString();

     var computerPicks = GetRandomPlay();

     ComputerPicksLbl.Text = computerPicks.ToString();

     WinnerLbl.Text = DetermineWinner(playerPicks, computerPicks).ToString();

 }

那时，我的三个按钮单击处理程序看起来像这样（这是 Rock 处理程序）：

 private void RockBtn_Click(object sender, EventArgs e)

 {

     PlayGame(HandSign.Rock);

 }

以这种方式构造事物的一个结果是将其扩展到“Rocks, Paper, Scissors, Lizard, Spock”，您需要做的就是向枚举添加两个条目，向两个条目添加HandSign更多逻辑DetermineWinner更多按钮/按钮处理程序。

另请注意，没有办法拼错“剪刀”，仍然可以编译！

Answer 3

在 Random.Next 上，上限是独占的，所以你应该这样做 RandomType = random.Next(0, 3);

此外在：

if (PlayerPicks == "Scissors" && Computer == "Scissor")你有“Scissor”而不是“Scissor s ”