[C++] Generare un numero random

[andbin]

Originariamente inviato da trallallero

vabbé non é quello che impedisce la ridefinizione di RAND_MAX. Pensavo intendessi qualcos'altro

Ci ho provato (#define RAND_MAX 10) mi da giustamente un warning sulla define
ma poi non me la caga proprio
Non mi considera proprio

A parte il fatto che se vuoi ridefinire una macro già esistente, devi prima fare un #undef ...

Comunque il problema non è quello!
Sono andato adesso a vedere il sorgente della rand() nel VC++. Eccolo:

int __cdecl rand (
        void
        )
{
#ifdef _MT

        _ptiddata ptd = _getptd();

        return( ((ptd->_holdrand = ptd->_holdrand * 214013L
            + 2531011L) >> 16) & 0x7fff );

#else  /* _MT */
        return(((holdrand = holdrand * 214013L + 2531011L) >> 16) & 0x7fff);
#endif  /* _MT */
}

Vedi che il valore di ritorno viene sempre troncato con un & 0x7fff??? Vuol dire che il comportamento di questa rand() è quello di ritornare un valore tra 0 e 32767.

E nell'include stdlib.h fornito con il VC++ c'è:
#define RAND_MAX 0x7fff

Se anche tu ridefinissi RAND_MAX, la funzione rand() continua a funzionare secondo il modo con cui è stata progettata e compilata!!!!!!

[trallallero]

Originariamente inviato da andbin

A parte il fatto che se vuoi ridefinire una macro già esistente, devi prima fare un #undef ...

vabbé quello é solo per evitare il warning

Originariamente inviato da andbin

Comunque il problema non è quello!
Sono andato adesso a vedere il sorgente della rand() nel VC++. Eccolo:

int __cdecl rand (
        void
        )
{
#ifdef _MT

        _ptiddata ptd = _getptd();

        return( ((ptd->_holdrand = ptd->_holdrand * 214013L
            + 2531011L) >> 16) & 0x7fff );

#else  /* _MT */
        return(((holdrand = holdrand * 214013L + 2531011L) >> 16) & 0x7fff);
#endif  /* _MT */
}

Vedi che il valore di ritorno viene sempre troncato con un & 0x7fff??? Vuol dire che il comportamento di questa rand() è quello di ritornare un valore tra 0 e 32767.

E nell'include stdlib.h fornito con il VC++ c'è:
#define RAND_MAX 0x7fff

Se anche tu ridefinissi RAND_MAX, la funzione rand() continua a funzionare secondo il modo con cui è stata progettata e compilata!!!!!!

ma bene! che bello vedere le costanti nel codice

[tomminno]

Se vi serve un generatore di numeri casuali che rispetti un minimo di requisiti di casualità andatevi a leggere i Numerical Recipes, senza perdere troppo tempo su rand().

[AngeL)]

ho un problema... il compilatore dice che null non è definito... questa è la funzione

int randgen()
{
srand(time(null));
return rand()%4+1;
}

help

[Energy++]

Semplice ti da errore perche lo devi scrivere tutto maiuscolo!!

NULL

ciao ciao

[AngeL)]

ehm.. scusa

[AngeL)]

ho scritto una funzione che scrive 100 volte il risultato di randgen e mi esce lo stesso numero... dove ho sbagliato?

[Energy++]

Originariamente inviato da AngeL)

ho scritto una funzione che scrive 100 volte il risultato di randgen e mi esce lo stesso numero... dove ho sbagliato?

così dovrebbe andare:

int randgen()
{
return rand()%4+1;
}

main(){
srand(time(NULL));
for(int i=0; i<101; i++)
cout <<randgen()<<endl;
system("pause");
}

ciao ciao

[Ziosilvio]

Originariamente inviato da AngeL)

il compilatore dice che null non è definito

null non è definito. NULL, invece, sì --- e il C è case-sensitive.

return rand()%4+1;

return (rand()/(RAND_MAX+1.0))*4+1;

qui

[Ziosilvio]

Originariamente inviato da AngeL)

ho scritto una funzione che scrive 100 volte il risultato di randgen e mi esce lo stesso numero... dove ho sbagliato?

Se non viene specificato altrimenti, il generatore pseudorandom viene sempre inizializzato col valore di default, quindi a ogni nuova esecuzione la sequenza è sempre la stessa.
Il problema si risolve inizializzandolo con un valore diverso a ogni esecuzione: il modo classico srand(time(NULL)), in base all'ora di sistema, ma ho visto fare anche srand(getpid()), in base al PID del processo in corso.

[AngeL)]

Originariamente inviato da Energy++

così dovrebbe andare:

int randgen()
{
return rand()%4+1;
}

main(){
srand(time(NULL));
for(int i=0; i<101; i++)
cout <<randgen()<<endl;
system("pause");
}

ciao ciao

il mio codice è uguale.

Originariamente inviato da ziosilvio

non

return rand()%4+1

ma

return (rand()/(RAND_MAX+1.0))*4+1;

che cambia?

edit: ho fatto cosi

#include <iostream>
#include <time.h>
using namespace std;

int randgen();

int main()
{
int i;
for(i=0;i<10;i++)
{
cout << randgen() << endl;
}
}

int randgen()
{
srand(time(NULL));
return (rand()/(RAND_MAX+1.0))*4+1;
}

ma restituisce sempre lo stesso numero :\

[andbin]

Originariamente inviato da AngeL)

ma restituisce sempre lo stesso numero :\

srand la devi chiamare 1 volta sola all'inizio del programma.

[AngeL)]

grazie

[Energy++]

Originariamente inviato da andbin

srand la devi chiamare 1 volta sola all'inizio del programma.

si infatti

[Ziosilvio]

Originariamente inviato da AngeL)

che cambia?

Che usi tutte le cifre usate da rand, anziché solo quelle meno significative.
In questo modo, la qualità del generatore che ottieni, è confrontabile con quella di rand; nell'altro modo, rischi di ottenere un generatore anche molto peggiore.