Heures, minutes, secondes... Résolu/Fermé

Question

Bonsoir, J'ai un petit problème en C. Je souhaite que mon programme renvoie, à partir des secondes entrées par l'utilisateur, l'équivalent en jours, heures, minutes, secondes. Tout fonctionne, sauf dans certains cas. Par exemple, lorsque j'entre 6849684141698168161 en secondes, j'obtiens -23001 jours, -12 heures, -52 minutes et -47 secondes Ou lorsque j'entre 100000000000000000000000000000000000000000 secondes, j'obtiens des 0 jours, 0 heures, 0 minutes, 0 secondes... Alors que par exemple avec 7445622 j'obtiens bien 8 jours, 14 heures, 49 minutes et 22 secondes. Le problème viendrait-il de la capacité de stockage de long int ? Voici le code : #include #include #include void clock(int* jours, int* heures, int* minutes, int* secondes); int main(int argc, char *argv[]) { long int jours = 0, heures = 0, minutes = 0, secondes; scanf("%d", &secondes); clock(&jours, &heures, &minutes, &secondes); printf("%d jours, %d heures, %d minutes et %d secondes",jours ,heures, minutes, secondes); return 0; } void clock(int* jours, int* heures, int* minutes, int* secondes) { *jours = (*secondes/3600)/24; *heures = *secondes / 3600; *minutes = *secondes / 60; *secondes = *secondes % 60; *minutes = *minutes % 60; *heures = *heures % 24; } Merci. <\EchoIsON>.

NHenry · Answer

long int jours = 0, heures = 0, minutes = 0, secondes;
void clock(int* jours, int* heures, int* minutes, int* secondes)

Ce ne serait pas un problème de type ?

fiddy · Answer

Bonjour,

Attention à la cohérence entre les types :
void clock(int* jours, int* heures, int* minutes, int* secondes);
Tu appelles cette fonction avec des long. Il est possible que cela ne change rien sur ta machine (int = long int sur certaines plateformes), mais ce n'est pas bon pour autant.

Sinon, les valeurs que tu as entrées sont trop grandes pour être contenues dans un long. Tu peux essayer avec long long (C90) si tu souhaites plus de place. Mais, ça restera limité...
Tu veux vraiment rentrer des valeurs aussi grandes ? Si oui, il va te falloir implémenter une bibliothèque type BIGNUM.

[Dal] · Answer

Salut EchoIsON,

Juste une remarque, qui ne répond pas à ta question mais qui concerne le nommage de ta fonction.

clock() est une fonction standard du C accessible par time.h et qui fait tout autre chose.

http://www.cplusplus.com/reference/ctime/clock/

Pour cette raison, je pense que tu devrais éviter d'appeler ta fonction "clock()".


Dal

Utilisateur anonyme · Answer

Merci à tous, [Dal] je prend en compte le conseil. En effet il y a une incohérence avec long int et int dans la fonction mais le problème était toujours présent même avant que je change int en long int. fiddy, est-ce-que je pourrais avoir plus de précisions sur la bibliothèque BIGNUM, svp ? Voici le code modifié : #include #include #include void heure(long int* siecles, long int* annees, long int* mois, long int* jours, long int* heures, long int* minutes, long int* secondes); int main(int argc, char *argv[]) { long int siecles = 0, annees = 0, mois = 0, jours = 0, heures = 0, minutes = 0, secondes; scanf("%d", &secondes); heure(&siecles, &annees, &mois, &jours, &heures, &minutes, &secondes); printf("%d siecles, %d ans, %d mois, %d jours, %d heures, %d minutes et %d secondes", siecles, annees, mois, jours ,heures, minutes, secondes); return 0; } void heure(long int* siecles, long int* annees, long int* mois, long int* jours, long int* heures, long int* minutes, long int* secondes) { *siecles = ((((*secondes/3600)/24)/30)/12)/100; *annees = (((*secondes/3600)/24)/30)/12; *mois = ((*secondes/3600)/24)/30; *jours = (*secondes/3600)/24; *heures = *secondes / 3600; *minutes = *secondes / 60; *secondes = *secondes % 60; *minutes = *minutes % 60; *heures = *heures % 24; *jours = *jours % 30; *mois = *mois % 12; *annees = *annees % 100; } /* J'ai rajouté certaines modifications entre temps...*/ EDIT : J'ai rajouté unsigned lont int, je pense que cela m'offre une solution partielle... Au moins je n'ai plus de valeurs négatives. Merci, EchoIsON.

Heures, minutes, secondes...

4 réponses

Discussions similaires