Discussion :

[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

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 :

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
 
#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)
}

[nicolas.sitbon]

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
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;
}

[Sve@r]

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
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...

[nicolas.sitbon]

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.

[yann_m]

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

Bonne journée.

[jeremy13]

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
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...

[exhortae]

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

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

Code :

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

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 :

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

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 :

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
#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;
}