Выполнение метода дважды - PullRequest
       4

Выполнение метода дважды

0 голосов
/ 21 февраля 2020

Я создаю «Угадайку» для проекта класса. Идея состоит в том, что пользователь попытается угадать число, а GUI покажет, сколько попыток у него осталось. MyCustomTooHighException и MyCustomTooLowException оба расширяют исключение. Итак, моя проблема в том, что всякий раз, когда я запускаю свой код, он кажется go через мой метод GuessCheckNumber дважды, и я не слишком уверен, почему.

Это мой метод GuessTheNumberMethod:

public class GuessTheNumber extends JFrame{

public int random;


public int attempts;
//public int guess;
private String title;

public GuessTheNumber(){
    attempts = 0;
}
public int checkGuess(int guess) {

    Random rand = new Random();
    random = rand.nextInt(100) +1;
    MyCustomException tj = new MyCustomException();


        try {
            System.out.println(guess);
            if (guess > random) {
                throw new MyCustomTooHighException(guess);
            } else if (guess < random) {
                throw new MyCustomTooLowException(guess);
            }

        } catch (MyCustomTooHighException e) {
            attempts++;
            JOptionPane.showMessageDialog(null, "The value is smaller", "Alert", JOptionPane.ERROR_MESSAGE);

        } catch (MyCustomTooLowException e) {
            attempts++;
            JOptionPane.showMessageDialog(null, "The value is bigger", "Alert", JOptionPane.ERROR_MESSAGE);

        } finally {


            if(guess == random)
            {
                JOptionPane.showMessageDialog(null, "You Won!", "Alert", JOptionPane.ERROR_MESSAGE);
            }else if(guess != random && attempts<10)
                JOptionPane.showMessageDialog(null, "Guess Again\nYou have "+ (10-attempts) +" attempts left." , "Alert", JOptionPane.ERROR_MESSAGE);
            if(attempts ==10){
                JOptionPane.showMessageDialog(null, "Game Over!", "Alert", JOptionPane.ERROR_MESSAGE);
            }
            return 10-attempts;
        }


 }

}

А это класс JPanel:

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.NumberFormat;

public class GuessGamePanel extends JPanel {

private GuessTheNumber guess;
JButton submitButton;
JTextField gField;
JLabel aField;
public GuessGamePanel(){

    setLayout(new GridLayout(5,2));
    setBackground(Color.lightGray);

    guess = new GuessTheNumber();
    add(new JLabel("Pick a number between 1 -10."));
    add(new JLabel("Guess:"));
    gField = new JTextField("0");
    add(gField);
    submitButton = new JButton("Submit");
    add(submitButton);
    submitButton.addActionListener(new ButtonListener());
    aField = new JLabel();
    add(new JLabel("Attempts Left:"));
    add(aField);
}

private class ButtonListener implements ActionListener{

    public void actionPerformed(ActionEvent e) {

        int playsguess;
        if(submitButton ==e.getSource()){
            playsguess=Integer.parseInt(gField.getText());
            guess.checkGuess(playsguess);
            aField.setText(String.valueOf(guess.checkGuess(playsguess)));
        }


    }



  }

}

1 Ответ

0 голосов
/ 21 февраля 2020

Строка

guess.checkGuess(playsguess);

не имеет смысла, так как метод снова вызывается в следующей строке:

aField.setText(String.valueOf(guess.checkGuess(playsguess)));

Просто удалите первую строку.

...