Hora del sistema

[baker1562]

Hola, necesito ayuda en el siguiente programa, bueno hice un programa con un menu de 2 opciones, la opcion 1 entra ve la hora del sistema y la guarda en un arreglo, y la opcion 2 muestra las horas guardadas, mi problema es que siempre se guarda la misma hora, y no se porque, lo que quiero es que guarde las diferentes horas que le entra y no se cual es mi error, apreciaria que alguien me echara una mano

Código C++:

#include <iostream>
#include <ctime>
#include <iomanip>
 
using namespace std;
 
int main()
{
    const char *Fecha[3];
    int opcion;
    int i=0;
    int controlador;
    string respuesta;
    
    while(respuesta!="N" && respuesta!="n")
    {
    
    cout<<"Presione 1 para entrar!!!"<<endl;
    cout<<"Presione 2 para mostrar!!!"<<endl;
    cin>>opcion;
    
    time_t tAct=time(NULL);
    
    system("cls");
    
    switch(opcion)
    {
    
    case 1:
         
         
         cout<<"Hora del sistema:"<<endl;
         cout<<asctime(localtime(&tAct));
         Fecha[i]=asctime(localtime(&tAct));
         controlador=i;
         i++;
         cout<<endl;
         
         system("pause");
         
         
         
    break;
    
    case 2:
         
         for(int i=0;i<(controlador+1);i++)
         {
                 cout<<"Fechas"<<setw(10)<<Fecha[i]<<endl;
                 cout<<endl;
                 
                 
                 }
                 
         system("pause");
         
    break;
}
    cout<<"Desea realizar una nueva accion: s/n?"<<endl;
    cin>>respuesta;
    
    system("cls");
    
}
    return 0;
    
}
 

[sam90]

Te recomiendo guardar la varaible time_t y no la cadena de caractes. Si guardas el time_t a la hora de mostrar los datos puede cambiar el formato rapidamentes. De la otra forma no.

Ademas no se como trabaja las funciones esas del tiempo. Pero imagino que el string que te estan devolviendo es statico. Eso quiere decir que el puntero que devuelven siempre apunta al mismo lado. Asi que el segundo llamado pisa al primero.
Aviso esta es una suposicion mia. Saludos

Código C++:

#include <iostream>
#include <ctime>
#include <iomanip>
 
using namespace std;
 
int main()
{
    const char *Fecha[3];
    time_t fechas[3];
    int opcion;
    int i=0;
    int controlador;
    string respuesta;
    
    while(respuesta!="N" && respuesta!="n")
    {
    
    cout<<"Presione 1 para entrar!!!"<<endl;
    cout<<"Presione 2 para mostrar!!!"<<endl;
    cin>>opcion;
    
    time_t tAct=time(NULL);
    
 
    
    switch(opcion)
    {
    
    case 1:
         
         
         cout<<"Hora del sistema:"<<endl;
         cout<<asctime(localtime(&tAct));
         fechas[i] = tAct;
         controlador=i;
         i++;
         cout<<endl;
         
 
         
         
         
    break;
    
    case 2:
         
         for(int i=0;i<(controlador+1);i++)
         {
                 cout<<"Fechas"<<setw(10)<<asctime(localtime(&fechas[i]))<<endl;
                 cout<<endl;
                 
                 
                 }
                 
 
         
    break;
}
    cout<<"Desea realizar una nueva accion: s/n?"<<endl;
    cin>>respuesta;
    
 
    
}
    return 0;
    
}
 

Hice la prueba de imprimir el puntero y efectivamente en dos llamados distintos el valor del puntero devuelto es el mismo asi que el segundo llamado te esta pisando al primero.

Si quieres manejar igualmente con cadenas deberias copiarla a un string o un array de caracteres.

[baker1562]

Iniciado por sam90

Te recomiendo guardar la varaible time_t y no la cadena de caractes. Si guardas el time_t a la hora de mostrar los datos puede cambiar el formato rapidamentes. De la otra forma no.

Ademas no se como trabaja las funciones esas del tiempo. Pero imagino que el string que te estan devolviendo es statico. Eso quiere decir que el puntero que devuelven siempre apunta al mismo lado. Asi que el segundo llamado pisa al primero.
Aviso esta es una suposicion mia. Saludos

Código C++:

#include <iostream>
#include <ctime>
#include <iomanip>
 
using namespace std;
 
int main()
{
    const char *Fecha[3];
    time_t fechas[3];
    int opcion;
    int i=0;
    int controlador;
    string respuesta;
    
    while(respuesta!="N" && respuesta!="n")
    {
    
    cout<<"Presione 1 para entrar!!!"<<endl;
    cout<<"Presione 2 para mostrar!!!"<<endl;
    cin>>opcion;
    
    time_t tAct=time(NULL);
    
 
    
    switch(opcion)
    {
    
    case 1:
         
         
         cout<<"Hora del sistema:"<<endl;
         cout<<asctime(localtime(&tAct));
         fechas[i] = tAct;
         controlador=i;
         i++;
         cout<<endl;
         
 
         
         
         
    break;
    
    case 2:
         
         for(int i=0;i<(controlador+1);i++)
         {
                 cout<<"Fechas"<<setw(10)<<asctime(localtime(&fechas[i]))<<endl;
                 cout<<endl;
                 
                 
                 }
                 
 
         
    break;
}
    cout<<"Desea realizar una nueva accion: s/n?"<<endl;
    cin>>respuesta;
    
 
    
}
    return 0;
    
}
 

Hice la prueba de imprimir el puntero y efectivamente en dos llamados distintos el valor del puntero devuelto es el mismo asi que el segundo llamado te esta pisando al primero.

Si quieres manejar igualmente con cadenas deberias copiarla a un string o un array de caracteres.

grax me sirvio eso era lo ue queria aunque me quedaron dudas de porque se guarda tAct y no asctime(localtime(&tAct))

[sam90]

Puedes guardar asctime(localtime(&tAct)) pero tienes que copiar su contenido a otro lado porque en el proximo llamado se modifica la memoria donde estaba el resultado previo.

Tienes ideas sobre punteros?? Esta funcion se reserva un pedazo de memoria para ella. Cuando la llamas escribe en esa memoria y te devuelve la direccion de la memoria. Cuando vuelves a llamar a la misma funcion escribe sobre este mismo lugar. Asi que cuando buscas la direccion que tenias antes fue modificado por el ultimo llamado.

[Trovaz]

Exactamente. sam90 tiene la boca llena de razon hehe. Básicamente lo que te estas copiando es la dirección de memoria. En lugar de utilizar una asignación directa, utiliza un strcopy o algo opr el estílo. Además te falta realizar la reserva de memoria, ya que solo estas reservando el espacio de memoria para almacenar los punteros, pero te falta reservar memoria para los datos.

Salu2, espero te sea de ayuda.

[baker1562]

Iniciado por Trovaz

Exactamente. sam90 tiene la boca llena de razon hehe. Básicamente lo que te estas copiando es la dirección de memoria. En lugar de utilizar una asignación directa, utiliza un strcopy o algo opr el estílo. Además te falta realizar la reserva de memoria, ya que solo estas reservando el espacio de memoria para almacenar los punteros, pero te falta reservar memoria para los datos.

Salu2, espero te sea de ayuda.

grax, si solo estaba el puntero no la variable a la cual apuntaba