Java converting integers to strings and calculating it

Question

Can anyone explain to me why my Homework keeps retuning a value of 1.0, I asked my teacher and he tells me "langauges are very similar I do not remember in this languge -.-..... you should not be learning this because you are in java 1 not java 2" ...... anyway im trying to get ahead and teach myself

              /**
 *  Author Rich Talcik, Do not Edit or modify
 *  Homework assignment2 
 *  09/11/13
 *  Problem 2.7 page 109
 *  
 */
private static final long serialVersionUID = 1L;
    JLabel jlabel;
    JFrame frame = new JFrame("home work");

    JTextField tfAnswer = new JTextField("Click to find travel Time");
    JTextField tfSpeed = new JTextField("Remvoe Text & Enter Speed here");
    JTextField tfDistance = new JTextField("Remove Text & Enter Distance Here");

    float dSpeed;
    float dDistance;
    String sTotal;
    String sSpeed;
    String sDistance;

     public shapeDrawing() {
         super(new BorderLayout());
         JButton button = new JButton("Calculate my Travel Time");

         //Converting the speed to an Integer from a String



         button.addActionListener(this);


         tfSpeed.addMouseListener(new MouseListener() {




            @Override
            public void mouseClicked(MouseEvent e) {
                // TODO Auto-generated method stub
                tfSpeed.setText("");
            }

            @Override
            public void mouseEntered(MouseEvent e) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mouseExited(MouseEvent e) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mousePressed(MouseEvent e) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mouseReleased(MouseEvent e) {
                // TODO Auto-generated method stub

            }
         });

         tfDistance.addMouseListener(new MouseListener() {


            @Override
            public void mouseClicked(MouseEvent arg0) {
                // TODO Auto-generated method stub
                tfDistance.setText("");
                tfAnswer.setText("");
            }

            @Override
            public void mouseEntered(MouseEvent arg0) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mouseExited(MouseEvent arg0) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mousePressed(MouseEvent arg0) {
                // TODO Auto-generated method stub

            }

            @Override
            public void mouseReleased(MouseEvent arg0) {
                // TODO Auto-generated method stub

            }

         });

         add(tfSpeed, BorderLayout.AFTER_LINE_ENDS);
         add(tfDistance, BorderLayout.LINE_START);

         add(tfAnswer, BorderLayout.PAGE_END);
         add(button, BorderLayout.PAGE_START);
         setBorder(BorderFactory.createEmptyBorder(20,20,20,20));

 }


     public void actionPerformed(ActionEvent e) {

         sSpeed = tfSpeed.getText();
         dSpeed = Integer.valueOf(sSpeed);

         //Converting Distance to integer from string
         sDistance = tfSpeed.getText();
         dDistance = Integer.valueOf(sDistance);             


         final float fTotal = (dDistance / dSpeed);


         tfAnswer.setText(String.valueOf(fTotal));
 }

can someone explained why tfAnswer.setText keeps returning the value of 1.0

Answer 1

You should read tfDistance instead of tfSpeed

         sSpeed = tfSpeed.getText();
         dSpeed = Integer.valueOf(sSpeed);

         //Converting Distance to integer from string
         sDistance = tfSpeed.getText(); //!!! copy-past error
         dDistance = Integer.valueOf(sDistance); 

Should be

         sSpeed = tfSpeed.getText();
         dSpeed = Integer.valueOf(sSpeed);

         //Converting Distance to integer from string
         sDistance = tfDistance.getText(); 
         dDistance = Integer.valueOf(sDistance); 

Answer 2

Basically, you're still dividing an integer by an integer (which is then cast to a float). Try this instead:

// Multiplying by 1.0 casts that answer to a float, so the
//     division by dSpeed will also result in a float.
float fTotal = (1.0 * dDistance) / dSpeed;