Output sum of element entered by user to an ArrayList Java

Question

i am currently working on a java code that takes input from user and outputs the size of arraylist, the sum of numbers entered, the average and the maximum number entered. i have not been able to complete the sum as the for loop is not calculating result. I would appreciate any suggestions. my code is as follows.

import java.util.ArrayList;
import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        // write your code here
        double size = 0;
        double a = -999;
        double total = 0;

        Scanner in = new Scanner(System.in); // scanner object for user input

        ArrayList<Double> inputs = new ArrayList<Double>();
        System.out.println("Enter a number, to terminate enter -999 :");
        while (in.hasNextDouble()) {
            //assign the nextDouble to a variable
            double tmp = in.nextDouble();
            //test the variable
            if (tmp != a) {
                //add the variable
                //if you want to add -999 to the inputs then this next line above the test.
                inputs.add(tmp);
                System.out.println("Enter a number, to terminate enter -999 :");
            } else {
                inputs.size(); // get size of input
                System.out.println("Numbers entered: " + inputs.size());
                break;
            }
        }
        for (double sum : inputs) {
            total += sum;
            System.out.println("The sum of numbers entered is " + total);
            break;
        }

    }
}

Carcigenicate · Accepted Answer · 2018-10-05 16:12:54Z

3

The break in the for loop causes the loop to exit. Here, it's causing the loop to exit on the first iteration, which is almost certainly not what you want.

Move the println outside the loop, and remove the break:

for (double sum : inputs) {
    total += sum;
}

System.out.println("The sum of numbers entered is " + total);

This allows the for loop to iterate the entire list while calculating the sum so you can print it after instead of exiting prematurely.

Also note that this:

inputs.size(); // get size of input

Isn't doing anything useful. size returns the size of inputs, then you don't do anything with that number, so it's lost. You should just remove that line since you call size again on the next line anyways.

edited Oct 5, 2018 at 16:12

answered Oct 5, 2018 at 15:19

Carcigenicate

46.6k12 gold badges81 silver badges134 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Rendón · Accepted Answer · 2018-10-05 15:32:52Z

0

The problem you have in your code is you have a break condition in your for loop

The correct code is the following

import java.util.ArrayList;
import java.util.Scanner;

public class Main {

public static void main(String[] args) {
    // write your code here
    double size = 0;
    double a = -999;
    double total = 0;

    Scanner in = new Scanner(System.in); // scanner object for user input

    ArrayList<Double> inputs = new ArrayList<Double>();
    System.out.println("Enter a number, to terminate enter -999 :");
    while (in.hasNextDouble()) {
        //assign the nextDouble to a variable
        double tmp = in.nextDouble();
        //test the variable
        if (tmp != a) {
            //add the variable
            //if you want to add -999 to the inputs then this next line above the test.
            inputs.add(tmp);
            System.out.println("Enter a number, to terminate enter -999 :");
        } else {
            inputs.size(); // get size of input
            System.out.println("Numbers entered: " + inputs.size());
            break;
        }
    }
    for (double sum : inputs) {
        total += sum;
    }

    System.out.println("The sum of numbers entered is " + total);

}
}

answered Oct 5, 2018 at 15:32

Rendón

3001 silver badge7 bronze badges

2 Comments

1256 Over a year ago

hi would you have any advise of how to find the largest number entered by user?#

Rendón Over a year ago

Only store it in a local variable in the while block

Collectives™ on Stack Overflow

Output sum of element entered by user to an ArrayList Java

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related