[SOLUCIONADO] Problemas funciones obtención hora y fecha local

[drzaius]

Hola a todos,

Estoy teniendo problemas con las funciones time y localtime para obtener la hora local. En el siguiente ejemplo que muestro intento obtener dos fechas diferentes con un delay de 5 seg entre ellas, usando variables independientes.
Antes de obtener la segunda fecha imprimo también en pantalla la primera para la comprobación.

Código C:

#include <stdio.h>
#include <time.h>
#include <windows.h>
 
#define TFECHA 30
 
int main ()
  {
  time_t t1,t2;
  struct tm *tinfo1, *tinfo2;
  char f1[TFECHA],f2[TFECHA];
 
  time(&t1);
  tinfo1=localtime(&t1);
  strftime(f1, TFECHA, "%d/%m/%Y %H:%M:%S", tinfo1);
  printf("%s\n", f1);
 
  Sleep(5000);
 
  time(&t2);
  tinfo2=localtime(&t2);
 
  strftime(f1, TFECHA, "%d/%m/%Y %H:%M:%S", tinfo1);
  strftime(f2, TFECHA, "%d/%m/%Y %H:%M:%S", tinfo2);
  printf("%s\n%s", f1,f2);
 
  return 0;
  }
 

Sin embargo al imprimir al final las dos últimas fechas, los valores que se obtienen son exactamente iguales. No entiendo por qué al ejecutar tinfo2=localtime(&t2) (linea 21) la estructura t1 también se ve alterada...

La salida que obtengo es:

Código:

25/12/2013 23:54:14
25/12/2013 23:54:19
25/12/2013 23:54:19

Si alguien pudiera aclarármelo se lo agradecería.

Saludos.

[vangodp]

no son *tinfo1, *tinfo2; punteros a una misma estructura: "tm"?
Me da la impresión que imprimes lo mismo pero con diferentes punteros XD
En todo caso seria
struct tm1 *tinfo1;
struct tm2 *tinfo2;

envés de: struct tm *tinfo1, *tinfo2;

Usas la misma estructura para guardar ambas fechas?

Digo que es lo que me parece, no estoy seguro XDD

[vosk]

@vangodp: 'struct tm' es una estructura estandar definida en time.h, pero tienes razon: usa dos punteros sobre una misma cosa.

El problema está en que la funcion 'localtime' escribe sobre una unica estructura interna a la libreria, pero solo te retorna un puntero a esa estructura. Para que se vea mejor puedes cambiar los nombres de las variables:

Código C:

time_t t1,t2;
struct tm *puntero1;
struct tm *puntero2;
 
time(&t1);
puntero1 = localtime(&t1);//guardas en una estructura interna unica y recibes el puntero
 
time(&t2);
puntero2 = localtime(&t2);//sobreescribes la estructura interna y recibes el puntero
 

Ambos punteros apuntan a una misma estructura, cuya direccion es constante para esta instancia de la aplicacion, eso significa que puntero1 apunta al mismo sitio que puntero2, o traducido a tu codigo significa que tinfo1 es lo mismo que tinfo2 despues de que ejecutes el segundo localtime.

Supongo que quieres calcular el tiempo de ejecucion. Puedes solucionarlo de dos formas: o bien creas un struct tm y copias literalmente el retorno de localtime, o usas los timestamps. Normalmente se usan los timestamps. Un ejemplo con las estructuras:

Código C:

time_t t1,t2;
struct tm *ptr1, *ptr2;
struct tm tinfo1, tinfo2;
char f1[TFECHA],f2[TFECHA];
 
time(&t1);
ptr1 = localtime(&t1);
memcpy(&tinfo1, ptr1, sizeof(struct tm));
 
strftime(f1, TFECHA, "%d/%m/%Y %H:%M:%S", &tinfo1);
printf("%s\n", f1);
 
Sleep(5000);
 
time(&t2);
ptr2 = localtime(&t2);
memcpy(&tinfo2, ptr2, sizeof(struct tm));
 
strftime(f1, TFECHA, "%d/%m/%Y %H:%M:%S", &tinfo1);
strftime(f2, TFECHA, "%d/%m/%Y %H:%M:%S", &tinfo2);
printf("%s\n%s", f1,f2);
 

Y ahora uno con los timestamps:

Código C:

clock_t t1, t2;
 
printf("Entre ahora... ");
t1 = clock();
 
Sleep(5000);
printf("y ahora han pasado ");
 
t2 = clock() - t1;
printf("%f segundos\n", ((float)t2) / CLOCKS_PER_SEC);
 

Una ultima cosa: cuando usas una libreria donde alguna funcion te retorna un puntero tienes que prever dos cosas. La primera, que puede haber en esa libreria una funcion para liberar la posible memoria bloqueda para el puntero. La segunda, que si no hay funciones de liberacion de memoria ni en la referencia de las funciones te indica que debes liberar la memoria entonces significa que ese puntero está apuntando a un sitio unico (o eso o la libreria está mal documentada, que tambien puede ser). En el caso de localtime te retorna un puntero pero no te dice que al finalizar su uso debes liberar la memoria: eso indica que te retorna un puntero a una direccion de memoria unica para la instancia de tu aplicacion, es decir que cada vez que llames a localtime se sobreescribirá esa direccion.

Este tipo de funciones se dice que no son thread-safe, es decir que no esperes que funcione cuando los uses en una aplicacion en multiprocesos sin semaforos o zonas criticas (ya se que el codigo que colgaste no es multithread, ahí funciona pero funciona mal). Te gustará saber que tambien la funcion strtok es de ese tipo.

Saludos
vosk

[vangodp]

Bien..pues ya sabemos algo mas :D
Saludos vosk.

[drzaius]

Aunque haya pasado algo de tiempo, muchas gracias a ambos, especialmente a vosk por tu genial explicación.
Saludos