Program znajdujący liczbę najbliższą średniej

Question

Cześć,

Wiem, że podobne tematy zostały już utworzone (czytałem je), ale mój problem jest inny.

Napisałem program który wczytuje X liczb typu float z klawiatury (chcę by był uniwersalny), następnie liczy średnią i podaje liczbę najbliższą średniej.

Kod:

#include <iostream>

using namespace std;

float czy_poprawnie(float x){
    bool stan_x;
    do{
        cin.clear();
        cin.ignore();
        cin>>x;
        stan_x=cin.good();
        if(stan_x==0)cout<<"Wprowadzono niepoprawne dane. Podaj ponownie: ";
    }while(stan_x==0);
    return x;
}

int main()
{
    cout<<"Ile liczb w tablicy(najpierw ENTER): ";
    int ile=czy_poprawnie(0);
    float liczba[ile],srednia,suma=0,szukana1=1,szukana2=2,x1,x2,y;
    for(int i=0;i<ile;i++){
        cout<<"Podaj "<<i+1<<" liczbe: ";
        liczba[i]=czy_poprawnie(0);
        suma=suma+liczba[i];
    }
    cout<<endl<<"\tSuma: "<<suma<<endl;
    srednia=suma/ile;
    cout<<"\tSrednia: "<<srednia<<endl;


    for(int i=0;i<ile;i++){         
        for(int j=0;j<ile;j++){
            if(liczba[i]<srednia&&liczba[i]>liczba[j]) // PROBLEM 
                szukana1=liczba[i];
        }
    }
    cout<<"Szuk1: "<<szukana1<<endl; // dla sprawdzenia

    for(int i=0;i<ile;i++){
        for(int j=0;j<ile;j++){
            if(liczba[i]>srednia&&liczba[i]<liczba[j]) // PROBLEM 
                szukana2=liczba[i];
        }
    }
    cout<<"Szuk2: "<<szukana2<<endl; // dla sprawdzenia

    x1=srednia-szukana1;
    x2=szukana2-srednia;
    cout<<"x1: "<<x1<<endl; // dla sprawdzenia
    cout<<"x2: "<<x2<<endl; // dla sprawdzenia

    y=x1-x2;
    if(y<0)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana1<<endl;
    else if(y>0)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana2<<endl;
    else if(x1==x2)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana1<<endl;

    return 0;
}

Niestety raz działa, raz nie. Co jest głupim podejściem (chcę mieć kontrolę nad programem).
Najprawdopodobniej coś w warunku jest nie ta (zaznaczyłem w kodzie), niestety nie mam pojęcia jak to obejść.

Bardzo prosił bym o pomoc.

Pozdrawiam,
Hinzeq

Answer 1

Strasznie to przekombinowane. Możesz skorzystać z funkcji abs i liczyć odleglosc liczby od sredniej jako abs(liczba - srednia).

Co do kodu to nazwa funkcji czy_poprawnie jest myląca. Jak programista widzi nazwe zaczynającą się na czy lub is to spodziewa się, że ta zmienna / funkcja odpowiada na pytanie czy, na które są dwie odpowiedzi tak lub nie - czyli jest typu bool. 

int ile=czy_poprawnie(0);
float liczba[ile];

to nie powinno się skompilować, ale najwidoczniej korzystasz z kompilatora który przymyka oko na łamanie standardu.

A ja bym to napisał tak:

int main()
{
	cout << "Ile liczb w tablicy(najpierw ENTER): ";
	int ile = wczytuj_liczbe_do_skutku();
	vector<double> liczby(ile);
	for (size_t i = 0; i < ile; i++) {
		cout << "Podaj " << i + 1 << " liczbe: ";
		liczby[i] = wczytuj_liczbe_do_skutku();
	}
	double srednia = std::accumulate(liczby.begin(),liczby.end(),0.0) / ile;
	cout << *max_element(liczby.begin(), liczby.end(), [srednia](double a, double b) {
		return abs(a - srednia) > abs(b - srednia);
	});
	return 0;
}

 

Answer 2

  cout<<"Ile liczb w tablicy(najpierw ENTER): ";
    int ile=czy_poprawnie(0);
    float liczba[ile]

Taka deklaracja tablicy nie jest poprawna, nieważne czy u Ciebie działa czy nie. Nazywa się to VLA, poniżej link dlaczego tak nie powinno się robić:

https://stackoverflow.com/questions/1887097/why-arent-variable-length-arrays-part-of-the-c-standard

Poprawną wersją jest

int ile=...
float *liczba=new float[ile];

Albo możesz użyć jakiegoś kontenera.

Co do samych warunków to ciężko mi je zrozumieć, są łatwiejsze sposoby na znalezienie liczby najbliższej średniej, np. użycie funkcji abs() zwracającej wartość bezwzgledną z wyrażenia, przykład poniżej.

if(abs(liczba[i]-srednia) < abs(liczba[i]+1 -srednia))
{
     liczbaNajblizszaSredniej=liczba[i];
}

 

Answer 3

Faktycznie.. wskaźniki..

Dodałem trochę warunków i działa teraz poprawnie:

#include <iostream>

using namespace std;

float wczytaj_liczbe_do_skutku(float x){
    bool stan_x;
    do{
        cin.clear();
        cin.ignore();
        cin>>x;
        stan_x=cin.good();
        if(stan_x==0)cout<<"Wprowadzono niepoprawne dane. Podaj ponownie: ";
    }while(stan_x==0);
    return x;
}

int main()
{
    cout<<"Ile liczb w tablicy(najpierw ENTER): ";
    int ile=wczytaj_liczbe_do_skutku(0);
    float *liczba=new float [ile];

    float srednia,suma=0,szukana1=1,szukana2=2,x1,x2,y;
    for(int i=0;i<ile;i++){
        cout<<"Podaj "<<i+1<<" liczbe: ";
        liczba[i]=wczytaj_liczbe_do_skutku(0);
        suma=suma+liczba[i];
    }
    cout<<endl<<"\tSuma: "<<suma<<endl;
    srednia=suma/ile;
    cout<<"\tSrednia: "<<srednia<<endl;


    for(int i=0;i<ile;i++){
        for(int j=0;j<ile;j++){
            if(liczba[i]<=srednia&&liczba[i]>liczba[j])
                szukana1=liczba[i];
        }
    }
    //cout<<"Szuk1: "<<szukana1<<endl; // dla sprawdzenia

    for(int i=0;i<ile;i++){
        for(int j=0;j<ile;j++){
            if(liczba[i]>=srednia&&liczba[i]<liczba[j])
                szukana2=liczba[i];
        }
    }
    //cout<<"Szuk2: "<<szukana2<<endl; // dla sprawdzenia

    x1=srednia-szukana1;
    x2=szukana2-srednia;
    //cout<<"x1: "<<x1<<endl; // dla sprawdzenia
    //cout<<"x2: "<<x2<<endl; // dla sprawdzenia

    y=x1-x2;
    if(y<0)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana1<<endl;
    else if(y>0)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana2<<endl;
    else if(x1==x2&&szukana1!=szukana2)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana1<<" i "<<szukana2<<endl;
    else if(x1==x2)
        cout<<"\tLiczba najblizej sredniej to: "<<szukana1<<endl;


    delete [] liczba;

    return 0;
}

Dziękuję za odpowiedzi, teraz zapoznam się z "vector" i "abs" i spróbuję w ten sposób.

Pozdrawiam,
Hinzeq :)