Structure, pointeur et fonction...

**Linaa** · 01/12/2005, 22h05

Bonsoir,

J'ai un problème avec mon programme.
Alors j'ai un .h avec

Code :

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

1
2
3
4
5
6
7
8
9
struct CLIENT {
  int * t_num;
  float * t_achats;
  int nb_max_clients;
};
 
typedef struct CLIENT TAB_CLIENTS;
 
void init (int, TAB_CLIENTS *);

C'est le fait de passer ma variable structure en pointeur à la fonction init qui pose problème!

Ma fonction init :

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
void init (int max_clients, TAB_CLIENTS * cli) {
 
  // declarations
  int i, num[max_clients];
  float achats[max_clients];
 
  cli->nb_max_clients=max_clients;
 
  // initialisation des numeros
  for(i=0;i<max_clients;i++)
    num[i]=0;
  cli->t_num = num;
 
  // initialisation des achats
  for(i=0;i<max_clients;i++)
    achats[i]=0.0;
  cli->t_achats = achats;
 
  // test d'affichage
  for(i=0;i<cli->nb_max_clients;i++)
    printf("n° %d : %f\n",cli->t_num[i],cli->t_achats[i]);
 
  printf("\n\n");
}

Avec le test d'affichage, tout est ok, ma variable structure est bien initialisée!

Le problème est dans mon main

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int main () {
  // declarations
  int i;
  TAB_CLIENTS les_clients;
 
  // appel a la fonction
  init(10,&les_clients);
 
  // test d'affichage qui foire!
  for(i=0;i<les_clients.nb_max_clients;i++)
    printf("n° %d : %f\n",les_clients.t_num[i],les_clients.t_achats[i]);
 
  return 0;
}

La variable les_clients que je recupère devrait etre initialisée, mais en fait elle l'est à moitié

Code :

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

1
2
3
4
5
6
7
8
9
10
n° 0 : 0.000000
n° 32 : 0.000000
n° 1107378272 : 0.000000
n° 1108287296 : 0.000000
n° 0 : 0.000000
n° 0 : 0.000000
n° 0 : 0.000000
n° 0 : 0.000000
n° 0 : 0.000000
n° 0 : 0.000000

J'ai réduit le nombre max, mais avec 30 par ex, c'est pire ! Il y a meme des nombres negatifs qui apparaissent!
Je ne vois pas où est l'erreur ?! Si vous pouvez m'éclairer.

Merci

**Médinoc** · 01/12/2005, 22h13

euh... TU passes à ta structure l'adresse de variables locales automatiques: quand tu quittes la fonction init(), c'est comme si elles n'existaient plus --> comportement indéfini, le plus souvent des trucs pas bien.

**Linaa** · 01/12/2005, 22h18

Comment puis-je faire alors pour initialiser mes 2 tableaux ?

J'ai essayé ceci

Code :

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

1
2
3
4
5
for(i=0;i<max_clients;i++)
    cli->t_num[i] = 0;
 
  for(i=0;i<max_clients;i++)
    cli->t_achats[i] = 0.0;

Mais cela ne fonctionne pas

Code :

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

Erreur de segmentation

**Pouic** · 01/12/2005, 22h30

Envoyé par Linaa

Comment puis-je faire alors pour initialiser mes 2 tableaux ?

J'ai essayé ceci

Code :

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

1
2
3
4
5
for(i=0;i<max_clients;i++)
    cli->t_num[i] = 0;
 
  for(i=0;i<max_clients;i++)
    cli->t_achats[i] = 0.0;

Mais cela ne fonctionne pas

Code :

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

Erreur de segmentation

Ben c'est normal, il faut quand même allouer de la mémoire pour stocker ces valeurs....

**Linaa** · 01/12/2005, 22h32

Envoyé par Pouic

Ben c'est normal, il faut quand même allouer de la mémoire pour stocker ces valeurs....

Euh... quelles valeurs ??

