0

I made a program that has a JFrame that contains a JTextField, a button, and two JLabels. When a number is typed into the JTextField, either pressing the enter key or clicking on the JButton should display the number in scientific notation on the second JLabel. When I hit the enter key, it works, however, when I click on the JButton, it does not. It gives me a NumberFormatException: empty string.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class MyMath extends JPanel implements ActionListener
{
   private JTextField textField;
   private static JLabel textArea;
   private static JLabel comment;
   private JButton button;
   private static JFrame frame;

public MyMath()
{
    comment = new JLabel("Enter a number");
    textField = new JTextField();
    textField.setSize(new Dimension(10 , 10));
    textField.addActionListener(this);
    button = new JButton("Go");
    button.addActionListener(this);
}

    public static void addComponentsToPane(Container pane) 
    {

    textArea = new JLabel(" ");
    pane.setLayout(new BoxLayout(pane, BoxLayout.PAGE_AXIS));
    pane.add(new MyMath().textField);
    pane.add(new MyMath().button);
    pane.add(new MyMath().comment);
    pane.add(textArea);
}

public void actionPerformed(ActionEvent evt) 
{
    String text = textField.getText();
    textArea.setText(SciNotation.convertToSciNotation(text));
    textArea.setHorizontalAlignment(SwingConstants.CENTER);
    comment.setText(text + " in Scientific Notation:");
    textField.selectAll();
}

private static void createAndShowGUI() 
{  
    frame = new JFrame("Scientific Notation");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    addComponentsToPane(frame.getContentPane());
    Container bg = frame.getContentPane();
    Dimension d = new Dimension(300, 150);   
    bg.setPreferredSize(d);
    frame.setResizable(false);
    frame.getContentPane().setBackground(Color.GREEN);
    Dimension screen = Toolkit.getDefaultToolkit().getScreenSize();
    Point screenCenter = new Point (screen.width/2 , screen.height/2);
    Point center = new Point(screenCenter.x - (150), screenCenter.y - (75));
    frame.setLocation(center);
    frame.pack();
    frame.setVisible(true);
}

public static void main(String[] args) 
{
    SwingUtilities.invokeLater(new Runnable() 
    {
        public void run() 
        {
            createAndShowGUI();
        }
    });
  }
}

Here is SciNotation.java

    import java.awt.Component;
    import java.awt.font.TextAttribute;
    import java.text.AttributedString;

public class SciNotation
{
  public static String convertToSciNotation(String num)
  {
    num = num.replaceAll("," , "");
    if (num.contains("E"))                    //not working
    {
        String[] partsE = num.split("E"); 
        String beforeE = partsE[0];
        String afterE = partsE[1];
        char first = num.charAt(0);
        num = first + beforeE;
    }

    double number = Double.parseDouble(num);
    double resultNumber = 0;
    int power = 0;
    String numString = Double.toString(number); 
    String[] parts = numString.split("\\.");
    String decimals = parts[1];
    int decimalInt = Integer.parseInt(decimals);
    int numberInt = (int) number;

    if(number > -1 && number < 1)
    {
        String[] low = numString.split("\\.");
        String afterLow = low[1];
        for(int i = 1; i < 10; i++)
        {
            String decNums = Integer.toString(i);
            afterLow = afterLow.replaceAll(decNums, "");
            int zeros  = afterLow.length();
            power = -1 * (zeros + 1);
            resultNumber = number * Math.pow(10, zeros + 1);  
            decimals = "";
        }
    }

    if(decimalInt == 0)
    {
        decimals = "";
    }

    if( number >= 10)
    {
        power = Integer.toString(numberInt).length() - 1;   
        resultNumber =  numberInt/(Math.pow(10,(power)));
    }

    if((number >= 1 && number < 10) || (number <= -1 && number > -10))
    {
        resultNumber = number;
        decimals = "";
    }

    if(number <= -10)
    {
        power = Integer.toString(numberInt).length() - 2;
        resultNumber =  numberInt/(Math.pow(10,(power)));
    }

    return resultNumber + decimals + " x 10^" + power;

}
 }
Metalskin
  • 3,998
  • 5
  • 37
  • 61
obamaself
  • 35
  • 1
  • 6
  • Please post the stack trace for your exception. Shot in the dark, check Double.parseNumber. Its probably trying to parse a string which cannot parsed as a double. – William Morrison Jan 21 '14 at 03:39
  • Instead of a `JTextField`, use a `JSpinner` as seen [here](http://stackoverflow.com/a/10021773/418556). And for better help sooner, post an [MCVE](http://stackoverflow.com/help/mcve) (like seen in the linked answer). It doesn't need to be that short, but it should be one source file, and contain only a text field or spinner, a button, and possibly a label to show the result. – Andrew Thompson Jan 21 '14 at 03:46

1 Answers1

5

This bug is pretty tricky. You constructed three MyMath instances. So your code was holding a wrong reference to your JTextFiled instance. That's why you couldn't get the right input.

public static void addComponentsToPane(Container pane) {

    textArea = new JLabel(" ");
    pane.setLayout(new BoxLayout(pane, BoxLayout.PAGE_AXIS));
    pane.add(new MyMath().textField);  //MyMath instance 1
    pane.add(new MyMath().button);  //MyMath instance 2
    pane.add(new MyMath().comment);  //MyMath instance 3
    pane.add(textArea);
}

Here is the right code:

public static void addComponentsToPane(Container pane) {

        MyMath myMath = new MyMath();

        textArea = new JLabel(" ");
        pane.setLayout(new BoxLayout(pane, BoxLayout.PAGE_AXIS));
        pane.add(myMath.textField);
        pane.add(myMath.button);
        pane.add(myMath.comment);
        pane.add(textArea);
    }

:)

Weibo Li
  • 3,565
  • 3
  • 24
  • 36