java - 我的 Hangman 代码不会立即记录错误的猜测，而是打印双下划线等等-6ren

java - 我的 Hangman 代码不会立即记录错误的猜测，而是打印双下划线等等

转载作者：搜寻专家更新时间：2023-11-01 03:18:54

我在复制 Hangman 游戏时遇到三个主要问题:

“填写”部分随机打印两次，正确的猜测随机记录为错误

并且不正确的猜测并不总是被计算在内
这是主类的代码:

    public class Main 
    {

        public static void main(String[] args) 
        {
            Hangman manHang = new Hangman();    
        }

    }

The Hangman class:

    import java.util.Random;


    public class Hangman
            {
                Random r;
                GetData get;
                String[] Bank = {"consider","minute","accord","evident","practice","intend","concern","commit","issue","approach","establish","utter","conduct","engage","obtain","scarce","policy","straight","stock","apparent","property","fancy","concept","court","appoint","ambiguous","arbitrary","alliteration","arrogant","benevolent","belligerent","boycott","cynical","connotation","cessation","contemporary","craving","grandiose","gratuitous","guile","harbinger","impetuous","incandescent","indigent","inexorable","injunction","insipid","insurgent","languish","magnate","abjure","abrogate","abstemious", "acumen", "antebellum","auspicious","belie","bellicose","bowdlerize","chicanery","chromosome","churlish","circumlocution","circumnavigate","deciduous","deleterious","diffident","enervate","enfranchise","epiphany","equinox","evanescent","expurgate","facetious", "fallacious"};
                String word;//Stores the random word used
                boolean finished = false;
                int incGuessCount = 0;
                boolean[] lettersFound;//Used to mark which letters have been found
                String guessedLetter=" ";//Used to store guesses
                boolean playerHasWon = false;

            public Hangman()
            {
                r = new Random();
                get = new GetData();
                word=Bank[r.nextInt(Bank.length)]; //Selects a random word and assigns word the value
                lettersFound = new boolean[word.length()]; //Creates a boolean array the length of the word
                int incGuessCount = 0;

                while(incGuessCount<5 && playerHasWon == false)
                {
                    drawGallows(); //Show the Gallows depending on how many incorrect guesses there are
                    displayWord();
                    getGuess();
                    checkGuess();
                    //checkVictory();

                }

                if (incGuessCount>=5)
                {
                    System.out.print('\u000C');//Clears the screen
                    fiveWrong();//Displays full Hangman
                    System.out.println("You have lost! The word was "+word);
                }
            }

        public void getGuess()
        {
            System.out.println("\u000C");
            System.out.println(" ");
            System.out.println("What letter do you guess?");
            System.out.println("You have "+(5-incGuessCount)+" guesses left.");
            System.out.print("Enter guess:");
            guessedLetter = get.aWord();//Uses scanners to take in the guesses
        }


        public boolean displayWord()            
        {
            boolean goodGuess = false;//Assumes guess is bad automatically
            char letter = guessedLetter.charAt(0);

            for(int i = 0;i<word.length();i++)//Goes through all the letters to check guess's status
                if (lettersFound[i]==true)//Checks if a letter was already revealed at that position
                {
                 System.out.print(word.charAt(i)+" ");
                }

                    else if (word.charAt(i)==letter)//Prints the correctly guessed letter at the position
                    {
                        System.out.print(word.charAt(i)+" ");
                        lettersFound[i] = true;
                        goodGuess = true;
                    }
                        else//Fills in non-applicable spaces with an underscore
                            System.out.print("_ ");


            return goodGuess;           
        }   

        public void checkGuess()
            {
                 if(!displayWord() && incGuessCount==5)
                     fiveWrong();
                 else if(!displayWord() && incGuessCount<5)
                    {
                     incGuessCount++;
                     drawGallows();
                     getGuess();
                     displayWord();
                    }

                 else
                 {
                     drawGallows();
                     getGuess();
                     displayWord();
                 }
            }   

        public void defaultMan()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |       ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void oneWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |       \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void twoWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void threeWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |     /| ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ "); 
            System.out.println(" ");
        }

        public void fourWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |     /|\\ ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void fiveWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |     ( ) ");                        
            System.out.println(" |     /|\\ ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
            System.out.println("You have lost! The word was "+word+".");
            System.out.println("Rerun the program to try again.");
            finished=true;
        }

        public void drawGallows()
        {
            if(incGuessCount==0)
            {
                defaultMan();                   
            }

            if(incGuessCount==1)
            {
                oneWrong();                   
            }

            if(incGuessCount==2)
            {
                twoWrong();                  
            }

            if(incGuessCount==3)
            {
                 threeWrong();                 
            }

            if(incGuessCount==4)
            {
                fourWrong();                   
            }

            if(incGuessCount==5)
            {
                fiveWrong();

            }

            }
        } 

