Discussion :

[defcon29]

bonjour,

je viens de commencer en C, par un programme tout simple: calculer Vitesse distance et temps:

1 - parametre manquant? vitesse distance ou temps

en fonction de la reponse, on donne les 2 autres parametres, , et le prog donne la valeur inconnue...

tres simple!
mais je n arrive pas a faire la comparaison entre "p", qui est le parametre manquant, et ce que vas taper l utilisateur (v d ou t)

je sais qu il y a des moyens plus simple, avec strcmp, mais pour bien voir le fonctionnement de base.

voici mon code, en cour d elaboration:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
#include <stdio.h>
 
int main ()
{
char p;
int vit,dist,tps;
 
 
    printf("Calcul de temps\n");
    printf("Paramètre manquant? V, D, T ?");
    scanf("%c", &p);
 
    if ( p == "V" )
    {
         printf("Distance KM?");
         scanf("%d",&dist);
 
         printf("Temps Min?");
         scanf("%d",&tps);
 
         vit= dist + tps;
 
         printf("Votre vitesse est de %d KM/H", vit); 
     }
 
     else 
         printf("en attente de creation\n");
 
      system("PAUSE"); 
      return 0;   
}

par avance merci pour vos conseil et suggestion ! mais je veux pas forcement les moyens les plus simples!

MERCI

[Invité(e)]

Bonjour,

Ici, tu compare UN caractère à une CHAINE de caractères :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( p == "V" )

Il faut écrire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(p == 'V')

Après, utiliser scanf n'est pas évident, je te conseille la lecture de http://xrenault.developpez.com/tutoriels/c/scanf/

[diogene]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

vit= dist + tps;

Accessoirement, la vitesse n'est pas égale à la distance + le temps .
On va avoir

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

vit= dist/tps;

toutes ces grandeurs ne devraient pas être des entiers mais des flottants

[defcon29]

super merci, pour la correction!!

sinon pour la formule, ben ........., on vas dire que j ai pas reflechis !!

Merci

[defcon29]

encore une petite question, on fais comment pour limiter l affichage d un float a 2, 3 4... chiffre apres la virgule ??

Est il possible de transformer, une donnée de l 'utilisateur debile (4,1234567876545678) en 4 ou 4,5 ou 4,2 selon ce dont j ai besoin ??

merci !

[Invité(e)]

encore une petite question, on fais comment pour limiter l affichage d un float a 2, 3 4... chiffre apres la virgule ??

Est il possible de transformer, une donnée de l 'utilisateur debile (4,1234567876545678) en 4 ou 4,5 ou 4,2 selon ce dont j ai besoin ??

merci !

http://www.developpez.net/forums/d98...at-via-printf/

[Sve@r]

je sais qu il y a des moyens plus simple, avec strcmp, mais pour bien voir le fonctionnement de base.

Non. strcmp() est une fonction qui compare deux chaines.

Faut bien comprendre qu'en C, une chaine n'existe pas en tant qu'entité manipulable. Tout ce que tu peux manipuler, c'est l'adresse de son premier caractère.
Ensuite, à partir de cette adresse, tu peux passer au caractère suivant et encore au suivant etc jusqu'à avoir traité toute la chaine.

Il s'ensuit que si tu ceux comparer 2 chaines, te faudra les comparer caractère par caractère. Ben c'est ce que fait strcmp(). Tout simplement.

par avance merci pour vos conseil et suggestion ! mais je veux pas forcement les moyens les plus simples!

Ben déjà, ton code de départ semble aller vers une super redondance. S'il te manque V alors tu fais saisir d et t. S'il te manque d, je présume que tu feras saisir V et t (donc double codage). etc.
Donc déjà, moi je partirais sur une fonction dédiée à la saisie de V, une autre dédiée à la saisie de d et une autre pour t. Ainsi, me suffit d'appeler les bonnes fonctions au bon endroit. Mais bon, ça c'est juste pour faire classe.

Le vrai problème de ton code, c'est scanf. Mais c'est pas de ta faute. Un débutant ne possède pas encore tous les outils qui vont bien pour faire de la belle saisie et donc on est obligé de lui parler de scanf() au-moins pour pouvoir faire les premiers TP.
Le problème de scanf, c'est qu'il attend un truc "formaté". Et que ce que tape l'utilisateur est tout sauf formaté. Même s'il est de bonne volonté, ça merdera quand-même. Parce que scanf ne traite que ce qu'il peut comprendre. Et ce qu'il ne comprend pas, ben il le laisse dans le tampon clavier.

Et donc imaginons que tu fasses saisir un age scanf("%d") puis un nom scanf("%s"). Tu entres l'age 18, tu entreras correctement '1' puis '8' puis <entrée>. Ben ce <entrée> n'étant pas du numérique, il ne sera pas pris en compte par le scanf("%d") et restera dans le tampon clavier. Ensuite, arrivé à la saisie du nom, là le "entrée" de la saisie précédente sera pris en compte et récupéré sans que le programme ne ce soit arrêté pour te laisser taper quoi que ce soit (puisqu'il a déjà qqchose).
Un autre problème est que dans la saisie du nom, l'utilisateur pourra rentrer autant de caractères qu'il a envie. scanf() ne limitera pas. Et tant pis si la zone du nom n'a pas la taille adaptée.

Et donc c'est pour tout ça que scanf n'est absolument pas adapté aux saisies clavier.

Une des méthodes les plus simples à mettre en oeuvre, pour éliminer ces problèmes, est de faire saisir que du texte en utilisant fgets(). Puis traiter ce texte via sscanf(). Déjà fgets() limite la saisie ce qui évite de trop en entrer. Et ça permet en plus un contrôle de saisie car sscanf() renvoie le nb d'éléments récupérés et qu'il est facile de comparer ce nb avec le nb attendu.

Exemple

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
#include <stdio.h>
int main()
{
    char nom[21];
    int age;
    char saisie[80];
 
    // Saisie de l'âge
    while (1)
    {
         printf("Entrez votre age :\n");
         fgets(saisie, 80; stdin);
 
         // Contrôle age ok - Si ok, on sort de la boucle
         if (sscanf(saisie, "%d", &age) == 1) break;
 
         // Saisie incorrecte
         printf("Saisie incorrecte - Recommencez\n");
     }
 
    // Saisie du nom
    while (1)
    {
         printf("Entrez votre nom :\n");
         fgets(saisie, 80; stdin);
 
         // Contrôle nom ok - Si ok, on sort de la boucle
         if (sscanf(saisie, "%20s", nom) == 1) break;
 
         // Saisie incorrecte
         printf("Saisie incorrecte - Recommencez\n");
     }
}

Voilà. J'espère que tu vois le principe qui peut être facilement adapté à la saisie des nombres flottants. Normalement il reste le risque que l'utilisateur entre plus de 80 caractères alors que la saisie n'en accepte que 80 (ou plus exactement 79 car fgets() garde une place pour le '\0'). Dans ce cas, le surplus reste dans le tampon clavier. Pour bien faire, faudrait rajouter une boucle à base de fgetc() pour vider ledit tampon mais j'ai pas voulu en rajouter trop...