Discussion :

[toopa]

Salut à tous !
Voilà, je me mets tranquillement au C en commençant par faire des petits programmes gentils, donc j'ai décidé après plusieurs programmes d'essayer de réaliser une calculatrice (simple juste avec +,-,*,/). Mais je me retrouve deja confronté a un problème :

j'utilise :

Code :

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

1
2
3
4
5
float a=0;
char b;
float c=0;
 
scanf("%f%c%f",&a,&b,&c);

ensuite je compare le char à +,-,*,/ et je fais l'opération adéquate. Jusque la pas de souci.. Mis à part que je ne peux faire des opérations que sur des floats à 1 chiffre !

j'ai bien pensé à utiliser (j'ai cru comprendre que cela marchait) :

Code :

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

scanf("%f2%c%f2",&a,&b,&c);

mais le problème est qu'à présent je ne peux faire des calculs que sur des floats à 2 chiffres ! J'ai aussi pensé à utiliser 3 scanf() à la suite, mais l'idée de devoir saisir chaque caractère indépendamment ne me séduit pas...

Voilà, donc je me demande s'il est possible de faire des saisies de longueur pouvant varier sur les floats ?

Quelqu'un peut-il m'aider ? (mis à part google, je sais qu'il est fort mais la ca devient un peu long comme question à lui taper...).

Merci toopa

[FunK92]

Salut,

Je ne suis pas sur de ce que j'annonce là mais il me semble que quand tu tape par exemple 12.5, il considère le '.' comme un caractère.

A mon avis tu devrait faire une fonction qui parser ton expression (qui sera pour le coup une chaîne de caractère-> %s). Et convertir ce que tu as parser avec des fonction tel que atoi, atol ou encore atof

[diogene]

J'utilise :

Code :

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

1
2
3
4
5
float a=0;
char b;
float c=0;
 
scanf("%f%c%f",&a,&b,&c);

ensuite je compare le char à +,-,*,/ et je fais l'opération adéquate. Jusque la pas de souci.. Mis à part que je ne peux faire des opérations que sur des floats à 1 chiffre !

Il n'y a pas de raisons :
si je tape 12x45 , j'obtient bien a = 12, b = 'x' et c = 45 !
Par contre, on ne peut taper 12 x 45

[toopa]

Ouais en fait ça marche très bien.. Je sais pas ce qu'il y avait j'avais du faire une autre erreur mais à présent tout fonctionne !

Code de la "calcoulette" :

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
32
33
 int main()
{
    float a=0;
    float b=0;
    char c;
    float d=0;
 
    printf("Entrez votre calcul :\n");
    scanf("%f%c%f",&a,&c,&b);
 
    if(strcmp(&c,"+")==0){
        d=a+b;
        printf("%f",d);
     }
 
    else{if(strcmp(&c,"-")==0){
        d=a-b;
        printf("%f",d);
        }
        else{if(strcmp(&c,"*")==0){
        d=a*b;
        printf("%f",d);
            }
            else{if(strcmp(&c,"/")==0){
            d=(float)a/b;
            printf("%f",d);
                }
                else{printf("ERROR");}
            }
        }
    }
    return 0;
}

Par contre je me sens un peu pouilleux avec mes 4 if/else je pense utiliser des case si j'arrive a comprendre comment ça marche.
En tout cas merci à vous 2 et bonne continuation

[Climoo]

Par contre je me sens un peu pouilleux avec mes 4 if/else je pense utiliser des case si j'arrive a comprendre comment ça marche.

C'est vrai que ici, un switch serait plus simple.
D'ailleurs, (mais je me trompe peut être), utiliser strcmp avec un pointeur vers un char qui n'est PAS une chaine de caractère, peut être un problème.
En effet, rien ne nous dit qu'en mémoire, après le 'c', on va bien trouver le 0 qui termine la chaine de caractères... (erreur de segmentation?).
Je trouve aussi la remarque de Funk92 pertinente, même si ce que tu fais fonctionne... De cette manière, tu pourrais vérifier que l'on rentre bien des nombres et ce serait plus "sûr".

Pour le switch, la syntaxe est

Code :

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

1
2
3
4
5
6
7
8
 
switch (variable) {
  case valeur1 : action1 [break;] 
  case valeur2 : action2 [break;]
  ...
  case valeurN : actionN [break;]
  default  : actionParDefaut [break;]
}

Les break à la fin ne sont pas toujours mis selon ce qu'on désire faire, mais dans ton cas, met les (je te laisse chercher pourquoi). Le break après le défaut peut toujours être enlevé. Ca donnerait un truc du type

Code :

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

1
2
3
4
5
6
7
8
 
switch (c) {
  case '+' : /*addition*/ break;
  case '-' : /*soustraction*/ break;
  case '*' : /*multiplication*/ break;
  case '/' : /*division*/ break;
  default : /*opérateur inconnu*/ 
}

Une dernière chose importante : Tu ne testes pas, si lors d'une division, le deuxième nombre vaut 0 ! Autant prévenir ce cas et afficher une erreur soit même, et ne pas demander au processeur de faire une division par 0.

[diogene]

....
Par contre je me sens un peu pouilleux avec mes 4 if/else je pense utiliser des case si j'arrive a comprendre comment ça marche.
....

Pour confirmer ce que dit Climoo :

Attention, tes if sont faux :

Code :

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

1
2
3
4
 
    char c;
....
    if(strcmp(&c,"+")==0){

strcmp compare deux chaînes de caractères "+" est bien une chaîne, mais &c ne l'est pas : c'est l'adresse d'UN caractère. Ce test échouera en général même si c vaut '+'

Pour comparer deux caractères, il suffit de tester c == '+' (entre guillemets simples)