Не проходит проверку, хотя все работает

перемудрил знаю)

Благодарю за ответ! Странно , я убрал 22,25,28 реально лишние но не совсем понимаю почему, ведь в прошлых задачках необходимо было создавать переменную класса .. А как через ифы ? вроде они стоят, и анализ кода не дает никаких нареканий..

public static int count = 0; статик - означает единственный экземпляр. Конкретный класс еще не создан а эта переменная уже существует. К нему есть доступ через "НАЗВАНИЕ КЛАССА".count, смысла создавать классы никакого. А теперь давай упростим твой код:

import java.io.BufferedReader;
import java.io.InputStreamReader;

/*
Положительные числа
*/

public class Solution {
  private static int count = 0;

  public static void main(String[] args) throws Exception {
    BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

    int n1 = Integer.parseInt(reader.readLine());
    int n2 = Integer.parseInt(reader.readLine());
    int n3 = Integer.parseInt(reader.readLine());

    if (n1 > 0)count++;
    if (n2 > 0)count++;
    if (n3 > 0)count++;

    System.out.println(count);
  }
}

Не надо усложнять ради усложнения.

Но иногда "простота хуже воровства", можно очень сильно упростить что времени для понимания потратишь слишком много. Как говорят, быстро писать программисту нет необходимости, так как большую часть времени он читает :)

import java.io.BufferedReader;
import java.io.InputStreamReader;

/*
Положительные числа
*/

public class Solution {
  private static int count = 0;

  public static void main(String[] args) throws Exception {
    BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

    for(int i = 0; i < 3; i++) {
      if(Integer.parseInt(reader.readLine()) > 0)count++;
    }

    System.out.println(count);
  }
}

Я усложняю изза непонимания))) Я понял что у меня не решалась задача т.к. я переменную count не создал. Михаил благодарю!

Вот это выглядит прям как джедайские штучки😎

подскажи как это выражение (Integer.parseInt(reader.readLine()) заменяет создание переменной string с занесением туда reader.readLine(). а после создание целочисл переменной и перевод строки в число, ведь bufferreader только строки читает.

Визуально это матрёшка, но реально это конвейер. Повсеместно используется в юникс системах. Правда там используют программы, в нашем случае классы и методы. Есть вход и есть выход и мы можем присоединить выход одной программы ко входу другой, таким образом провести обработку данных (чаще всего в текстовом виде) В нашем случае есть ограничение у метода. В заголовке мы пишем что он принимает ( в скобочках) и что отдает (перед названием метода). BufferedReader reader - это мы подключились к стандартному входу компьютера ( в нашем случае прием данных с терминала) reader.readLine() , доки класса - читает строку текста ( тип String) , теперь передаем эту строку в метод Integer.parseInt(String s) опять же доки Этот метод преобразует строку в число (если не получится программа остановит работу с ошибкой). Теперь полученное число сравнивается в if( число > 0) и если это так, то выполняет инкрементирование счетчика. Да и в этом примере зря я не перенес счетчик внутрь, как временную переменную. Она нигде больше нам не понадобится.

String snumber1 = reader.readLine(); Здесь результат ридера присуждается конкретному стрингу. То есть ты даешь ему адрес по которому ты можешь обращаться, но нам в дальнейшем не будет нужен эта строка, поэтому берем то что надо (обрабатываем) и забываем об этой строке.

Михаил, сделал так по твоей подсказке:

import java.io.BufferedReader;
import java.io.InputStreamReader;

public class Solution {

    public static void main(String[] args) throws Exception {

        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int count = 0;
        int a = 3;
        while (a > 0) {
            if (Integer.parseInt(reader.readLine()) > 0) {
                count++;
            }
            a--;
        }
        System.out.println(count);
    }
}

Не понял, почему у тебя count в отдельном методе стоит (private static int count = 0;), а не просто создать переменную в main (int count = 0;)? И еще по твоему коду у меня получалось бесконечное считывание с консоли через цикл фор, а через wile все ок получилось. Тоже не понимаю почему. Заодно кажется понял как использовать BufferedReader и InputStreamReader. Только не пойму, почему не используем Scanner, кажется удобнее. Или привычнее.

count в отдельном, потому что автор вопроса его вообще в другой класс поместил, а я показал что это не обязательно. Но ты прав. Это проходная переменная и её можно было и не выносить. По поводу Scanner, введен он был только в 1.5, но поддержка остальных методов остается. Это как с классом Vector, он поддерживается. но пользоваться им не обязательно. Главный плюс Scanner , он сам обрабатывает ошибки при вводе. Но BufferedReader , метод read() Reads a single character. Просто и удобно. Ты точно знаешь что читается символ. В общем всё дело в привычке. Тут как с инструментами, надо брать подходящий, чтобы молотком не забивать шурупы.