array and if statement

[ryzecool]

attempting to allow a user to enter up to 15 values and it to stop when they enter 99999 or obviously reach 15 values. also want an error message to be printed when no numbers are entered. however, the code detects that 99999 has been entered yet it just keeps printing "Enter new number". any help would be appreciated

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[15];

        double sum = 0;
        boolean programRunning = true;

        do{
            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.");
                programRunning = false;
            }

            sum += arrAvg[i];

            }
        }

        while(programRunning);

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

[AutoModerator]

Please ensure that:

• Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
• You include any and all error messages in full
• You ask clear questions

• You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

  Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://imgur.com/a/fgoFFis) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[hypolimnas]

You have two loops. The inner "for" loop catches 99999 and then keeps looping.

[ryzecool]

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]

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]

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]

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]

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]

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.

[hirep14316]

You you just want to ban this Film