Moja pierwsza zabawa z funkcjami, sprawa dosyć prosta. Zależy mi na sprawdzeniu jakości kodu - dla mnie jako laika jest to dosyć istotne. Chcę wiedzieć na co zwracać uwagę w przyszłości.
Opis zadania:
Napisz program, który wylosuje 999 liczb całkowitych z zakresu od 4 do 10 włącznie, wypisze te wartości na ekran, po czym zsumuje je i wynik wypisze na ekran. Wszelkie operacje mają zostać zrealizowane w oparciu o funkcje. Tablica ma być zadeklarowana w funkcji main. Oczekiwane funkcje:
- Wypełnienie tablicy o podanym rozmiarze losowymi liczbami z przedziału podawanego przez argumenty funkcji.
- Sumowanie określonej ilości liczb znajdujących się w tablicy i zwrot wyniku.
- Wypisywanie zawartości tablicy.
Przykładowa implementacja funkcji main:
przykład jest taki sam jak u mnie w funkcji main.
#include <iostream>
#include <time.h>
#include <cstdlib>
using namespace std;
//////////////////////////////////////////////////////////
int wylosujLiczby(int tab[], int rozmiar, int b, int c)
{
srand (time(NULL));
int los = 0, ilosc = 0, x;
x = (c-b)+1;
while(ilosc <= rozmiar)
{
los = (rand ()%x)+b;
tab[ilosc]=los;
ilosc++;
}
}
////////////////////////////////////////////////////////
int wypiszLiczby(int tab[], int rozmiar)
{
int ilosc = 0;
int a = 0;
cout<<"Wypis wszystkich liczb z tablicy: "<<endl;
while (ilosc <= rozmiar)
{
cout<<tab[ilosc]<<" ";
ilosc++;
}
}
////////////////////////////////////////////////////////
int obliczSume(int tab[], int rozmiar)
{
int ilosc = 0, a;
while (ilosc <= rozmiar)
{
a = a + tab [ilosc++];
}
return a;
}
///////////////////////////////////////////////////////
int main()
{
int tablica[ 999 ];
wylosujLiczby (tablica, 999, 4, 10);
wypiszLiczby( tablica, 999 );
int iSuma = obliczSume( tablica, 999 );
cout << endl << "Suma liczb wynosi: " << iSuma <<endl;
return 0;
}
Edit:
Znalazłem pierwszy błąd, w sumowaniu...