petit exercice - améliorable ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : petit exercice - améliorable ?

ritzle

Bonjour à tous.

Je dois faire un petit exercice :
Écrire un programme qui calcule la durée d'un trajet connaissant l'heure de départ et d'arrivée. On se contente
des heures et des minutes, la durée totale ne dépassera jamais 24 heures.

voici mon code :

Code :

#include <stdio.h>
int main( void )
{
struct {
unsigned int h, m, mins;
} depart, arrivee, duree;
printf( "\n==Calcul du temps de trajet==\n\n" );
do {
printf( "heure de depart (HH-MM) : " );
scanf( "%d-%d", &depart.h, &depart.m );
} while( depart.h>23 || depart.m>59 );
do {
printf( "\nheure d'arrivee (HH-MM) : " );
scanf( "%d-%d", &arrivee.h, &arrivee.m );
} while( arrivee.h>23 || arrivee.m>59 );
depart.mins = 60*depart.h + depart.m;
arrivee.mins = 60*arrivee.h + arrivee.m;
if( depart.mins <= arrivee.mins )
{
//depart <= arrivee
duree.mins = arrivee.mins - depart.mins;
}
else
{
//depart > arrivee
duree.mins = 24*60 - (depart.mins - arrivee.mins);
}
duree.h = duree.mins / 60;
duree.m = duree.mins % 60;
printf( "\n\nDuree de trajet : %d h %d mn\n", duree.h, duree.m );
return 0;
}

Ca fonctionne sans problème, mais je voudrais simplement savoir s'il y a des points à améliorer ou à optimiser.

Merci

Message cité 2 fois

Publicité

skelter

tu peux déjà améliorer la saisie, c'est pénible pour un utilisateur de voir le programme partir en boucle infinie à cause d'une mauvaise saisie

http://mapage.noos.fr/emdel/notes.htm#saisie

c'est mieux d'utiliser puts ou fputs à la place de printf quand tu n'as pas de chaine formatée à afficher et penses aux fflush(stdout) entre l'affichage d'une chaine qui ne se termine pas par un '\n' et une saisie

http://mapage.noos.fr/emdel/notes.htm#fflush_stdout

Emmanuel Delahaye

C is a sharp tool

ritzle a écrit :

Ca fonctionne sans problème, mais je voudrais simplement savoir s'il y a des points à améliorer ou à optimiser.

Mes remarques préliminaires :

L'indentation est cohérente (quoique les do-while ne sont pas traités comme le if-else...), mais je la trouve horrible (c'est personnel, et tu n'y peux rien). De toutes façons, avec Code::Blocks, on a un indenteur intégré, alors tout va bien...
J'ai tapé 'a'<enter> au lieu d'un nombre et c'est parti en boucle. Problème de saisie... (scanf() mal utilisé, probablement...)
La structure monolithique du code ne permet pas de faire de test unitaire facilement car le code de calcul n'est pas isolé...

#include <stdio.h>

int main( void )
{

/* -ed-
struct
{
unsigned int h, m, mins;
}
depart, arrivee, duree;

j'evite la factorisation des donnees.
*/
struct heure
{
unsigned h;
unsigned m;
unsigned mins;
}
depart;
struct heure arrivee;
struct heure duree;

printf( "\n==Calcul du temps de trajet==\n\n" );

do
{
printf( "heure de depart (HH-MM) : " );
scanf( "%d-%d", &depart.h, &depart.m );
/* -ed- NON! */
}
while ( depart.h > 23 || depart.m > 59 );

Pas le temps de finir. J'essayerais demain soir...

En attendant, lit ça :

http://mapage.noos.fr/emdel/notes.htm#saisie

Message édité par Emmanuel Delahaye le 07-02-2006 à 21:52:38

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

Sve@r

ritzle a écrit :

Tout convertir en minutes est une méthode qui coûte du calcul. Tu peux essayer de soustraire directement les minutes et heures avec retenue

m=m_fin - m_deb
si (m < 0)
{
m=m + 60
h_fin = h_fin - 1
}

h=h_fin - h_deb
si (h < 0)
h=h + 24

Durée = concaténation {h} heures + {m} minutes

Tu évites les divisions et multiplications gourmandes en ressources...

Message cité 1 fois

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

ritzle

Ah ouais quand même :whistle:
Je vais améliorer ca en tenant compte des conseils de ton site, emmanuel

Sve@r a écrit :

Tout convertir en minutes est une méthode qui coûte du calcul. Tu peux essayer de soustraire directement les minutes et heures avec retenue

Tu évites les divisions et multiplications gourmandes en ressources...

Justement j'avais commencé comme ca mais je pensais que ca faisait plus de calculs...

FORUM HardWare.fr

Programmation

petit exercice - améliorable ?

Sujets relatifs
petit probleme javascript	[ORACLE] petit problème avec les procédures..
Petit pb simple à resoudre (je pense)	petit probleme d'animation
[Visual C++] Pourquoi ce que j'imprime est tout petit	Petit problème avec Psexec
petit sondage : options de compilation avec gcc	Petit soucis page web !!
Petit jeu Image :)	Petit problème avec vb6 et mysql
Plus de sujets relatifs à : petit exercice - améliorable ?

Page générée en 0.061 secondes