Infinite loop

Question about the task

Console-based piggy bank

Java Syntax, Level 5, Lesson 12

Resolved

I get an infinite loop..i also tried with if and break but still got an infinite loop.. what am i missing thanks

Use the keyboard to enter numbers and calculate their sum until the user enters the word "sum".
Display the resulting sum on the screen.

Hint: to read from the keyboard until the string 'exit' is input, do this:

BufferedReader buffer = new BufferedReader(new InputStreamReader(System.in));
while (true)
{
	String s = buffer.readLine();
	if (s.equals("exit"))
		break;
}

Requirements:

The program must read data from the keyboard.
The program should stop reading data from the keyboard after the user enters the word "sum" in lowercase letters and presses Enter.
The program should work correctly if the user enters one number and the word "sum".
The program should work correctly if the user enters two numbers and the word "sum".
The program should work correctly if the user enters more than two numbers and the word "sum".
The program should display text on the screen.

package com.codegym.task.task05.task0529; import java.io.BufferedReader; import java.io.InputStreamReader; /* Console-based piggy bank */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader buf = new BufferedReader(new InputStreamReader(System.in)); String str = buf.readLine(); int sum=0; while(!str.equals("sum")){ sum = sum + Integer.parseInt(str); } System.out.println(sum); //write your code here } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Minh Phạm Đức

Level 22 , Cedar Falls, United States

2 May 2019, 10:28

solution

the str only get read one from user input it didn't change through out the loop so it'll loop forever. do something like while (true) { take user input and so the sum check condition then break; }

dean

Level 8 , Deutschland, Germany

2 May 2019, 12:47

thanks man that solved it for me!!! i forgot it doesn't reset the user input unless it's in a loop