• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Zadanie domowe funkcje - sprawdzenie i komentarz

VPS Starter Arubacloud
0 głosów
325 wizyt
pytanie zadane 23 marca 2016 w C i C++ przez Matheyas Obywatel (1,520 p.)

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

2 odpowiedzi

0 głosów
odpowiedź 23 marca 2016 przez criss Mędrzec (172,590 p.)
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.
komentarz 23 marca 2016 przez Matheyas Obywatel (1,520 p.)
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 :)
1
komentarz 24 marca 2016 przez criss Mędrzec (172,590 p.)
A, faktycznie.

Btw. nie ma czegoś takeigo jak interator. Int, czy integer jak już musisz.
komentarz 24 marca 2016 przez Matheyas Obywatel (1,520 p.)
Wstyd... dzięki ! :)
0 głosów
odpowiedź 23 marca 2016 przez Boshi VIP (100,240 p.)
Wypis- typ void, dostajesz ostrzeżenie bo deklarujesz returna na int a nic nie zwracasz.
komentarz 24 marca 2016 przez Matheyas Obywatel (1,520 p.)

Czyli funkcja:

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

jest źle skonstruowana?

komentarz 24 marca 2016 przez Boshi VIP (100,240 p.)
Pisałem o  funkcji wypisującej liczby.

Podobne pytania

+1 głos
1 odpowiedź 329 wizyt
pytanie zadane 26 maja 2021 w SQL, bazy danych przez ANONIM2137 Początkujący (300 p.)
0 głosów
2 odpowiedzi 897 wizyt
pytanie zadane 17 października 2015 w C i C++ przez parimple Początkujący (260 p.)
+1 głos
2 odpowiedzi 962 wizyt

92,453 zapytań

141,262 odpowiedzi

319,086 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...