Discussion :

[coco.cohen]

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
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    char  choix[1];   
 
    printf("VIEW   : v ou V --> Consulter les informations\n");
    printf("UPDATE : u ou U --> Mettre à jour les informations d'un demandeur\n");
    printf("DELETE : d ou D --> Supprimer un demandeur\n");
    printf("ADD    : a ou A --> Ajouter un demandeur\n");
    printf("SHOW   : s ou S --> Montrer la représentation de l'arbre binaire\n");
 
 
    printf("Commande:");
    scanf("%s",choix);
 
    if (choix = 'v')
     {
                printf("cool");
     }
 
    else if (choix = 'v')
     {
                printf("cool2");
     }
  system("PAUSE");	
  return 0;
}

Lorsque je compile le compilateur DEV C++ me dit qu'il y a des erreurs de compilations

mon programme ne veut pas compiler
je me rapel plus comment on fait

dsl je me suis tromper de prog

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
40
41
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    char  choix[1];   
 
    printf("VIEW   : v ou V --> Consulter les informations\n");
    printf("UPDATE : u ou U --> Mettre à jour les informations d'un demandeur\n");
    printf("DELETE : d ou D --> Supprimer un demandeur\n");
    printf("ADD    : a ou A --> Ajouter un demandeur\n");
    printf("SHOW   : s ou S --> Montrer la représentation de l'arbre binaire\n");
 
 
    printf("Commande:");
    scanf("%s",choix);
 
    if (choix = 'v')
     {
                printf("cool");
     }
 
    else if (choix = 'u')
     {
                printf("cool2");
     }
     else if (choix = 'd')
     {
                printf("cool3");
     }
    else if (choix = 'a')
     {
                printf("cool4");
     }
    else if (choix = 's')
     {
                printf("cool5");
     }
  system("PAUSE");	
  return 0;
}

[dany9]

hello !


regarde bien tes tests, ici :

Code :

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

  if (choix = 'v')

et ici :

Code :

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

else if (choix = 'v')

y a comme un défaut ...

[Gruik]

Salut,

Comme dirait Manu, un compilateur bien configuré t'aurais averti du problème

[Neitsa]

Code :

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

1
2
3
4
5
6
7
 
    char  choix[1];   
 
/* [...] */
 
 
    scanf("%s",choix);

Attention à scanf et à tous ses problèmes... Attention au dépassement de tampon ! (i.e : que va-t-il se passer si l'utilisateur entre une chaîne assez longue plutôt qu'une simple lettre ? )

[diogene]

De toute façon

Code :

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

if (choix = 'v')

est incorrect :
- il faudrait ==
- choix est un tableau pas un char

[fredo3il]

tu dois avoir un fros trou de mémoire alors

[jeremy13]

Tiens c'est bientôt Noël... :

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
40
41
42
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    char  choix;
 
    printf("VIEW   : v ou V --> Consulter les informations\n");
    printf("UPDATE : u ou U --> Mettre ` jour les informations d'un demandeur\n");
    printf("DELETE : d ou D --> Supprimer un demandeur\n");
    printf("ADD    : a ou A --> Ajouter un demandeur\n");
    printf("SHOW   : s ou S --> Montrer la reprisentation de l'arbre binaire\n");
 
 
    printf("Commande:");
    choix=getchar();
 
        choix=tolower(choix);
 
        switch(choix)
        {
                case 'v' :
                                        printf("cool-1\n");
                                        break;
                case 'u' :
                                        printf("cool-2\n");
                                        break;
                case 'd' :
                                        printf("cool-3\n");
                                        break;
                case 'a' :
                                        printf("cool-4\n");
                                        break;
                case 's' :
                                        printf("cool-5\n");
                                        break;
                default :
                                        printf("boufon! t'as tapi la mauvaise lettre!\n");
                                        break;
        }
        return 0;
}

Tu devrais mettre un while pour éviter de faire terminer ton binaire...



PS : testé sous Solaris avec gcc...

[nicolas.sitbon]

Pour les réglages du compilateur c'est par ici :
http://emmanuel-delahaye.developpez....fg_compilo_gcc

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
40
41
42
43
44
45
46
47
48
49
#include <stdio.h>
 
