[SOLUCIONADO] Pasar de segundos a horas,minutos y segundos correspondientes.

[anggiebs]

Hola,el programa consiste en que el usuario introducirá un número de segundos y se mostrará a cuánto equivale ese número en horas,minutos y segundos.
He avanzado esto ,lo he probado con ciertos números como:
3601 segundos = 1h 0min 1 segundo.
7204 segundos = 2h 0min 4 segundos.
pero a la hora de probarlo con :
10806 debería de darme 3h 0min 6 segundos y me sale 2h 0min 6 segundos.

Creo que mi error es sencillo pero no lo veo :/ .Espero puedan ayudarme,gracias :) .

Código C++:

int s=0,horas=0,min=0;
int aux;
 
cout << "Introduce los segundos:" ;
cin >> s;
 
if(s>3600){
        aux = s-3600;   
        if(aux > 0){    
            horas = 1;
            if(aux % 60 == 0){  //Si es múltiplo de 60.
                min = aux/60;
                if(min >=60){
                    horas++;
                    min = 0;
                }
                s = 0;
            }
            else{   
                if(aux >60){
                    s = aux % 60; //El resto de dividir entre 60 será s.
                    aux = aux - s;
                    min= aux/60;
                    if(min >= 60){
                        horas++;
                        min=0;
                    }
                }
                else{
                    s=aux;
                }
            }
        }
    }
 

[dehm]

Creo que el error lo tienes en que sólo incrementas como mucho la variable "horas" en dos unidades.
En la línea 10:

Código C++:

horas = 1;
 

y en las líneas 14 ó 25,

Código C++:

horas++;
 

dependiendo de por donde discurra el programa
Pero en esas líneas, la variable "horas" sólo se incrementa en una ud si la variable min>=60.

Pero...qué pasa si la variable min>=120? Eso son 2 horas, y tú sólo incrementas el valor de horas en una ud, sólamente con la condición de que min sea >= a 60.

Por otro lado, la verdad es que en mi opinión el método seguido es un poco lioso. Seguro que lo puedes mejorar un poco

[patilanz]

Te complicas demasiado la vida con este codigo. Porque no haces algo como esto:

Código Javascript:

function a(s){
   var horas = Math.floor(s/3600);
   var minutos = Math.floor((s - horas*3600)/60);
   var secundos = (s - horas*3600 - minutos*60);
   return [horas,minutos,secundos];
}
 

[anggiebs]

Apenas pocas líneas de código...tengo que practicar mucho para hacerlo así.
Gracias a ambos por la ayuda :)!

[ecfisa]

Hola anggiebs.

Ese cáculo lo podes realizar con operaciones básicas sin necesidad de usar ciclos:

Código C++:

void segsToHMS(const unsigned int sgs, int& h, int& m, int& s) {
  h = sgs / 3600;
  m = sgs / 3600 % 60;
  s = sgs % 3600 % 60;
}
 

Saludos.

[Fw190]

Te corrijo un poco:

Código C:

void segsToHMS(const unsigned int sgs, int& h, int& m, int& s)
{
    h = sgs / 3600;
    m = (sgs % 3600) / 60;
    s = sgs % 60;
}
 

[ecfisa]

Tenes razón, al escribir invertí la operación en la segunda línea.

Debería haber sido:

Código C++:

void segsToHMS(const unsigned int sgs, int& h, int& m, int& s) {
  h = sgs / 3600;
  m = sgs % 3600 / 60;
  s = sgs % 3600 % 60;
}