Input continuously collecting

[Heather Carey]

This programs problem is that it will not allow more than one input and then it goes into no where land. If I do type a zero initially it takes me to the output statements. I'm to demonstrate the use of a method, that is why I'm averaging out my amount below and returning them to main.

I can't figure out why it won't continuously ask me to input a number until I input zero. Does the problem have something to do with my average method?

Thanks,
Heather

    public static void main(String[] args) {
        //Main Module
        //Present program detail
       JOptionPane.showMessageDialog(null,"This program will gather numbers and determine the sum and average of those numbers.");
       
       //Initialize variables
       float sum = (float) 0.0;
       int count = 0;
       float average = avg(sum, count); 
   
       String dataString = JOptionPane.showInputDialog(null,"Enter a float value (press 0 to exit):  ");
       float data = Float.parseFloat(dataString);
         while (data != 0) {
         sum += data; 
         count++;
         
         }
       
       //Print out the values
       JOptionPane.showMessageDialog(null, "The amount of numbers you entered is:  " + count);
       JOptionPane.showMessageDialog(null, "The sum of all your numbers you entered is:  " + sum);
       JOptionPane.showMessageDialog(null, "The average of all the numbers you entered is:  " + average);  
       
    }
       
       public static float avg(float sum, int count) {
           float result = (float) 0.0;
           float average = sum/count;
                    
           return result;
                     
         }
        
    }

[Joanne Neal]

In your while loop you never change the value of data so the loop will never exit. (In the case of zero it never enters the loop) You need to move your prompt and calculation inside the loop and probably make it a do while loop

float data;
do {
    String dataString = JOptionPane.showInputDialog(null,"Enter a float value (press 0 to exit):  ");  
    data = Float.parseFloat(dataString);  
    if  (data != 0) {
         sum += data;   
         count++;  
    } 
} while (data != 0);

[Heather Carey]

Thanks! I found a guy a work who knew JAVA scripting (I'm not sure what the difference is) and he said the exact same thing as you did about doing a DO WHILE loop. I'm so glad that you responded though because of all the notes I took from him that was one I forgot to write down.

Thanks!
Heather

[Campbell Ritchie]

Unlikely to be JAVAscripting. More likely JavaScript. A completely different language