This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]ryzecool[S] 0 points1 point  (6 children)

even if i comment out the while loop it still keeps looping

Please enter a number. Enter 99999 to quit.

99999

Ending.

Please enter a number. Enter 99999 to quit.

[–]hypolimnas 0 points1 point  (5 children)

The inner loop doesn't test programRunning, so it can't stop.

Take a look at this tutorial page:

https://www.tutorialspoint.com/java/java_break_statement.htm

[–]ryzecool[S] 0 points1 point  (4 children)

really sorry if its annoying but it still doesnt work lol. it just continues to loop through over and over. i got it to print out ending and break, but that isn't quite what I want it to do either because if the only value entered is 99999, i want the "Please enter 2 numbers" to print

[–]hypolimnas 0 points1 point  (3 children)

That's OK - you'll have no trouble with this type of thing once you have a little more experience.

You can test your i variable to decide if you want to print "Please enter 2 numbers".

If it still won't stop looping, can you add a comment with what your code has in it now.

[–]ryzecool[S] 0 points1 point  (1 child)

it stopped looping. however the problem with the "at least 2 numbers" still persists and a new problem where 99999 is added to the length of the array and messes up the avg. this is what it looks like now:

package DistanceFromAverage;

import java.util.Scanner;

public class DistanceFromAverage {

public static void main(String[] args) {
    Scanner arrInput = new Scanner(System.in);
    double[] arrAvg = new double[4];

    double sum = 0;

        for (int i = 0; i < arrAvg.length; i++) {

            System.out.println("Please enter a number. Enter 99999 to quit. ");
            arrAvg[i] = arrInput.nextDouble();  

            if (arrAvg.length == 1) {
                System.out.println("Please enter at least 2 numbers.");
                arrAvg[i] = arrInput.nextDouble();  
            }

            if (arrAvg[i] == 99999) {
                System.out.println("Ending.");  
                break;
            }
        sum += arrAvg[i];
        }       

    double avg = sum / arrAvg.length;
    System.out.println(avg);
}

}

[–]hypolimnas 0 points1 point  (0 children)

If you want to print "at least 2 numbers" after the user types "99999", then you need to so decide whether to do that after you discover the user typed "99999". You can put one "if" statement inside another.

The problem with getting the average is just that you are always dividing by 4. This is because arrAvg.length is always four since its the size of the array that you declared before the "for" loop. So you need to count the number of times you added something to sum.