[C] pointeurs de pointeurs [RESOLU]

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] pointeurs de pointeurs [RESOLU]

Noobs69

Bonjour,

J'étais en train d'écrire un petit programme de test pour bien comprendre les principes des pointeurs en C en i686 en gcc, tout se compile bien, les données sont lisibles dans les fonctions, mais plus à leur sortie, je ne comprend pas mon erreur.

Pouvez-vous me corriger ce code ?
Merci.

settings.h

Code :

#ifndef __SETTINGS_H__
#define __SETTINGS_H__
typedef struct __test {
int a;
char b;
char c[20];
char *d;
} _TEST;
void setinit(_TEST *);
#endif

settings.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "settings.h"
void setinit(_TEST *test)
{
test=malloc(sizeof(_TEST));
if (test != NULL)
{
test->a=34;
test->b='B';
sprintf(test->c, "%s", "test c ok" );
test->d=malloc((strlen("test d ok" ) + 1) * sizeof(char));
if (test->d != NULL)
test->d="test d ok";
printf("setinit [%d]\n", test->a);
printf("setinit [%c]\n", test->b);
printf("setinit [%s]\n", test->c);
printf("setinit [%s]\n", test->d);
}
}

main.h

Code :

#ifndef __MAIN_H__
#define __MAIN_H__
typedef struct __essai {
int a;
char b;
char c[20];
char *d;
} _ESSAI;
#endif

main.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "main.h"
#include "settings.h"
_TEST *test;
_ESSAI *essai;
void maininit(_ESSAI *essai)
{
essai=malloc(sizeof(_ESSAI));
if (essai != NULL)
{
essai->a=12;
essai->b='b';
sprintf(essai->c, "%s", "essai c ok" );
essai->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
if (essai->d != NULL)
essai->d="essai d ok";
printf("maininit [%d]\n", essai->a);
printf("maininit [%c]\n", essai->b);
printf("maininit [%s]\n", essai->c);
printf("maininit [%s]\n", essai->d);
}
}
int main()
{
setinit(test);
maininit(essai);
if (test != NULL)
{
printf("test [%d]\n", test->a);
printf("test [%c]\n", test->b);
printf("test [%s]\n", test->c);
printf("test [%s]\n", test->d);
}
if (essai != NULL)
{
printf("essai [%d]\n", essai->a);
printf("essai [%c]\n", essai->b);
printf("essai [%s]\n", essai->c);
printf("essai [%s]\n", essai->d);
}
free(essai);
essai=NULL;
free(test);
test=NULL;
return(0);
}

Le résultat d'exécution donne ceci :

Code :

setinit [34]
setinit [B]
setinit [test c ok]
setinit [test d ok]
maininit [12]
maininit [b]
maininit [essai c ok]
maininit [essai d ok]

Il manque donc bien les lignes d'impression du main.

Merci pour votre aide

Message édité par Noobs69 le 25-04-2011 à 18:18:30

Publicité

WiiDS

Connardophobe

Si je comprends bien (tu aurais pu faire plus court comme test )

Tu espères pouvoir modifier tes pointeurs test et essai juste en les passant dans une fonction qui les assigne au résultat de malloc ?

Ca ne peut pas marcher pour une raison simple: Quand tu fais passer un paramètre à une fonction, cette fonction va en réalité travailler avec une copie de ce paramètre. A la fin de ta fonction, ce paramètre va donc être détruit et évidemment tu auras donc perdu le pointeur que t'a retourné malloc.

Si tu voulais pouvoir modifier ce pointeur "a distance", il faudrait que tu fasses passer en paramètre l'adresse du pointeur (donc setinit(&essai) et maininit(&test)), que tu modifies bien évidemment le prototype de ces fonctions en conséquence et une fois cela fait, tu aurais alors un pointeur sur pointeur.

Ce que ça change ? Tu pourras alors modifier l'adresse du pointeur essai et test en faisant une assignation toute simple comme *essai = NULL; ou *test = malloc() ou ce que tu veux

