Задачу решил, но запутался как

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

Обсуждается

Доброго времени суток, господа. Задачу решил буквально переменой мест слагаемых, объясните пожалуйста логику, почему при перемене места строк 18 и 19

i = Integer.parseInt(suka.readLine());
s = s+1;

меняется результат? BTW Даже при перемене этих строк работает корректно, но ругается на условие "После ввода -1 программа должна корректно завершить работу.", что я упускаю?

Вводить с клавиатуры числа и вычислить среднее арифметическое.
Если пользователь ввел -1, вывести на экран среднее арифметическое всех чисел и завершить программу.
-1 не должно учитываться.

Примеры:
а) при вводе чисел
1
2
2
4
5
-1
получим вывод
2.8

б) при вводе чисел
4
3
2
1
-1
получим вывод
2.5

Подсказка: один из вариантов решения этой задачи, использовать конструкцию:

while (true) {
    int number = считываем число;
    if (проверяем, что number -1)
        break;
}

Требования:

Программа должна считывать данные с клавиатуры.
Программа должна выводить данные на экран.
После ввода -1 программа должна корректно завершить работу.
Если последовательно ввести: 1, 2, 2, 4, 5 и -1, программа должна вывести 2.8.
Если последовательно ввести: -100, 0, 100 и -1, программа должна вывести 0.0.
Если последовательно ввести: 1 и -1, программа должна вывести 1.0.
Выведенный результат должен соответствовать заданию для любых входных данных.

package com.javarush.task.task05.task0507; /* Среднее арифметическое */ import java.io.BufferedReader; import java.io.InputStreamReader; public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код BufferedReader suka = new BufferedReader(new InputStreamReader(System.in)); float s = 0; int i = Integer.parseInt(suka.readLine()), n = 1; while (true){ i = Integer.parseInt(suka.readLine()); s = s+i; n++; if (i == -1) break; } System.out.println(s / (n-1)); //BufferedReader r = new BufferedReader(new InputStreamReader(System.in)); //float s = 0; int i = Integer.parseInt(r.readLine()), n = 1; //for (; i != -1;) { //s = (s + i); //i = Integer.parseInt(r.readLine()); //n++; // } //System.out.println(s / (n - 1)); } }

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

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

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

Алексей Захаров

Уровень 7

28 августа 2019, 08:34

import com.sun.org.apache.xpath.internal.objects.XNumber;

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 step = 0;
        double sum = 0;
        while (true){
            int  Number = Integer.parseInt(reader.readLine());
            if (Number == -1)
                {
                    break;
                }
            else
                {
                    sum = sum + Number;
                    step++;
                }
        }
        //System.out.println("Колличество цифр: "+step);
        //System.out.println("Сумма цифр: "+sum);
        //System.out.print("Среднее арифметическое цифр: "+ sum / step);
        System.out.print(sum / step);

Майк

Уровень 18

23 февраля 2019, 07:51

s = s+i;
n++;
if (i == -1)
    break;

Соглашусь с Evgesha728. Когда вы вводите "-1", программа прибавляет его к сумме, однако в условии сказано, что "-1" не должно учитываться. Естественно, n++ тоже следует перенести после break. В таком случае костыль с (n-1) в конце вам не пригодится. <spoiler> Еще пара замечаний (имхо*) по поводу кода: 1. Когда выкладываете код, представьте, что его кто-то будет читать, приведите его в порядок. В вашем конкретном случае

s**a.readline()

очень путает, приходится при чтении смотреть что это за s**a переменная такая. Не надо так. 2. Почему одна переменная float, а другая int? Это костыль, чтобы получались дробные значения?

float s = 0;
int i = Integer.parseInt(suka.readLine()), n = 1; //не совсем понятно зачем n изначально равно 1.

Читатель ожидает что s в коде уже хранит значения с плавающей точкой. Почему бы на объявить переменные так:

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
int s = 0, n = 0;
int input = 0;

while(true) {
    input = Integer.parseInt(reader.readLine());
    //дальнейшая логика
}

(за одно сразу отделим объявление переменных и считывание данных) 3. Гораздо удобнее, если ответ будет получен уже на этапе функции. Если заранее объявить еще и float result. А потом его уже вывести, код будет читаться легче. Зачем я говорю все это? Ошибка в вашем коде основана на невнимательности и костылях (имхо*). Если бы вы сразу "причесали" свой код, то наверняка ваш вопрос бы отпал сам собой. </spoiler> *имхо - по моему скромному мнению

Konstantin

Уровень 9

23 февраля 2019, 17:53

Спасибо большое! Учту замечания, особенно с названием ридера, писал для себя и даже не обратил внимания, когда постил. Костыли и невнимательность от отсутствия глубинного понимания =(

Maxim724 Java Developer

22 февраля 2019, 22:30

Присмотрись в цикл. Ты по сути увеличиваешь, а потом только проверяешь, не -1 ли

Konstantin

Уровень 9

23 февраля 2019, 17:49

Спасибо!