Discussion :

[kernerteam]

Salut tout le monde

J'ai un probleme. Je souhaite comparer deux chaines de caractére l'un de type char * string l'autre contenu dans une structure. strcmp ne semble pas accéder au champs de la structure, et gcc me renvoie une erreur du genre qu'il ne peut pas comparer une chaine de caracteres et un pointeur... Alors comment faire ???? Voila je vous remercie!!

[David.Schris]

Alors comment faire ????

Deux options :

• soit tu nous montres ton code
• soit tu nous offres une ...

[_kal_]

Exact, on ne peut rien dire si on ne voit pas ton code

[kernerteam]

Voila mon code comme vous me l'avez demander.

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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
 
 
 
 
 
//Pour l'instant tout est fait de façon trés naive
//Je n'ai pas fait un programme modulaire exprés
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define L 23//Longeur du tableau
 
 
int 
main(void){
 
    //Tableau de pointeurs pour les noms de station
    char *stat[L];
 
    stat[0] = "la_defense";
    stat[1] = "esplanade_de_la_defense";
    stat[2] = "pont_de_neuilly";
    stat[3] = "les_sablons";
    stat[4] = "porte_maillot";
    stat[5] = "argentine";
    stat[6] = "charles_de_gaule_etoile";
    stat[7] = "georgesV";
    stat[8] = "franklin_d.roosvelt";
    stat[9] = "champs_elysees_clemenceau";
    stat[10] = "concorde";
    stat[11] = "tuileries";
    stat[12] = "palais_royale_musee_du_louvre";
    stat[13] = "louvre_rivoli";
    stat[14] = "chatelet";
    stat[15] = "hotel_de_ville";
    stat[16] = "saint_paul";
    stat[17] = "bastille";
    stat[18] = "gare_de_lyon";
    stat[19] = "reully_diderot";
    stat[20] = "nation";
    stat[21] = "porte_de_vincennes";
    stat[22] = "saint_mande";
    stat[23] = "berault";
    stat[24] = "chateau_de_vincennes";
 
    int i = 0;
    int j = 0;
 
    //La structure pour la liste chainé
    typedef struct list
    {
	int ligne;
	char *station;
	struct list *next;
    }List;
 
    List *t;
    List *courant;
    List *q;
 
    t = (List*)malloc(sizeof(List));
    t->ligne = 2;
    t->station = stat[i];
    t->next = NULL;
    courant = t;
 
    while(i<L)
    {
	q = (List*)malloc(sizeof(List));
	q->ligne = 1;
	q->station = stat[i+1];
	q->next = NULL;
 
	t->next = q;
	t = q;
	i++;
    }
 
    while(j<L)
    {
	printf("*** %s ***\n",courant->station);
	courant = courant->next;
	j++;
    }
 
 
    char nom_station[BUFSIZ];
 
    puts("\n--------------------------\n");
    puts("Verifier une station!\n");
    puts("Pour cela ecrivez le nom d'une station quelqueconque.\n");
    puts("Le programme vous repondra par oui ou par non.\n");
 
    scanf("%s",nom_station);
 
    while(courant != NULL)
    {
	//Là je veux comparer le nom de station entré par l'utilisateur au nom de station present dans les differents "wagons" de la liste
	if( (strcmp(nom_station,courant->station) > 0))//Si ils sont différents ils me renvoie NON
       {
	   printf("NOM");
       }
      else("OUI");
	courant = courant->next;//Pour pouvoir passer à l'autre "wagon"
 
    }
 
    return EXIT_SUCCESS;
}

Voila j'espere avoir été assez claire cette fois. Merci

[Médinoc]

1. Pas de scanf("%s",nom_station);. Très mauvais.
2. strcmp() peut retourner une valeur inférieure à zéro si les chaînes sont différentes. Utilise != au lieu de >.
3. Pas de cast au retour de malloc() : c'est inutile en C (quoi que puissent en dire des vieux professeurs).
4. BUFSIZ ne sert pas à cela. Utilise un #define perso pour ta taille de buffer.

[FamiDoo]

stat[0] = "la_defense";
stat[1] = "esplanade_de_la_defense";
stat[2] = "pont_de_neuilly";

ca se fait ca ??

Pour comparer 2 chaines, tu peux utiliser la fonction strcmp ou stricmp( ca dépend si tu veux prendre en compte la casse), mais pour cela il faut que cela soit 2 char *, il faut donc que tu fasses un cast vers char * de ton string.

[millie]

ca se fait ca ??

Sur le coup, j'eus cru que ça n'allait pas marché, mais en fait, si: c'est bien un
char * truc[L] donc ça marche.

[Franck.H]

Pour fair mieux tu peux aussi virer la structure du main, ca se fait pas vraiment, surtout si en plus tu en fait une petite liste chainée !

[thewho]

Bonjour,

Sans aller très loin, je vois au moins 2 erreurs

Code :

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

#define L 23//Longeur du tableau

Ne jamais faire ça ! : le commentaire qui suit la valeur numérique est inclu dans la définition !!!

donc, plus tard quand tu écris

Code :

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

char *stat[L];

ça revient à écrire

Code :

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

