CODEWITHSUNDEEP
javauser-interfaceparseintnumberformatexception
Nivalen
Nivalen

Reputation: 29

Can not resolve java.lang.NumberFormatException thrown by Integer.ParseInt

I am getting this error, but I can not seem to find want might be causing it. It says the error is on the line where I initialize the variable bet. If I remove that line and Integer.parseInt(betTextFieldAmount), then it says the error is on that initialization.

public class RollTheDice extends JFrame implements ActionListener {

    JButton roll = new JButton("Roll the Dice");
    JButton bet1 = new JButton("5$");
    JButton bet2 = new JButton("10$");
    JButton bet3 = new JButton("25$");
    JButton bet4 = new JButton("50$");
    JButton bet5 = new JButton("100$");

    JTextField betTextField = new JTextField(5);
    String betTextFieldAmount = betTextField.getText();
    int bet = Integer.parseInt(betTextFieldAmount);

    int money = 100;

    JLabel currentBet = new JLabel();
    JLabel currentMoney = new JLabel("Your money: " + money + "$");

    int myDice = new Random().nextInt(12) + 1;
    int AIDice = new Random().nextInt(12) + 1;

    public static void RollTheDice() {
        int myDice = new Random().nextInt(12) + 1;
        int AIDice = new Random().nextInt(12) + 1;
    }

    public RollTheDice() {
        setLayout(new FlowLayout());

        add(currentMoney);

        add(bet1);
        bet1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (money < 5) {
                    JOptionPane.showMessageDialog(null, "Not enough money!");
                } else {
                    money -= 5;
                    bet +=5;
                }
                currentMoney.setText(money + "$");
            }           
        });

        add(bet2);
        bet2.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (money < 10) {
                    JOptionPane.showMessageDialog(null, "Not enough money!");
                } else {
                    money -= 10;
                    bet +=10;
                }
                currentMoney.setText(money + "$");
            }           
        });

        add(bet3);
        bet3.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (money < 25) {
                    JOptionPane.showMessageDialog(null, "Not enough money!");
                } else {
                    money -= 25;
                    bet +=25;
                }
                currentMoney.setText(money + "$");
            }           
        });

        add(bet4);
        bet4.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (money < 50) {
                    JOptionPane.showMessageDialog(null, "Not enough money!");
                } else {
                    money -= 50;
                    bet +=50;
                }
                currentMoney.setText(money + "$");
            }           
        });

        add(bet5);
        bet5.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (money < 100) {
                    JOptionPane.showMessageDialog(null, "Not enough money!");
                } else {
                    money -= 100;
                    bet +=100;
                }
                currentMoney.setText(money + "$");
            }           
        });

        add(betTextField);
        betTextField.setEditable(false);
        add(roll);
        roll.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                if (myDice > AIDice) {
                    money += bet;
                    JOptionPane.showMessageDialog(null, "You won " + bet + "$");
                } else {
                    JOptionPane.showMessageDialog(null, "You lost " + bet + "$");
                }
                currentMoney.setText(money + "$");
                RollTheDice();
            }           
        });
    }

    @Override
    public void actionPerformed(ActionEvent arg0) {
        // TODO Auto-generated method stub

    }
}

Upvotes: 1

Views: 464

Answers (3)

Vilt&#233;r Panitz
Vilt&#233;r Panitz

Reputation: 84

Try to verify if you betTextField.getText() is not null. I'm presuming that your field has no value, so the getText will return null and the parse will fail

Upvotes: -1

Amit Bhati
Amit Bhati

Reputation: 5659

Integer.parseInt(betTextFieldAmount)is causing this error.
Check betTextFieldAmount to see whether it has a valid integer value or not.
Ensure that betTextFieldAmount should not be null, or contain any special characters or white spaces at the begining or end.

Upvotes: 3

codebox
codebox

Reputation: 20264

This is caused by the value of betTextFieldAmount not being a valid integer, so when you try to convert it using Integer.parseInt(betTextFieldAmount); an exception is thrown.

You should surround the parseInt call with a try...catch statement and display an appropriate error message to the user if this happens.

Even better, try to prevent the user entering non-numeric values in the first place by restricting which characters can be entered in the field, and add an emptiness check to catch the case where no value has been entered in the field.

Upvotes: 3

Related Questions