j'ai un trou de memoire

Version imprimable

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
#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:

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

20/11/2007, 19h25
dany9

hello !

regarde bien tes tests, ici :

Code:

if (choix = 'v')

et ici :

Code:

else if (choix = 'v')

y a comme un défaut ... ;)
20/11/2007, 19h27
Gruik

Salut,

Comme dirait Manu, un compilateur bien configuré t'aurais averti du problème
20/11/2007, 19h37
Neitsa
Code:

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 ? )
20/11/2007, 19h48
diogene

De toute façon

Code:

if (choix = 'v')

est incorrect :
- il faudrait ==
- choix est un tableau pas un char
20/11/2007, 19h56
fredo3il

tu dois avoir un fros trou de mémoire alors:mouarf:

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

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

:salut:

PS : testé sous Solaris avec gcc...

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

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

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

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

20/11/2007, 21h52
nicolas.sitbon

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

Code:

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

:mrgreen:

donc pour résumer, ça donnerait :

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
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"...:king:

20/11/2007, 22h38
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
20/11/2007, 22h46
jeremy13

Citation:

Envoyé par 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

:furieux: 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...
20/11/2007, 22h49
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.
20/11/2007, 22h56
jeremy13

Citation:

Envoyé par 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.

tu te rappelles des fonctions où l'on peut sentir la différence ?
20/11/2007, 22h59
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.
20/11/2007, 23h03
Gruik

Citation:

Envoyé par jeremy13

:furieux: 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)
20/11/2007, 23h08
Sve@r

Citation:

Envoyé par nicolas.sitbon

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