Почему я получаю предупреждение о возможной потере данных при заполнении генератора случайных чисел по времени (NULL)? - PullRequest
5 голосов
/ 03 октября 2011

я изучаю векторы и сделал немного кода, который выбирает случайные числа, которые я могу использовать для покупки лотерейных билетов здесь, в Нидерландах.Но хотя он работает, компилятор предупреждает меня о «преобразовании из time_t в unsigned int, возможной потере данных».

Может кто-нибудь определить, что вызывает это?Я даже не определил ни одного беззнакового целого в этом коде;int i по умолчанию является int со знаком, как я понимаю.Спасибо за понимание.

#include <iostream>
#include <vector>
#include <string>
#include <ctime>
using namespace std;

void print_numbers();
string print_color();

int main() {
srand(time(NULL));
print_numbers();
string color = print_color();
cout << color << endl;

system("PAUSE");
return 0;
}

//Fill vector with 6 random integers. 
//
void print_numbers() {
vector<int> lucky_num;

for (int i = 0; i < 6; i++) {
    lucky_num.push_back(1 + rand() % 45);
    cout << lucky_num.at(i) << endl;
}
}

//Select random color from array.
//
string print_color() {
string colors[6] = {"red", "orange", "yellow", "blue", "green", "purple"};
int i = rand()%6;
return colors[i];
}

Точное сообщение компилятора: предупреждение C4244: «аргумент»: преобразование из «time_t» в «unsigned int», возможная потеря данных.Строка 11.

Ответы [ 6 ]

7 голосов
/ 03 октября 2011

Поскольку размер time_t больше, чем unsigned int на вашей конкретной платформе, вы получаете такое предупреждение. Преобразование из «большего» в «меньший» тип включает в себя усечение и потерю данных, но в вашем конкретном случае это не имеет большого значения, потому что вы просто заполняете генератор случайных чисел, и в случае дата в очень далеком будущем.

Приведение его к unsigned int явно должно подавить предупреждение:

srand((unsigned int) time(NULL));
1 голос
/ 03 октября 2011

srand(time(NULL)); <- эта строка. time возвращает целое число типа time_t, и вы конвертируете это в unsigned int. time_t имеет тип unsigned [максимально возможное значение int]. </p>

0 голосов
/ 03 октября 2011
void srand ( unsigned int seed );
time_t time ( time_t * timer );
typedef long int __time_t;

long int - это не то же самое, что и unsigned int. Отсюда и предупреждение.

(из stackoverflow

0 голосов
/ 03 октября 2011
srand(time(NULL));

Эта строка может переполниться, если возвращаемое значение из time превышает диапазон представления unsigned int, что, безусловно, возможно.

0 голосов
/ 03 октября 2011

время возвращает объект time_t.

и ожидает целое число без знака.

0 голосов
/ 03 октября 2011

time_t - это 64-битное значение на многих платформах, чтобы предотвратить временное наложение времени эпохи, в то время как unsigned int составляет 32 бита.

В вашем случае вам все равно, потому что вы просто засеваетегенератор случайных чисел.Но в другом коде, если ваше программное обеспечение когда-либо работает с датами после 2038 , вы можете усечь time_t до 32-битной даты до 2038, когда вы приведете к 32-битному значению.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...