Discussion :

[JEAN-GREGOIRE]

Bonjour à tous
j'ai un soucie avec une procédure devant m'afficher une structure crée au préalable, celle ci m'affiche toutes la structures sans problèmes mais lorsque qu'il doit afficher des chaines de caractere celle ci m'affiche des métacaractères

Code :

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

1
2
3
4
5
6
7
8
9
10
 
ITEM *buildItem(int id, char *category,char *name, float prise)//construit un item
{
    ITEM *a=malloc(sizeof(ITEM));
    a->id=id;
    a->name=name;
    a->category=category;
    a->price=prise;
    return a;
}

Code :

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

1
2
3
4
5
6
 
void printItem (ITEM *item)ma procédure qui affiche ma structure
{
    printf("\n %d %s (%s) : %f \n", item->id,item->name, item->category, item->price);
    printf("%s",item->name);
}

Code :

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

1
2
3
4
5
6
7
8
 
typedef struct ITEMtag//voici ma structure a afficher
{
	int id;
	char *name;
	char *category;
	float price;
} ITEM;

si je construit un item avec builditem et que je l'éxécute dans le main celle ci m'affichera 4 Ï#Ö (Ï#Ö) : 12.360000

[Médinoc]

Comment gères-tu la durée de vie des chaînes de caractères que tu passes à tes objet ITEM?
En fait, ne devrait-ce pas être à ces objets de gérer la durée de vie des chaînes de caractères qu'ils contiennent?
(même aux prix d'une recopie; ne pas oublier, l'optimisation prématurée est la racine de tous les maux)

Question subsidiaire: Ces chaînes sont-elles censées pouvoir être modifiées une fois l'ITEM construit?

[JEAN-GREGOIRE]

elles ne sont pas censé être modifier lors de leurs créations pour leur durée de vie je n'en ai aucune idée pour être honnête avec vous.

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
 
int main()
{
 
 
 
    char*voiture=malloc(256*sizeof(char));//j'alloue dans mon main leurs espacements mémoires
    char *toto=malloc(256*sizeof(char));
    ITEM *test=buildItem(4,voiture,toto,12.36);
    printItem(test);
 
 
 
    return 0;
}

[dalfab]

Euh, dans l'exemple les chaines ne sont jamais remplies, elles seront donc affichées comme elles ont été remplies : aléatoirement.

[JEAN-GREGOIRE]

mdr sincérement désolée a force de lire des lignes et des lignes de code je n'ai pas fait attention a ce détails merciii!!!!

[stephane78l]

TU dois utiliser strcpy dans la fonction buildItem pour copier name et category dans a->name et a->category car ce sont des chaines de caracteres...

[stephane78l]

et d'ailleurs aussi il aurait fallu les allouer toutes les deux dans buildItem avec malloc ou les declarer comme char name[256] par exemple, dans la structure ITEMtag

[Médinoc]

L'idéal, ce serait un truc de ce genre, avec constructeur et destructeur, plus les fonctions jouant le rôle de new et delete, gérant à la fois la mémoire et la construction/destruction:

Code C :

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
static void ITEM_constructor(ITEM* pThis, int id, char const *category, char const *name, float price)
{
	pThis->id = id;
	pThis->name = strdup(name);
	pThis->category = strdup(category);
	pThis->price = price;
}
 
ITEM * ITEM_new(int id, char const *category, char const *name, float price)//construit un item
{
	ITEM *pThis = malloc(sizeof *pThis);
	if(pThis != NULL)
	{
		ITEM_constructor(pThis, id, category, name, price);
	}
	return pThis;
}
 
static void ITEM_destructor(ITEM* pThis)
{
	free(pThis->name), pThis->name=NULL;
	free(pThis->category), pThis->category=NULL;
}
 
void ITEM_delete(ITEM* pThis)
{
	if(pThis != NULL)
	{
		ITEM_destructor(pThis);
		free(pThis);
	}
}

Ainsi, tant qu'on crée les ITEM uniquement par ITEM_new() et qu'on les détruit uniquement par ITEM_delete(), la durée de vie des chaînes de caractères dans l'objet est correctement gérée.

PS: Si tu n'as pas de strdup() sur la plate-forme que tu utilises (c'est une fonction POSIX plutôt qu'une fonction du C standard), elle est facile à implémenter:

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
char* strdup(char const *src)
{
	size_t lengthInChars = strlen(src);
	size_t sizeInChars = lengthInChars + 1;
	char* ret = calloc(sizeInChars, sizeof *ret);
	if(ret != NULL)
	{
		memcpy(ret, src, sizeInChars * sizeof *ret);
	}
	return ret;
}

[JEAN-GREGOIRE]

Merci pour votre aide je vais tenter les modifications et voir les résultats