Discussion :

[Pikwik]

Bonjour,

Je voulais savoir si en C on peut reconnaitre une chaine "constante" d'une chaine dynamique.

j'ai une fonction :

Code :

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

1
2
 
void addUrl( dataBase *pDB, char *pUrl, char *pType);

Elle peut deux pointeur vers des chaines, donc, je peux lui donner soit une chaines "en dur", soit une chaine venant d'un fichier par exemple.

Le problème se pose pour faire un "free"...

j'ai une autre fonction :

Code :

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

1
2
 
void destroyDataBase(dataBase *pDB);

Elle a pour but de tout "nettoyer" avec les free necessaires.
Tout ce corse quand je me retouve a faire un free sur une constante...

Est-il possible de les différencier ?

[Skyounet]

Bah si c'est toi qui a codé tu dois savoir quelles sont les chaines à liberer et celle à ne pas liberer.

[Pikwik]

Oui !

C'était plus une question à titre informatif
Je me posais la question, par exemple dans le cas d'une lib !?

[Gruik]

Si tu as pas envie de te prendre la tête, tu peux allouer une nouvelle chaine et y copier la constante litterale
http://man.developpez.com/man3/strdup.3.php

Comme ça, toute "chaine" sera a libérer

[Médinoc]

Par contre, une fonction ne peut pas différencier si on ne lui dit pas.
Et les chaînes constantes devraient être déclarées en char const *.


Quand j'ai ce genre de problèmes, je passe deux pointeurs à la fonction:

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
#include <stdlib.h>
#include <assert.h>
 
void addUrl( dataBase *pDB, char const *sczUrl, char *szFreeUrl, char const *sczType, char *szFreeType)
{
assert(szFreeUrl==NULL || szFreeUrl==sczUrl);
assert(szFreeType==NULL || szFreeType==sczType);
 
/*
 Traitement utilisant sczUrl et sczType
 ...
*/
 
 
free(szFreeUrl); /* Aucun effet ni aucune erreur si szFreeUrl est NULL */
free(szFreeType);
}

[Gruik]

C'est pas con Medinoc

[Pikwik]

Médinoc je viens de look le man de assert, c'est pas mal, mais je ne comprend pas superbien ce que ca fait, tu peux expliquer un peut ?

[Skyounet]

Le probleme avec assert c'est que dans un programme en production il ne doit pas y avoir de assert (ou du moins ces dernieres seront eliminées lors de la compilation si on définie la constante NDEBUG [-DNDEBUG]).

Donc voila en gros assert regarde la condition que tu lui donnes, si la condition est fausse lors de l'execution du programme ca affiche la ligne et le message d'erreur

genre

Code :

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

1
2
3
 
char *test = NULL;
assert(test != NULL);

Affichera à l'execution : line x : test = NULL
Ou un truc du genre.

[crocodilex]

Je ne vois pas trop où est ton problème.
Tu appelles free() uniquement lorsque tu alloues de la mémoire de manière dynamique, c'est à dire avec malloc(), calloc(), realloc()...

[Emmanuel Delahaye]

Bonjour,

Je voulais savoir si en C on peut reconnaitre une chaine "constante" d'une chaine dynamique.

j'ai une fonction :

Code :

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

1
2
 
void addUrl( dataBase *pDB, char *pUrl, char *pType);

Elle peut deux pointeur vers des chaines, donc, je peux lui donner soit une chaines "en dur", soit une chaine venant d'un fichier par exemple.

Le problème se pose pour faire un "free"...

j'ai une autre fonction :

Code :

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

1
2
 
void destroyDataBase(dataBase *pDB);

Elle a pour but de tout "nettoyer" avec les free necessaires.
Tout ce corse quand je me retouve a faire un free sur une constante...

Est-il possible de les différencier ?

Non. Je connais 2 solutions pour résoudre ton problème.

• Tout passer en dynamique
• Utiliser une structure avec un indicateur

Code :

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

1
2
3
4
5
6
 
struct chaine
{
   char *s;
   int dyn; /* 0=statique 1=dynamique */
};

La première solution est plus légère...

[Médinoc]

Et la seconde ne compile pas en -Werror -Wwrite-strings, si on lui passe une constante chaîne (je ne connais pas d'autre terme français pour "string literal").

[Pragmateek]

Code :

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

int dyn;

Pourquoi pas un "char dyn" plus léger?

[Médinoc]

Code :

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

1
2
3
typedef enum e_bool { FAUX, VRAI } E_BOOL;
 
E_BOOL dyn;

:p

Edit: Joli mon cher cochon, vous m'avez battu

[Gruik]

Code :

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

1
2
3
4
5
struct chaine
{
   char *s;
   enum { TC_DYN, TC_CONST } type;
};