Sprawdzenie kodu

Question

Czy ktoś by mógł pomóc i wytłumaczyć dlaczego kod nie działa poprawnie.  Bardzo dziękuję za pomoc dopiero zaczynam szlify :)  

 

 

#include <iostream>
#include <windows.h>
#include <time.h>

using namespace std;

int main()
{

    srand(time(NULL));

int ile_pytan=20;
int ile_wylosowac=5;
int ile_juz_wylosowano=0;
int *wylosowane = new int[ile_wylosowac+1];
bool losowanie_ok;

for(int i=1; i<=ile_wylosowac; i++)
{
    do

    {
    int liczba=rand()%ile_pytan+1;
    losowanie_ok=true;

    for(int j=1; j<=ile_juz_wylosowano; i++)
{
    if(liczba==wylosowane[j]) losowanie_ok=false;
}
    if(losowanie_ok==true)
{

    ile_juz_wylosowano++;
    wylosowane[ile_juz_wylosowano]=liczba;
}

    }while (losowanie_ok!=true);

}

    cout<<"Wylosowane numer: ";
    for(int i=1; i<=ile_wylosowac; i++)

    cout<<wylosowane[i]<<" ";

 

    return 0;
}

Answer 1

Zamiast fora po losowaniu liczby powinien być if, który sprawdza czy ile_juz_wylosowane jest większe od zera i czy liczba, którą wylosowano się nie powtarza. Oto cały, przepisany kod, który powinien działać: 

#include <iostream>
#include <windows.h>
#include <time.h>

using namespace std;

int main()
{

    srand(time(NULL));

int ile_pytan=20;
int ile_wylosowac=5;
int ile_juz_wylosowano=0;
int *wylosowane = new int[ile_wylosowac];
bool losowanie_ok;

for(int i=0; i<ile_wylosowac; i++) {


    do {

        int liczba=rand()%ile_pytan+1;
        losowanie_ok=true;

        if (ile_juz_wylosowano > 0 && liczba == wylosowane[ile_juz_wylosowano]) {

            losowanie_ok=false;
        }

        if(losowanie_ok==true) {

            ile_juz_wylosowano++;
            wylosowane[ile_juz_wylosowano]=liczba;
        }

    }while (!losowanie_ok);

    }

    cout<<"Wylosowane numer: ";
    for(int i=1; i<=ile_wylosowac; i++)

    cout<<wylosowane[i]<<" ";

    return 0;
}

Mam nadzieję, że pomogłem. Pozdrawiam.

Comments

Tak bardzo dziękuję! Również pozdrawiam

Answer 2

Kod daj w ramke i powiedz co chcesz uzyskać.

Comments

chciałbym żeby program generował 5 losowych liczb a 4 ostatnie cały czas pokazują wartości 0 , czy mógłbyś określić dokładniej gdzie dać kod w ramkę. Przepraszam ale dopiero zaczynam