j'ai un trou de memoire

**coco.cohen** · 20/11/2007, 19h15

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** · 20/11/2007, 19h25

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** · 20/11/2007, 19h27

Salut,

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

**Neitsa** · 20/11/2007, 19h37

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** · 20/11/2007, 19h48

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** · 20/11/2007, 19h56

tu dois avoir un fros trou de mémoire alors

**jeremy13** · 20/11/2007, 20h22

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** · 20/11/2007, 20h42

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** · 20/11/2007, 21h43

Envoyé par 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
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** · 20/11/2007, 21h52

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

**Gruik** · 20/11/2007, 21h55

Code :

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

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

**jeremy13** · 20/11/2007, 22h17

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** · 20/11/2007, 22h38

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** · 20/11/2007, 22h46

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

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** · 20/11/2007, 22h49

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** · 20/11/2007, 22h56

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 ?

**nicolas.sitbon** · 20/11/2007, 22h59

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** · 20/11/2007, 23h03

Envoyé par jeremy13

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** · 20/11/2007, 23h08

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

j'ai un trou de memoire

C

Discussions similaires

Partager

Partager