Gdzie poprawić i kilka łatwych pytań

Question

#include <iostream>
#include <time.h>
#include <cstdlib>
using namespace std;

int main()
{
    int los;
    int przydzial [999];
    int ilosc=0;
    int ilosc2=0;
    int suma=0;
    srand (time(NULL));

    while(ilosc <= 999)
        {
         los = (rand () %7)+4;
         przydzial [ilosc] = los;
            cout<< przydzial [ilosc]<<" ";
         ilosc++;
        }

    while (ilosc2 <= 999)
{
        
        suma = suma + przydzial [ilosc2++];
}


            cout<<endl<<"suma losow wynosi: "<<suma;

    return 0;
}

 

1) Można jakoś skrócić ten kod? Bo gdy próbuje połączyć "while" w jedno to w "sumie" wyskakują mi monstrualne liczby. (bez pętli "for", ew z pętla - takie zadanie :) )

2)  Przy drugiej pętli while nie mogłem użyć warunku zmiennej "ilosc" (tj. w pierwszej pętli while) wyskakiwały mi dziwne znaki. Przed pętlą mam zadeklarowane że "ilosc = 0" więc po skończeniu pierwszej pętli program kontynuuje podróż i nie bierze pod uwagę początkowej deklaracji?

W celu rozwiązania tego problemu użyłem nowej zmiennej "ilosc2" dobra decyzja? mogłem podjąć inną? (Nie licząc, że jest poprawna - program działa).

3) Największy problem miałem z sumowaniem wylosowanych liczb przy użyciu tablic. Sądziłem że gdy dam:

    while (ilosc2 <= 999)
{
        ilosc2++;
        suma = suma + przydzial [ilosc2];
}

Będzie to samo co "przydzial [ilosc2++] ale powyższy kod daje 2x większą cyfrę. Myślałem, że "ilosc2++" jest konieczny do warunku pętli while.

Mam nadzieję, że jakoś poprawnie to wszystko ująłem :)

Answer 1

Ad 1) Jedną petlą for lub while (jak kto woli, ja bym dał for, ale to czy będzie for czy while, nie jest istotne) można załatwić losowanie i sumowanie. Kodu mniej wyjdzie jakby nie patrzeć.

Ad 2) Jak najbardziej mogłeś użyć zmiennej ilosc, pod warunkiem, że przed użyciem w drugiej pętli while, wyzerowałbyś ją (przypisał do niej zero). Bo tak jak masz teraz w kodzie, to zaczyna liczyć od 999. Z tego faktu, zapewne wynikają te monstrualnie wielkie liczby, bo przypisujesz wartości z komórek pamięci, które znajdują się poza tablicą.

Ad 3) Co do fragmentu kodu, który na końcu postu zamieściłeś. Różni się on w działania od tego zamieszczonego wcześniej. Sumowanie wykonywane jest od elementu [1] tablicy nie od [0], gdyż inkrementacje ilosc2 robisz przed zsumowaniem.

Primo ultimo) Ustawienie warunku ilosc <= 999 w pętlach jest błedem, gdyż kończy się to wyjściem poza tablicę. Tablica 999-elementowa, zawiera elementy od [0] do [998]. Więc, żeby było git, to albo <=998 albo <999.

Comments

Dzięki za istotne uwagi.

Najważniejsze pytanie:

Nie jest to rozbudowany program, ale jest możliwość skrócenia. Skracać, czy brać się za następny (kolejne lekcje, bo ta jest już zaliczona)?

---

Za wiele z tego programu nie ućkniesz (jedna prosta pętla i jedna zmienna mniej). Ważniejsze, żebyś przeanalizował błędy jakie się w nim pojawiły (że nie działał, jak powinien) i ich przyczyny.