Foro de elhacker.net

 #include <iostream>
 
using namespace std;
 
int main()
{
    int d,m,a;
    cout<<"Ingresa una fecha en formato DD-MM-AAAA"<<endl;
    cin>>d>>m>>a;
    cout<<"Ingresaste.."<<d<<"/"<<m<<"/"<<a<<endl;
    int D,M,A;
    cout<<"Ingresa una segunda fecha para comparar con la primera.."<<endl;
    cin>>D>>M>>A;
    cout<<"Ingresaste.."<<D<<"/"<<M<<"/"<<A<<endl;
    cout<<"Comparar?? 1=si y 2=no"<<endl;
    int t;
    cin>>t;
 
    Eleccion:
    switch(t) {
    case '1':
    goto calculo;
    break;
    case '2':
    goto rehacer;
    break;
    default:
    cout <<"Error,Reingresa los datos"<<endl;
    break;
    }
    calculo:
    {
            cout<<"Aki iria el algoritmo para la comparacion"<<endl;
                //El cual aun no se como hacer ^^^^^^^^^^^^^
    }
 
    rehacer:
    {
        cout<<"Ingresa una fecha en formato DD-MM-AAAA"<<endl;
    cin>>d>>m>>a;
    cout<<"Ingresaste.."<<d<<"/"<<m<<"/"<<a<<endl;
    cout<<"Ingresa una segunda fecha para comparar con la primera.."<<endl;
    cin>>D>>M>>A;
    cout<<"Ingresaste.."<<D<<"/"<<M<<"/"<<A<<endl;
    cout<<"Comparar?? 1=si y 2=no"<<endl;
    goto Eleccion;
    }
    cin.ignore();
    cin.get();
    return 0;
}
 

Estoy creando este programa que calcula el tiempo que paso entre 2 fechas.. necesito que me tiren alguna indirecta para el algoritmo del calculo.. y por favor,diganme si tengo algo mal en el codigo hasta ahora.. Creo que el switch lo tengo mal porque cada vez que pongo 1 por ejemplo,ejecuta el cout y luego vuelve al principio en lugar de terminar el programa.. gracias de antemano

No entiendo siquiera que es lo que quieres, si te explicas bien creo que seria mejor, verifica tambien que el codigo que tengas sea compilable... Evita los GoTo, no estamos en Basic. para eso existen las funciones, los bucles y las sentencias condicionales.

Sinceramente,estaba leyendo uno de los libros que tengo y fue la unica manera en la que se me ocurrio.. La idea es comprar dos fechas diferentes y calcular la cantidad de dias que pasaron entre fecha1 y fecha2.. El codigo compila sin errores ni warnings asi que supongo que esta bien..

1) el codigo tiene un error cuando se introducen las fechas... verificalo.
2) El Switch tiene un error, siempre sale default porque solicitas un int y luego lo comparas con un char sin realizar la conversion... Mira la Tabla ASCII.

Compara primero los años, buscales la diferencia, despues los meses y despues los dias, entonces toma en consideracion la longitud del año con respecto a los meses y si el año es o no es bisiesto.

Un consejo: Si el libro que estas leyendo te recomienda GoTo inmediatamente deja de leerlo... te aseguro que quien escribio el libro a suerte debe de saber basic.

No es que lo recomienda.. en un momento aparece un ejemplo con el goto.. Pero no dice nada al respecto.. es como una mencion de que existe.. ahora me fijo y lo corrijo.. Gracias por la ayuda RNA  :D

EDIT: Revisando el Codigo,hice algunas modificaciones y el codigo compila y funciona bien.. Ahora mi problema es como reemplazar los goto.. Deberia usar void x () siendo x una funcion hecha por mi?? Si es asi aun no estoy seguro de como hacerlo, y si no es asi no tengo ni idea.. ayuda?

