Help, ive broken my brain!

Question about the task

Resolved

Ok I simply may have been at all this for too long today, but I cannot get this to work in anyway whatsoever. I've tried re-writing code based on previous help questions/answers and none of it has worked. not even making changes marked "solution". so I wrote it this way which I thought would be fine but it still does not meet the final requirement.

Write a program that will read numbers from the keyboard.
The code for reading numbers from the keyboard must be in the readData method.
Wrap the code inside readData in a try-catch block.
If the user enters text instead of a number, then the method should catch an exception and then display all previously entered numbers.
Display each number on a new line in the same order it was entered.

Requirements:

The program must read data from the keyboard.
Don't change the main method.
The main method should not throw exceptions.
The readData method must contain a try-catch block.
If the user enters text rather than a number, the program should display all previously entered numbers.

package com.codegym.task.task09.task0921; import java.io.BufferedReader; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.List; /* Method in a try-catch */ public class Solution { public static void main(String[] args) { readData(); } public static void readData() { ArrayList<Integer> list = new ArrayList<>(); try{ BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int x; while (true){ x=Integer.parseInt(reader.readLine()); list.add(x); System.out.println(list); } } catch (Exception e){ System.out.println(e); } //write your code here } }

Comments (5)

Popular
New
Old

You must be signed in to leave a comment

IvanP

Level 15 , Barcelona, Spain

14 July 2020, 19:15

You don't display the numbers when you should, instead of printing the list at line 27 or printing the exception at line 31, you should print the whole array:

catch (Exception e){
    for(int i = 0; i < list.size(); i++) {
        System.out.println(list.get(i));
    }
}

Thomas

Level 41 , Bayreuth, Germany

8 July 2020, 07:12

solution

Looks not that bad. Or better, it's nearly finished. you should delete line 27, that's within the loop and you should display all numbers when you enter text and not after each input. When you enter text, a NumberFormatException gets thrown. That's what we need to catch and then do the output. You catch all Exceptions, I do not know if validation accepts that as it says to print the result only when you not enter a number. So I'd change the catched exception class to NumberFormatException. Then you need another catch block or you add throws IOException to your main methods signature, cause the readLine() method can throw exceptions, too. So we have now two catch blocks, one (the first) with NumberFormatException, the other with IOException. In the first don't output the exception (as you do now). We do not want to print problems but a result. Loop over your list and display all elements.

Brad Reed

Level 19 , Hartford, United States

8 July 2020, 16:28

well that dindt work...looked like it was going to. trying again from scratch (attempt 6) in case i screwed something else up.

Thomas

Level 41 , Bayreuth, Germany

8 July 2020, 16:35

good luck ;)

Brad Reed

Level 19 , Hartford, United States

8 July 2020, 16:37

So i tested by adding text and everything works as it should...yet it wont pass. I made your recommended changes and it catches the exceptions and prints only the numbers before the text...still fails...wtf. if I run it, its fine, the code commenter has nothing to say but wont verify...so either I'm missing something or I simply didn't do it the way the verification wants....