Problème dans mon main

Version imprimable

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:

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 !

07/12/2007, 17h45
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' )"

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
07/12/2007, 17h52
corentin59
Citation:
Envoyé par pikaTuX

Code:

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
07/12/2007, 17h57
siegfried64

Suppression des réponses erronées
d'ailleur :
Code:

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:

bool sign = true;

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

Code:

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]

voici une base de départ.

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

07/12/2007, 18h09
corentin59
Citation:
Envoyé par siegfried64

d'ailleur :

Code:

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 ||
08/12/2007, 03h57
souviron34
@siegfried64 :
Citation:
Envoyé par siegfried64

d'ailleur :

Code:

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 /*.
:evilred:
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...

Citation:

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'....
10/12/2007, 09h56
corentin59

Citation:

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...
10/12/2007, 14h03
souviron34

Citation:

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...
10/12/2007, 14h15
Thierry Chappuis

Citation:

Envoyé par siegfried64

Code:

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
10/12/2007, 14h21
corentin59

Citation:

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.