int main (void)
{
   char choix;
   int ret;
   puts ("VIEW   : v ou V --> Consulter les informations\n"
         "UPDATE : u ou U --> Mettre à jour les informations d'un demandeur\n"
         "DELETE : d ou D --> Supprimer un demandeur\n"
         "ADD    : a ou A --> Ajouter un demandeur\n"
         "SHOW   : s ou S --> Montrer la représentation de l'arbre binaire");
 
   do
   {
      char temp[2];
      printf ("Commande:");
      ret = scanf ("%1[vVuUdDaAsS]", temp);
      scanf ("%*[^\n]"), getchar ();
      if (ret == 1)
         choix = temp[0];
   }
   while (ret != 1);
 
   switch (choix)
   {
   case 'v':
   case 'V':
      puts ("cool");
      break;
   case 'u':
   case 'U':
      puts ("cool2");
      break;
   case 'd':
   case 'D':
      puts ("cool3");
      break;
   case 'a':
   case 'A':
      puts ("cool4");
      break;
   case 's':
   case 'S':
      puts ("cool5");
   }
 
   getchar ();
   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
switch (choix)
   {
   case 'v':
   case 'V':
      puts ("cool");
      break;
   case 'u':
   case 'U':
      puts ("cool2");
      break;
   case 'd':
   case 'D':
      puts ("cool3");
      break;
   case 'a':
   case 'A':
      puts ("cool4");
      break;
   case 's':
   case 'S':
      puts ("cool5");
   }

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
switch (tolower(choix))
   {
   case 'v':
      puts ("cool");
      break;
   case 'u':
      puts ("cool2");
      break;
   case 'd':
      puts ("cool3");
      break;
   case 'a':
      puts ("cool4");
      break;
   case 's':
      puts ("cool5");
      break;
   }

[nicolas.sitbon]

Je sais mais j'évite un appel de fonction pour si peu.

[Gruik]

Code :

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

1
2
3
switch (choix >= 'A' && choix <= 'Z' ? choix + ('a' - 'A') : choix)
{
...

[jeremy13]

donc pour résumer, ça donnerait :

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
40
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    char  choix;
 
    printf("VIEW   : v ou V --> Consulter les informations\n");
    printf("UPDATE : u ou U --> Mettre ` jour les informations d'un demandeur\n");
    printf("DELETE : d ou D --> Supprimer un demandeur\n");
    printf("ADD    : a ou A --> Ajouter un demandeur\n");
    printf("SHOW   : s ou S --> Montrer la reprisentation de l'arbre binaire\n");
 
 
    printf("Commande:");
    choix=getchar();
 
        switch (choix >= 'A' && choix <= 'Z' ? choix + ('a' - 'A') : choix)
        {
                case 'v' :
                                        printf("cool-1\n");
                                        break;
                case 'u' :
                                        printf("cool-2\n");
                                        break;
                case 'd' :
                                        printf("cool-3\n");
                                        break;
                case 'a' :
                                        printf("cool-4\n");
                                        break;
                case 's' :
                                        printf("cool-5\n");
                                        break;
                default :
                                        printf("boufon! t'as tapi la mauvaise lettre!\n");
                                        break;
        }
        return 0;
}

je pense qu'on ne peux pas mieux niveaux "optimisation"...

[Gruik]

Non c'était un blague, vaut mieux utiliser toupper/tolower qu'ecrire cette ribambelle de code illisible (ou alors avoir une macro qui fait ça), mais franchement économiser UN appel de fonction c'est comme se tater entre prendre un int ou un short pour une variable locale : c'est completement hors de propos

[jeremy13]

Non c'était un blague, vaut mieux utiliser toupper/tolower qu'ecrire cette ribambelle de code illisible (ou alors avoir une macro qui fait ça), mais franchement économiser UN appel de fonction c'est comme se tater entre prendre un int ou un short pour une variable locale : c'est completement hors de propos

Alors ça c'est pas très marrant...au début je doutais...puis j'me suis dit mais non c'est le modérateur, il sait ce qu'il dit quand même...mais d'un côté il faudrait peut-être testé, peut-être qu'on gagne quelques µs...qui sait ? Mais là j'ai pas le courage ce soir...

[nicolas.sitbon]

Déjà tester, avec tolower() on perd rien parcontre il y a d'autre fonction parfois ou...
C'est juste une habitude que j'avais prise à cause du boulot ou on faisait des gros calculs en boucle et la le gain se faisait sentir mais bien sûr dans ce cas précis ça ne sert strictement à rien.

[jeremy13]

Déjà tester, avec tolower() on perd rien parcontre il y a d'autre fonction parfois ou...
C'est juste une habitude que j'avais prise à cause du boulot ou on faisait des gros calculs en boucle et la le gain se faisait sentir mais bien sûr dans ce cas précis ça ne sert strictement à rien.

tu te rappelles des fonctions où l'on peut sentir la différence ?

[nicolas.sitbon]

Je pourrais en citer plusieurs mais ça ne voudrait strictement rien dire dans la mesure ou cela peut varier d'une plateforme à une autre, d'un compilateur à un autre, d'une librairie à un autre, bref c'est à toi de faire les tests qui vont bien et voir les gains ou pertes de performance.

[Gruik]

Alors ça c'est pas très marrant...au début je doutais...puis j'me suis dit mais non c'est le modérateur, il sait ce qu'il dit quand même...mais d'un côté il faudrait peut-être testé, peut-être qu'on gagne quelques µs...qui sait ? Mais là j'ai pas le courage ce soir...

Désolé 8) C'était aussi pour montrer un peu de technique..
Apres pour le gain de temps, je voulais dire ceci : éviter d'avoir ce genre de considérations, enfin faut éviter de perdre du temps à se poser des questions là dessus, vaut mieux chercher la simplicité du code (le plus court, le plus simple, le plus naturel, le moins tordu)

[Sve@r]

Déjà tester, avec tolower() on perd rien parcontre il y a d'autre fonction parfois ou...

Boaf ça n'a pas influencé mon code mais juste pour info: c'est utile de savoir que tolower() c'est plus souvent une macro qu'une fonction...