srand (time (null)) avverte il compilatore: la conversione implicita perde la precisione di un numero intero

Question

Ci scusiamo se questa domanda ha già avuto risposta.

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

using namespace std; 

int main() { 

srand(time(NULL)); 
cout << rand(); 
} 

"conversione implicita perde precisione intero: 'time_t' (aka 'lunga') per 'unsigned int'"

È il messaggio di errore Im ottenendo quando eseguo il codice di cui sopra. Sto usando xcode 4.6.1. Ora quando uso un altro compilatore come quello di codepad.org, esso esegue perfettamente bene ciò che sembra come numeri casuali, quindi presumo che sia un problema di xcode che devo risolvere?

Ho appena iniziato la programmazione quindi sono un principiante assoluto quando si tratta di questo. C'è un problema con il mio codice o è il mio compilatore?

Qualsiasi aiuto sarebbe apprezzato!

Answer 1

"conversione implicita perde precisione intero: 'time_t' (aka 'lunga ') per 'unsigned int'"

stai perdendo precisione implicitamente perché time() restituisce una long che è più grande di un unsigned int sul vostro obiettivo. Per risolvere questo problema, è necessario lanciare esplicitamente il risultato (eliminando in tal modo la "perdita implicita di precisione"):

srand(static_cast<unsigned int>(time(NULL))); 

---

Dato che è ora il 2017, sto modificando questa domanda a suggerire che si considerare le funzionalità fornite da std::chrono::* definite in <chrono> come parte di C++ 11. Il tuo compilatore preferito fornisce C++ 11? In caso contrario, dovrebbe davvero!

Per ottenere l'ora corrente, si consiglia di utilizzare:

#include <chrono> 

void f() { 
    const std::chrono::time_point current_time = std::chrono::system_clock::now(); 
} 

Perché dovrei perdere tempo con questo quando time() opere?

IMO, una sola ragione è sufficiente: tipi chiari, espliciti. Quando si affrontano programmi di grandi dimensioni tra team abbastanza grandi, sapere se i valori trasmessi rappresentano intervalli di tempo o tempi "assoluti" e quali sono le grandezze critiche.Con std::chrono è possibile progettare interfacce e strutture dati che sono portatili e saltare sul blues is-that-time-out-a-dead-mill-sec-time-or-wait-was-is-secondi.

Answer 2

La funzione srand ha unsigned uns come tipo di argomento, time_t è di tipo long. i 4 byte superiori da lunghi vengono eliminati, ma non ci sono problemi. srand randomizzerà l'algoritmo rand con 4 byte inferiori di tempo, quindi fornirai più dati del necessario.

se si ottiene un errore, provare a lanciare solo in modo esplicito il tipo time_t a unsigned int:

srand((unsigned int) time(NULL)); 

Un'altra cosa interessante è che se si esegue il programma due volte nello stesso secondo, si otterrà la stesso numero casuale, che a volte può essere indesiderato, perché se si semina l'algoritmo di Rand con gli stessi dati, genererà la stessa sequenza casuale. Oppure può essere desiderabile durante il debug di un certo pezzo di codice e la necessità di testare ancora una volta lo stesso comportamento ... allora è sufficiente utilizzare qualcosa come srand(123456)

Answer 3

Questo è non un errore. Il codice è valido e il suo significato è ben definito; se un compilatore rifiuta di compilarlo, il compilatore non è conforme alla definizione della lingua. Più probabilmente, è un avviso , e ti sta dicendo che lo scrittore del compilatore pensa che potresti aver fatto un errore. Se insisti a eliminare i messaggi di avviso, potresti aggiungere un cast, come altri hanno suggerito. Non sono un grande fan della riscrittura di un codice valido e significativo per soddisfare la nozione di buon stile del produttore di compilatori; Spegnerei l'avviso. Se lo fai, però, potresti trascurare altri luoghi in cui una conversione perde i dati che non hai inteso.

Answer 4

Come menzionato da "nio", una soluzione pulita sarebbe quella di digitare espressamente il cast. spiegazione

Deeper:

Lo srand() richiede un unsigned int come parametro (srand(unsigned int)), ma il tempo() restituisce un long int (long int time()) e questo non è accettata dal srand() in modo al fine per risolvere questo problema, il compilatore deve semplicemente typecast (convert) "long int" in "unsigned int".

MA nel tuo caso il compilatore ti avverte invece (come i progettisti del compilatore pensavano dovresti essere a conoscenza che è tutto).

Quindi, un semplice

srand((unsigned int) time(NULL));

farà il trucco!

(mi perdoni se ho fatto qualcosa di sbagliato, questa è la mia prima risposta su StackOverflow)

Answer 5

#include <stdlib.h> 
#include <iostream>   //rand 
#include <time.h>  //time 

float randomizer(int VarMin, int VarMax){ 
    srand((unsigned)time(NULL)); 
     int range = (VarMax - VarMin); 
     float rnd = VarMin + float(range*(rand()/(RAND_MAX + 1.0))); 
return rnd; 
}