NumberFormatException и другие проблемы при создании калькулятора в Java

1

Поэтому я делаю калькулятор Java и хожу в некоторые ошибки при нажатии функциональных кнопок (он отлично компилируется, хотя). Ошибка в 1-й строке моего CMD была исключением NumberFormat Exception. После некоторого поиска я понял, что где-то в своем коде я пытался получить двойную строку из пустой строки (в основном то, что я пытаюсь сделать, это разобрать двойной текст с надписью). Я предполагаю, что эта часть:

// if statement that puts the labels text into the first or second number
    if(firstNumber)
    {
        number1 = Double.parseDouble(label.getText().trim());
    } else {
        number2 = Double.parseDouble(label.getText().trim());
    }

В основном проблемы возникают, когда я нажимаю кнопку /* + или - я получаю NumberFormatExeption: для строки ввода "/" и т.д. Я не могу понять, как мне исправить эту ошибку (я все еще (несколько) начинающий на Java). Дальше по строке ошибок (было довольно много) было множество ошибок, которые я не понимал, таких как EventDispatchThread, EventQueue и многие другие. Я не мог найти объяснения своего уровня опыта, поэтому я прошу помочь здесь.

Цифровые кнопки работают нормально.

При запуске: [ http://gyazo.com/71cb4dde449ccf7ece44017388a71a0f]

Ввод чисел: [ http://gyazo.com/5c7ab6c54ac6da180845c66866d66f8f]

Все остальные кнопки дают ошибки в моем CMD.

Здесь мой код (промежуток может быть испорчен в некоторых частях):

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;
// import for Nimbus look
import javax.swing.UIManager.*;

public class Calculator extends JFrame implements ActionListener
{
    private JPanel bottom = new JPanel(); private JPanel top = new JPanel();
    private JLabel label = new JLabel(" ");
    private JButton[] buttons = new JButton[16];
    // booleans for calculator functions
    boolean add = false, substract = false, devide = false, multiply = false, firstNumber = true;
    // numbers that will be calculated
    double number1, number2;

    public Calculator()
    {
        setLayout(new BoxLayout(getContentPane(), BoxLayout.PAGE_AXIS));

        // bottom panel
        bottom.setPreferredSize(new Dimension(100,100));
        bottom.setLayout(new BorderLayout());

        //add bottom panel to frame
        add(bottom);

        // top panel
        top.setPreferredSize(new Dimension(300,400));
        top.setLayout(new GridLayout(4,4,3,3));
        top.setBackground(Color.BLACK);
        //dont add top panel to frame: you want top to be on bottom

        // add top panel to bottom panel
        bottom.add(top);

        // label
        label.setFont(new Font("Courier", Font.PLAIN, 20));
        label.setBackground(Color.BLACK);
        label.setForeground(Color.WHITE);
        label.setHorizontalAlignment(SwingConstants.RIGHT); // text is right-aligned
        label.setOpaque(true);

        // add the label to the bottom panel
        bottom.add(label, BorderLayout.NORTH);

        // creating buttons
        for(int i = 0; i < buttons.length; i++)
        {
            buttons[i] = new JButton("789/456*123+c0=-".substring(i, i+1));
            buttons[i].addActionListener(this);
           // add them to the top panel
           top.add(buttons[i]);
        }

        // Nimbus look
        try {
            for (LookAndFeelInfo info : UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (Exception e) {
           // If Nimbus is not available, you can set the GUI to another look and feel.
           //set to default somehow o.o
        }

        // frame setters
        setTitle("Calculator");
        setSize(400,400);
        setVisible(true);
        setResizable(false);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
    }

    public void Clear()
    {
        label.setText(" ");
    }

    @Override
    public void actionPerformed(ActionEvent e)
    {
        //if for all function buttons
        if(e.getSource() == buttons[3])
        {
           // devide
           devide = true;

           substract = false;
           add = false;
           multiply = false;

           firstNumber = false;
           Clear();

           label.setText("/");
        }

        if(e.getSource() == buttons[7])
        {
           // multiply
           multiply = true;

           substract = false;
           devide = false;
           add = false;

           firstNumber = false;
           Clear();

           label.setText("*");
        }

        if(e.getSource() == buttons[11])
        {
           // add
           add = true;

           substract = false;
           devide = false;
           multiply = false;

           firstNumber = false;
           Clear();

           label.setText("+");
        }

        if(e.getSource() == buttons[12])
        {
           // clear
           label.setText("0");
           number1 = 0.00;
           number2 = 0.00;

           add = false;
           substract = false;
           devide = false;
           multiply = false;

           firstNumber = true;
           Clear();
        }

        if(e.getSource() == buttons[15])
        {
           // substract
           substract = true;

           add = false;
           devide = false;
           multiply = false;

           firstNumber = false;
           Clear();

           label.setText("-");
        }

        // for loops that add the numbers on the buttons to the label
        for(int i = 0; i < 3; i++)
        {
           if(e.getSource() == buttons[i])
           {
               if(label.getText() == "0")
               {
                   label.setText("");
                   label.setText(label.getText() + buttons[i].getText());
               } else {
                   label.setText(label.getText() + buttons[i].getText());
               }
           }
        }

        for(int i = 4; i < 7; i++)
        {
           if(e.getSource() == buttons[i])
           {
               if(label.getText() == "0")
               {
                   label.setText("");
                   label.setText(label.getText() + buttons[i].getText());
               } else {
                   label.setText(label.getText() + buttons[i].getText());
               }
           }
        }

        for(int i = 8; i < 11; i++)
        {
           if(e.getSource() == buttons[i])
           {
               if(label.getText() == "0")
               {
                label.setText("");
                label.setText(label.getText() + buttons[i].getText());
               } else {
                label.setText(label.getText() + buttons[i].getText());
               }
           }
        }

        for(int i = 13; i < 14; i++)
        {
           if(e.getSource() == buttons[i])
           {
             if(label.getText() == "0")
             {
                label.setText("");
                label.setText(label.getText() + buttons[i].getText());
             } else {
                label.setText(label.getText() + buttons[i].getText());
             }
         }
     }

     // if statement that puts the labels text into the first or second number
     if(firstNumber)
     {
        number1 = Double.parseDouble(label.getText().trim());
     } else {
        number2 = Double.parseDouble(label.getText().trim());
     }

     // calculation
     if(e.getSource() == buttons[14])
     {
        // calculate
        if(devide){number1 = ((double)(number1) / (double)(number2)); }
        if(multiply){number1 = ((double)(number1) * (double)(number2)); }
        if(add){number1 = ((double)(number1) + (double)(number2)); }
        if(substract){number1 = ((double)(number1) - (double)(number2)); }
        label.setText(Double.toString(number1));
     }

 }

 public static void main(String[] args)
 {
    new Calculator();
 }
}

Наконец, вычисления, выполненные калькулятором, неверны. Я также не могу обернуть вокруг себя то, что вызывает это. Пожалуйста, имейте в виду, что я начинающий на Java, и это мой первый вопрос о stackoverflow. Заранее спасибо за то, что помог мне, кто бы ни был :)

UPDATE: я исправил ошибки, поставив свой код следующим образом:

if(e.getSource() == buttons[15])
    {
        // substract
        substract = true;

        add = false;
        devide = false;
        multiply = false;

        firstNumber = false;
        isNumberKey = false;

        if(isNumberKey)
        {
            if(firstNumber)
            {
                label.setText(label.getText().replace("/",""));
                label.setText(label.getText().replace("*",""));
                label.setText(label.getText().replace("+",""));
                label.setText(label.getText().replace("-",""));
                number1 = Double.parseDouble(label.getText().trim());
            } else {
                label.setText(label.getText().replace("/",""));
                label.setText(label.getText().replace("*",""));
                label.setText(label.getText().replace("+",""));
                label.setText(label.getText().replace("-",""));
                number2 = Double.parseDouble(label.getText().trim());
            }
        }

        Clear();

        label.setText("-");
    }

Все, что мне нужно сделать, это исправить расчеты... Спасибо за помощь всем!

  • 2
    Добро пожаловать на ТАК! Всякий раз, когда вы сталкиваетесь с необработанными исключениями, вы найдете трассировку стека с ценной информацией о том, в какую строку было сгенерировано исключение. Добавьте это исключение к вопросу, отредактировав его. Кроме того, укажите, какая именно это строка, поскольку у нас нет номеров строк.
  • 0
    @Magnilex Спасибо! под stacktrace вы имеете в виду список ошибок? я должен отредактировать свой пост или мой код? Я добавлю строки в секунду.
Показать ещё 3 комментария
Теги:
calculator
numberformatexception

2 ответа

0
Лучший ответ

Невозможно вникать в вашу логику, но здесь есть некоторые намеки:

Во-первых, вам не нужно догадываться, где ошибка. В случае неперехваченного исключения, как здесь, поток, который его создал, будет печатать трассировку стека на консоль. Это выглядит так:

Exception in thread "AWT-EventQueue-0" java.lang.NumberFormatException: For input string: "/"
at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1222)
at java.lang.Double.parseDouble(Double.java:510)
at mypackage.Calculator.actionPerformed(Calculator.java:229)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6297)

Отсюда вы видите, что исключение произошло в потоке с именем AWT-EventQueue-0, который представляет собой стандартный поток диспетчеризации swing (все приложения GUI обрабатывают события в одном потоке). Еще одна вещь, которую вы ищете в трассировке стека, - это найти, какую строку в вашем коде бросили. Вот он:

at mypackage.Calculator.actionPerformed(Calculator.java:229)

В источнике это строка:

number2 = Double.parseDouble(label.getText(). trim());

так что вы догадались. Что вы можете сделать, так это добавить еще один булев, который будет установлен только при нажатии кнопки с цифрой, а затем измените эту часть кода на что-то вроде:

if( isNumber){
  if(firstNumber)
    {
        number1 =     Double.parseDouble(label.getText().trim());
    } else {
        number2 = Double.parseDouble(label.getText().trim());
    }
}

Кроме того, вместо использования нескольких булевых флагов для операции используйте тип перечисления Java

enum Operation {devide, substract, add, multiply};

ваш код будет более читабельным и "java". Хорошая отправная точка, просто вперед!

  • 0
    Большое спасибо за объяснение! я только что понял, что на самом деле делал мой код ...> <Я пока не знаю с enum в java, так что я, вероятно, пока не использую это, но спасибо за предложение, я посмотрю на него.
0

Отказ от ответственности: я нахожусь на своем телефоне, поэтому трудно разобрать ваш код; с учетом сказанного, я считаю, что ваше числовое исключение относится к тому, что хранится в тексте метки.

Из внешнего вида, если вы нажмете кнопку оператора (например, умножьте), labelText будет "*", тогда, если вы нажмете цифровую кнопку (например, 5), labelText будет "* 5". Наконец, если вы нажмете кнопку ввода, чтобы перейти к вычислению ответа, будет Double.parseDouble(label.getText().trim()), который будет вызывать исключение NumberFormatException. Используя флаги операций, похоже, что вам даже не нужно хранить оператора (если вы не пытаетесь отобразить оператора для пользователя); в этом случае вам нужно удалить оператор из строки, прежде чем пытаться разобрать двойной.

label.setText(label.getText().replace("/",""));
label.setText(label.getText().replace("*",""));
label.setText(label.getText().replace("+",""));
label.setText(label.getText().replace("-",""));

Что касается других исключений, в eclipse вы можете установить точку останова для остановки программы, когда возникает конкретное исключение. Это поможет вам отладить ваш код дальше.

подробнее об этом здесь

  • 0
    Спасибо за ваш ответ! Ваше описание верно, знаете ли вы какие-нибудь причудливые трюки, чтобы удалить операторов? Все, что я могу думать, это .replace ("/, *, -, +", "") <не уверен, работает ли это.
  • 0
    Я добавил код, который вы редактировали, теперь я получаю NumberFormatException: пустая строка gyazo.com/c8b19902af9fc95e6690b9ee6ec6759c
Показать ещё 3 комментария

Ещё вопросы

Сообщество Overcoder
Наверх
Меню