**Pouic** · 01/12/2005, 22h35

Envoyé par Linaa

Envoyé par Pouic

Ben c'est normal, il faut quand même allouer de la mémoire pour stocker ces valeurs....

Euh... quelles valeurs ??

Dans ta struct CLIENT, tes pointeurs ne sont pas initialisés... Et après, tu espère mettre des valeurs via tes boucles...

Code :

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

cli->t_num[i] = 0;

Ceci ne peut que planter si tu n'as pas fait le malloc qui va bien auparavant...

**Linaa** · 01/12/2005, 22h45

Je vois...

Mais dans ma structure CLIENT

Code :

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

1
2
3
4
5
struct CLIENT {
  int * t_num;
  float * t_achats;
  int nb_max_clients;
};

t_num et t_achats sont des tableaux dont je veux fixer la taille à max_clients, qui peut varier.
C'est pour cela que je passe par des pointeurs, sinon j'aurais fait

Code :

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

1
2
3
4
5
struct CLIENT {
  int  t_num[10];
  float t_achats[10];
  int nb_max_clients;
};

En fait j'voudrais un pointeur sur un tableau de taille max_clients.
Ce que j'ai fait au début, mais à la sortie de la fonction, plus rien...

**Pouic** · 01/12/2005, 22h48

Envoyé par Linaa

Je vois...

Mais dans ma structure CLIENT

Code :

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

1
2
3
4
5
struct CLIENT {
  int * t_num;
  float * t_achats;
  int nb_max_clients;
};

t_num et t_achats sont des tableaux dont je veux fixer la taille à max_clients, qui peut varier.

Et qu'est ce qui te pose problème avec malloc ??

Code :

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

1
2
3
4
5
6
7
 
struct CLIENT t;
t.max_clients = 10;
t.t_num = malloc (t.max_clients);
if (!t.t_num){
....
}

:

**Linaa** · 01/12/2005, 23h12

En fait j'y pensais pas au malloc!
Mais j'ai pas trop compris, ça alloue une taille de 10 octets (dans l'exemple) c'est ça ??
Sinon, ça fonctionne.

Mais là, j'ai aussi un problème de free
En faisant dans le main

Code :

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

free (les_clients.t_num);

Me suis-je trompée ?

**Jack_serious** · 02/12/2005, 00h19

Envoyé par Linaa

En fait j'y pensais pas au malloc!
Mais j'ai pas trop compris, ça alloue une taille de 10 octets (dans l'exemple) c'est ça ??
Sinon, ça fonctionne.

Mais là, j'ai aussi un problème de free
En faisant dans le main

Code :

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

free (les_clients.t_num);

Me suis-je trompée ?

Ca te fais quoi comme erreur ? chunk pointer ou not allocated memory ? ou pire ?

**Linaa** · 02/12/2005, 13h21

Pire !! Enfin, je ne sais pas à quoi cela peut etre dû

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x0804a008 ***
Abandon

**Emmanuel Delahaye** · 02/12/2005, 13h35

Envoyé par Linaa

Pire !! Enfin, je ne sais pas à quoi cela peut etre dû

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x0804a008 ***
Abandon

J'ai un peu du mal à suivre.

Montre le code complet, compilable mais réduit au minimum qui montre le problème (avec malloc() / free()).

**Linaa** · 02/12/2005, 13h40

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
void init (int max_clients, TAB_CLIENTS * cli) {
 
  int i, num[max_clients];
  float achats[max_clients];
 
  cli->nb_max_clients=max_clients;
 
  [b]cli->t_num = malloc (max_clients);[/b]
  for(i=0;i<max_clients;i++)
    cli->t_num[i]=0;
 
  [b]cli->t_achats = malloc (max_clients);[/b]
  for(i=0;i<max_clients;i++)
    cli->t_achats[i]=0.0;
 
  printf("\n\n");
}
 