J'espère avoir répondu à ta question :jap:

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray | Darky, enfoiré :( | Pseudonyme TF2 : [T4D] Jer0m3_K3rv1el

Noobs69

Merci mais je n'arrive pas à compiler correctement à présent.

J'ai fait ça comme modifs :

settings.h

Code :

...
void setinit(_TEST **);
...

settings.c

Code :

...
void setinit(_TEST **test)
{
*test=malloc(sizeof(_TEST));
if (test != NULL)
{
test->a=34;
...

main.c

Code :

...
void maininit(_ESSAI **essai)
{
*essai=malloc(sizeof(_ESSAI));
if (essai != NULL)
{
essai->a=12;
...
int main()
{
setinit(&test);
maininit(&essai);
...

Je vois bien que l'erreur est dans l'affectation à présent mais je ne vois pas comment écrire ça correctement.

Message édité par Noobs69 le 25-04-2011 à 10:35:37

WiiDS

Connardophobe

Pour te simplifier les choses (tu pourras comprendre après ): Tu as bien modifié le prototype, par contre dans ta fonction à chaque fois que tu as "test" ou "essai" il faut qu tu remplaces par *test et par *essai (donc tu mets une étoile chaque fois, ce n'est pas l'adresse du pointeur de pointeur que tu veux modifier mais l'adresse du premier pointeur)

Donc par exemple, tu remplaces if (essai != NULL) par if (*essai != NULL) etc ...

Message édité par WiiDS le 25-04-2011 à 10:42:12

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray | Darky, enfoiré :( | Pseudonyme TF2 : [T4D] Jer0m3_K3rv1el

Noobs69

Je creuse pour comprendre pourtant !!

Ca fonctionne pour malloc et le test, mais je ne comprend pas la syntaxe à adopter pour l'affectation :

Code :

essai->a=12; //???
*essai->a=12; //???
essai[0]->a=12; //CA, ça compile, mais j'y crois pas du tout !!
*essai->b='b';

Je me prend des :
error: request for member `a' in something not a structure or union

Message édité par Noobs69 le 25-04-2011 à 10:47:11

WiiDS

Connardophobe

Tu as essayé (*essai)->a=12; ? Faut faire gaffe à l'endroit ou on met les *, faut pas déréférencer n'importe quoi

Parce que je pense qu'en faisant *essai->a = 12, ton compilateur doit comprendre essai->*a = 12

Message édité par WiiDS le 25-04-2011 à 11:04:45

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray | Darky, enfoiré :( | Pseudonyme TF2 : [T4D] Jer0m3_K3rv1el

Noobs69

J'essaye ça de suite !!

Moi, j'en étais là pour contourner :

Code :

{
_ESSAI *tmp;
tmp=malloc(sizeof(_ESSAI));
if (tmp != NULL)
{
tmp->a=12;
tmp->b='b';
sprintf(tmp->c, "%s", "essai c ok" );
tmp->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
if (tmp->d != NULL)
tmp->d="essai d ok";
printf("maininit [%d]\n", tmp->a);
printf("maininit [%c]\n", tmp->b);
printf("maininit [%s]\n", tmp->c);
printf("maininit [%s]\n", tmp->d);
}
*essai=tmp;

Merci de ton aide précieuse et claire, j'avance vite avec une aide qui explique et ne met pas des commentaires lapidaires et inutiles comme souvent sur les forums, j'apprécie.

Noobs69

Ca fonctionne !!!

Et ton analyse du préprocesseur est correcte, il comprend l'inverse, belle blague celle-là, merci grandement de ta pédagogie.

Je mets tout au propre et je post ici mon code complet, si ça peut en aider d'autres à "naviguer" dans les pointeurs de pointeurs en C ...

settings.h

Code :

#ifndef __SETTINGS_H__
#define __SETTINGS_H__
#include <stdbool.h>
typedef struct __test {
int a;
char b;
char c[20];
char *d;
} _TEST;
void setinit(_TEST **);
#endif

settings.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "settings.h"
void setinit(_TEST **test)
{
*test=malloc(sizeof(_TEST));
if (*test != NULL)
{
(*test)->a=34;
(*test)->b='B';
sprintf((*test)->c, "%s", "test c ok" );
(*test)->d=malloc((strlen("test d ok" ) + 1) * sizeof(char));
if ((*test)->d != NULL)
(*test)->d="test d ok";
printf("maininit [%d]\n", (*test)->a);
printf("maininit [%c]\n", (*test)->b);
printf("maininit [%s]\n", (*test)->c);
printf("maininit [%s]\n", (*test)->d);
}
}

main.h

Code :

#ifndef __MAIN_H__
#define __MAIN_H__
typedef struct __essai {
int a;
char b;
char c[20];
char *d;
} _ESSAI;
#endif

main.c

Code :

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include "main.h"
#include "settings.h"
_TEST *test;
_ESSAI *essai;
void maininit(_ESSAI **essai)
{
*essai=malloc(sizeof(_ESSAI));
if (*essai != NULL)
{
(*essai)->a=12;
(*essai)->b='b';
sprintf((*essai)->c, "%s", "essai c ok" );
(*essai)->d=malloc((strlen("essai d ok" ) + 1) * sizeof(char));
if ((*essai)->d != NULL)
(*essai)->d="essai d ok";
printf("maininit [%d]\n", (*essai)->a);
printf("maininit [%c]\n", (*essai)->b);
printf("maininit [%s]\n", (*essai)->c);
printf("maininit [%s]\n", (*essai)->d);
}
}
int main()
{
setinit(&test);
maininit(&essai);
if (test != NULL)
{
printf("test [%d]\n", test->a);
printf("test [%c]\n", test->b);
printf("test [%s]\n", test->c);
printf("test [%s]\n", test->d);
}
if (essai != NULL)
{
printf("essai [%d]\n", essai->a);
printf("essai [%c]\n", essai->b);
printf("essai [%s]\n", essai->c);
printf("essai [%s]\n", essai->d);
}
free(essai);
essai=NULL;
free(test);
test=NULL;
return(0);
}

Résultat du code :

Code :

maininit [34]
maininit [B]
maininit [test c ok]
maininit [test d ok]
maininit [12]
maininit [b]
maininit [essai c ok]
maininit [essai d ok]
test [34]
test [B]
test [test c ok]
test [test d ok]
essai [12]
essai [b]
essai [essai c ok]
essai [essai d ok]

Message édité par Noobs69 le 25-04-2011 à 11:24:45

Noobs69

Ca n'existe pas RESOLU sur ce forum ou je vois mal ?

WiiDS

Connardophobe

:jap:

Tu peux éditer le premier post et changer le titre si tu y tiens vraiment, mais effectivement il n'y a pas de politique concernant le titre des sujets

Bonne continuation :hello:

---------------
"I can cry like Roger. It's just a shame I can't play like him" - Andy Murray | Darky, enfoiré :( | Pseudonyme TF2 : [T4D] Jer0m3_K3rv1el

gilou

Modérateur
It's the only NEET thing to do

Je vois pas trop ce que tu veux faire, mais si j'avais à utiliser ce type de structure, je procéderais ainsi:
(Bon, à reformater, la page web ayant pas la même notion de tabulations que mon emacs en mode electric-c)

Code :

/* settings.h*/
#ifndef __SETTINGS_H__
#define __SETTINGS_H__
typedef struct {
int a;
char b;
char c[20];
char *d;
} TEST;
TEST *test_create_empty(void);
TEST *test_create(int a, char b, char *c, char *d);
bool test_set(TEST *test, int a, char b, char *c, char *d);
void test_print(TEST *test);
#endif

Code :

/* settings.c*/
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <stdbool.h>
#include "settings.h"
TEST *test_create_empty(void)
{
TEST *test = malloc(sizeof(TEST));
if (test) {
test->a = 0;
test->b = 0;
(test->c)[0] = 0;
test->d = NULL;
}
return test;
}
bool test_set(TEST *test, int a, char b, char *c, char *d)
{
if (test) {
test->d = malloc((strlen(d) + 1));
if (test->d) {
test->a = a;
test->b = b;
strncpy(test->c, c, 20);
strcpy(test->d, d);
return true;
}
}
return false;
}
TEST *test_create(int a, char b, char *c, char *d)
{
TEST *test = malloc(sizeof(TEST));
if (test) {
test->d = malloc((strlen(d) + 1));
if (test->d) {
test->a = a;
test->b = b;
strncpy(test->c, c, 20);
strcpy(test->d, d);
}
else {
free(test);
test = NULL;
}
}
return test;
}
void test_print(TEST *test)
{
printf("a: [%d]\nb: [%c]\nc: [%s]\nd: [%s]\n", test->a, test->b, test->c, test->d);
}

Code :

/* main.c*/
#include <stdlib.h>
#include "settings.h"
int main()
{
TEST *test = test_create_empty();
if (test) {
if (test_set(test, 34, 'B', "test c ok", "test d ok" )) {
test_print(test);
}
free(test);
return EXIT_SUCCESS;
}
return EXIT_FAILURE;
}

In Hope It Helps,
A+,

Message édité par gilou le 25-04-2011 à 13:42:38

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

Publicité

FORUM HardWare.fr

Programmation

[C] pointeurs de pointeurs [RESOLU]

Sujets relatifs
Affectation illicite de pointeurs en assembleur	[C] soustraire deux pointeurs
[boost.serialization] Sérialization, templates et pointeurs	pointeurs dans un conteneur - comment éviter leur destruction ?
Passage d'un tableau de pointeurs de fonctions en argument	tableaux de pointeurs
Structures, pointeurs..affichage?	Cas où les références remplacent mal les pointeurs ?
Tableau de pointeurs en c	[résolu] parcours d'un char** sans connaitre le nombre de chaines
Plus de sujets relatifs à : [C] pointeurs de pointeurs [RESOLU]

Page générée en 0.385 secondes