chaîne de caractère en condition d'un IF

Version imprimable

Bonjour,

pour ma part très habitué au fortran, je découvre le C avec lequel je souhaite réecrire un gros code que j'avais développez en fortran. Je fais, pour me familiariser avec ce nouveau language quelques exercices simples, et je bloque sur une petite chose :evilred:

Voila je saisis 2 entiers (a, b), et un charactère (+ ou -), si la saisie de charactère est "-" alors j'effectue l'opération c=a-b sinon c=a+b.

Le bout de code qui suit compile, mais quoique je tappe l'opération effectuée est c=a+b...

Si quelqu'un a une idée, merci d'avance.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 #include<stdio.h> int a,b,c; char signe[1]; int main() { printf("Valeur de a : \n"); scanf("%d",&a); printf("Valeur de b : \n"); scanf("%d",&b); printf("Quel opération : \n"); scanf("%s",&signe); printf("MOINS %c \n",(char)signe); if (signe != "-") c=a+b; else c=a-b; printf("%d %s %d = %d \n",a,signe,b,c) }

Code:

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
34
35
36
37
38
39
#include <stdio.h>
 
int main (void)
{
   int a, b, ret;
   char signe;
 
   do
   {
      puts ("Valeur de a : ");
      ret = scanf ("%d", &a);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      puts ("Valeur de b : ");
      ret = scanf ("%d", &b);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      char temp[2];
      puts ("Quel opération : ");
      ret = scanf ("%1[+-]", temp);
      scanf ("%*[^\n]"), getchar ();
      if (ret == 1)
      {
         signe = temp[0];
      }
   }
   while (ret != 1);
 
   printf ("%d %c %d = %d \n", a, signe, b, signe == '-' ? a - b : a + b);
   return 0;
}

Citation:

Envoyé par nicolas.sitbon

Code:

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
34
35
36
37
38
39
#include <stdio.h>
 
int main (void)
{
   int a, b, ret;
   char signe;
 
   do
   {
      puts ("Valeur de a : ");
      ret = scanf ("%d", &a);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      puts ("Valeur de b : ");
      ret = scanf ("%d", &b);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      char temp[2];
      puts ("Quel opération : ");
      ret = scanf ("%1[+-]", temp);
      scanf ("%*[^\n]"), getchar ();
      if (ret == 1)
      {
         signe = temp[0];
      }
   }
   while (ret != 1);
 
   printf ("%d %c %d = %d \n", a, signe, b, signe == '-' ? a - b : a + b);
   return 0;
}

Ben pour un débutant tu pourrais lui expliquer un peu ce que tu fais et pourquoi tu le fais (je parle surtout de tes scanf() finaux qui me semblent incompréhensibles pour qui n'a pas déjà un bon niveau en C). Par ailleurs, moi je sortirais le "signe=temp[0]" hors de la boucle ce qui évite un test inutile...

21/11/2007, 14h40
nicolas.sitbon

Citation:

Envoyé par Sve@r

Ben pour un débutant tu pourrais lui expliquer un peu ce que tu fais et pourquoi tu le fais (je parle surtout de tes scanf() finaux qui me semblent incompréhensibles pour qui n'a pas déjà un bon niveau en C). Par ailleurs, moi je sortirais le "signe=temp[0]" hors de la boucle ce qui évite un test inutile...

Non je préfère limiter la portée de temp, c'est un choix comme un autre, chacun sa façon de coder.
21/11/2007, 14h45
yann_m

Merci a vous, je vais essayer de comprendre tout ça.

Bonne journée.

Citation:

Envoyé par nicolas.sitbon

Code:

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
34
35
36
37
38
39
#include <stdio.h>
 
int main (void)
{
   int a, b, ret;
   char signe;
 
   do
   {
      puts ("Valeur de a : ");
      ret = scanf ("%d", &a);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      puts ("Valeur de b : ");
      ret = scanf ("%d", &b);
      scanf ("%*[^\n]"), getchar ();
   }
   while (ret != 1);
 
   do
   {
      char temp[2];
      puts ("Quel opération : ");
      ret = scanf ("%1[+-]", temp);
      scanf ("%*[^\n]"), getchar ();
      if (ret == 1)
      {
         signe = temp[0];
      }
   }
   while (ret != 1);
 
   printf ("%d %c %d = %d \n", a, signe, b, signe == '-' ? a - b : a + b);
   return 0;
}

c'est un peu lourd tout ça...:mouarf3::mouarf3::mouarf3:

Citation:

Envoyé par yann_m

Bonjour,

pour ma part très habitué au fortran, je découvre le C avec lequel je souhaite réecrire un gros code que j'avais développez en fortran. Je fais, pour me familiariser avec ce nouveau language quelques exercices simples, et je bloque sur une petite chose :evilred:

Déjà pourquoi déclarer signe comme un tableau vu qu'il ne contient qu'un seul caractère, autant le declarer

Code:

char signe;

Ensuite quand tu as un tableau de caractère tel que signe[1], et bien que scanf ne soit pas du tout approprié pour la saisie de données non formatées. il faut donner à scanf un pointeur, or le nom d'un tableau de caractère (ici signe) est déjà un pointeur, donc ton scanf n'a pas besoin d'adresse.

Code:

scanf ("%s", signe);

Sinon quand au problème que tu as il va falloir soit que tu étudies l'utilisation de fgets à la place de scanf http://c.developpez.com/faq/?page=clavier_ecran, soit que tu trouves des petites astuces bien pourries tels que le code suivant :
Code:

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 #include<stdio.h> int a,b,c; char signe; int main(void) { printf("Valeur de a : \n"); scanf("%d",&a); printf("Valeur de b : \n"); scanf("%d",&b); c = getchar (); printf("Quel opération : \n"); scanf("%c", &signe); printf("MOINS %c \n",(char)signe); if (signe != '-') c=a+b; else c=a-b; printf("%d %c %d = %d \n",a,signe,b,c); return 0; }