And the GetData class:

    import java.util.Scanner;

    public class GetData
    { 
      private Scanner input; 

      public GetData()//Produces a scanner to take in input
      { input = new Scanner(System.in); } 

      public String aWord()//Gets the input as a guess/string
      { return input.next(); } 

      public int aNumber()//Gets the input as a number
      { return input.nextInt(); }

    }

抱歉组织不力，我是编码新手。在多人的帮助下，我已经研究了我的代码数小时，但我们就是无法弄清楚我的逻辑在哪里失败了，而且显然至少在几个方面是这样。感谢任何帮助，即使它只是指出更多次要的逻辑缺陷或无用变量。谢谢。

更新:非常感谢那些做出贡献的人，这些答案帮助我修复了程序的所有错误。现在，我的获胜检测又遇到了一个问题。我创建了一个变量 gameOver，当用户超过他的错误猜测次数时它存储 true 并且我还让它在打印字母而不是下划线时递增。然后我创建了两个 if 语句:

if(corLetters == word.length())
                    gameOver = true;

            if (incGuessCount<5 && gameOver)
            {
                System.out.println("\u000c");
                System.out.println("Congratulations!");
                System.out.println("You have won!");
                System.out.println("Rerun the program to try again.");
            }

但游戏并未记录获胜。再次感谢您的所有帮助，如果您有任何建议来修复或改进我的获胜检测，我们将不胜感激 :)

这是我在此线程中解决问题后的新 Hangman 类(class):

import java.util.Random;


