Fonction/pointeur/langage c
Résolu/Fermé
steflemaister
Messages postés
19
Date d'inscription
lundi 11 août 2008
Statut
Membre
Dernière intervention
27 décembre 2008
-
30 août 2008 à 12:54
java4ever - 30 août 2008 à 16:00
java4ever - 30 août 2008 à 16:00
A voir également:
- Fonction/pointeur/langage c
- Fonction si et - Guide
- Langage ascii - Guide
- Pointeur souris - Guide
- Fonction somme excel - Guide
- Fonction moyenne excel - Guide
5 réponses
steflemaister
Messages postés
19
Date d'inscription
lundi 11 août 2008
Statut
Membre
Dernière intervention
27 décembre 2008
30 août 2008 à 12:59
30 août 2008 à 12:59
dans le main, juste après le scanf, je mettrai bien un while(nombre<=0);
afin qu il me redemande chaque fois si j introduit un nombre négatif...
Merci de me lire
afin qu il me redemande chaque fois si j introduit un nombre négatif...
Merci de me lire
Mahmah
Messages postés
496
Date d'inscription
lundi 17 septembre 2007
Statut
Membre
Dernière intervention
22 juin 2010
125
30 août 2008 à 13:21
30 août 2008 à 13:21
Bonjour,
La déclaration courte de la fonction h_m_s doit être terminée par un ;
Les types et les paramètres dans les déclarations ou définitions de fonctions doivent être séparés par des virgules et non des point-virgules.
Je suis d'accord sur le principe du post précédent, bien-que je préférerais dans ce cas saisir un nombre positif et vérifier que la saisie a été correcte :
Certains compilateurs sont pénible avec ce point mais la fonction main devrait avoir un type de retour spécifié. De préférence un int. A partir de là, le main peut renvoyer un code d'erreur. Petite convention, le succès est exprimé par la valeur 0 et tous les autres nombres servent à préciser une erreur.
Dans le scanf, la variable nombre n'existe pas.
Dans le printf il conviendrait de mettre un 's' à "seconde". Remarque similaire dans l'autre printf... ^^"
Un dernier point qui est correct mais peut être fait "mieux". On chipotte là !
Dans h_m_s, la variable locale tmp est inutile, le paramètre nb est déjà une copie de la valeur passée en paramètre. On peut donc changer directement sa valeur au cours de notre fonction sans danger et sans passer par une variable supplémentaire.
Les return ne permettent que de renvoyer qu'une seule valeur, ta fonction h_m_s est donc toute indiquée pour donner plusieurs valeurs à la fois. Les alternatives sont de faire trois fonctions h, m et s (avec de meilleurs nom si possible) ou de faire une structure si tu as déjà vu. Une structure permet de grouper plusieurs valeurs dans une seule entité. on fait ensuite : mesValeurs.h, mesValeurs.m et mesValeurs.s. Le passage par pointeur reste le meilleur à mon goût.
Le type void est donc bien selon moi.
M.
void h_m_s(int nb;int *heure;int *min;int *sec )
La déclaration courte de la fonction h_m_s doit être terminée par un ;
Les types et les paramètres dans les déclarations ou définitions de fonctions doivent être séparés par des virgules et non des point-virgules.
Je suis d'accord sur le principe du post précédent, bien-que je préférerais dans ce cas saisir un nombre positif et vérifier que la saisie a été correcte :
unsigned int nb;
while ( scanf( "%u", &nb ) != 1 )
{
printf( .......
}
Certains compilateurs sont pénible avec ce point mais la fonction main devrait avoir un type de retour spécifié. De préférence un int. A partir de là, le main peut renvoyer un code d'erreur. Petite convention, le succès est exprimé par la valeur 0 et tous les autres nombres servent à préciser une erreur.
Dans le scanf, la variable nombre n'existe pas.
Dans le printf il conviendrait de mettre un 's' à "seconde". Remarque similaire dans l'autre printf... ^^"
Un dernier point qui est correct mais peut être fait "mieux". On chipotte là !
Dans h_m_s, la variable locale tmp est inutile, le paramètre nb est déjà une copie de la valeur passée en paramètre. On peut donc changer directement sa valeur au cours de notre fonction sans danger et sans passer par une variable supplémentaire.
Les return ne permettent que de renvoyer qu'une seule valeur, ta fonction h_m_s est donc toute indiquée pour donner plusieurs valeurs à la fois. Les alternatives sont de faire trois fonctions h, m et s (avec de meilleurs nom si possible) ou de faire une structure si tu as déjà vu. Une structure permet de grouper plusieurs valeurs dans une seule entité. on fait ensuite : mesValeurs.h, mesValeurs.m et mesValeurs.s. Le passage par pointeur reste le meilleur à mon goût.
Le type void est donc bien selon moi.
M.
steflemaister
Messages postés
19
Date d'inscription
lundi 11 août 2008
Statut
Membre
Dernière intervention
27 décembre 2008
30 août 2008 à 13:47
30 août 2008 à 13:47
un grand merci,
Donc je résume mes erreurs : -oubli du point virgule après ma première déclaration
- dans cette même déclaration, virgule et non point virgule (faute d inatention sur ce coup la ;)
-ok pour le tmp, je vois bien ce que tu veux dire
-pour etre sur, mettre en return(0) avant la fin de main? ainsi si j 'ai bien compris, si mon programme est correct, il ira jusqu au return 0 donc succès! On travaille en c++!
Sinon avec ce prof on a pas vu les structures! J ai bien vu les arbres, fiches, structures avec un autre prof, mais rien avoir avec cet examen la!
Un tout grand merci de m 'avoir lu!
ps: tu seras encore là tantot, j aurai surment besoin de ton aide :) Sur ce je retourne potasser, bonne journée
Donc je résume mes erreurs : -oubli du point virgule après ma première déclaration
- dans cette même déclaration, virgule et non point virgule (faute d inatention sur ce coup la ;)
-ok pour le tmp, je vois bien ce que tu veux dire
-pour etre sur, mettre en return(0) avant la fin de main? ainsi si j 'ai bien compris, si mon programme est correct, il ira jusqu au return 0 donc succès! On travaille en c++!
Sinon avec ce prof on a pas vu les structures! J ai bien vu les arbres, fiches, structures avec un autre prof, mais rien avoir avec cet examen la!
Un tout grand merci de m 'avoir lu!
ps: tu seras encore là tantot, j aurai surment besoin de ton aide :) Sur ce je retourne potasser, bonne journée
steflemaister
Messages postés
19
Date d'inscription
lundi 11 août 2008
Statut
Membre
Dernière intervention
27 décembre 2008
30 août 2008 à 13:53
30 août 2008 à 13:53
en fait, je dois mettre ds mon scanf betement nb au lieu de nombre, j espère que je serai plus attentif à l exam ;)
Vous n’avez pas trouvé la réponse que vous recherchez ?
Posez votre question
bonjour,
je m'étais amusé compiler ton programme, et du coup quelqu'un t'as (bien) répondu avant moi.
Mais je rajoute ma sauce : Je te conseille de tester la compilation (et l'exécution) de ton programme avant de remettre ton exercice. Personne n'a un compilateur dans la tête.
Ceci m'a permis de voir, qu'en plus des erreurs que mahmah a vues, il en existe d'autres (un autre point virgule manquant à la fin, et des point virgulesà remplacer par des virgules sur trois lignes différentes)
Pour cela j'ai utilisé gcc que tu peux télécharger gratuitement (chercher le site MinGW.org).
Tu dis que c'est du C++, mais en fait ce code est du C pur et dur, même s'il la syntaxe est compatible C++.
Remarque s'ajoutant à celles de mahmah : mets de commentaires (pertinents) dans tes programmes. en environnement industriel, c'est capital, et je pense qu'un prof ne peut que l'apprécier.
En cadeau, le code garanti compilable sous gcc
#include <stdio.h>
/*
Programme en langage C mettant en oeuvre le oncept de fonction et le concept de pointeur
Ce programme a été compilé en utilisant gcc 3.4.5
Fait à partir de la version originale de steffemaister
*/
/********************************************************
* DECLARATIONS
********************************************************/
void h_m_s(int nb,int *heure,int *min,int *sec );
/********************************************************
* PROGRAMME PRINCIPAL
*******************************************************/
int main()
{
// Variables locales
int s,m,h,nombre;
// rajouter ici un test de nombre > 0 ou caner un unsigned selon remarque
printf("entrer le nbre de secondes :\n");
scanf("%d",&nombre);
// Appel de la méthode en passant les variables heure, minutes et secondes par pointeur
// De façon à récupérer la valeur modifiée
h_m_s(nombre,&h,&m,&s);
printf("%d = %d heure %d minutes %d secondes",nombre,h,m,s);
return 0;
}
/********************************************************
* FONCTION de conversion de nombre de secondes en h,m,s
* Description :
* Implémentation de la fonction qui convertit un nombre
* de secondes en heure, minutes, secondes
*******************************************************/
//
void h_m_s(int nb,int *heure,int *min,int *sec )
{
int tmp;
*heure=nb/3600;
tmp=nb%3600;
*min=tmp/60;
*sec=tmp%60;
}
je m'étais amusé compiler ton programme, et du coup quelqu'un t'as (bien) répondu avant moi.
Mais je rajoute ma sauce : Je te conseille de tester la compilation (et l'exécution) de ton programme avant de remettre ton exercice. Personne n'a un compilateur dans la tête.
Ceci m'a permis de voir, qu'en plus des erreurs que mahmah a vues, il en existe d'autres (un autre point virgule manquant à la fin, et des point virgulesà remplacer par des virgules sur trois lignes différentes)
Pour cela j'ai utilisé gcc que tu peux télécharger gratuitement (chercher le site MinGW.org).
Tu dis que c'est du C++, mais en fait ce code est du C pur et dur, même s'il la syntaxe est compatible C++.
Remarque s'ajoutant à celles de mahmah : mets de commentaires (pertinents) dans tes programmes. en environnement industriel, c'est capital, et je pense qu'un prof ne peut que l'apprécier.
En cadeau, le code garanti compilable sous gcc
#include <stdio.h>
/*
Programme en langage C mettant en oeuvre le oncept de fonction et le concept de pointeur
Ce programme a été compilé en utilisant gcc 3.4.5
Fait à partir de la version originale de steffemaister
*/
/********************************************************
* DECLARATIONS
********************************************************/
void h_m_s(int nb,int *heure,int *min,int *sec );
/********************************************************
* PROGRAMME PRINCIPAL
*******************************************************/
int main()
{
// Variables locales
int s,m,h,nombre;
// rajouter ici un test de nombre > 0 ou caner un unsigned selon remarque
printf("entrer le nbre de secondes :\n");
scanf("%d",&nombre);
// Appel de la méthode en passant les variables heure, minutes et secondes par pointeur
// De façon à récupérer la valeur modifiée
h_m_s(nombre,&h,&m,&s);
printf("%d = %d heure %d minutes %d secondes",nombre,h,m,s);
return 0;
}
/********************************************************
* FONCTION de conversion de nombre de secondes en h,m,s
* Description :
* Implémentation de la fonction qui convertit un nombre
* de secondes en heure, minutes, secondes
*******************************************************/
//
void h_m_s(int nb,int *heure,int *min,int *sec )
{
int tmp;
*heure=nb/3600;
tmp=nb%3600;
*min=tmp/60;
*sec=tmp%60;
}
