Java Simple Guessing Game GameZone

Question

これを実行しようとすると問題が発生し、ユーザーに推測を尋ねると、あまりにも低すぎると状態が低すぎたり、高すぎたり、正しくなったりします。番号が何であったかを述べる。これはこれまでのコードです。

import javax.swing.JOptionPane; 
public class RandomGuess2 { 
    public static void main(String[] args) { 
     // TODO Auto-generated method stub 
     int random; 

     final int MIN = 1; 
     final int MAX = 10; 

     int userNum; 
     String userInput; 

     userInput = JOptionPane.showInputDialog(null, "Please guess a number between 1 and 10"); 
     random = 1 + (int)(Math.random() * MAX); 
     userNum = Integer.valueOf(userInput); 
     int userDiff = (random - userNum); 

     if (userDiff < 0) { 
      JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO HIGH"); 

      if (userDiff > 0) 
       JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO LOW");   
      if (userNum == random) 
       JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is correct! Congratulations!"); 
      else 
       JOptionPane.showMessageDialog(null, "The number was " + random + "\nTry again Next time!"); 
     } 
    } 
} 

ご協力いただければ幸いです。

Answer 1

あなたのチェックが間違っていたので、それが働いていなかった理由は、それがusernumが高すぎると、それはそれからだった場合、それがチェックするかどうあなたが見て最初にチェックして、この

if (userNum == random){ 
      JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is correct! Congratulations!"); 
    } 
    else{ 

     if (userDiff < 0) { 
     JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO HIGH"); 
     } 
     if (userDiff > 0) 
     { 
      JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO LOW"); 
     } 
     JOptionPane.showMessageDialog(null, "The number was " + random + "\nTry again Next time!"); 
    } 

ようにする必要がありました

ユーザーナンバーが正しかったか低すぎるかを確認してください。ただし、numが高すぎる場合にのみコードが実行されました。

Answer 2

if-else-structureが正しくありません。私はこれがあなたが望むものだと思う：

if (userDiff < 0) { 
    JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO HIGH"); 
} else if (userDiff > 0) { 
    JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is TOO LOW"); 
} 

if (userNum == random) { 
    JOptionPane.showMessageDialog(null, "Your guess of " + userNum + " is correct! Congratulations!"); 
} else { 
    JOptionPane.showMessageDialog(null, "The number was " + random + "\nTry again Next time!"); 
}