Problem z pętlą, niezamierzony efekt

Question

Witam, od jakiegoś czasu męczę się z następującym problemem:

deklaruję funkcję

float RandomTable(int length)
{
    int * tablica = new int[51];
    srand(time(NULL));
    for(int i = 0;i<50;i++)
    {
        tablica[i]=rand();
        cout << tablica[i] << endl;
    }
    cout << endl << "koniec" <<endl;
    for(int i=0;i<2;i++)
    {
        cout << i << endl;
    }
}

a następnię wywołuję ją w mainie:

int main()
{
    RandomTable(50);
    return 0;
}

pierwsza pętla for działa dobrze, wykonuje 50 operacji i przestaje działać, druga pętla natomiast działa w nieskończoność. Tą samą drugą pętlę wywołałem w mainie i wyliczyła mi dwie liczby - poprawnie.

Założenie było takie, że druga pętla wypisuje mi elementy z tablicy stworzonej wcześniej, ale wypisywała mi całą tablicę i leciała dalej wypisując jakieś randomowe elementy z pamięci, więc postanowiłem wypisywanie tablicy zrobić w 1 pętli, a w drugiej dla eksperymentu dać polecenie "cout << i << endl;", pętla działa w nieskończoność.

Czy robię coś źlę? A może to jakiś błąd w codeblocks?:D

Pozdrawiam

Comments

Zapomniałeś zwrócić cokolwiek z funkcji, masz typ float, a nie zwracasz nic. Zamień typ na void, jak nic nie chcesz zwracać.

---

Poza zwracaniem floata to też tutaj nic dziwnego nie widzę - popraw na void i daj znać jak wciąż jest źle.

Answer 1

Dzięki za pomoc, rzeczywiście winą był float, chciałem rozbudować funkcję w przyszłości żeby zwracała liczbę i nie spodziewałem się że to może mieć wpływ na samo działanie funkcji.

Mam jeszcze jedno pytanko przy okazji, a mianowicie napisałem funkcję:

srand(time(NULL));
    int r;
    do
    {
        r = rand();
    }
    while (r>length-1);
    return r;

chcę ją wywołać w mainie bądź innej funkcji, żeby uzupełniała tablicę kolejnymi losowymi ale nie takimi samymi liczbami, korzystając z pętli:

for (int i=0; i<10; i++)
    {
        liczba=LosujIndeks(100);
        cout << endl << liczba;
    }

niestety program wyrzuca w każdym cyklu pętli te same liczby, po każdym uruchomieniu programu liczby są inne ale między sobą się nie różnią. Jak mogę wywołać funkcję, ażeby za każdym razem od początku losowała liczbę a nie wyrzucała to co wylosowała "w pierwszym podejściu"?

E: już sobie poradziłem, problemem było wywoływanie za każdym razem "srand(time(NULL));" w funkcji, przeniesienie wywołania sranda do maina rozwiązało problem. Pozdrawiam serdecznie.

Comments

Tak jest, ziarno losujemy tylko raz.

Btw. float winą, ale tak w zasadzie kompilator już powinien krzyczeć w trakcie kompilacji, że nie masz nic zwracane z funkcji. Korzystałem z VS, ale pewnie i w CodeBlocks da się ustawić, by kompilator sam zgłaszał takie błędy i nie kompilował, dopóki tego się nie naprawi.