JavaRush/Помощь по Java задачам/Всем привет! Подскажите, в чём ошибка??? уже стопятьсот р...

8 уровень

08.01.2018
1547просмотров
9комментариев

Всем привет! Подскажите, в чём ошибка??? уже стопятьсот раз всё переделал

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

Решен

import java.io.*;
public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader x = new BufferedReader(new InputStreamReader(System.in));
        int b = 0;
        int a = 0;
        for( ;true; ) {
            int i = Integer.parseInt(x.readLine());
            if (a!=-1) {
            a = a+i;
            b++;
            }
            else {
        double c = (a/b);
        System.out.println(c);
            }
            break;
        }



    }
}

Вводить с клавиатуры числа и вычислить среднее арифметическое.
Если пользователь ввел -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.*; public class Solution { public static void main(String[] args) throws Exception { BufferedReader x = new BufferedReader(new InputStreamReader(System.in)); int b = 0; int a = 0; for( ;true; ) { int i = Integer.parseInt(x.readLine()); if (a!=-1) { a = a+i; b++; } else { double c = (a/b); System.out.println(c); } break; } } }

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

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

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

hidden #1278682

Уровень 32

8 января 2018, 10:07

ты его запускал? что выдает твой код?

Vladislav

Уровень 8

8 января 2018, 10:15

да, говорит задача не прошла тестирование

Стас Пасинков Software Developer в Zipy

Master

8 января 2018, 11:47

for( ;true; )

есть же цикл while для таких случаев! почему не использовать его?

if (a!=-1)

вы уверены, что вам надо именно переменную a проверять на -1? просто трудно будет подловить такой момент, когда в а получится именно -1... лучше переименуйте их и дайте нормальные названия, чтобы вы сами не путались.

double c = (a/b);

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

Vladislav

Уровень 8

8 января 2018, 12:47

Спасибо!!! да, запутался в переменных

hidden #1278682

Уровень 32

8 января 2018, 14:18

запустить - значит нажать зеленый треугольник. Он запустит код на выполнение и можно будет увидеть результат вашего кода

Стас Пасинков Software Developer в Zipy

Master

8 января 2018, 20:49

момент с делением целого числа на другое целое число - разобрались как пофиксить?)

Vladislav

Уровень 8

9 января 2018, 09:13

да, с делением тоже упростил, как Вы посоветовали) только насчёт цикла не понял немного...разницы же нет while или for использовать, принцип работы одинаковый же, или есть какие-то отличия?

Стас Пасинков Software Developer в Zipy

Master

10 января 2018, 08:51

фор используется для случаев, где надо перебрать поочередно элементы какой-нибудь структуры. если надо просто закольцевать выполнение до определенного события - тогда while/do-while подойдет лучше. ведь зачем-то придумали 2 разных типа цикла) хотя можно было бы все делать одним) принцип работы одинаковый, да. тут скорее момент красоты и понятности кода. а момент с делением целого на целое - то тут прикол, что тоже получится целое в результате. такова специфика оператора / поэтому чтобы не терять дробную часть результата - обычно какую-то из частей множат просто на 1.0 (что в результате дает дабл, и получается уже деление целого на дробное и ответ тоже дробный тогда), ну или приводят к типу дабл напрямую :) поэтому когда вы делали

double c = (a/b);

то выполнялось целочисленное деление, получался ответ в виде целого числа (с потерянной дробной частью), а потом уже он приводился к дробному виду чтобы попасть в переменную с. получается, при а = 3, b = 2, с = 3/2 = 1 => 1.0 а должно было получиться 1.5

Vladislav

Уровень 8

12 января 2018, 12:19

Ааа...теперь понятно! думал, что при объявлении double дробная часть автоматически возвращается. Благодарю!!!!!))))