char *stat[23//Longeur du tableau];

et la fin de ta ligne est commentée, comment le compilateur pourrait-il s'y retrouver ?

D'autre part, tu as mis L = 23
or, plus loin, je vois

Code :

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

stat[24] = "chateau_de_vincennes";

il faut donc que L = 25

[Médinoc]

Tu es sûr de ton post, là ?

Je n'ai jamais eu ce problème ni sous Visual ni sous GCC...
Edit: Peut-être que si l'option "keep comments" du préprocesseur de GCC est activée... Je n'ai pas testé cela...

[fearyourself]

Bonjour,

Sans aller très loin, je vois au moins 2 erreurs

Code :

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

#define L 23//Longeur du tableau

Ne jamais faire ça ! : le commentaire qui suit la valeur numérique est inclu dans la définition !!!

Ah bon ?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#include <stdio.h>
 
#define L 23//hello
 
int main()
{
        printf("%d\n",L);
        return 0;
}

Me sors :

~/tmp$ gcc define.c -Wall -Wextra
~/tmp$ ./a.out
23

et gcc -E me sors bien :

Code :

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

1
2
3
4
5
6
 
int main()
{
 printf("%d\n",23);
 return 0;
}

[EDIT]
En effet, l'option -CC garde les commentaires mais les transforme en /**/
ce qui donne :

gcc -E -CC define.c


int main()
{
printf("%d\n",23/*hello*/);
return 0;
}

[/EDIT]
Jc

[thewho]

Bonjour,

Ok, j'ai testé et ça marche.

Je ne fais jamais ça, et je n'avais jamis essayé, car on m'a toujours appris que "dans un #define tout ce qui suit l'identificateur est inclu dans la définition"
Il est possible que les anciens compilateurs n'aient pas su jouer avec les commentaires ajoutés sur la même ligne.

Il reste au moins le problème de la valeur de L

[Lunixinclar]

Bonjour, j'ai une question sur une partie du code qui se trouve dans le main() :

while(i<L)
{
q = (List*)malloc(sizeof(List));
q->ligne = 1;
q->station = stat[i+1];
q->next = NULL;

t->next = q;
t = q;
i++;
}

Est-ce que free() est implicite à la sortie de la fonction?

[Trap D]

Ben non, le free n'est JAMAIS implicite en C.

[Médinoc]

Et en C, le cast au retour de malloc() doit être supprimé.
Il n'est pas en soi nuisible, à cela près qu'il peut cacher un oubli d'inclusion de <stdlib.h> si on compile sans warnings.

[Lunixinclar]

Oui voilà comment avec 50 lignes de codes on se retrouve avec 4Ko de fuite dans la mémoire sur chaque exécution.

Chaque allocation dynamique de mémoire demande une libération manuelle avec free().
Ca personne ne le fera à la place du programmeur, c'est une des très rares choses de sa responsabilité vis à vis du système d' exploitation, en langage C.

Besoin d 'aide, incontestablement. Ca ne s'invente pas.
En se basant exclusivement sur ce que tu nous montres, le prog se résume à quelques lignes:

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
61
62
63
64
#include <stdio.h>
#include <string.h>
 
#define L 25
 
int main()
{
  char strin [256];
  unsigned char n, m=0;
 
  typedef struct
  {
    unsigned int ligne;
    char *station;
  } uneListe;
 
  uneListe maListe[L]={
            {1,"la_defense"},
            {1,"esplanade_de_la_defense"},
            {1,"pont_de_neuilly"},
            {1,"les_sablons"},
            {1,"porte_maillot"},
            {1,"argentine"},
            {1,"charles_de_gaule_etoile"},
            {1,"georgesV"},
            {1,"franklin_d.roosvelt"},
            {1,"champs_elysees_clemenceau"},
            {1,"concorde"},
            {1,"tuileries"},
            {1,"palais_royale_musee_du_louvre"},
            {1,"louvre_rivoli"},
            {1,"chatelet"},
            {1,"hotel_de_ville"},
            {1,"saint_paul"},
            {1,"bastille"},
            {1,"gare_de_lyon"},
            {1,"reully_diderot"},
            {1,"nation"},
            {1,"porte_de_vincennes"},
            {1,"saint_mande"},
            {1,"berault"},
            {1,"chateau_de_vincennes"}
  };
 
  puts("\n--------------------------\n");
  puts("Verifier une station!\n");
  puts("Pour cela ecrivez le nom d'une station quelconque.\n");
 
  gets (strin);
 
  for (n=0; n<25; n++)
  {
    if (strcmp(strin, maListe[n].station)==0)
    {
      m=1;
      break;
    }
  }
 
  if (m) printf("%s existe.\n",strin);
  else   printf("%s n'existe pas.\n",strin);
 
  return 0;
}

gets() est dangereux, lui aussi. Quelqu'un indiquera la méthode adéquate.

[Trap D]

Ouh le vilain paresseux

gets() est dangereux, lui aussi. Quelqu'un indiquera la méthode adéquate.

Il suffisait d'écrire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
fgets(strin, sizeof(strin), stdin);
{
  char *p;
  if ((p = strchr(strin, '\n')) != NULL)
    *p = 0;
  else
  {
     // buffer de saisie trop court
     // a voir ce qu'on veut faire
  }
}

En supposant que strin est un tableau de caractères et non un pointeur vers une chaîne de caractères.

[Lunixinclar]

lol Trap, en bon marseillais ça me fait toujours plaisir de donner un coup de main aux informaticiens de la RATP.

[Trap D]

Reste Assis T'es Payé

C'est ça

[Foobar1329]

C'est ça

Y'a ça aussi : Rentre Avec Tes Pieds

A+