Problème dans mon main

**pikaTuX** · 07/12/2007, 17h36

Au secours !!!
Bon voila, pour mon projet on doit faire l'implémentation d'un tas. Mais j'ai quelques problèmes dans le main, notamment pour la saisie des valeurs. Voici le code :

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
50
51
52
53
54
55
56
57
58
59
60
int
main(int argc, char *argv[])
{
  tas T;
 // Recupere le type de valeur en entree
  // int ou string
  char x;
  printf("Quels types de valeurs dans le tas ?\n");
  printf("c pour des chaines,\nn pour des entiers.\n");
  scanf("%c", &x);
  /*
  if (x != "c" || x != "n")
    {
      fprintf(stderr,"Entiers ou chaines !\n");
      exit(EXIT_FAILURE);
    }
  */
  // Recuperation de la taille du tas
  printf("\nCombien de valeurs dans le tas ? ");
  int n = 0;
  scanf("%d", &n);
 
  // Entree des valeurs
  printf("\nEntrez les %d valeurs :\n", n);
 
  int w =0;
  bool sign = true;
  while(sign == true)
    {
      void* z = NULL;
      if (&x == "c") // Entre une chaine
	{
	  //char z = '\0';
	  scanf("%s",(char*) &z);
	  tas_inserer(T, z);
	}
      if (&x == "n") // Entre un entier
	{
	  //int z = 0;
	  scanf("%d",(int*) &z);
	  tas_inserer(T, z);
	}
      w++;
      if(w == n)
	sign = false;
    }
 
  if (argc == 1)
    T=tas_creer(ordre_lexico_normal);
  if (argc == 2 && argv[1] == "-i")
    T=tas_creer(ordre_lexico_inverse);
  if (argc == 2 && argv[1] == "-n")
    T=tas_creer(ordre_numerique_ascendant);
  if (argc == 3 && argv[1] == "-i" && argv[2] == "-n")
    T=tas_creer(ordre_numerique_descendant);
  else
    usage(argv[0]);
 
  return EXIT_SUCCESS;
}

Merci beaucoup !

**ced236** · 07/12/2007, 17h45

Il y a plusieurs erreur dans ton code par exemple celle la "if (&x == "c")"
tu dis "si l adresse de x = c " tu devrais plutot remplacer par "if (x == 'c' )"

le symbolle "&" permet d utiliser la valeur gauche d un variable ( si je dis pas de betise )
La valeur gauche est son adresse

Tu devrai revoir ton code il me semble qu il y a plusieur erreurs plus ou moins grosses

**corentin59** · 07/12/2007, 17h52

Envoyé par pikaTuX

Code :

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

1
2
3
4
5
6
7
void* z = NULL;
      if (&x == "c") // Entre une chaine
	{
	  //char z = '\0';
	  scanf("%s",(char*) &z);
	  tas_inserer(T, z);
	}

il faut allouer de la mémoire pour une chaine de caractères d'une certaine longueur (et faire toutes les vérifications nécessaires) pour pouvoir y copier ce que tu tapes. En plus, ton "&z" alors que z est un pointeur, me parait louche

**siegfried64** · 07/12/2007, 17h57

d'ailleur :

Code :

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

1
2
3
4
5
6
7
 /*
  if (x != "c" || x != "n")
    {
      fprintf(stderr,"Entiers ou chaines !\n");
      exit(EXIT_FAILURE);
    }
  */

je vois pas pourquoi t a mis ca en commentaire

, je suppose que c'est juste un erreur, faut enlever les /*.

Code :

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

bool sign = true;

si tu travail sur c le type bool n'est pas defini, tu dois le definir toi meme.

Code :

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

if (&x == "c") // Entre une chaine

la tu essaie de comparer un addresse a un char, apart ca " est reservé aux chaines de caracteres.
le bon code serait :[ANERIES SUPPRIMEES]

**corentin59** · 07/12/2007, 18h08

