Discussion :

[ouste4863]

Bonjour, j'ai un souci avec le champ pointeur d'une structure sur une autre, j'ai deux structures dont l'une contient un pointeur vers vers, maintenant je veux accéder aux champs de la structure pointée en passant par la structure qui contient le pointeur. Vs avez ici les codes pour mieux cerner de quoi je parle.

déclaration des structure

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
struct pixel
{
	unsigned short R,G,B;
}; typedef struct pixel pixel;
struct imagePPM
{
	char *nombreMagique[3];
	char col,lig;
	short max;
	pixel * pixels;
};typedef struct imagePPM imagePPM;
pixel initialise(unsigned short r, unsigned short g, unsigned b);
void initialise2(unsigned short r, unsigned short g, unsigned b, pixel *p);
imagePPM *creerImagePPM(unsigned char col, unsigned char lig, unsigned char max);
void affichePixels(imagePPM *image);
void save(imagePPM *image, char *nomFic);

les fonctions

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
magePPM * creerImagePPM(unsigned char col, unsigned char lig, unsigned char max)
{
	imagePPM *image = malloc(sizeof(imagePPM));
	image->col=col;
	image->lig=lig;
	image->max=max;
	image->nombreMagique[0]="P";
	image->nombreMagique[1]="3";
	return image;
}
 
void affichePixels(imagePPM *p)
{
	printf("les codes de couleurs: %d %d %d\n",p->pixels->R,p->pixels->G,p->pixels->B);
}

epuis dans le main j'appelle la fonction affichePixels j'ai une erreur de segmentation

Code :

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

1
2
3
4
5
6
7
8
9
#include <stdio.h>
#include "tp3.h"
#include <stdlib.h>
int main(int argc, char argv[])
{
	imagePPM *image;
	image = creerImagePPM(2,2,5);
	affichePixels(image);
}

Je voudrais afficher les valeurs par défaut des champs p->pixels->R,p->pixels->G,p->pixels->B

[dalfab]

Bonjour,

Un pointeur, ça doit pointer quelque part. A aucun moment dans le code que tu fournis, le champ pixels n'est initialisé. Il pointe donc à un endroit aléatoire, l'utiliser donnera donc un comportement aléatoire

[ouste4863]

Merci Dalfab pour votre réponse, mais je me rends compte que quand je fais de cette manière
dans la fonction qui me créé mon imagePPM, ça ne s'execute pas aussi cette fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
imagePPM * creerImagePPM(unsigned char col, unsigned char lig, unsigned char max)
{
	imagePPM *image = malloc(sizeof(imagePPM));
	image->col=col;
	image->lig=lig;
	image->max=max;
        image->pixels->G=0;
        image->pixels->R=0;
        image->pixels->B=0;
	image->nombreMagique[0]="P";
	image->nombreMagique[1]="3";
	return image;
}

je ne sais pas si j'ai mal fait mais je ne sais pas comment faire autrement.

[ouste4863]

je viens de faire quelque chose dans ma fonction qui créé imagePPM, ça marche mais je me demande si c'est normal

Code :

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

1
2
3
4
5
6
7
8
9
10
11
imagePPM * creerImagePPM(unsigned char col, unsigned char lig, unsigned char max)
{
	imagePPM *image = malloc(sizeof(imagePPM));
	image->pixels= malloc(sizeof(pixel));//est-il normal de faire un malloc ici pour le champ pointeur??
	image->col=col;
	image->lig=lig;
	image->max=max;
	image->nombreMagique[0]="P";
	image->nombreMagique[1]="3";
	return image;
}

j'ai fait un malloc pour le champ pixels avec sizeof de la structure pixel sur laquelle il pointe.

[foetus]

Tu peux initialiser un pointeur soit avec une adresse (avec l'esperluète) soit avec un malloc.

Donc c'est bon pour le malloc, mais ton initialisation est fausse parce que tu initialises un tableau de dimension col x lign

Et il te manque la désallocation de ta structure parce que tu as une fuite mémoire ou

[ouste4863]

Vous parlez dans ce cas de ma fonction en générale ou bien de la partie qui concerne le champ pixels?

[ouste4863]

je comprends pour la des-allocation mais quant à l’initialisation d'un tableau bidimensionnel, je ne suis pas sûr de vous avoir compris.

[ouste4863]

si je comprends bien feotus, je dois faire de cette manière.

Code :

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

image->pixels = malloc(col*lig*sizeof(pixel));

[dalfab]

Oui, il faut allouer tous les pixels de la table

[ouste4863]

Merci à tous, mon problème a été résolu.