public class Hangman
        {
            Random r;
            GetData get;
            String[] Bank = {"consider","minute","accord","evident","practice","intend","concern","commit","issue","approach","establish","utter","conduct","engage","obtain","scarce","policy","straight","stock","apparent","property","fancy","concept","court","appoint","ambiguous","arbitrary","alliteration","arrogant","benevolent","belligerent","boycott","cynical","connotation","cessation","contemporary","craving","grandiose","gratuitous","guile","harbinger","impetuous","incandescent","indigent","inexorable","injunction","insipid","insurgent","languish","magnate","abjure","abrogate","abstemious", "acumen", "antebellum","auspicious","belie","bellicose","bowdlerize","chicanery","chromosome","churlish","circumlocution","circumnavigate","deciduous","deleterious","diffident","enervate","enfranchise","epiphany","equinox","evanescent","expurgate","facetious", "fallacious"};
            String word;//Stores the random word used
            boolean finished = false;
            int incGuessCount = 0;
            int corLetters = 0;
            boolean[] lettersFound;//Used to mark which letters have been found
            String guessedLetter=" ";//Used to store guesses
            boolean gameOver = false;


            public Hangman()
            {
                r = new Random();
                get = new GetData();
                word=Bank[r.nextInt(Bank.length)]; //Selects a random word and assigns word the value
                lettersFound = new boolean[word.length()]; //Creates a boolean array the length of the word


                do
                {
                    drawGallows(); //Show the Gallows depending on how many incorrect guesses there are
                    displayWord();
                    getGuess();
                    checkGuess();

                }
                while(incGuessCount<5 && gameOver == false);

                if (incGuessCount>=5)
                {

                    fiveWrong();//Displays full Hangman

                }

                if(corLetters == word.length())
                    gameOver = true;

                if (incGuessCount<5 && gameOver)
                {
                    System.out.println("\u000c");
                    System.out.println("Congratulations!");
                    System.out.println("You have won!");
                    System.out.println("Rerun the program to try again.");
                }


            }

        public void getGuess()
        {
            System.out.println("\u000C");
            System.out.println(" ");
            System.out.println("What letter do you guess?");
            System.out.println("You have "+(5-incGuessCount)+" guesses left.");
            System.out.print("Enter guess:");
            guessedLetter = get.aWord();//Uses scanners to take in the guesses
        }


        public boolean displayWord()            
        {
            boolean goodGuess = false;//Assumes guess is bad automatically
            char letter = guessedLetter.charAt(0);

            for(int i = 0;i<word.length();i++)//Goes through all the letters to check guess's status
                if (lettersFound[i]==true)//Checks if a letter was already revealed at that position
                {
                 System.out.print(word.charAt(i)+" ");
                 corLetters++;
                }

                    else if (word.charAt(i)==letter)//Prints the correctly guessed letter at the position
                    {
                        System.out.print(word.charAt(i)+" ");
                        lettersFound[i] = true;
                        goodGuess = true;
                        corLetters++;
                    }
                        else//Fills in non-applicable spaces with an underscore
                            System.out.print("_ ");


            return goodGuess;           
        }   

        public void checkGuess() {
             boolean disW = displayWord();

             if (!disW && incGuessCount == 5)
              fiveWrong();
             else if (!disW && incGuessCount < 5) {
              incGuessCount++;

             }
            }


        public void defaultMan()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |       ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void oneWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |       \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void twoWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |      | ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void threeWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |     /| ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ "); 
            System.out.println(" ");
        }

        public void fourWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |      | ");                        
            System.out.println(" |     /|\\ ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
        }

        public void fiveWrong()
        {
            System.out.println('\u000C');
            System.out.println(" ________");
            System.out.println(" |      | ");
            System.out.println(" |     ( ) ");                        
            System.out.println(" |     /|\\ ");                 
            System.out.println(" |     / \\ ");                   
            System.out.println(" |_________ ");
            System.out.println(" ");
            System.out.println("You have lost! The word was "+word+".");
            System.out.println("Rerun the program to try again.");
            gameOver=true;
        }

        public void drawGallows()
        {
            if(incGuessCount==0)
            {
                defaultMan();                   
            }

            if(incGuessCount==1)
            {
                oneWrong();                   
            }

            if(incGuessCount==2)
            {
                twoWrong();                  
            }

            if(incGuessCount==3)
            {
                 threeWrong();                 
            }

            if(incGuessCount==4)
            {
                fourWrong();                   
            }

            if(incGuessCount==5)
            {
                fiveWrong();

            }

            }
        }

最佳答案

“填充”部分可以打印两次的原因是因为您正在使用来打印单词并确定字符是否正确。我的意思是当你在 checkGuess() 中有这个时:

if(!displayWord() && incGuessCount==5)
             fiveWrong();
else if(!displayWord() && incGuessCount<5)

您正在 if 语句的决策部分调用 displayWord()(打印单词并在猜测正确时返回 true)。这会导致单词打印在您的 checkGuess() 方法中。这可以通过制作 checkWord() 来解决，例如:

public boolean checkWord()            
{
    boolean goodGuess = false;//Assumes guess is bad automatically
    char letter = guessedLetter.charAt(0);
    for(int i = 0;i<word.length();i++){//Goes through all the letters to check guess's status
        if (word.charAt(i)==letter)
        {
            lettersFound[i] = true;
            goodGuess = true;
        }
    }

    return goodGuess;           
}

这将用于 if 语句的决策部分。