Si, usar funciones seria lo ideal. Tal vez te sirva revisar este capitulo de conclase (http://c.conclase.net/curso/?cap=003).

Saludos

 #include <iostream>
 
using namespace std;
 
int d,m,a,D,M,A;
    int calculo();
    int datos();
    void eleccion(int t);
 
int main()
{
    int datos();
    {
        cout<<"Ingresa una fecha en formato DD-MM-AAAA"<<endl;
    cin>>d>>m>>a;
    cout<<"Ingresaste.."<<d<<"/"<<m<<"/"<<a<<endl;
 
    cout<<"Ingresa una segunda fecha para comparar con la primera.."<<endl;
    cin>>D>>M>>A;
    cout<<"Ingresaste.."<<D<<"/"<<M<<"/"<<A<<endl;
    }
 
    int eleccion(int t);
    {
     cout<<"Comparar?? 1=si, 2=no y 3=salir"<<endl;
       int t;
    cin>>t;
 
        switch(t)
        {
        case 1:
        break;
 
        case 2:
        break;
 
        case 3:
        cout <<"Algun mensaje de salida.."<<endl;
        cin.get();
        return 0;
 
        default:
        cout <<"Error,Reingresa los datos"<<endl;
        break;
 
        }
    }
 
 
}

No se si voy bien pero por ahora tengo esto.. Alguien me podria decir si de lo de las funciones esta bien? si lo estan hago rapido el algoritmo para el calculo,relleno las funciones que me faltan y lo compilo a ver si funciona..

Hola, la idea es esa pero no lo haces bien. Donde tienes definidas las funciones escribes su código y luego dentro del main solo tienes que llamar a la función:

 #include <iostream>
 
using namespace std;
 
void datos(int &d, int &m, int &a, int &D, int &M, int &A) //Los parametros son de salida (por referencia)
    {
    //Aqui el code, que en tu caso seria:
    cout<<"Ingresa una fecha en formato DD-MM-AAAA"<<endl;
    cin>>d>>m>>a;
    cout<<"Ingresaste.."<<d<<"/"<<m<<"/"<<a<<endl;
 
    cout<<"Ingresa una segunda fecha para comparar con la primera.."<<endl;
    cin>>D>>M>>A;
    cout<<"Ingresaste.."<<D<<"/"<<M<<"/"<<A<<endl;
    }
 
 
void calculo(int d, int m, int a, int D, int M, int A)
    {
    //Esta funcion calcula el tiempo que ha pasado entre las 2 fechas y lo muestra
    //Aqui el code
    }
 
 
int main()
{
    int d, m, a, D, M, A, t;
 
    //Esto yo lo haria asi
    do{
        datos(d, m, a, D, M, A); //Aqui ya se almecenan en las variables los valores
        cout<<"Comparar?? 1=si, 2=no y 3=salir"<<endl;
        cin>>t;
 
        switch(t)
            {
            case 1: //Llamas a la funcion calcular y listo
            calcular(d, m, a, D, M, A);
            break;
 
            case 2: //Por ejemplo pides que se vuelvan a introducir los datos
            cout<<"Vuelve a introducir los datos...."<<endl;
            break;
 
            case 3: 
            cout <<"Algun mensaje de salida.."<<endl;
            cin.get();
            return 0;
 
            default:
            cout <<"Error"<<endl;
            break;
            }
    }while(t!=3);
 
return 0;
}

Che, pero por que primero no intentas hacer el codigo que compare dos fechas?

btw: Tenes que validar que las 2 fechas esten bien o se supone que van a ingresar fechas correctas?

Hola

Eso lo tiene que hacer él, es su tarea.

Y claro que puede comprobar que las fechas estén bien, pero eso ya depende de él. De esta forma tienes que confiar en que la fecha introducida sea válida, pero así está bien, eso ya depende de cómo quiera hacerlo, mientras más comprobaciones "mejor" (nótense las comillas) pero si no se lo piden....

Saludos

Eso de la comprobacion de la voy a agregar mas adelante.. primero quiero lo basico y se lo mando a mi amiga que me lo pidio.. y ahi le pregunto si quiere esa comprobacion,si me dice que no,el programa esta bien,si las kiere,solamente tengo que agregarlas y listo! Gracias por la ayuda :)

