Do not pass the 3rd requirement.

Question about the task

Positive number

Java Syntax, Level 4, Lesson 7

Resolved

Why don't I pass the third requirement?

Use the keyboard to enter three integers. Display the number of positive numbers in the original set.

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

b) if you enter the numbers
-6
-6
-3
then we display
0

c) if you enter the numbers
0
1
2
then we display
2

Requirements:

The program should read the numbers from the keyboard.
The program must display a number on the screen.
The program should display the number of positive numbers in the original set.
If there are no positive numbers, the program should display "0".
Note that 0 is not a positive or negative number.

package com.codegym.task.task04.task0428; /* Positive number */ import java.io.*; public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int a = Integer.parseInt(reader.readLine()); int b = Integer.parseInt(reader.readLine()); int c = Integer.parseInt(reader.readLine()); int count = 0; if (a > 0) { count ++; if (b > 0) { count ++; if (c > 0) { count ++; } } } else if (b > 0) { count ++; if (c > 0) { count ++; } } else if (c > 0) count ++; System.out.println(count); } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

horst

Level 26 , Not in list

30 March 2020, 13:12

solution

It took me a moment, but I found at least one flaw in your conditions. Consider e.g. the case 2, -3, 4 (i.e. a>0, b<0, c>0) This is what happens in your code (lines 20ff):

if (a > 0) { // true
            count ++; // correct
            if (b > 0) { // false, therefore the whole rest of the statement will not be executed
                count ++;
                if (c > 0) { // not reachable in this case
                    count ++;
                }
            } // Here is the end of your b>0-statement
        } // all the following else-if-cases will be ignored, because the first if (a>0) was true

My advice: Keep it simple. Instead of trying to write code for all possible scenarios with various if-else-statements, just see it this way: If the variable a is bigger than zero, then your count goes up - completely regardless of b and c. (so no else-if or nested if-statements needed) If b is bigger than zero, we count up - who cares about a and c, right? If c is bigger than zero, same thing. Hope this helps, otherwise don't hesitate to ask questions.