looping looping looping and system won't let me place break at loop

Question about the task

Arithmetic mean

Java Syntax, Level 5, Lesson 5

Resolved

Very sad. Won't leave loop.

Use the keyboard to enter numbers, and then calculate the arithmetic mean.
If the user enters -1, display the arithmetic mean of all entered numbers and end the program.
-1 should not be included in the calculation.

Here are some examples:
a) if you enter the numbers
1
2
2
4
5
-1
then we display
2.8

b) if you enter the numbers
4
3
2
1
-1
then we display
2.5

Hint: one of the solutions to this problem uses the following construct:

while (true) {
    int number = read the number;
    if (check whether the number is -1)
        break;
}

Requirements:

The program must read data from the keyboard.
The program should display data on the screen.
After entering -1, the program must correctly terminate.
If you sequentially enter the numbers 1, 2, 2, 4, 5 and -1, the program should display 2.8.
If you sequentially enter the numbers -100, 0, 100, and -1, the program should display 0.0.
If you sequentially enter the numbers 1 and -1, the program should display 1.0.
The displayed result should match the task conditions for any input data.

package com.codegym.task.task05.task0507; import java.io.*; /* Arithmetic mean */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String x1 = reader.readLine(); double mean = 0; int sum = 0; int i = 0; int x = Integer.valueOf(x1); while(x != -1){ sum = sum + x; i++; } mean = sum/i; System.out.println(mean); //write your code here } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Branka

Level 41 , Belgrade, Serbia

18 April 2020, 17:53

You have to consider that your x value doesn't change. You have entered it only once. You have to keep on entering numbers until you enter -1. Because of this the contents of your while() are never false and you have an infinite loop.

BlueJavaBanana

Level 37

19 April 2020, 13:07

^^^ Just to add to the above comment look at when you collect the value for x1. At the very start of the program. Your code then never collects this again. You will need to put your code for collecting x1 inside your loop. Make sure you also include a way to convert your x1 string into your x integer too.