jaja a wazzap le decia que lo haga
Porque hizo toda una estructura todo ordenadito, pero primero que nada yo haria el programa funcional, y despues le doy mejor formato

Intenta hacer el codigo que compare las 2 fechas y decinos que tal te fue

Ok.. les voy a ser 100% Sincero.. No tengo NI IDEA de que hice XD Solo se que compila y funciona bien.. pero no creo que el algoritmo para las fechas este bien.. Alguien corrige? porque ya lo revise como 20 veces y no se cual podria ser un error..

 #include <iostream>
 
using namespace std;
    void datos(int &d,int &m,int &a,int &D,int &M,int &A)
    {
    cout<<"Ingresa una fecha en formato DD-MM-AAAA"<<endl;
    cin>>d>>m>>a;
    cout<<"Ingresaste.."<<d<<"/"<<m<<"/"<<a<<endl;
 
    cout<<"Ingresa una segunda fecha para comparar con la primera.."<<endl;
    cin>>D>>M>>A;
    cout<<"Ingresaste.."<<D<<"/"<<M<<"/"<<A<<endl;
    }
    void calculo(int d,int m,int a,int D,int M,int A,int m2,int a2,int r)
     {
        if (a>=A)
        {
            a2=a-A;
        }
 
      if (a<=A)
    {
        a2=A-a;
    }
    if (m>=M)
    {
        m2=M-m;
    }
        else if (m<=M)
        {
           m2=M-m;
        }
    r=(a2*365)+(m2*30)+(D-d);
    cout<<"El resultado seria: "<<r<<"dias.."<<endl;
    }
 
int main()
{
   int d,m,a,D,M,A;
   int a2=0,m2=0,r=0;
   datos(d,m,a,D,M,A);
   int t;
   cout<< "Comparar..? 1=si 2=no-quiero cambiar las fechas 3=salir"
   <<endl;
   cin>>t;
   do
     {
       switch(t)
       {
            case 1:
            calculo(d,m,a,D,M,A,a2,m2,r);
            return 0;
            break;
            case 2:
            cout<<"Revisa las fechas.."<<endl;
            datos(d,m,a,D,M,A);
            calculo(d,m,a,D,M,A,a2,m2,r);
            return 0;
            break;
            default:
            return 0;
            break;
       }
    }while(t!=3);
 
}

Ahh y disculpa satu pero use tu idea.. Una pregunta,es necesario ubicar los & dentro del void()??

Pero los meses no tienen todo 30 dias

Vos lo que queres hacer es algo simplesito que calcule aproximadamente, o tiene que ser exacto?
Porque tampoco estas teniendo en consideracion años bisiestos

Mira, yo empece hace poquito a programar, y lo que haria es lo siguiente:
dividi el calculo en 3 etapas
Primero calcular cuantos dias faltan hasta el fin de año de la primer fecha
despues culcular cuantos años hay entre las 2 fechas
y por ultimo cuantos dias pasaron del ultimo año

Esto te funcionaria solo para fechas en las que los años no son el mismo
pero el otro caso lo haces a parte y listo

No se si estoy ayudando demasiado, que alguien me diga si es el caso, soy nuevo en el foro

Aclaro por las dudas: El programa es para una amiga,en ningun caso debe ser 100% Exacto. Ya le pregunte el tema de la exactitud del programa y me dijo que no hacia falta.. Si en algun momento me pide que lo haga mas exacto usaria un par de Ifs mas para los meses que tengan 30/31/28/29 dias y los años bisiestos

Igual,Ayudes demasiado o no,quiero darte las gracias :) Todo esto me ayuda mas que a realizar el ejercicio(Que sinceramente no tiene tanta importancia) Me ayuda a pensar como programador,y creo que eso no se puede aprender de ningun lado, Solo tu interes y tus ganas de ser un programador pueden servir :) -Solo mi opinion-

Saludos del Wazzp  :rolleyes:

bueno, si no tiene que ser exacto esta bien


Te diria que intentes volver a hacerlo siguiendo los tips de las 3 etapas
Y si no va a ser tan exacto, tampoco te gastes en fijarte cual es la fecha anterior a la otra
Que en la primer variable se ingrese la fecha mas antigua y en la segunda la mas actual

