Program wskazujący wartość najbliższą średniej nie działa poprawnie

Question

Jest to zadanie domowe z odcinka 11 kursu C++. Użytkownik ma wprowadzić 5 liczb, a program ma mu podać średnią i liczbę, która jest najbliżej wartości średniej. Podczas podawania niektórych wartości program podaje złą liczbę np. 1 2 3 5 4 (w tej kolejności). Proszę o zlokalizowanie błędu. Z góry dziękuje.

#include <iostream>

using namespace std;
float a[5]; //WARTOSCI
float srednia; //SREDNIA
float odleglosc[5]; //ODLGELOSCI DANYCH LICZB OD SREDNIEJ
float najblizej; //LICZBA NAJBLIZEJ SREDNIEJ
int main()
{
    cout<<"Podaj 5 liczb oddzielonych spacja i zatwierdz enterem: "; 
    cin>>a[0]>>a[1]>>a[2]>>a[3]>>a[4];
    srednia=(a[0]+a[1]+a[2]+a[3]+a[4])/5;

    for (int i=0; i<5; i++)
    {
        odleglosc[i]=a[i]-srednia;
        if (odleglosc[i]<0) 
            odleglosc[i]=odleglosc[i]*-1;
    }
    najblizej=a[0];

    for (int i=1; i<5; i++)
        {
            if (odleglosc[i]<odleglosc[i-1]) 
                najblizej=a[i];
        }

    cout<<"Srednia wynosi: "<<srednia<<endl;
    cout<<"Najblizej jest: "<<najblizej<<endl;

    return 0;
}

 

Answer 1

W zasadzie to cały blok od 24-26 linijki jest błędny. Ty sprawdzasz tylko różnice między aktualną wartością a poprzednią, a powinieneś sprawdzać między aktualną a globalnie najmniejszą.

Trochę zmodyfikowałem ten kod:

#include <iostream>
 
using namespace std;
float a[5]; //WARTOSCI
float srednia; //SREDNIA
float odleglosc[5]; //ODLGELOSCI DANYCH LICZB OD SREDNIEJ
float najblizej; //LICZBA NAJBLIZEJ SREDNIEJ
int indeks;

int main() {
    cout<<"Podaj 5 liczb oddzielonych spacja i zatwierdz enterem: "; 
    cin>>a[0]>>a[1]>>a[2]>>a[3]>>a[4];
    srednia=(a[0]+a[1]+a[2]+a[3]+a[4])/5;
 
    for (int i=0; i<5; i++) {
        odleglosc[i]=a[i]-srednia;
        if (odleglosc[i]<0) odleglosc[i]=odleglosc[i]*-1;
    }

    najblizej = odleglosc[0];
    indeks = 0;

    for (int i=1; i<5; i++) {
        if (odleglosc[i] < najblizej) {
            najblizej = odleglosc[i];
            indeks = i;
        }
    }

    cout<<"Srednia wynosi: "<<srednia<<endl;
    cout<<"Najblizej jest: "<< a[indeks] <<endl;
 
    return 0;
}

 

 

Comments

Bardzo dziękuje za odpowiedź :)

Answer 2

Błąd znajduje się w 24 linijce.

Comments

heh wyprzedziłeś mnie ;)