Поэтому я делаю калькулятор 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("-");
}
Все, что мне нужно сделать, это исправить расчеты... Спасибо за помощь всем!
Невозможно вникать в вашу логику, но здесь есть некоторые намеки:
Во-первых, вам не нужно догадываться, где ошибка. В случае неперехваченного исключения, как здесь, поток, который его создал, будет печатать трассировку стека на консоль. Это выглядит так:
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". Хорошая отправная точка, просто вперед!
Отказ от ответственности: я нахожусь на своем телефоне, поэтому трудно разобрать ваш код; с учетом сказанного, я считаю, что ваше числовое исключение относится к тому, что хранится в тексте метки.
Из внешнего вида, если вы нажмете кнопку оператора (например, умножьте), 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 вы можете установить точку останова для остановки программы, когда возникает конкретное исключение. Это поможет вам отладить ваш код дальше.