int main () {
 
  int i;
  TAB_CLIENTS les_clients;
 
  init(10,&les_clients);
 
  les_clients.t_num[0] = 1;
  les_clients.t_achats[0] = 23.4;
 
  for(i=0;i<les_clients.nb_max_clients;i++)
    printf("n° %d : %f\n",les_clients.t_num[i],les_clients.t_achats[i]);
 
  [b]free (les_clients.t_num);
  free (les_clients.t_achats);[/b]
 
  return 0;
}

La structure est celle de mon premier post

**Emmanuel Delahaye** · 02/12/2005, 13h51

Envoyé par Linaa

La structure est celle de mon premier post

Ok.

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
 
main.c:18: error: implicit declaration of function `malloc'
main.c:18: warning: nested extern declaration of `malloc'
<internal>:0: warning: redundant redeclaration of 'malloc'
main.c:18: warning: passing arg 1 of `malloc' as unsigned due to prototype
main.c:22: warning: passing arg 1 of `malloc' as unsigned due to prototype
main.c:26: error: implicit declaration of function `printf'
main.c:26: warning: nested extern declaration of `printf'
<internal>:0: warning: redundant redeclaration of 'printf'
main.c:13: warning: unused variable `num'
main.c:14: warning: unused variable `achats'
main.c: At top level:
main.c:29: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:40: warning: nested extern declaration of `printf'
<internal>:0: warning: redundant redeclaration of 'printf'
main.c:42: error: implicit declaration of function `free'
main.c:42: warning: nested extern declaration of `free'

Merci de poster du code COMPILABLE.

Ton code fonctionne, mais les données sont mal organisées. Je propose ça :

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
 
/* x.h */
#ifndef H_X
#define H_X
 
struct client
{
    int num;
    float achats;
};
 
struct TAB_CLIENTS
{
    struct client *a;
    size_t nb;
};
 
int init (size_t nb, struct TAB_CLIENTS *);
 
#endif

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
 
/* x.c */
#include <stdlib.h>
 
#include "x.h"
 
int init (size_t max_clients, struct TAB_CLIENTS *p_cli)
{
    int err=0;
    int i;
 
    p_cli->nb=max_clients;
 
    p_cli->a = malloc (max_clients);
 
    if (p_cli->a != NULL)
    {
 
        for(i=0;i<p_cli->nb;i++)
        {
            p_cli->a[i].num=0;
            p_cli->a[i].achats=0.0;
        }
    }
    else
    {
        err=1;
    }
    return err;
}

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
 
#include "x.h"
 
#include <stdio.h>
#include <stdlib.h>
 
int main (void)
{
    struct TAB_CLIENTS les_clients = {0};
 
    init(10,&les_clients);
 
    les_clients.a[0].num = 1;
    les_clients.a[0].achats = 23.4;
 
    {
        int i;
        for(i=0;i<les_clients.nb;i++)
        {
            printf("%d : %f\n"
            ,les_clients.a[i].num
            ,les_clients.a[i].achats);
        }
    }
    free (les_clients.a), les_clients.a=NULL;
 
    return 0;
}

**Linaa** · 02/12/2005, 13h55

Code :

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

1
2
3
#include <stdio.h>
#include "client.h"
#include <stdlib.h>

Dans le fichier client.h (la structure + prototype de la fonction)

Code :

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

1
2
3
4
5
6
7
8
9
struct CLIENT {
  int * t_num;
  float * t_achats;
  int nb_max_clients;
};
 
typedef struct CLIENT TAB_CLIENTS;
 
void init (int, TAB_CLIENTS *);

**Linaa** · 04/12/2005, 13h12

Envoyé par Emmanuel Delahaye

Ton code fonctionne, mais les données sont mal organisées.

Mais pourquoi de mon côté ça ne fonctionne pas ?!
A quoi correspond cette erreur ?

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x0804a008 ***
Abandon

Structure, pointeur et fonction...

C

Discussions similaires

Partager

Partager