Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

warning C4244: 'argument' : conversion from 'time_t' to 'unsigned int', possible loss of data -- C++

Tags:

c++

I made a simple program that allows the user to pick a number of dice then guess the outcome... I posted this code before but with the wrong question so it was deleted... now I cannot have any errors or even warnings on this code but for some reason this warning keeps popping and I have no clue how to fix it... "warning C4244: 'argument' : conversion from 'time_t' to 'unsigned int', possible loss of data"

#include <iostream>
#include <string>
#include <cstdlib>
#include <time.h>

using namespace std;

int  choice, dice, random;

int main(){
    string decision;
    srand ( time(NULL) );
    while(decision != "no" || decision != "No")
    {
        std::cout << "how many dice would you like to use? ";
        std::cin >> dice;
        std::cout << "guess what number was thrown: ";
        std::cin >> choice;
         for(int i=0; i<dice;i++){
            random = rand() % 6 + 1;
         }
        if( choice == random){
            std::cout << "Congratulations, you got it right! \n";
            std::cout << "Want to try again?(Yes/No) ";
            std::cin >> decision;
        } else{
            std::cout << "Sorry, the number was " << random << "... better luck next  time \n" ;
            std::cout << "Want to try again?(Yes/No) ";
            std::cin >> decision;
        }

    }
    std::cout << "Press ENTER to continue...";
    std::cin.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
    return 0;
}

This is what I am trying to figure out, why am I getting this warning: warning C4244: 'argument' : conversion from 'time_t' to 'unsigned int', possible loss of data

like image 771
Gal Appelbaum Avatar asked Feb 12 '12 04:02

Gal Appelbaum


3 Answers

That's because on your system, time_t is a larger integer type than unsigned int.

  • time() returns a time_t which is probably a 64-bit integer.
  • srand() wants an unsigned int which is probably a 32-bit integer.

Hence you get the warning. You can silence it with a cast:

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

In this case, the downcast (and potential data loss) doesn't matter since you're only using it to seed the RNG.

like image 196
Mysticial Avatar answered Oct 22 '22 07:10

Mysticial


This line involves an implicit cast from time_t which time returns to unsigned int which srand takes:

srand ( time(NULL) );

You can make it an explicit cast instead:

srand ( static_cast<unsigned int>(time(NULL)) );
like image 10
Pubby Avatar answered Oct 22 '22 09:10

Pubby


time() returns a time_t, which can be 32 or 64 bits. srand() takes an unsigned int, which is 32 bits. To be fair, you probably won't care since it's only being used as a seed for randomization.

like image 2
Ignacio Vazquez-Abrams Avatar answered Oct 22 '22 09:10

Ignacio Vazquez-Abrams