voici une base de départ.

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
50
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[]) {
    //tas T;
    char x, ligne[256];
    int n = 0, w = 0, z = 0;
 
    printf("Quels types de valeurs dans le tas ?\n");
    printf("c pour des chaines,\nn pour des entiers.\n");
    scanf("%c",&x);
    while(getchar()!='\n'); // pour vider le buffer
    if ( x != 'c' && x != 'n' ) {
        fprintf(stderr,"Entiers ou chaines !\n");
        exit(EXIT_FAILURE);
    }
    printf("Vous avez choisi %c\n",x);
 
    printf("\nCombien de valeurs dans le tas ? ");
    scanf("%d",&n);
    while(getchar()!='\n'); // pour vider le buffer
    printf("\nEntrez les %d valeurs :\n",n);
 
    while ( w < n ) {
        void* z = NULL;
        if ( x == 'c' ) {
            fgets(ligne,256,stdin);
            //tas_inserer(T, z);
        } else if ( x == 'n' ) {
            scanf("%d",&z);
            while(getchar()!='\n'); // pour vider le buffer
            //tas_inserer(T,z);
        }
        w++;
    }
 
    if (argc == 1) {
        //T=tas_creer(ordre_lexico_normal);
    } else if (argc == 2 && argv[1] == "-i") {
        //T=tas_creer(ordre_lexico_inverse);
    } else if (argc == 2 && argv[1] == "-n") {
        //T=tas_creer(ordre_numerique_ascendant);
    } else if (argc == 3 && argv[1] == "-i" && argv[2] == "-n") {
        //T=tas_creer(ordre_numerique_descendant);
    } else {
        //usage(argv[0]);
    }
 
    return EXIT_SUCCESS;
}

j'ai mis en commentaires les fonctions que tu n'as pas posté

**corentin59** · 07/12/2007, 18h09

Envoyé par siegfried64

d'ailleur :

Code :

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

1
2
3
4
5
6
7
 /*
  if (x != "c" || x != "n")
    {
      fprintf(stderr,"Entiers ou chaines !\n");
      exit(EXIT_FAILURE);
    }
  */

je vois pas pourquoi t a mis ca en commentaire

, je suppose que c'est juste un erreur, faut enlever les /*.

peut-être parce que ça ne marchait pas. Et de fait, il faut && et non ||

**souviron34** · 08/12/2007, 03h57

@siegfried64 :

Envoyé par siegfried64

d'ailleur :

Code :

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

1
2
3
4
5
6
7
 /*
  if (x != "c" || x != "n")
    {
      fprintf(stderr,"Entiers ou chaines !\n");
      exit(EXIT_FAILURE);
    }
  */

je vois pas pourquoi t a mis ca en commentaire

, je suppose que c'est juste un erreur, faut enlever les /*.

Ne raconte pas n'importe quoi sur ce forum, s'il te plait, si c'est pour répondre.

Ca fait 2 fois que je lis une c...rie de ta part en 2 posts..

Si tu ne sais pas, tu te te tais...

Ensuite...

Envoyé par corentin59

peut-être parce que ça ne marchait pas. Et de fait, il faut && et non ||

Et surtout que si x est un char, alors "c" est une chaine.....

Si on veut comparer à un char, c'est 'c'....

**corentin59** · 10/12/2007, 09h56

Envoyé par souviron34

Et surtout que si x est un char, alors "c" est une chaine.....

Si on veut comparer à un char, c'est 'c'

Ben je sais, regarde le code que j'ai posté avant de répondre trop vite...

**souviron34** · 10/12/2007, 14h03

Envoyé par corentin59

Ben je sais, regarde le code que j'ai posté avant de répondre trop vite...

oui mais ça n'est pas ce que tu dis ni dans tes commentaires ni comme raison...

Et pourtant c'est important...

**Thierry Chappuis** · 10/12/2007, 14h15

Envoyé par siegfried64

Code :

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

bool sign = true;

si tu travail sur c le type bool n'est pas defini, tu dois le definir toi meme.

bool est défini en C99 dans le fichier d'en-tête standard stdbool.h.

Thierry

**corentin59** · 10/12/2007, 14h21

Envoyé par ced236

Il y a plusieurs erreur dans ton code par exemple celle la "if (&x == "c")"
tu dis "si l adresse de x = c " tu devrais plutot remplacer par "if (x == 'c' )"

@ souviron34 : dans ce if, il y avait effectivement deux erreurs. La première a été signalée par ced236 et il me semble qu'on n'est pas censé répéter ce qui a été dit dans les posts précédents.

Problème dans mon main

C

Discussions similaires

Partager

Partager