Czemu ta funkcja nie dziala ?

Question

co znowu robię źle :/?

czemu tablica sie nie wczytuje ?

czemu losowanie nie dziala ?

 

#include <iostream>
#include <cstdlib>
#include <ctime>


using namespace std;



void wylosujLiczby(int t[],int ile,int start,int stop)
{


    cout<<"Losowanie "<<ile<<" liczb z zakresu od "<<start<<" do "<<stop<<endl;

    int i=0;

    do{
          srand(time(NULL));
         t[i]=((rand()%(stop-start+1)+4));

    cout<<"TEST "<<i+1<<"    "<<t[i]<<endl;//test
    i++;
    }while(i<ile);

}

int main()
{

    int i;
     int tablica[ 999 ];
    wylosujLiczby( tablica, 999, 4, 10 );
    do
    {
        cout<<"TEST 2"<<tablica[i]<<endl;
        i++;

    }while(i<999);
    return 0;
}

 

Comments

Ziarno przenieś nad "do".

---

ok losowanie zadziałało a co z tym ze main nie pobiera tablic ?

---

Twoje i jest zerem.

Edit:

Skrót myślowy, "i" nie jest niczym, nie jest zainicjalizowana.

---

Ok rozumiem. Masakra mój błąd:/

---

Wspieraj się flagami kompilatora, uratują trochę czasu. W czym piszesz?

---

obecnie w CodeBlocks ale to był ostatni raz teraz przechodzę na Visual Studio 2019

 

tylko mam problem ze ciagle AVAST mi skanuje programy ;/ jest to irytujace

---

https://www.google.com/search?q=disable+avast+for+visual+studio&oq=avast+disale+for+vi&aqs=chrome.1.69i57j0l7.5839j0j7&sourceid=chrome&ie=UTF-8

VS sam inicializuje zmienne, jak dobrze pamiętam pod 0xCCC... w trybie debug, na co musisz uważać...

Answer 1

Jak napisał przedmówca, przeniesienie srand(); na początek funkcji losuj lub na początek funkcji main.

W funkcji main nie masz przypisanej wartości do "i". Lepszym rozwiązaniem byłoby skorzystanie z pętli for. Wtedy "i" z przyzwyczajenia być zainicjował :)

#include <iostream>
#include <cstdlib>
#include <ctime>


using namespace std;



void wylosujLiczby(int t[],int ile,int start,int stop)
{
    srand(time(NULL));

    cout<<"Losowanie "<<ile<<" liczb z zakresu od "<<start<<" do "<<stop<<endl;

    int i=0;

    do{

         t[i]=((rand()%(stop-start+1)+4));

    cout<<"TEST "<<i+1<<"    "<<t[i]<<endl;//test
    i++;
    }while(i<ile);

}

int main()
{


    int i = 0;
     int tablica[ 999 ];
    wylosujLiczby( tablica, 999, 4, 10 );
    do
    {
        cout<<"TEST 2"<<tablica[i]<<endl;
        i++;

    }while(i<999);
    return 0;
}