Discussion :

[clinic]

Bonjour, j'ai un petit problème avec les vectors, je n'arrive pas à le passer en paramètre d'une fonction.


voilà mon code ( un résumé )
ma fonction :

Code :

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

1
2
3
4
5
6
7
8
9
 
int GrapheGen(Stack* pStackA, std::vector<std::vector<unsigned char>> vector2d)
{
...
...
vector2d[count][k] = 1;
...
...				
}

et je l'appelle comme ca dans mon main

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
std::vector < std::vector<unsigned char>> vector2d;
 
	// lines
	vector2d.resize(NBNODE);
	// columns
	for( indexvect=0 ; indexvect < vector2d.size() ; indexvect++)
	{
		vector2d[indexvect].resize(SIZE);
	} 
 
	GrapheGen(&stackA,vector2d);

le vector est bien rempli dans ma fonction GrapheGen, mais dès qu'on sort de la fonction, vector2d est vide ...
j'ai du mal à voir mon erreur, merci à ceux qui pourront jeter un petit coup d'oeil.

[loufoque]

Ton code est syntaxiquement incorrect, tu dois mettre un espace entre les >> dans les templates.
Ensuite tu devrais utiliser le constructeur plutôt que de faire ton initialisation bidon.

Pour finir, ton problème vient simplement du fait que tu prends ton vecteur par copie. Normal donc qu'il ne soit pas modifié...

[clinic]

hum ok, je vais voir pour le constructeur.

par contre comment dois je passer mon vecteur ? Je suppose qu'il faudrait plutot lui donner un pointeur, mais j'ai du mal à trouver la syntaxe, tous ce que j'ai essayer ne compile pas.

comme tu as pu le remarquer, le c/cpp n'est pas du tout ma spécialité ...

[JolyLoic]

comme tu as pu le remarquer, le c/cpp n'est pas du tout ma spécialité

Normal, puisque le c/cpp n'existe pas... C'est comme si tu disais que le smalltalk/ruby ou la fraise/côte de boeuf n'est pas ta spécialité.

Sinon, pour reprendre les remarques 1 et 3 de Loufoque dans du code (la 2, je suis d'accord aussi, mais c'est plus long à écrire la bonne version) :

Code :

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

1
2
 
int GrapheGen(Stack* pStackA, std::vector<std::vector<unsigned char> > &vector2d)

[loufoque]

GrapheGen pourrait aussi retourner ton vecteur par valeur.

Pour l'initilisation il suffit de faire

Code :

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

std::vector < std::vector<unsigned char> > vector2d(NBNODE, std::vector<unsigned char>(SIZE));

Ce qui est plus performant.

[Matthieu Brucher]

Ou pas... Le mieux est sans doute de passer par référence pour éviter les copies inutiles.