Discussion :

[tintin72]

Bonjour,

Pour initialiser les pointeurs de mon tableau de pointeurs à zero je fais comme ceci:

Code :

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

1
2
 
char *array[30]={0};

ça fonctionne, mais j'ai lu quelque part que ce n'était pas une trés bonne solution (mais je ne me souviens plus du tout pourquoi ).
Aussi, j'aimerais savoir si c'est une bonne pratique ou si il y a
une autre méthode plus sûr et plus portable.

Merci

[Phoenix_Blue]

je ne suis pas sur mais je pense qu'il serait mieux de créer un pointeur sur pointeur ...

Code :

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

1
2
3
typedef char *car;
car *array=new car[30];
array=NULL;//initialisation sur rien

[Faiche]

je ne suis pas sur mais je pense qu'il serait mieux de créer un pointeur sur pointeur ...

Code :

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

1
2
3
typedef char *car;
car *array=new car[30];
array=NULL;//initialisation sur rien

t'as honte la, non ?

tintin72 : si tu veux initialiser les cases a 0, tu parcours les tableau en mettant chaque case à 0, comme ça t'es sur.
Sinon tu alloue dynamiquement, et tu utilises calloc au lieu de malloc.

[Phoenix_Blue]

en fait j'initialisait sur rien pour ne pas lire à des adresses interdites
c'est vrai que je n'ai initialiser que le premier pointeur je m'en excuse
il est en effet possible d'initialiser comme suit:

Code :

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

1
2
for(int i=0;i<30;i++)
array[i]=NULL;

là ça va?

[Faiche]

en fait j'initialisait sur rien pour ne pas lire à des adresses interdites
c'est vrai que je n'ai initialiser que le premier pointeur je m'en excuse
il est en effet possible d'initialiser comme suit:

Code :

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

1
2
for(int i=0;i<30;i++)
array[i]=NULL;

là ça va?

sachant que array[i] est un char, le mettre a NULL ça n'a rien de malin. Note que ça passera quand meme puisque NULL vaut 0, mais que je vois pas ça dans un code hein.

[olive_le_malin]

memset
Sets buffers to a specified character.

void *memset( void *dest, int c, size_t count );

[Faiche]

memset

oui ben vu le niveau, je sais pas si c'est recommandé de les faire utiliser memset.

[olive_le_malin]

memset(array, 0, sizeof(char*) * 30);

[Phoenix_Blue]

oui ben vu le niveau, je sais pas si c'est recommandé de les faire utiliser memset.

je comprends rien

[olive_le_malin]

utilise ça (si c'est bien ce que tu veux faire ...)
et pourt'en convaincre regarde au debuggeur le comportement de ton tableau.

Regarde aussi l'aide de cette fonction.

[Luc Hermitte]

Ce qui n'est pas terrible comme solution, c'est de gérer des tableaux de chaines de la sorte. Rien de tel pour avoir des programmes pénibles à maintenir instables après la moindre refactorisation.

Sinon, c'est une initialisation valable. Cela ne pourra en revanche jamais réinitialiser quoique ce soit.

[loufoque]

Bonjour,

Pour initialiser les pointeurs de mon tableau de pointeurs à zero je fais comme ceci:

Code :

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

1
2
 
char *array[30]={0};

ça fonctionne, mais j'ai lu quelque part que ce n'était pas une trés bonne solution (mais je ne me souviens plus du tout pourquoi ).
Aussi, j'aimerais savoir si c'est une bonne pratique ou si il y a
une autre méthode plus sûr et plus portable.

Merci

Ça me semble être parfaitement correct.

[YmYannick]

Pourquoi faire un :

Code :

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

1
2
3
 
for(int i=0;i<30;i++)
array[i]=NULL;

au lieu d'un :

Code :

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

1
2
 
char *array[30]={0};

Toutes les cases seront initialisées à zéro anyway non ?

[tintin72]

Ca y est! J'ai retrouvé l'article qui déconseillait (plus ou moins) cette façon de faire:
http://www.game-lab.com/index.php?se...showtut&id=212
l'argument est que cela stocke le tableau en entier dans le code du programme, ce qui fait grossir la taille de l'exécutable.
Mais bon, tant pis pour la taille de l'exe car de toute façon il est impératif que tous mes pointeurs soit initialisés à zero sinon, avec les tests conditionnel qui suivent c'est le plantage assuré.
Par contre je ne sais pas si pour une réinitialisation (le tableau doit être réinitialisé à zero aprés chaque utilisation) je peux réutiliser:

Code :

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

1
2
 
char *array[30]={0};

sinon je peux toujours utiliser memset qui est aussi une bonne alternative.

[Luc Hermitte]

Non, tu ne peux pas. (et +1 pour ma théorie des questions qui arrivent en troupeaux d'elles même)

Mais que veux tu stocker ? Si c'est un tableau de chaines, cette solution est archaique en C++ -- pour ne pas dire maladroite, source à bugs, pénible à maintenir, non-flexible, ...

[tintin72]

Non, tu ne peux pas.

Pourquoi ?
Quelle méthode puis-je utiliser alors ?

Mais que veux tu stocker ? Si c'est un tableau de chaines, cette solution est archaique en C++ -- pour ne pas dire maladroite, source à bugs, pénible à maintenir, non-flexible, ...

Oui ce sont des chaine de caractères. Je sais, en C++ on utilise des strings, mais le pb c'est que les widgets (input text, output text etc..) de la GUI que j'utilise (fltk) attendent des const char* comme arguments.
Je sais qu'on peut convertir les string en char et vice versa, mais pour l'instant mon prog est suffisament compliqué Aussi je préfère pour l'instant travailler avec des char* quitte aprés a tout
reconvertir en string.

[David Fleury]

Je sais qu'on peut convertir les string en char et vice versa, mais pour l'instant mon prog est suffisament compliqué

Il serait pourant plus simple si tu utilisais des std:string (avec c_str() ) pour les injecter dans tes widgets.

[rogerio]

je suis ce topic parce que il m'interesse aussi

mais si tu ne renseignes pas ton tableau à sa declaration

son contenu est-il considéré comme null ?

ou est-il considéré déjà a 0 ?

rog

[tintin72]

Ni l'un ni l'autre. Si tu n'itinialise pas ton tableau de pointeurs lors de sa déclaration, tes pointeurs vont pointer sur tout et n'importe quoi, d'où l'intérêt de l'initialisation.

[swirtel]

L'utilisation de type primitif char * n'est pas à conseiller dans un langage orienté objet. Il faut réaliser de l'abstraction qui est plus facile à maintenir.

L'utilisation de std::string est à prescrire au lieu de char *. Et plus le programme deviendra complexe et plus tu te casseras les dents.

Un bête exemple, la majorité des développeurs oublient d'inclure la sécurité dans leur soft en se disant "je le ferai en dernier", et pourtant ça serait plus simple de le faire dès la naissance du projet. On gagne du temps.

Même constatation avec les tests unitaires.