Not sure what I'm doing wrong here

Pytania dotyczące zadania

Rozwiązane

Could someone please have a look? Don't know why this code is not working - the result is always wrong (works only for input -100, 0, 100 and -1, output 0.0).

Za pomocą klawiatury wprowadź liczby i oblicz ich średnią arytmetyczną.
Jeśli użytkownik wprowadzi -1, zostanie wyświetlona średnia arytmetyczna wprowadzonych liczb i program się zakończy.
-1 nie powinno być włączane do obliczeń.

Oto kilka przykładów:
a) jeśli wprowadzisz liczby
1
2
2
4
5
-1
to wyświetlimy
2.8

b) jeśli wprowadzisz liczby
4
3
2
1
-1
to wyświetlimy
2.5

Wskazówka: jedno z rozwiązań zawiera następującą konstrukcję:

while (true) {
    int liczba = wczytaj liczbę;
    if (sprawdza, czy liczba jest -1)
        break;
}

Wymagania:

Program musi odczytywać dane z klawiatury.
Program powinien wyświetlać dane na ekranie.
Po wprowadzeniu -1 program musi prawidłowo się zakończyć.
Jeśli wprowadzisz kolejno liczby 1, 2, 2, 4, 5 i -1, program powinien wyświetlić 2.8.
Jeśli wprowadzisz kolejno liczby -100, 0, 100 i -1, program powinien wyświetlić 0.0.
Jeśli wprowadzisz kolejno liczby 1 i -1, program powinien wyświetlić 1.0.
Wyświetlany tekst musi spełniać warunki zadania dla danych wejściowych.

package pl.codegym.task.task05.task0507; /* Średnia arytmetyczna */ import java.util.Scanner; public class Solution { public static void main(String[] args) throws Exception { //tutaj wpisz swój kod Scanner scanner = new Scanner(System.in); int sum = 0; int count = 0; while (true) { int number = scanner.nextInt(); sum = sum + number; count += 1; if (number == -1) break; } double avg = sum / count; System.out.println(avg); } }

Komentarze (4)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

zimme

Poziom 14

9 czerwca 2020, 18:25

hi, you break the loop as soon as you find -1 and then, so to speak, program forget everything you added up and the initial value of the initialization of the variable is printed, 0, a new value never reaches it. try to rearange your code like this: while(true){ int i=Integer.parseInt.... if(i == -1) { break; } else { sum += i; or sum = sum+i count++; } } System.out.println(sum/count);

Angelika Tsouti

Poziom 5 , Poznan, Poland

11 czerwca 2020, 14:48

thank you! I fixed that - it makes a lot of sense! it works better, although it still gives me a wrong result (when I put 1, 2, 2, 4, 5, -1 I get 2.0, not 2.8) - I'm not sure why?

import java.util.Scanner;

public class Solution {
    public static void main(String[] args) throws Exception {
        //tutaj wpisz swój kod
        Scanner scanner = new Scanner(System.in);

        int sum = 0;
        int count = 0;

        while (true) {
            int number = scanner.nextInt();
            if (number == -1)
                break;
            else {
sum += number;
            count++;
}
        }

        double avg = sum / count;

        System.out.println(avg);
    }
}

zimme

Poziom 14

11 czerwca 2020, 15:08

in java when you divade int by int you get int (in your case 14/5 = 2 (actually 2.0 because you print the int result as double) instead of 2.8, 0.8 is discarded) So to solve this problem you can eg declare the variable sum as double sum = 0.0

Angelika Tsouti

Poziom 5 , Poznan, Poland

11 czerwca 2020, 17:10

you are so right! thank you very much for your help!!!