Wie kann ich die Warnung mit rand() loswerden? (C++)

Question

Jedes Mal, wenn ich die Funktion rand in C++ verwenden:

#include<iostream> 
#include<time.h> 
#include<stdlib.h> 
using namespace std; 
int main(){ 
srand(time(0)); 
int n=(rand()%6)+1; 
cout<<"The dice roll is "<<n<<"."<<endl; 
} 

ich eine Warnung über Umwandlung von Time_t erhalten in Zeile int 5:

srand(time(0)); 

Gibt es eine Möglichkeit, um loszuwerden, diese Warnung?

Answer 1

Eigentlich sollten Sie eine eine unsigned int with srand() werden:

srand((unsigned) time(0)); 

Answer 2

Verwenden Sie eine explizite Umwandlung der Warnung loszuwerden:

srand((int)time(0)); 

Answer 3

auf einem anderen Ton, dieser Code:

rand()%6 

wird allgemein als eine schlechte Praxis angesehen. die unteren Bits von rand() sind signifikant weniger zufällig als die höheren Bits. Sie erhalten eine bessere Zufälligkeit, wenn Sie folgendes tun:

(rand() >> 8)%6 

zum Beispiel.

EDIT:

Weitere Einzelheiten hierzu finden Sie this note und auch dieser Artikel von Dr. Dobbs Zeitschrift, die zumindest Hinweis auf den Grund:

Hinweis: Verwenden Sie KEINE

y = rand() % M; 

wie dies auf die unteren Bits von Rand() konzentriert. Für lineare Kongruenz zufällige Zahlengeneratoren, die rand() oft ist, sind die unteren Bytes viel weniger zufällig als die höheren Bytes. In der Tat das niedrigste Bit zwischen 0 und 1 Zyklen. So kann rand() zwischen geraden und ungerade (probieren Sie es aus). Anmerkung rand() muss nicht ein linearer kongruenter Zufallszahlengenerator sein. Es ist perfekt zulässig, damit es etwas besser ist, das dieses Problem nicht hat.

DDJ:

Der wichtigste Punkt ist, dass die unteren Bits der Ausgabe aus der üblichen (linearen Kongruenz) Zufallszahlengeneratoren die am wenigsten sind "random". Das heißt, Muster in den unteren Bits sind üblich. Daher ist die Ausgabe von der Routine in Ihrer Diskussion nicht überraschend. Außerdem ist es vermeidbar, indem man sich auf die oberen Bits bezieht, um die Ganzzahl zurückzugeben.

Zum Beispiel, wenn Sie wollten einen zufälligen „true“ oder „false“ Wert wählen, und Sie verwendete den Code:

rand() % 2 

Dann könnten Sie das Muster der Ergebnisse am Ende sehen:

1,0,1,0,1,0,1,0,1,0 (etc)

Dies ist offensichtlich nicht so zufällig, aber es ist eine Eigenschaft des Kongruenzgenerator, der in sein könnte benutzen. Ein besseres Schema (für C++) könnte die Verwendung der Boost.Random-Bibliothek sein, die Unterstützung für alle Arten von steckbaren Zufallsgeneratoren bietet (einschließlich Mersenne Twister, die diesen Fehler nicht aufweist).

Answer 4

Zwei seitliche Hinweise:

• Die Standardmethode C-Header in C++ von darunter ist wie folgt: #include <cstdio>.
• Der an time() übergebene Parameter ist ein Zeiger, und viele Leute denken, dass NULL ein besser lesbarer Nullzeiger als 0 ist. Auch

Answer 5

,

rand() % 6 

eine kleine Vorspannung einzuführen. Da RAND_MAX% 6 1 ist, werden Null und Eins etwas häufiger auftauchen als zwei bis sechs. In diesem Fall werden sie 5462-mal für 5461-mal zurückgegeben, die höheren Zahlen werden zurückgegeben, also werden Sie es wahrscheinlich nicht bemerken. Wenn jedoch der Bereich der gewünschten Zahlen groß ist, kann die Verzerrung signifikant sein. Wenn Sie zum Beispiel rand() % 32000 hätten, würde die Zahl im Bereich 0 - 767 doppelt so häufig erscheinen wie diese 768 - 32000.

Answer 6

Um die Warnung loszuwerden, sollten Sie eine statische Umwandlung in eine Ganzzahl ohne Vorzeichen verwenden.

Zu einem verwandten Hinweis sollten die Ergebnisse von Rand nach rechts verschoben werden, um Verzerrungen in den unteren Bits zu entfernen.

int n = ((rand() >> 8) % 6) + 1; 

schließlich in C++ die C Zeit und Standardbibliotheken sollten enthalten sein wie:

#include <ctime> 
#include <cstdlib> 

Diese Funktionen in dem entsprechenden Namespace platzieren wird, 'std'.