Can anybody shed light on where I am going wrong with this approach?

Question about the task

Resolved

I think it's to do with my sum but haven't been able to fathom it out. Much appreciated :)

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 myBufferedReader = new BufferedReader(new InputStreamReader(System.in)); int total = 0; String b = "sum"; String s = myBufferedReader.readLine(); while (s != b) { int a = Integer.parseInt(s); total = a + total; if (myBufferedReader.readLine().equals("sum")){ System.out.println(total); break; } } } }

Comments (9)

Popular
New
Old

You must be signed in to leave a comment

Aleksander

Level 6 , Slupsk, Poland

24 April 2020, 22:25

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 reader = new BufferedReader(new InputStreamReader(System.in)); int total = 0; while(true){ String s = reader.readLine(); if(s.equals("sum")){ break; }else{ int ss = Integer.parseInt(s); total = total + ss; } } System.out.println(total); } }

// Java Poser

Level 18 , Cincinnati, United States

10 April 2019, 13:11

For starters, you should use the code they provided and then go from there

sean

Level 8 , United Kingdom

10 April 2019, 13:55

Hey, Thanks for getting back to me. That was, as I am sure you can imagine, my initial approach. However, I got me to the same point that I am at now and therefore I opted to try something else. Here is my initial approach to solving this : public static void main(String[] args) throws Exception { BufferedReader myBufferedReader = new BufferedReader(new InputStreamReader(System.in)); int total = 0; while (true) { String s = myBufferedReader.readLine(); total = Integer.parseInt(s) + total; if (myBufferedReader.readLine().equals("sum")){ break; } } System.out.println(total); } }

// Java Poser

Level 18 , Cincinnati, United States

10 April 2019, 14:06

Ok your initial approach was very close, great job! just change your logic a bit. try doing the "if" statement before you add to your total. do the Integer.parseInt AFTER you've verified that s is not equal to "sum"

// Java Poser

Level 18 , Cincinnati, United States

10 April 2019, 14:08

solution

while (true)
        {
            String s = buffer.readLine();
            if (s.equals("exit"))
                break;
            else {
                int x = Integer.parseInt(s);
                sum += x;
            }
        }/code>

sean

Level 8 , United Kingdom

10 April 2019, 15:13

Many thanks sir! :)

// Java Poser

Level 18 , Cincinnati, United States

10 April 2019, 15:21

No problem (:

Ntuthuko Xaba

Level 18 , Johannesburg, South Africa

14 January 2020, 07:20

I don't understand how you would pass 's' which is a string into 'x' which is an integer - " int x = Integer.parseInt(s); " I tried using your logic but I'm getting tons of errors. BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int total = 0; while (true) { String input = reader.readLine(); if (input.equals("sum")) break; else{ int number = Integer.parseInt(reader.readLine()); total = total + number ; } } System.out.println(total);

Abdul-Hamed Mohammed

Level 8 , canton, USA

21 February 2020, 08:33

You are passing reader.readLine() in Integer.parseInt this is wroung: int number = Integer.parseInt(reader.readLine()); Right way is: int number = Integer.parseInt(input);