2016-05-31 10 views
1
if(letterGuessBoolean == true) { 
     System.out.println("Nice job! That was correct!"); 
     for (position = 0; position < pickRandomWord.length(); position++) { 
     if (pickRandomWord.charAt(position) == letterGuess) { 
      System.out.print(letterGuess); 
     } 
     else { 
      System.out.print(unknownLetters); 
     } 
     } 
    } 

ループは正しく単語を保存しますが、未知の文字は決して正しい方法を保存しません。JAVA - 未知の文字で正しい文字を入力し続けるハングマンゲーム

pickRandomWord.charAt(position) == letterGuess 

あなたは文字が以前に推測されている、何とかしてどこかに覚えておく必要があります。これはあなたがチェックしている唯一のものですので、

+0

申し訳ありませんが、updateLetterGuess = letterGuess;を無視してください。それは何もせず、削除することを忘れてしまった。 –

+0

残りのコードを参照する必要があります。推測された文字はどのように格納されていますか? –

+0

forループのインデックスをインクリメントする外に変数を割り当てるものは表示されません。あなたのコードをもう少し見せてくれますか? – RamenChef

答えて

1

あなたは、唯一のlatestly推測文字をプリントアウトしています。 1つの亜種は次のようなものです:

String pickRandomWord = ""; // select your random word 
char[] displayOutput = new char[pickRandomWord.length()]; 
for(int n = 0; n < displayOutput.length; ++n) 
    displayOutput[n] = '-'; 

/* ... */ 

if(letterGuessBoolean) // do not compare against true, if it is already boolean! 
{ 
    for(int position = 0; position < pickRandomWord.length(); ++position) 
    { 
     if (pickRandomWord.charAt(position) == letterGuess) 
     { 
      displayOutput[position] = letterGuess; 
     } 
     System.out.print(displayOutput[position]); 
    } 
} 
+0

私はブール値のフラグについて考えていましたが、char []ではおそらく値を変更するのが100倍簡単でした。それはあなたのためにそれをoverthinkingそれは –

+0

@Aconcaguaはとにかくあなたが時間があれば本当に迅速にあなたにメッセージすることができますか?私はほとんどそれを持っていると思うが、それでも問題がなければまだ助けが必要ですか? –

+0

@EvanVukasinovic申し訳ありませんが、[ここ](http://meta.stackexchange.com/questions/431/any-way-to-send-a-personal-message-to-another-user)または[here] (http://meta.stackexchange.com/questions/57537/how-do-i-contact-other-users)。ここでコメントを残しているオプション(またはここでは2:45ですが、今は寝る予定です...)。 – Aconcagua

関連する問題