JavaRush/Помощь по Java задачам/Как упростить?

24 уровень

18.10.2018
797просмотров
11комментариев

Как упростить?

Вопрос по задаче

Обсуждается

if (a>0) q++;
if (b>0) q++;
if (c>0) q++;

Сделал три цикла, но понимаю что это нерационально. Видел решение с одним циклом, но не могу понять его принцип:

if (Integer.parseInt(reader.readLine()) > 0)
    count++;

Блок условий считывает введенные переменные по одной, в порядке "живой очереди", и сравнивает с нулем. Если да, то выполняет условие, так? Остается неясным, как должны быть введены переменные? Если они введены через сканнер и считаны как scan.nextInt(), сработает ли это решение или нужно как-то иначе вводить? Проверить не могу - задача решена, до среды разработки пока не могу добраться. Прошу не кидать камнями, спасибо!

Комментарии (11)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Alexandr_Kiss

Уровень 5

27 октября 2018, 16:42

Не знаю как на счет упрощения, но есть альтернативный и не сложный вариант. Просто посчитай длину массива ArrayList.

for (int i = 0; i < 3; i++) {
            a = Integer.parseInt(reader.readLine());
            if (a>0) b.add (a);
            if (i==2) System.out.println(b.size());
}

где:

ArrayList b = new ArrayList();

и не забудь:

import java.util.ArrayList;

Zafar

Уровень 12

18 октября 2018, 19:10

/* Комментарий удален */

Sergey PleshkoBackend Developer в Sberbank

18 октября 2018, 19:25

Разве в этом случае цикл отработает правильно? Мне кажется он трижды сделает +1, даже если положительное число всего одно..

Zafar

Уровень 12

19 октября 2018, 07:51

if (Integer.parseInt(reader.readLine()) > 0)
    count++;

Это максимально упрощенное решение. Во первых, нет необходимости присваивать каждый раз введенное значение новой переменной. Во вторых, можно сразу же проверять его на выполнимость условия. При каждом вызове в цикле reader.readLine() - выполняется чтение строки, проверка на условие, и тут же перевод курсора на следующую строку (и так 3 раза).

Илья Геков

Уровень 40

Master

18 октября 2018, 18:28

Переменные вообще не нужны, readLine или nextInt возвращают значение, его можно сразу проверять на положительность, не присваивая никаким переменным

Sergey PleshkoBackend Developer в Sberbank

18 октября 2018, 19:23

Я некорректно выразился, в данном предложении следует читать переменные=вводимые числа. Я правильно понял что при использовании readLine введенные числа поочередно сравниваются с условием и цикл выполняется N раз, в зависимости от кол-ва введенных чисел?

Илья Геков

Уровень 40

Master

18 октября 2018, 19:44

Не очень понимаю, что вы имеет в виду. Вот как примерно это выглядит: - переменная счетчика положительных чисел; - цикл (до 3-х) { если ((считываем значение) > 0) счетчик полож. чисел++; } Таким образом, благодаря циклу, который проходит 3 шага, 3 раза вызывается метод по считыванию значения (не важно сканер или буф. ридер), после чего каждое считанное значение сразу проверяется на положительность, если да, то счетчик увеличивается.

Sergey PleshkoBackend Developer в Sberbank

18 октября 2018, 19:51

Scanner sc = new Scanner(System.in);
       int a = sc.nextInt();
       int b = sc.nextInt();
       int c = sc.nextInt();
        int q = 0;
        if (sc.nextInt() > 0) {
            q++;

        }
System.out.println(q);

Сложно сформулировать то, что не понимаешь.. Попробую на примере - почему это не работает?

Илья Геков

Уровень 40

Master

18 октября 2018, 19:59

полезный

Метод nextInt() считывает значение из потока ввода. Вы его считали три раза, сохранили в переменные, дальше что вы пытаетесь сделать этой строчкой?

if (sc.nextInt() > 0)

Ввести новое значение и сравнить его с 0? Зачем? Три значения же уже введены. Тут надо или возвращаться к тому решению, которое уже у вас было, используя переменные a, b, c и последовательно их сравнивая с нулем. Либо вводить цикл, убирать переменные a, b, c и считывать значения в цикле. Если есть желание, могу в личку прислать вам как это будет выглядеть с использованием цикла.

Sergey PleshkoBackend Developer в Sberbank

18 октября 2018, 20:01

Да, был бы благодарен.

Riccio

Уровень 35

Master

19 октября 2018, 00:06

Можно так:

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
int x = 0;
for (int i = 0; i < 3; i++) if (Integer.parseInt(reader.readLine()) > 0) x++;