Zadanie domowe z kursu Zelenta - odcinek 11

Question

#include <iostream>

using namespace std;

float a,b,c,d,e,x,m,w;
float a1,b1,d1,c1,e1;

int main()
{
    cout << "Podaj 5 liczby rozdzielone spacja: ";
    cin>>a>>b>>c>>d>>e;
    x = (a+b+c+d+e)/5;
    cout << x <<endl;
    a1=x-a;
    b1=x-b;
    c1=x-c;
    d1=x-d;
    e1=x-e;
    cout << "liczba:" <<endl;
    if(a1<0) a1=-a1;
    if(b1<0) b1=-b1;
    if(c1<0) c1=-c1;
    if(d1<0) d1=-d1;
    if(e1<0) e1=-e1;
    m=a1;
    if(m>b1) w=b; m=b1;
    if(m>c1) w=c; m=c1;
    if(m>d1) w=d; m=d1;
    if(m>e1) w=e; m=e1;
    cout << w;
    return 0;
}

Co myślicie o takim rozwiązaniu? Program szuka najbliższej liczby do średniej wszystkich podanych 5. Pisałem to 2h, zastanawiając się nad rozwiązaniem.

Answer 1

1) Nie używaj zmiennych globalnych, zmienne powinieneś zadeklarować wewnątrz maina

2) 5 liczb to już wystarczająco żeby użyć tablic i pętli - będzie krótsze

3) Zamiast jakichś zmian znaków w C++ jest funkcja fabs(), która zwraca moduł z liczby zmiennoprzecinkowej

Comments

"Nie używaj zmiennych lokalnych" Chyba globalnych

---

Tfu! Już edytuje! Taki skrót myślowy mi się wkradł :D jako "nie używaj globalnych, używaj lokalnych"

Answer 2

1. użyj funkcji abs

2. użyj tablicy i pętli

będzie prościej i uniwersalniej ( a co jak 100 licz , albo 1000 ktoś zażyczy ? )

Comments

W tym przypadku fabs(), abs() jest dla liczb całkowitych :)

Answer 3

Każde rozwiązanie oparte na masie warunków jest złe. Myślę, że można to sprawdzić w pęli na spokojnie.

Answer 4

1. Nie znam jeszcze funkcji fabs z odcinków Zelenta, więc chyba logiczne, że miało się obyć bez nich.

2. Pokażcie mi jak to wygląda z tablicami

3. Jak jest różnica między tym rozwiązaniem a z tablicami? Szybkość, czy tylko kwestia czytelności kodu?