Где проблема в цикле? (

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

Решен

Проходит по массиву и выводит количество положительных чисел 2 раза ( Как решить сие проблему?(

Ввести с клавиатуры три целых числа. Вывести на экран количество положительных и количество отрицательных чисел в исходном наборе, в следующем виде:
"количество отрицательных чисел: а", "количество положительных чисел: б",
где а, б - искомые значения.

Пример:
а) при вводе чисел
2
5
6
получим вывод
количество отрицательных чисел: 0 количество положительных чисел: 3

Пример:
б) при вводе чисел
-2
-5
6
получим вывод
количество отрицательных чисел: 2 количество положительных чисел: 1

Требования:

Программа должна считывать числа c клавиатуры.
Программа должна выводить текст на экран.
Программа должна выводить количество отрицательных чисел в исходном наборе.
Программа должна выводить количество положительных чисел в исходном наборе.
Если отрицательных чисел нет, программа должна вывести "количество отрицательных чисел: 0".
Если положительных чисел нет, программа должна вывести "количество положительных чисел: 0".
Учесть, что число "0" не относится ни к положительным, ни к отрицательным числам.

package com.javarush.task.task04.task0429; 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 num1 = Integer.parseInt(reader.readLine()); int num2 = Integer.parseInt(reader.readLine()); int num3 = Integer.parseInt(reader.readLine()); int count = 0; int count1 = 0; int[] mass = new int[]{num1, num2, num3}; for(int i=0;i<mass.length;i++){ if(mass[i]>0){ count++; System.out.println("количество положительных чисел:"+count);} if(mass[i]<0){ count1++; System.out.println("количество отрицательных чисел:"+count1);} } } }

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

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

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

Alex V. Mel

Уровень 26

9 апреля 2021, 20:20

IskanderM как бы Вы отнеслись к такому варианту? не обращайте внимание на конструкцию

try (BufferedReader reader = new BufferedReader(new InputStreamReader(System.in))) {

} catch (NumberFormatException | IOException ex) {
           ex.printStackTrace();
}

если Вы не знаете что это такое или этого ещё не было в лекциях. Вы также можете открыть поток и закрыть его "вручную", а перед фигурной скобкой написать throw IOException. Главное (на мой взгляд) здесь то, что видна некая структура и есть чёткое разделение ответственности. Хотя кода больше.

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

public class Solution {
    public static void main(String[] args) {
        Solution solution = new Solution();
        Integer[] numbers = solution.getNumbers();
        int[] count = solution.getCount(numbers);
        System.out.println("количество отрицательных чисел: " + count[0]);
        System.out.println("количество положительных чисел: " + count[1]);
    }

    private Integer[] getNumbers() {
        Integer[] numbers = new Integer[3];

        try (BufferedReader reader = new BufferedReader(new InputStreamReader(System.in))) {
            for (int i = 0; i < 3; i++) {
                numbers[i] = Integer.parseInt(reader.readLine());
            }
        } catch (NumberFormatException | IOException ex) {
            ex.printStackTrace();
        }

        return numbers;
    }

    private int[] getCount(Integer[] numbers) {
        int[] count = new int[]{0, 0};
        int current;
        for (Integer number : numbers) {
            current = number;
            if (current < 0) {
                ++count[0];
            } else if (current > 0) {
                ++count[1];
            }
        }
        return count;
    }
}

FrUshak Java Developer

9 апреля 2021, 18:32

Выводите результаты после цикла, а не в каждом проходе

IskanderM

Уровень 13

9 апреля 2021, 18:41

вывел результат за пределы цикла. но практически ничего не изменилось (

FrUshak Java Developer

9 апреля 2021, 18:43

Можете в коммент скинуть новый код метода? Выглядит странно, если вы действительно вынесли вывод за пределы цикла

IskanderM

Уровень 13

9 апреля 2021, 18:47

package com.javarush.task.task04.task0429; 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 num1 = Integer.parseInt(reader.readLine()); int num2 = Integer.parseInt(reader.readLine()); int num3 = Integer.parseInt(reader.readLine()); int count = 0; int count1 = 0; int[] mass = new int[]{num1, num2, num3}; for(int i=0;i<mass.length;i++){ if(mass[i]>0){ count++;} if(mass[i]<0){ count1++;} System.out.println("количество положительных чисел:"+count); System.out.println("количество отрицательных чисел:"+count1); } } }

FrUshak Java Developer

9 апреля 2021, 18:48

Вот код рабочего метода, просто вынес вывод за цикл ничего больше не меняя. Вы не вынесли вывод за пределы цикла :)

public static void main(String[] args) throws Exception {
     BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
     int num1 = Integer.parseInt(reader.readLine());
     int num2 = Integer.parseInt(reader.readLine());
     int num3 = Integer.parseInt(reader.readLine());
     int count = 0;
     int count1 = 0;
     int[] mass = new int[]{num1, num2, num3};
     for (int i = 0; i < mass.length; i++) {
         if (mass[i] > 0) {
             count++;
         }
         if (mass[i] < 0) {
             count1++;
         }
     }
     System.out.println("количество положительных чисел:" + count);
     System.out.println("количество отрицательных чисел:" + count1);
 }

Ksenia Volkova Java Developer в DXC

Master

9 апреля 2021, 18:49

Где у тебя тут цикл заканчивается?

IskanderM

Уровень 13

9 апреля 2021, 18:50

блин! Фигургая скобка... будь она неладна ))) Чувствую себя овощем! ))) спасибо большое )

FrUshak Java Developer

9 апреля 2021, 18:54

Не за что :) Совет - форматируйте код так, что бы скобки наглядно показывали уровень вложенности :)