导致图表未及时更新的第二件事是您在 checkGuess() 中做的太多。 您要求他们提供新的猜测，而这实际上应该由主要的 while{}。编辑后的版本可能如下所示:

public void checkGuess()
{
    if(!checkWord() && incGuessCount==5)
        fiveWrong();
    else if(!checkWord() && incGuessCount<5)
    {
        incGuessCount++;
    }
}

此外，在您的构造函数中，您创建了另一个计数器，这使得循环在玩家失败后无法结束(因为在全局变量之前使用了同名的局部变量)请参见以下代码:

 r = new Random();
 get = new GetData();
 word=Bank[r.nextInt(Bank.length)];
 lettersFound = new boolean[word.length()];
 int incGuessCount = 0; // <<< This shouldn't be here

剩下的唯一事情就是进行获胜检测，我认为你已经准备好了!

编辑:

对于您的获胜检测，我喜欢您决定计算用户猜对的字符数的方式。唯一的问题是，在您的 displayWord() 中，每次打印单词时都会添加正确字母的数量，而不是仅在添加新的正确字符时添加。此更改将:

if (lettersFound[i]==true)//Checks if a letter was already revealed at that position
{
    System.out.print(word.charAt(i)+" ");
    corLetters++;
}
else if (word.charAt(i)==letter)//Prints the correctly guessed letter at the position
{
    System.out.print(word.charAt(i)+" ");
    lettersFound[i] = true;
    goodGuess = true;
    corLetters++;
}

进入这个:

if (lettersFound[i]==true)//Checks if a letter was already revealed at that position
{
    System.out.print(word.charAt(i)+" ");
    //Deleted line here
}
else if (word.charAt(i)==letter)//Prints the correctly guessed letter at the position
{
    System.out.print(word.charAt(i)+" ");
    lettersFound[i] = true;
    goodGuess = true;
    corLetters++;
}

另一个问题是您只检查游戏是否从用户赢外主要do while。这可以通过将 if 语句移动到 checkGuess() 中来解决。如:

public void checkGuess() {
    boolean disW = displayWord();

    if (!disW && incGuessCount == 5)
        fiveWrong();
    else if (!disW && incGuessCount < 5) {
        incGuessCount++;

    }

    if (corLetters == word.length()){
        gameOver = true;
    }
}

关于java - 我的 Hangman 代码不会立即记录错误的猜测，而是打印双下划线等等，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/37581445/

文章推荐： java - 复杂类型中的通用通配符边界

文章推荐： iphone - 是否可以将单词包装在 UITableViewSectionIndex 中？

