Zadanie domowe funkcje - sprawdzenie i komentarz

Question

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...

Answer 1

W sumowaniu nie wiem gdzie widzisz błąd. Btw. użyj operatora +=, będzie ładniej :D We wszystkich pętlach zamien <= na <. Wychodzisz za tablice. Reszta wydaje się w porządku.

Comments

41 linijka interator "a" nie ma przypisanej wartości.

Gdy pozmieniałem wielkość tablicy i cyfr od - do, na mniejsze to suma losowych liczb wykazywała jakieś brednie. Gdy zmieniłem na a=0, pokazało poprawnie.

W każdym razie updatowałem program i teraz sam wpisuje wielkość tablic i od -do :)

---

A, faktycznie.

Btw. nie ma czegoś takeigo jak interator. Int, czy integer jak już musisz.

---

Wstyd... dzięki ! :)

Answer 2

Wypis- typ void, dostajesz ostrzeżenie bo deklarujesz returna na int a nic nie zwracasz.

Comments

Czyli funkcja:

int obliczSume(int tab[], int rozmiar)
{
    int ilosc = 0, a;
    while (ilosc <= rozmiar)
        {
            a = a + tab [ilosc++];
        }
        return a;
}

jest źle skonstruowana?

---

Pisałem o  funkcji wypisującej liczby.