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

Gdzie poprawić i kilka łatwych pytań

0 głosów
78 wizyt
pytanie zadane 3 marca 2016 w C i C++ przez Matheyas Obywatel (1,520 p.)
#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 :)

1 odpowiedź

+1 głos
odpowiedź 3 marca 2016 przez Krawiec91 Pasjonat (19,580 p.)
wybrane 3 marca 2016 przez Matheyas
 
Najlepsza

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.

komentarz 3 marca 2016 przez Matheyas Obywatel (1,520 p.)
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)?
komentarz 3 marca 2016 przez Krawiec91 Pasjonat (19,580 p.)
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.

Podobne pytania

0 głosów
1 odpowiedź 152 wizyt
0 głosów
5 odpowiedzi 247 wizyt
pytanie zadane 23 listopada 2016 w HTML i CSS przez lapa19904 Stary wyjadacz (11,590 p.)
0 głosów
1 odpowiedź 120 wizyt

87,940 zapytań

136,519 odpowiedzi

304,383 komentarzy

58,305 pasjonatów

Motyw:

Akcja Pajacyk

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

Sklep oferujący ćwiczenia JavaScript, PHP, rozmowy rekrutacyjne dla programistów i inne materiały

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...