ctime возвращает NULL случайным образом

Question

Я использую ctime. Однако он всегда возвращает NULL. Таким образом, он работает на линии sprintf. Он работал раньше. Итак, не уверен, почему он случайно возвращает NULL?

У меня есть следующий фрагмент кода:

int main() 
{ 
    char avp_val[50]; 

    uint32_t date_value=1477069401; 

    sprintf(avp_val,"%s",ctime((time_t*)(&date_value))); 

    return; 
} 

Answer 1

Не бросить указатель uint32_t к time_t. Используйте фактическую time_t, так и на системы с 64-битной time_tctime не читает четыре байта мусора, как часть времени эпохи:

int main() 
{ 
    char avp_val[50]; 

    time_t date_value=1477069401; 

    sprintf(avp_val,"%s",ctime(&date_value)); 
    // Or, because it's what you're doing anyway, skip sprintf: 
    // strcpy(avp_val, ctime(&date_value)); 

    return 0; 
} 

литейного (time_t*) заглушить предупреждения компилятора замолчать предупреждения, он не сделал решить проблему.

Answer 2

Ну, ваш код не имеет кучу включаемых файлов:

#include <stdlib.h> /* for the types you use below, like uint32_t */ 
#include <stdio.h> /* for a prototype for sprintf, which you use below */ 
#include <time.h> /* for a prototype for ctime() see NOTE 1 */ 

Вы также должны учитывать, что литье должно быть сделано со значениями данных, а не с указателями, как отливка указатель только минует тип компилятора проверяя систему и делает ваш код более подверженным ошибкам. В этом случае преобразование с uint32_t * в time_t * слишком опасно, если оба типа имеют разные размеры, так как вы будете сокращать значение, используя только половину необходимых бит. Это что-то знает как Undefined Behavior, и вы должны его избегать.

Вы также должны вернуть явное значение из основного. Это ошибка компиляции не делать этого, так что ваш пример не может выполнить, так как он не компилируется (и таким образом, вы не можете получить NULL от ctime(3))

Кстати, я попробовал ваш код с uint32_t date_value в Mac OS/X, и он напечатал правильное значение времени, поэтому, вероятно, time_t - это 32-битное значение, но вам лучше использовать time_t, поскольку он определяется по этой причине.

Примечание 1

Наиболее важным в данном случае является прототипом CTime(), которая возвращает указатель на char и без прототипа, компилятор предполагает, что это возвращает int. По крайней мере, на 64-битных платформах оба типа имеют разные размеры и не могут быть переданы таким образом.

Пожалуйста, всегда отправляйте пример Complete, minimum, verifiable, так как вы можете скрыть точный источник ошибки, отредактировав свой пример для показа.