Liczba najbliższa średniej - poprawność kodu

Question

Witam! 

Zrobiłem zadanie domowe z odc. 11 kursu C++ Pana Mirosława Zelenta. Proszę Was bardzo o sprawdzenie poprawności kodu i czy da się w nim jeszcze dokonać refaktoryzacji. 
 

#include <iostream>
#include <cmath>

using namespace std;

float l[5],srednia,wb[5],m;

int main()
{
    cout<<"Podaj 5 liczb po spacji : ";
    for(int i=0; i<5; i++)
    {
        cin>>l[i];
    }

    for(int i=0; i<5; i++)
    {
        srednia+=l[i];
    }
    srednia/=5;
    cout<<endl<<srednia<<endl;

    for(int i=0; i<5; i++)
    {
        wb[i]=fabs(srednia-l[i]);
    }

    m=wb[0];
    for(int i=1; i<5; i++)
    {
        if(wb[i]<m) m=wb[i];
    }

    for(int i=0; i<5; i++)
    {
        if(wb[i]==m) cout<<l[i]<<endl;
    }

    return 0;
}

Z góry dziękuję. Pozdrawiam!

Answer 1

Wywal zmienne globalne i nazywaj zmienne tak, aby człowiek nieznający kodu był w stanie powiedzieć za co one odpowiadają. Nazwy l, wb i m nie mówią nic.

Comments

Czyli mam rozumieć, że zamiast zmiennych globalnych należy używać lokalnych i nazywać te zmienne w sposób czytelny ? :)

Dziękuję za odpowiedź!

Answer 2

Po co dajesz dwie takie same pętle po sobie? Wrzuć to do jednej i już masz refaktoryzacje.

Comments

Może nie zna pojęcia funkcji :)