Discussion :

[Nalido]

Salut,

J'ai essayer de mettre en pratique la définition d'une structure avec pointeur mais ca bug
voici une partie du code dans lequel je defini et initialise mes variables

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
 
 
typedef struct test{
        short    *outLtable;
        short    *outMtable;
        short    *outRtable;
        short    *outStable;
        double   *outDtable;
} test;
 
test *option ;
 
if (option = malloc(sizeof(test)) == NULL) {
        perror("malloc failed");
        exit(EXIT_FAILURE);
}
 
    option -> outDtable = NULL;
    option -> outLtable = NULL;
    option -> outMtable = NULL;
    option -> outRtable = NULL;
    option -> outStable = NULL;

il me dit uhhandle exception

pourtant il me semblait que c'est ainsi qu'il fallait initialiser. Le problem semble venir des lignes option -> outDtable = NULL; mais je ne comprens pas pourquoi.

merci pour votre aide

[Médinoc]

Eh bien je ne vois pas d'où peut venir l'erreur...

Tu peux essayer avec sizeof(*option) au lieu de sizeof(test), mais je ne vois pas ce que ça changerait...

[millie]

En principe on met la définition de la structure hors des fonctions. Ensuite essaye avec ça :

Code :

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

1
2
3
4
5
option = malloc(sizeof(test));
if (option == NULL) {
        perror("malloc failed");
        exit(EXIT_FAILURE);
}

Chez moi ça passe très bien.

Sinon, si malloc renvoit bien une valeur non nulle, il n'y aura aucun problème avec les assignations : option -> outDtable = NULL;

[edit] en principe avec malloc, on force la valeur de retour avec (test*) dans ce cas. (enfin ici, il n'y a qu'une variable, il n'y aura pas de problème, mais si tu déclares un tableau comme ça, il faut mettre un cast (test*)

[Foobar1329]

Hello,

Salut,

J'ai essayer de mettre en pratique la définition d'une structure avec pointeur mais ca bug
voici une partie du code dans lequel je defini et initialise mes variables

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
typedef struct test{
        short    *outLtable;
        short    *outMtable;
        short    *outRtable;
        short    *outStable;
        double   *outDtable;
} test;

test *option ;

/* if (option = malloc(sizeof(test)) == NULL) { */
if ( (option = malloc(sizeof(*option)) ) == NULL) {
        perror("malloc failed");
        exit(EXIT_FAILURE);
}

    option -> outDtable = NULL;
    option -> outLtable = NULL;
    option -> outMtable = NULL;
    option -> outRtable = NULL;
    option -> outStable = NULL;

il me dit uhhandle exception

pourtant il me semblait que c'est ainsi qu'il fallait initialiser. Le problem semble venir des lignes option -> outDtable = NULL; mais je ne comprens pas pourquoi.

merci pour votre aide

Warf !!

Ce qui est affecté à option, ce n'est pas la valeur de pointeur vers la mémoire allouée dynamiquement mais le résultat de l'expression (malloc(sizeof(*option)) == NULL), donc 1 ou 0. Regardes bien les parenthèses manquantes plus haut. PB de priorité d'opérateur. Conséquence supplémentaire, la zone allouée est perdue, une belle fuite mémoire.

Voilà donc une bonne raison pour ne pas mettre des affectations dans des expressions conditionnelles

A+

[Foobar1329]

Hello,

Eh bien je ne vois pas d'où peut venir l'erreur...

Tu peux essayer avec sizeof(*option) au lieu de sizeof(test), mais je ne vois pas ce que ça changerait...

C'est exactement le même problème que :

Code :

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

1
2
3
4
5
 
/*...*/
int c;
while (c = fgetc(truc) != EOF) { /*...*/ }
/*...*/

A+

[Nalido]

merci
ca fonctionne mieux comme ça!