Прошу распишите каждое действия начиная res.. почему мы делим и от i - 2?

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

Решен

package com.javarush.task.task05.task0507;
import java.io.*;
/*
Среднее арифметическое
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        //напишите тут ваш код
BufferedReader buf = new BufferedReader(new InputStreamReader(System.in));
int a = 0;
double res = 0;
double sum = 0;
int i = 1;
for (i = 1; a != -1; i++) {
	a = Integer.parseInt(buf.readLine());
	sum += a;
	// res = (sum+1)/i;
}
res = (sum+1) / (i - 2);
System.out.println(res);
    }
}

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

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

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

Ярослав Java Developer

Master

22 февраля 2019, 19:51

Комментарии кода скинули уже внизу, но скажу так, это ужасный код :)

Riccio

Уровень 35

Master

22 февраля 2019, 20:14

Ярослав, товарищ просит помощи - хочет разобраться и понять. Так что вы уж дайте развернутый ответ - что не так и как надо было сделать.

Ярослав Java Developer

Master

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

1) Имена переменных - ужас. Какие ещё а, res? Вот не читайте код сейчас, прочитайте только названия переменных, которые я назвал тут, и скажите: вы сможете сказать хотя бы приблизительно их значение? Я вот не могу. 2) Если sum - double число, почему мы суммируем только целые числа? Только или sum нужно сделать int типом, или использовать Double.parseDouble() 3) Использование цикла for - ужасно, и обусловлено только кривой реализацией счетчика. Вынесите переменную отдельную int count = 1;, потом используйте цикл while (a != -1) { ...; count++ } 4) Особое дополнение к первому пункту: зачем переменная res? Одно дело, если она может каким-то образом конкретизировать то, что мы хотим вывести, как результат, однако res - даже не сразу можно понять, что это значит result, поленились добавить три лишние буквы? И непонятно сразу, что за результат. Эта переменная не имеет смысла и можно было спокойно писать выражение прямо в System.out.println. Единственное, как можно было бы воспользоваться отдельной переменной для результата, это если дать ей нормальное имя: double average = ...; Тогда понятно, что это среднее арифметическое, у этого имени есть смысл. 5) Зачем все переменные пихать прямо в начало метода? Это нормально в каком-то там Javascript, C, однако во втором это связано с отсутствием ООП, а в первом с какими-то привычками. В Java имеет смысл объявлять переменные тогда, когда они нужны, а не в начале метода на весь метод вперёд. Потому переменную double average = ...; следовало бы разместить прямо в конце метода перед выводом. Или вообще не размещать.

Сергеев Виктор

Уровень 40

Master

23 февраля 2019, 12:57

с большинством согласен, но стоит учитывать уровень спрашивающего. Пока человеку важно решить задачу а не написать красивый код. Ну а если уж придираться, то что это за переменная average средняя чего? Средняя температура по больнице? средняя площадь космоса на 1 человека? res - ну тут у каждого свое восприятие, но что ещё можно понять по res. Ресурсы, рестрикция, реструктуризация. Я чаще всего воспринимаю res как result автоматом. Не правильно конечно, но совсем не критично. На этом уровне люди чаще всего использую 1 буквенные переменные, а тут уже есть смысл в именах. "Если sum - double число, почему мы суммируем только целые числа?" - например, чтобы потом не приводить к дробному, при вычислении среднего. По сути решение правильное - 4) Особое дополнение к первому пункту: зачем переменная res? Одно дело, если она может каким-то образом конкретизировать то, что мы хотим вывести, как результат, однако res - даже не сразу можно понять, что это значит result, поленились добавить три лишние буквы? Все таки всегда стоит различать "я не могу понять" и "никто не может понять". Опять же на мой взгляд отделить вычисления от вывода - вполне логично и практично при дебаге, чем делать это сразу в выводе. С остальным согласен )

Ярослав Java Developer

Master

23 февраля 2019, 15:46

В числовых стримах как раз есть метод average(), который возвращает среднее значение :) Про то, что для того, чтобы не приводить, в принципе согласен. Для меня res не сразу был очевиден, как result. Только когда досмотрел код до вывода значения. И я не говорил, что автор ужасно пишет, и я даже не уверен, что код автора, учитывая то, что он пришел сюда с вопросом "ребята, а как это работает"? XD В принципе, для этого уровня - это нормально, все с этого начинали :))

Igor Kotljar

Уровень 14

23 февраля 2019, 21:03

К сожалению мои попытки кода сошлись к 0, взял этот и разбирал, и после упомянутой выше функции не понял что дальше)

Riccio

Уровень 35

Master

22 февраля 2019, 19:49

полезный

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

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader buf = new BufferedReader(new InputStreamReader(System.in));
        int a = 0;
        double res = 0;
        double sum = 0;

        //объявление переменной - счетчика итераций i
        //вынесено за пределы цикла, чтобы ее значение сохранилось и после дейтствия цикла
        int i = 1;

        //цикл для считывания строк
        //так же он проверяет, не введено ли -1
        //только эта проверка происходит перед началом следующей итерации
        for (i = 1; a != -1; i++) {

            //считывается строка и парситься в число
            //это число присваивается переменной а
            a = Integer.parseInt(buf.readLine());

            //к переменной sum добавляется значение а
            sum += a;
            // res = (sum+1)/i;
        }

        //результат равен сумме плюс единица (так как -1, когда было введено
        //тоже добавилось к переменной sum, а по условию оно не должно участвовать
        //в вычислении среднего арифметического
        //i-2 потому, что стартовал уже с "первой итерацией" и потому, что засчитал лишнюю -
        //когда ввели "-1"
        res = (sum+1) / (i - 2);

        //вывод результата
        System.out.println(res);
    }
}

Igor Kotljar

Уровень 14

22 февраля 2019, 20:17

Спасибо, очень помогло! Скажите, это нормально что 30% информации, кода остаётся не понятным и становится ясным когда код написан или часть кода взята у других ребят?

Riccio

Уровень 35

Master

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

По логике, после прочтения условия у вас в голове должно быть несколько путей решения. Но если задача после новой темы и вы не знаете что делать - значит, информации недостаточно. Гуглите.