Kurs C++ Lekcja 11 Zadanie Domowe

Question

Cześć,

Zrobiłem zadanie z odcinka 11 programowanie C++. 

Program poprawnie liczy wynik dla każdego z trzech przypadków podanych w filmiku.

Moje pytanie czy jest poprawnie napisany? I czy ewentualnie można jeszcze go zmniejszyć? Jeśli tak to poproszę jakieś wskazówki, bez poprawnego rozwiązania ;) (tak, abym mógł sam spróbować zmienić).

Dzięki!

#include <iostream>
#include <cmath>
#include <cstdlib>

using namespace std;
float sr,najblizsza,m,suma;
float tb[5];

int main()
{
    for (int i=0; i<5; i++)
    {
        cout<<"Podaj "<<i+1<<" liczbe: ";
        cin>>tb[i];
        suma=suma+tb[i];
    }
    sr=suma/5;
    najblizsza=tb[0];
    m=abs(tb[0]-sr);

    for (int i=0; i<=5; i++)
    {
        if (abs(tb[i]-sr)<m)(najblizsza=tb[i])&&(m=abs(tb[i]-sr));
    }
    cout<<endl;
    cout<<"Srednia to: "<<sr<<endl;
    cout<<endl;
    for (int i=0;i<5;i++)
    {
        if (najblizsza==tb[i]) i=i+1;
        if((abs(najblizsza-sr))==(abs(tb[i]-sr)))
            cout<<"Dwie najblizsze liczby to: "<<najblizsza<<" i "<<tb[i];
        else cout<<"Najblizsza to: "<<najblizsza;
            cout<<endl;
            exit(0);
    }
    return 0;
}

 

Answer 1

1) Powinieneś inicjalizować zmienną suma

2) Wydaje mi się, że dobrym nawykiem przy działaniu na liczbach zmiennoprzecinkowych jest pisanie w formie typue: 
sr = suma / 5.0; To nie znaczy, że tak jak masz jest źle, tylko po prostu to poprawia czytelność, bo od razu wiadomo, że działasz na float / double bez patrzenia na definicje
3) 

 if((abs(najblizsza-sr))==(abs(tb[i]-sr)))

Nie wszystkie nawiasy są potrzebne, za dużo nawiasów pogarsza czytelność 

 

Tyle ode mnie :) Pozdrawiam :)

Comments

Dzięki bardzo za uwagi :).

Ad 1) Wskazywać w samym programie, że np suma=0; , dobrze rozumiem?

Ad 2) W sumie racja, przy robieniu floatów, może być to czytelniejsze w dłuższym kodzie. Dzięki:)

Ad 3) Masz na myśli jak poniżej?

if(abs(najblizsza-sr))==(abs(tb[i]-sr))

 

---

1) Dokładnie :)

3) 

if( abs(najblizsza-sr) == abs(tb[i]-sr) )

 

Answer 2

Dzięki za odpowiedzi! :)

Zastosuję propozycje na pewno!