Discussion :

[3aychoucha]

Bonjour,
J'ai une structure test

Code :

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

1
2
3
4
5
6
 
struct test
{
	char *ch;
	int i;
};

je veux avoir un tableau de struct qui contient 5 éléments, par exemple, de ma structure test

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
 
std::vector<test> v;
void mTest()
{
	for(int i=0; i<5; i++)
	{
		char buf[255];
		sprintf(buf,"%d",i);
 
		//v.clear();
		test t;
		t.ch = buf;
		t.i = i;
		std::cout << t.ch << "   " << t.i << std::endl;
		v.push_back(t);
	}
}

puis dans le main j'affiche le tableau de struct

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int _tmain(int argc, _TCHAR* argv[])
{
	mTest();
	for(int i=0; i<5; i++)
	{
		std::cout << v.at(i).ch << "   " << v.at(i).i << std::endl;
	}
	system("pause");
	return 0;
}

le problème est que lorsque j'affiche les éléments du tableau, le contenu de la chaîne est perdu ou je sais pas,
voici le résultat

0 0
1 1
2 2
3 3
4 4
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╚█←g☺╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠Þ ╠╠╠╠╠╠╠╠ 0
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╚█←g☺╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠Þ ╠╠╠╠╠╠╠╠ 1
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╚█←g☺╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠Þ ╠╠╠╠╠╠╠╠ 2
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╚█←g☺╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠Þ ╠╠╠╠╠╠╠╠ 3
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╚█←g☺╠╠╠╠╠╠╠╠╠╠╠ ╠╠╠╠╠╠╠╠Þ ╠╠╠╠╠╠╠╠ 4
Appuyez sur une touche pour continuer...

comment puis-je corriger cette erreur
Merci d'avance

[oodini]

Pourquoi ne pas utiliser des std::string plutôt que des tableaux de caractères ?

[koala01]

Salut,

Déjà, y a-t-il une raison particulière pour l'utilisation d'un char * au lieu d'une std::string

Tu t'éviterais surement bien des soucis en manipulant cette dernière

autrement, si pour une raison qui ne dépend que de toi (mais que l'on pourrait sans doute démonter ), tu te trouves face à la nécessité absolue d'utiliser un char *, il faut penser au fait que les collections de la stl gèrent... des copies des objets que l'on insère dedans (sauf quand on crée une collection de pointeurs)

Il faudrait donc veiller à définir un constructeur par copie efficace qui effectue une "deep copy" de ch...

en outre, et c'est sans doute le pire, buf est créé de manière statique (par opposition à "dynamique", avec new ) et arrête donc d'exister ... à l'accolade fermante qui englobe le contenu de la boucle !

il n'y a donc rien d'étonnant à ce que les pointeurs ch de tes structures pointent sur des adresses invalides

[arash]

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
std::vector<test> v;
void mTest()
{
        for(int i=0; i<5; i++)
        {
                char buf[255];
                sprintf(buf,"%d",i);

                //v.clear();
                test t;
                t.ch = strdup(buf);
                t.i = i;
                std::cout << t.ch << "   " << t.i << std::endl;
                v.push_back(t);
        }
}

buf est une variable locale et est perdu après mtest().
donc t.ch pointe sur une adresse invalide.
vous pouvez utiliser strdup(). mais n'oubliez pas de libérer la mémoire

[jblecanard]

vous pouvez utiliser strdup(). mais n'oubliez pas de libérer la mémoire

Conseiller des APIs C alors que Koala a fourni tous les éléments de réponse en C++, ce n'est franchement pas une bonne idée.

[3aychoucha]

merci
pb résolu avec le std::string