Hmmm... Wow.. Se me acaba de ocurrir una muy buena idea o.o!!! Tu comment fue disparador! Graciaas!! Cuando termine el code lo posteo por si a alguien le interesa usarlo :) (Voy a ponerle lo de los años bisiestos y los meses antes de postearlo asi es 99.99% Exacto)

EDIT:

 #include <iostream>
 
using namespace std;
    void datos(int &d,int &mcont,int &a,int &D,int &Mcont,int &A,int &t)
    {
    cout<<"Ingresa una fecha(DD/MM/AAAA. Debe ser la mas antigua!)"<<endl;
    cin>>d>>mcont>>a;
    cout<<"Ingresaste.."<<d<<"/"<<mcont<<"/"<<a<<endl;
 
    cout<<"Ingresa una segunda fecha. Debe ser la mas reciente."<<endl;
    cin>>D>>Mcont>>A;
    cout<<"Ingresaste.."<<D<<"/"<<Mcont<<"/"<<A<<endl;
    cout<< "Comparar..? 1=si 2=no-quiero cambiar las fechas 3=salir"
   <<endl;
   cin>>t;
    }
 
    void ABis(int a,int A,int m,int M)
    {
        if ( ( ( a % 4 == 0 ) && (a % 100 != 0 ) ) || ( (a % 400 ==0)))
 
            m=1;
             else
                m=2;
 
       if ( ( ( A % 4 == 0 ) &&( A % 100 != 0 ) ) || ( (A % 400 ==0) ) )
 
           M=1;
            else
                M=2;
    }
 
    void meses(int m,int mb,int Mb,int M,int mcont,int Mcont)
    {
        if ((mcont==1)||(mcont==3)||(mcont==5)||(mcont==7)||(mcont==8)||(mcont==10)||(mcont==12))
        {
             mb=31;
        }
 
            else if ((mcont==4)||(mcont==6)||(mcont==9)||(mcont==11))
            {
                mb=30;
 
        };
    if ((Mcont==1)||(Mcont==3)||(Mcont==5)||(Mcont==7)||(Mcont==8)||(Mcont==10)||(Mcont==12))
        {
            Mb=31;
        }
        else if ((Mcont==4)||(Mcont==6)||(Mcont==9)||(Mcont==11))
        {
            Mb=30;
        };
    if ((m==1)&&(mcont==2))
    {
        mb=29;
        }
        else if((m==2)&&(mcont==2))
        {
            mb=28;
        }
 
    }
 
    void calculo(int d,int m,int a,int X,int mcont,int mb,int Mb,int mb1,int A,int M,int Mcont,int R,int Y)
    //Calcularia la cantidad de dias hasta el fin del año
    {
       {
         while (mcont<=12)
         ABis(a,A,m,M);
       meses(m,mb,Mb,M,mcont,Mcont);
     X+=mb;
     mcont++;
     };
     {
          while (Mcont>=1)
          ABis(a,A,m,M);
       meses(m,mb,Mb,M,mcont,Mcont);
     Y+=Mb;
     Mcont--;
     };
     R=(X-Y);
     cout<<"La respuesta seria: "<<R<<" Dias."<<endl;
     }
 
   int main()
{
   int d,a,D,A,t,m,M;
   int mb=0;int Mb=0;int mcont=0;
   int X=0;int Mcont=0;
   int Y=0;int mb1=0;
   int R=0;
   datos(d,mcont,a,D,Mcont,A,t);
 
   do
     {
       switch(t)
       {
            case 1:
            calculo(d,m,a,X,mcont,mb,Mb,mb1,A,M,Mcont,R,Y);
            break;
 
            case 2:
            cout<<"Revisa las fechas.."<<endl;
            calculo(d,m,a,X,mcont,mb,Mb,mb1,A,M,Mcont,R,Y);
            return 0;
            break;
 
            default:
            return 0;
            break;
       }
    }while(t!=3);
return 0;
}
 

Por que se cuelga en lugar de empezar cuando pongo introduzco Comparar en switch?