Zadanie domowe z odcinka 11 kursu C++

Question

Cześć, przerabiam własnie kurs C++ i dotarłam do zadania domowego w odcinku 11. Chciałam jednak sobie to nieco utrudnić więc zrobiłam nie dla 5 liczb tylko w tablicy żeby program działal zarówno dla 2 jak i 100 :)
Czy można ten kod jakoś jeszcze uprościć? Jakoś go zoptymalizować?

 

#include <iostream>
#include <math.h>
using namespace std;

double srednia(int ile, double *tab)
{
	double suma=0;
	for (int i = 0; i<=ile; i++)
	{
		suma += tab[i];
		
	}
	double srednia = (suma / ile);
	return srednia;
}

int main()
{
	int ile;
	cout << " Ile elementow bedzie miala Twoja tablica?" << endl;
	cin >> ile;
	double *tablica = new double[ile];

	cout << "Podaj elementy tablicy:" << endl;
	for (int i = 0; i < ile; i++)
	{
		cin >> tablica[i];
	}

	double sr = srednia(ile, tablica);
	cout << "Srednia to: " << sr << endl;
	
	double m;
	double *wyniki = new double[ile];
	for (int i = 0;i < ile;i++)
	{
		wyniki[i] = abs(sr - tablica[i]);
	}
	cout << "Wyniki odejmowania liczby od sredniej: " << endl;
	for (int i = 0; i < ile; i++)
	{
		cout << i + 1 << ".     " << wyniki[i] << endl;
	}
	
	m = wyniki[0];
	for (int i = 0; i <= ile-1;i++)
	{
		
		if (wyniki[i] < m) m = wyniki[i];
			
	}
	for (int i = 0; i < ile; i++)
	{
		if (m == wyniki[i])  cout << "Liczba najblizej sredniej to: " << tablica[i] << endl;
	}



	delete[] tablica;
	delete[] wyniki;

	return 0;
}

 

Answer 1

Hmmm, no niby możesz przy wczytywaniu liczby do tablicy od razu zliczać je do jakiejś sumy, którą potem użyjesz w funkcji średnia() zamiast liczyć tam w pętli jeszcze raz. To skróci trochę kod ale jednocześnie ucierpi trochę na czytelności więc tak jak jest, jest ok :)

Answer 2

Znalazłem jeden błąd w programie, a mianowicie w funkcji srednia() przy takim warunku pętli for do zmiennej suma zostanie dodany element znajdujący się poza tablicą.
 

    for (int i = 0; i<=ile; i++)
    {
        suma += tab[i];
         
    }

Możliwe, że przez przypadek ten błąd się wkradł, bo inne pętle są w porządku pod tym względem.
Tak poza tym, w zasadzie nie ma za bardzo z czego ten program odchudzić.

Comments

Hm... niby ma Pan rację ale jak odpalę program i podam 5 elementów każdy o wartości 2 to średnia nadal jest 2. Może dlatego nie zobaczyłam tego błędu.

---

Powiem tak, ten błąd znalazłem zanim ten kod u siebie skompilowałem. Program uruchomiłem i wszystko dobrze wygląda. Tylko dlatego dobrze działało, że jako szósty element do średniej brana była bardzo mała liczba (u mnie to było około 10^-9), więc dodanie praktycznie zera do sumy i podzielenie tego przez parametr ile, dawało i tak poprawny wynik.

Answer 3

Jedyne co na pewno bym skrócił to poniższe

    double srednia = (suma / ile);
    return srednia;

Zamieniłbym to na 

    return suma / ile;

Ale nie powinno to mieć wpływu na szybkość.

Answer 4

Yo

może lepiej by było zredukować jedną pętlę, i wrzucić to do jednej, czyli zrobić z tego:

for (int i = 0;i < ile;i++)

    {

        wyniki[i] = abs(sr - tablica[i]);

    }

    cout << "Wyniki odejmowania liczby od sredniej: " << endl;

    for (int i = 0; i < ile; i++)

    {

        cout << i + 1 << ".     " << wyniki[i] << endl;

    }

     

 

TO:

 cout << "Wyniki odejmowania liczby od sredniej: " << endl;

for (int i = 0;i < ile;i++)

    {

        wyniki[i] = abs(sr - tablica[i]);

        cout << i + 1 << ".     " << wyniki[i] << endl;

    }