Pojawiające się "0" przy pseudo losowaniu liczby

Question

Treść zadania : 

• Napisz funkcję, która losuje liczbę z przedziału od 50 do 60 włącznie. Wywołaj funkcję kilka razy (wypisz wylosowane wartości na ekran) w celu przetestowania czy działa ona poprawnie.
• Mój KOD : 

  #include <iostream>
  #include <time.h>
  #include <cstdlib>
  int losowanie()
  {
      int ile;
      std::cin>>ile;

      srand( time( NULL ) );

      while(ile)
      {
          std::cout<<rand()%10+50<<std::endl;
          ile--;
      }

  }

  int main()
  {

      std::cout<<"Podaj ile razy chcesz losowac liczbe : ";
      int wynik=losowanie();
      std::cout<<wynik;

  
      return 0;
  }
   

• Problem polega na tym że przy losowaniu np 3 liczb na końcu pojawia sie "0" nie mam pojęcia dlaczego, próbowałem juz zmieniać KOD na różne sposoby ale nic z tego jedynie raz zero zmieniło się na 1 ale to nadal nie rozwiązuje problemu, Proszę o radę :) 

• 

Answer 1

Uwaga porządkowa: kod źródłowy wstawiamy w bloczek "code" (ostatni przycisk w edytorze).

Twoja funkcja losowanie() zwraca wartość int, którą później wypisujesz w linii std::cout<<wynik;. W swoim kodzie w ogóle nie uwzględniasz zwracania wartości z funkcji losowanie. Najpierw dopisz tam:

int losowanie() 
{ 
    int ile; 
    std::cin>>ile;

    srand( time( NULL ) );

    while(ile) 
    { 
        std::cout<<rand()%10+50<<std::endl; 
        ile--; 
    }
    return 9000;
}

i sprawdź, co otrzymujesz. :) Potem zmień sygnaturę rzeczonej funkcji tak, żeby zwracała void i usuń tego returna oraz wypisanie. I sprawdź co otrzymujesz. :)

Comments

Dziękuję bardzo, faktycznie nie zauważyłem braku return  poprawiłem na void i wszystko działa :)

---

I wyrzuć sranda z funkcji losującej (przerzuć do maina) :P Inicjalizacja seeda przy każdym wywołaniu zdecydowanie mija się z celem.

---

I wyrzuć sranda z funkcji losującej (przerzuć do maina) :P Inicjalizacja seeda przy każdym wywołaniu zdecydowanie mija się z celem.

Criss, zauważ że mamy tutaj tylko jedno wywołanie funkcji losowanie().

---

Anyway "brzydkie programistycznie" :P

---

Kolega Krystian_n się jeszcze wyrobi. :)

---

Po prostu nie można z góry zakładać, że funkcja zostanie wywołana raz i tym bazować reszte kodu. Każdy się wyrobi, ale tylko wtedy jak będzie się mu wskazywać błędy :)

Answer 2

Napisz funkcję, która losuje liczbę z przedziału od 50 do 60 włącznie. Wywołaj funkcję kilka razy (wypisz wylosowane wartości na ekran) w celu przetestowania czy działa ona poprawnie.

Jakiego więc ma typu być wartość zwracana? Typu int czy unsigned int ? :-)

Inną sprawą jest użycie bardzo już przestarzałej, tworzącej błędy i niezalecanej od 2011 metody losowania. Obecnie do losowania liczb z podanego zakresu używa się nagłówka <random> oraz odpowiednich funkcji rozkładu. U Ciebie będzie to rozkład równomierny http://en.cppreference.com/w/cpp/numeric/random/uniform_int_distribution

Jeśli jednak powinieneś upierać się w metodzie "z srand() i rand()", to pętlę przenieś do main() bo losowanie() ma zwrócić po 1 wywołaniu 1 liczbę.Wtedy wywołanie srand(NULL), powinno pojawić się także w main() bo inicjalizację tego generatora przeprowadzasz 1 raz.

Zmień nagłówek <time.h> na <ctime> oraz informuję Cię że w C++ nie ma obowiązku zwracania na końcu main() zera. Kompilator robi to sam (ale pamiętaj że tylko main() ma taką "magiczną właściwość" :-)) 

Dodatkowo przemyśl co się stanie jeśli zamiast podania ilości wywołań zostanie wprowadzony np. napis lub liczba ujemna :-)

Answer 3

Co według ciebie zwraca funkcja losowanie()?
Bo to właśnie wypisujesz na ekran: std::cout << wynik;

Answer 4

Przecież funkcja losowanie() nic nie zwraca (brak return), a tylko wyświetla wylosowaną liczbę. Potem wyświetlasz wartość zmiennej wynik, która jest zerem (żadna wartość z funkcji losowanie() nie jest przypisywana, gdyż funkcja nic nie zwraca).

Ogólnie jeśli nie użyjemy w funkcji instrukcji return, wartość zwracana będzie przypadkowa (dostaniemy śmieci z pamięci).

Answer 5

Dodatkowo zakres liczb losowanych jest od 50 do 59.

std::cout<<rand()%11+50<<std::endl;

Teraz jest dobry zakres

Answer 6

zrobienie  z funkcji losowanie() voida i zamienienie tego kawałka kodu

int wynik=losowanie(); 
    std::cout<<wynik;

na ten 

losowanie(); 

powinno pomóc