android - 立即 toast
我希望在我的应用程序下载信息时显示 Toast 消息，但即使我将它放在我的代码之前，它也不会在下载完成后出现。将我的代码放在一个单独的线程中会引起很多麻烦，但是将 toast 放在一个单独的线程中也不
android - 应用内更新(立即)不重新启动应用
面临即时应用更新模式的问题。成功完成应用程序更新后，一切都关闭并且不重新启动应用程序。这就是问题所在。但是android文档说: A full screen user experience that
javascript - 时间选择器“立即”按钮不起作用
我有一张 table 我有一个 anchor 标记，
java - 如何强制*立即*将新记录保存到数据库中？
我正在开发一个具有两个线程的 Java/Seam/Hibernate/Glassfish 应用程序: 线程 #1 发送各种消息并将摘要信息写入 MySQL 数据库。线程 #2 定期轮询数据库(由 S
Java 三元(立即 if)求值
我找不到规范的相关部分来回答这个问题。在 Java 中的条件运算符语句中，是否同时评估真假参数？以下是否会抛出 NullPointerException Integer test = null; t
java - 我们如何在 JVM 启动时加载类(立即)？
大家下午好，我想知道是否有办法使类的静态 block 运行，即使类本身没有被引用？我知道它是延迟加载的，因此只需调用该类的任何函数即可开始启动该类，但是，我希望该类在任何调用之前启动，换句话说，
javascript - jquery prop() 立即(重点？)
我正在尝试使用 jQuery prop() 方法禁用元素(表单字段)。有两个字段，一个叫做fee，一个叫做currency。每当 fee 设置为 0 时，第二个字段 currency 将被禁用。这样做
swift - 立即 UIButton 旋转变换，带动画比例
我想为 UIButton 的缩放设置动画，并在完成后再次为缩放设置动画，但让它在没有动画的情况下旋转。我尝试将旋转变换放在没有持续时间的动画调用中，但不知何故它仍然成为缩放动画的一部分或替换缩放动画。
javascript - 根据用户输入使 Javascript 立即/立即可互换
在 js 代码中，我创建了 3 个按钮 --- 按钮 1...按钮 2...按钮 3和 3 个输入字段 --- 输入字段 1...输入字段 2...输入字段 3 从脚本开始所有按钮都被禁用只有当输入
Python:线程何时终止？显然不会(立即)返回
我正在使用一个 threading.Thread() 来完成它的工作并返回。它的返回记录在打印语句中，所以我确信有时候是这样的。然而，依靠 threading.active_count() 和 th
java - IntelliJ 的“立即”窗口
我正在使用 IntelliJ 9，我很好奇是否有任何与 Visual Studio“即时”调试窗口等效的 IntelliJ。在编辑器中选择所需的表达式，然后 ALT-F8 来评估表达式，但我希望能够在
jsf-2 - JSF 立即 ="true"取消按钮不起作用
我有一个两个标签页，一个标签是记录列表，点击记录会切换到编辑标签，编辑标签中有保存和取消按钮。现在，我单击记录 #1，进行一些编辑，然后单击取消按钮。当然我不想验证表单，因为它被取消了，所以我设置了
ios - 立即 dimissViewController 后，presentViewController 时为空白
我有一个 A viewController，首先，我呈现 B viewController，经过一些工作后，我需要关闭 B viewController 并呈现 C viewController，所以
java - 如何在没有按钮的情况下主动(立即)检查 JFrame 表单中文本框的值？
我希望能够在文本框中输入内容，当用户在文本框中输入内容时，我希望程序无需单击按钮即可自动读取文本框。例子:用户类型:“abcd”当用户输入时，程序会显示每个字母对应的数字。程序输出:“1234”
javascript - 如果任何表单输入发生更改，则更改(立即)提交按钮文本 [JS]
如果任何表单输入发生更改，如何立即更改提交按钮文本？ //This applies to whole form $('#test').change(function() { $("#send").
javascript - javascript 中的clearInterval 和 setInterval 立即
主要功能: var interval; function refreshId(sessio
R data.table 立即 reshape 列 block
假设我有一个包含这些列的 data.table nodeID hour1aaa hour1bbb hour1ccc hour2aaa hour2bbb hour2ccc .
javascript - Vimeo JS API - Froogaloop - 立即/过早完成事件调用？
根据vimeo js-api doc ，事件 finish - 当视频播放结束时触发。出于某种原因，我无法让它工作，finish 事件总是立即调用，我做错了什么吗？我试图让嵌入的视频在播放完毕后消
javascript - slideDown 立即 li 并向上滑动其他 li 元素
我想滑动当前ul元素下的所有li元素和slideDown li元素 $(document).ready(function(){ $("li").slideUp(); }); $(".nav")
javascript - Backbone Marionette 取消 itemview 上的所有事件 - 立即
我有一个表-compositeView，其中有行-itemView。每行都有许多事件 - 单击、更改等等。在某些状态下，我想“锁定”表。禁用按钮并取消事件。是否有一种好方法可以立即取消 itemv

搜寻专家

个人简介

我是一名优秀的程序员,十分优秀！

作者热门文章

滴滴打车优惠券免费领取

全站热门文章

首页

博学

6Ren·AI

商城

java - 我的 Hangman 代码不会立即记录错误的猜测，而是打印双下划线等等