gestionnaire de signets

**Delnir** · 08/12/2008, 22h02

Bonsoir,
je doit créer un gestionnaire de signets (URL,description et nombre d'étoile)
Le compilateur me sort une erreur de segmentation, je cherche mais ne trouve pas (le soucis vient d'affiche et erase).
la fonction affiche marche dans la création d'un signet mais pas dans le programme principal, je ne comprend pas du tout pourquoi ! ( je passe bien l'adresse de s donc s est modifié)

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
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
typedef struct _signet{
  char *url;
  char *info;
  int star;
}signet;
 
void init(signet *s){
  s->url=NULL;
  s->info=NULL;
  s->star=0;
}
void saisie(char *url,char *info,int *star){
  printf("saisir une adresse, la description et le nombre d'étoile:");
  scanf("%s %s %d",url,info,star);
}
void affiche(signet s){
  printf("%s\n",s.url);
  printf("%s\n",s.info);
  printf("%d\n",s.star);
}
void ajoute_signet(signet *s,char *url,char *info,int star){
  s=malloc(sizeof(signet));
  s->url=malloc(sizeof(char *)*strlen(url));;
  s->url=url;
  s->info=malloc(sizeof(char *)*strlen(info));
  s->info=info;
  s->star=star;
  /*affiche(*s) marche très bien ici*/
}
 
void erase(signet *s){
  free(s->url);
  free(s->info);
  free(s);
}
 
void modifie_signet(signet *s){
  char url[50];
  char info[50];
  int star;
  saisie(url,info,&star);
  s->url=realloc(s->url,sizeof(char *)*strlen(url));
  s->url=url;
  s->info=realloc(s->info,sizeof(char *)*strlen(info));
  s->info=info;
  s->star=star;
}
int main(void){
  signet s;
  init(&s);
  char url[50];
  char info[50];
  int star;
  saisie(url,info,&star);
  ajoute_signet(&s,url,info,star);
  affiche(s);/*affiche n'inporte quoi*/
  modifie_signet(&s);  
  affiche(s);
  erase(&s);/*erreur invalide pointer*/
  return 0;
}

Meric d'avance pour votre aide.

Invité(e) · 09/12/2008, 10h17

Bonjour,

Deux choses :

La fonction ajoute_signet commence par créer un signet... mais le garde pour elle : la mémoire allouée dans cette fonction n'est plus libérable.
Les copies de chaines de caractère ne se font pas avec = mais avec strcpy, strcat, strdup...

**3DArchi** · 09/12/2008, 10h44

Je rajouterais:
-> l'échec du réalloc n'est pas géré
-> Pourquoi dans ajoute_signet faire un malloc sur le paramètre signer alors que l'adresse passée en paramètre est celle d'une variable sur la pile?

**Delnir** · 09/12/2008, 16h35

ok merci, j'ai comprit l'erreur, j'ai modifié mon programme afin d'entrer plusieurs signets:
quand je compile il me sort:

signet.c: In function ‘cree_signet’:
signet.c:46: attention : return from incompatible pointer type
signet.c: In function ‘ajoute_signet’:
signet.c:51: erreur: invalid use of undefined type ‘struct signet’
signet.c:51: erreur: déréférencement d'un pointeur de type incomplet

Pourquoi mon return s ne renvoie pas ma création de signet?

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>
#include<string.h>
#include<ctype.h>
 
typedef struct _signet{
  char *url;
  char *info;
  int star;
}signet;
 
typedef struct _lessignets{
  struct signet *tab;
  int taille;
 
}signets;
 
void init2(signets *s){
  s->tab=NULL;
  s->taille=0;
}
void init(signet *s){
  s->url=NULL;
  s->info=NULL;
  s->star=0;
}
void saisie(char *url,char *info,int *star){
  printf("saisir une adresse, la description et le nombre d'étoile:");
  scanf("%s %s %d",url,info,star);
 
 
signet* cree_signet(char *url,char *info,int *star){
  signet s;
  init(&s);
  s=malloc(sizeof(signet));
  saisie(url,info,star);
  s.url=(char *)malloc(sizeof(char *)*strlen(url));;
  strcpy(s.url,url);
  s.info=(char *)malloc(sizeof(char *)*strlen(info));
  strcpy(s.info,info);
  s.star=*star;
  return s;
}
 
void ajoute_signet(signets *s2,char *url,char *info,int *star){
  s2->tab=realloc(s2->tab,sizeof(signet)*s2->taille+1);
  s2->tab[s2->taille]=cree_signet(url,info,star);
  s2->taille++;
}

**diogene** · 09/12/2008, 17h19

Code :

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

1
2
3
4
5
void saisie(char *url,char *info,int *star){
  printf("saisir une adresse, la description et le nombre d'étoile:");
  scanf("%s %s %d",url,info,star);
}
signet* cree_signet(char *url,char *info,int *star){

Code :

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

1
2
3
4
5
6
7
8
9
signet* cree_signet(char *url,char *info,int *star){
...
//  s.url=(char *)malloc(sizeof(char *)*strlen(url));; 
// sizeof(char), pas sizeof(char*)
// strlen(url)+1  pour le zéro terminal 
  s.url=malloc(strlen(url)+1); 
...
//  s.info=(char *)malloc(sizeof(char *)*strlen(info));
// idem

**Delnir** · 09/12/2008, 18h35

Je cherche mais pas moyen de trouver mon erreur : Help

signet.c: In function ‘ajoute_signet’:
signet.c:52: erreur: invalid use of undefined type ‘struct signet’
signet.c:52: erreur: déréférencement d'un pointeur de type incomplet
tab est un pointeur sur un struct signet donc logiquement je peut le faire pointer sur un struct signet qui est créer_signet !!

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
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<ctype.h>
 
typedef struct _signet{
  char *url;
  char *info;
  int star;
}signet;
 
typedef struct _lessignets{
  struct signet *tab;
  int taille;
 
}signets;
 
void init2(signets *s){
  s->tab=NULL;
  s->taille=0;
}
void init(signet *s){
  s->url=NULL;
  s->info=NULL;
  s->star=0;
}
void saisie(char *url,char *info,int *star){
  printf("saisir une adresse, la description et le nombre d'étoile:");
  scanf("%s %s %d",url,info,star);
}
void affiche(signet *s){
  printf("%s\n",s->url);
  printf("%s\n",s->info);
  printf("%d\n",s->star);
}
 
signet cree_signet(char *url,char *info,int *star){
  signet s;
  init(&s);
  saisie(url,info,star);
  s.url=(char *)malloc(sizeof(char)*strlen(url)+1);
  strcpy(s.url,url);
  s.info=(char *)malloc(sizeof(char)*strlen(info)+1);
  strcpy(s.info,info);
  s.star=*star;
  return s;
}
 
 
void ajoute_signet(signets *s2,char *url,char *info,int *star){
  s2->tab=realloc(s2->tab,sizeof(signet)*s2->taille+1);
  s2->tab[s2->taille]=cree_signet(url,info,star); /*ERREUR ICI*/
  s2->taille++;
}
 
 
int main(void){
  signet s;
  signets s2;
  init2(&s2);
  char url[50];
  char info[50];
  int star;
  ajoute_signet(&s2,url,info,&star);
  return 0;
}

**diogene** · 09/12/2008, 20h19

signet.c:52: erreur: invalid use of undefined type ‘struct signet’

Ce qui est défini c'est :
1- struct _signet
et
2- signet
mais pas struct signet :

Code :

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

1
2
3
4
typedef struct _lessignets{
  signet *tab;
  int taille; 
}signets;

**Delnir** · 09/12/2008, 20h44

ha ..... je cherchai pas l'erreur au bon endroit, sa marche merci !

**Delnir** · 09/12/2008, 23h00

j'ai un problème dans une de mes fonctions:
voici ce que donne le compilateur:
signet.c:74: attention : comparaison entre élément signé et élément non signé
signet.c:74: attention : comparaison entre élément signé et élément non signé

Code :

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

1
2
 
while((test==1) && (cpt<strlen(m2.info)) && (cpt<strlen(m1.info))){

le problème ce situe ici, j'ai fait des test avec un printf sur les strlen et y'a bien un entier
donc je compare bien des entiers ! (je supose que le strlen est un long int ou truc du genre mais alors pourquoi

)

voici la fonction:

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
 
void trie_signet(signets *s2){
  int i,j,test,cpt;
  signet m1,m2,save;
  if(s2->taille<2)
    return;
  for(i=0;i<s2->taille-1;i++){
    m1=s2->tab[i];
    for(j=i+1;j<s2->taille;j++){
      m2=s2->tab[j];
      if(m2.star<m1.star){
	save=s2->tab[i];
	s2->tab[i]=s2->tab[j];
	s2->tab[j]=save;
      }
      else{
	if(m2.star==m1.star){
	  test=1;
	  cpt=0;
	  while((test==1) && (cpt<strlen(m2.info)) && (cpt<strlen(m1.info))){
	    if(m2.info[cpt]<m1.info[cpt]){
	      save=s2->tab[i];
	      s2->tab[i]=s2->tab[j];
	      s2->tab[j]=save;
	      test=0;
	    }
	    else{
	      if(m2.info[cpt]>m2.info[cpt])
		test=0;
	    }
	  }
 
	}
      }
    }
  }
}

le programme entier si sa peut servir:

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
110
111
112
113
114
115
116
117
118
119
120
121
122
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<ctype.h>
 
typedef struct _signet{
  char *url;
  char *info;
  int star;
}signet;
 
typedef struct _lessignets{
  signet *tab;
  int taille;
 
}signets;
 
void init2(signets *s){
  s->tab=NULL;
  s->taille=0;
}
void init(signet *s){
  s->url=NULL;
  s->info=NULL;
  s->star=0;
}
void saisie(char *url,char *info,int *star){
  printf("saisir une adresse, la description et le nombre d'étoile:");
  scanf("%s %s %d",url,info,star);
}
void affiche(signet s){
  printf("Site:%s\n",s.url);
  printf("Description du site:%s\n",s.info);
  printf("nombre d'étoile:%d\n",s.star);
}
 
 signet cree_signet(char *url,char *info,int *star){
  signet s;
  init(&s);
  saisie(url,info,star);
  s.url=(char *)malloc(sizeof(char)*strlen(url)+1);
  strcpy(s.url,url);
  s.info=(char *)malloc(sizeof(char)*strlen(info)+1);
  strcpy(s.info,info);
  s.star=*star;
  return s;
}
 
 
void ajoute_signet(signets *s2,char *url,char *info,int *star){
  s2->tab=realloc(s2->tab,sizeof(signet)*s2->taille+1);
  s2->tab[s2->taille]=cree_signet(url,info,star);
  s2->taille++;
}
 
void trie_signet(signets *s2){
  int i,j,test,cpt;
  signet m1,m2,save;
  if(s2->taille<2)
    return;
  for(i=0;i<s2->taille-1;i++){
    m1=s2->tab[i];
    for(j=i+1;j<s2->taille;j++){
      m2=s2->tab[j];
      if(m2.star<m1.star){
	save=s2->tab[i];
	s2->tab[i]=s2->tab[j];
	s2->tab[j]=save;
      }
      else{
	if(m2.star==m1.star){
	  test=1;
	  cpt=0;
	  while((test==1) && (cpt<strlen(m2.info)) && (cpt<strlen(m1.info))){
	    if(m2.info[cpt]<m1.info[cpt]){
	      save=s2->tab[i];
	      s2->tab[i]=s2->tab[j];
	      s2->tab[j]=save;
	      test=0;
	    }
	    else{
	      if(m2.info[cpt]>m2.info[cpt])
		test=0;
	    }
	  }
 
	}
      }
    }
  }
}
 
 
 
 
 
void affiche_all(signets s){
  int i;
  for(i=0;i<s.taille;i++){
    affiche(s.tab[i]);
    printf("\n");
  }
}
 
 
int main(void){
  signet s;
  signets s2;
  init2(&s2);
  char url[50];
  char info[50];
  int star;
  ajoute_signet(&s2,url,info,&star);
  ajoute_signet(&s2,url,info,&star);
  affiche_all(s2);
  printf("Gestionnaire de signets triés:\n");
  printf("\n");
  trie_signet(&s2);
  affiche_all(s2);
  return 0;
}

merci d'avance pour votre aide.

Invité(e) · 10/12/2008, 09h11

Bonjour,

strlen renvoie un type size_t qui est un entier non signé.

cpt est de type int, donc signé.

pour faire disparaitre le warning, il suffirait de déclarer cpt comme size_t. Attention dans ce cas que la conception du programme le permette.

Au passage, je n'ai pas l'impression que cpt change beaucoup dans le programme.

**Delnir** · 10/12/2008, 17h07

Merci pour l'aide j'ai corrigé avec un (int) strlen et l'oublie sur le cpt++

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
void trie_signet(signets *s2){
  int i,j,test,cpt;
  signet m1,m2,save;
  if(s2->taille<2)
    return;
  for(i=0;i<s2->taille-1;i++){
    m1=s2->tab[i];
    for(j=i+1;j<s2->taille;j++){
      m2=s2->tab[j];
      if(m2.star<m1.star){
	save=s2->tab[i];
	s2->tab[i]=s2->tab[j];
	s2->tab[j]=save;
      }
      else{
	if(m2.star==m1.star){
	  test=1;
	  cpt=0;
	  while((test==1) && (cpt<(int)strlen(m2.info)) && (cpt<(int)strlen(m1.info))){
	    if(m2.info[cpt]<m1.info[cpt]){
	      save=s2->tab[i];
	      s2->tab[i]=s2->tab[j];
	      s2->tab[j]=save;
	      test=0;
	    }
	    else{
	      if(m2.info[cpt]>m2.info[cpt])
		test=0;
	    }
            cpt++;
	  }
 
	}
      }
    }
  }
}

gestionnaire de signets

C

Discussions similaires

Partager

Partager