Discussion :

[mister3957]

Bonjour à tous,

J'ai un tableau et j'aimerai lui attribuer une taille que je ne connais pas encore lors de la compilation, est-ce que c'est possible?

int size = _vector->size();
int tab[size];

Le compilateur me signale une erreur "expected constant expression".

Qu'est-ce que je peux faire?

Merci beaucoup

A bientôt

Aurélien

[Médinoc]

Utiliser un pointeur et de l'allocation dnamique...

Code :

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

1
2
3
4
int *tab = new int[size];
 
//et quand tu as fini
delete[] tab

[JolyLoic]

Utiliser std::vector :

Code :

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

1
2
3
std::vector<int> tab(size);
// et quand tu as fini
// rien à faire de spécial

[Laurent Gomila]

http://c.developpez.com/faq/cpp/?page=STL#STL_vector

[mister3957]

Oups, je crois que je me suis mal exprimé.

En fait je veux transformer un vector en tab[], c'est pour l'utiliser par la suite dans un memcpy...

[Aurelien.Regat-Barrel]

tu peux utiliser std::copy, qui s'utilise naturellement avec vector. memcpy est plutôt à éviter en C++.
Sinon, un vector peut s'utiliser sans rien faire de spécial là où un tableau C est attendu:

Code :

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

1
2
3
 
vector<int> vect( 10 );
int *tab = &vect[ 0 ];

[mister3957]

Parceque en fait je fais une apply avec DirectX. Là je m'occupe du générateur de map. Ma class (Ground) prends des valeurs comme la longueur, la hauteur, la largeur, le nombre de "cases" de la map etc...

J'ai une fonctions qui crée les Vertex dynamiquement, c'est pour ça que j'utilise un vector, puisque je ne connais pas la taille du tableau tout de suite, donc j'ai un algo qui crée les vertex et les ajoute au vector.

Voici le vector:

Code :

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

1
2
3
 
//Création du vector
std::vector<CUSTOMVERTEX*>* _vertexVector = new std::vector<CUSTOMVERTEX*>();

Dans mon algo il y a plusieurs appel du genre:

Code :

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

1
2
3
4
5
6
7
8
9
 
//Création d'un Vertex
CUSTOMVERTEX* vertex = new CUSTOMVERTEX();
vertex->x = (*j)->points[k].x;
vertex->y = (*j)->points[k].y;
vertex->z = (*j)->points[k].z;
vertex->color = 0xff000000;
//Ajout du Vertex dans le vector
_vertexVector->push_back(vertex);

Donc ici, j'ai un vector avec les vertex que j'ai besoin.
La procédure pour envoyer ces vertex à la carte graphique est:

Code :

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

1
2
3
4
5
6
7
 
//on blok le buffer
_vertexBuffer->Lock(0, sizeof(vertexTab), (void**)&pVertices, 0);
//on copy les vertex dedant
memcpy(pVertices, vertexTab, sizeof(vertexTab));
//on débloque le buffer
_vertexBuffer->Unlock();

vertexTab doit être un tableau CUSTOMVERTEX[].

Là je cherche à partir du vector<CUSTOMVERTEX*> à faire un tableau CUSTOMVERTEX[]. Mais la taille initiale d'un tableau doit être constante, je peux pas faire CUSTOMVERTEX vertexTab[_vertexVector->size()]...

J'ai fait des recherches sur std::copy mais je sais pas si il pourrait m'être utile, ou alors je sais pas comment l'utiliser pour mon cas...

[Laurent Gomila]

Code :

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

std::vector<CUSTOMVERTEX*>* _vertexVector

Pourquoi pas std::vector<CUSTOMVERTEX> ?

Là je cherche à partir du vector<CUSTOMVERTEX*> à faire un tableau CUSTOMVERTEX[]. Mais la taille initiale d'un tableau doit être constante, je peux pas faire CUSTOMVERTEX vertexTab[_vertexVector->size()]...

Code :

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

1
2
3
4
5
6
7
8
//on bloque le buffer 
_vertexBuffer->Lock(0, 0, reinterpret_cast<void**>(&pVertices), 0); 
 
//on copie les vertices dedans
std::copy(_vertexVector.begin(), _vertexVector.end(), pVertices); 
 
//on débloque le buffer 
_vertexBuffer->Unlock();

PS : évite les préfixage par un underscore, par convention c'est reservé aux implémentations de la SL.

[mister3957]

Voilà:

Code :

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

1
2
 
std::vector<CUSTOMVERTEX> vertexVector;

Code :

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

1
2
3
4
5
6
7
 
CUSTOMVERTEX vertex;
vertex.x = (*j)->points[k].x;
vertex.y = (*j)->points[k].y;
vertex.z = (*j)->points[k].z;
vertex.color = 0xff000000;
vertexVector.push_back(vertex);

Code :

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

1
2
3
4
 
_vertexBuffer->Lock(0, sizeof(vertexTab), (void**)&pVertices, 0);
memcpy(pVertices, vertexTab, sizeof(vertexTab));
_vertexBuffer->Unlock();

Mais bon, le problème est le même, dans le memcpy je peux pas mettre le vector<CUSTOMVERTEX>, il me faut un CUSTOMVERTEX[].

Code :

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

1
2
3
 
//error C2057: expected constant expression
CUSTOMVERTEX vertexTab[vertexVector.size()];

[Aurelien.Regat-Barrel]

Oublie memcpy, et relis le code de loulou.

[mister3957]

En fait j'ai une solution mais bon, ça me paraît un peu trop bricolage foireux:

Faire un tableau de 10 cases par exemple
Faire une boucle sur le vector
Si le tableau est plein, on fait un malloc de 10 par exemple
On met le Vertex à l'index courant
Une fois sorti de la boucle, on redimensionne le tableau pour qu'il n'y ai plus de place vide...

Ca sent pas bon comme solution ça :s

[mister3957]

Ha oui exact, j'vaais pas vu que loulou avait mis ça

Je test...

[Aurelien.Regat-Barrel]

Oublie malloc, oublie même new. vector s'occupe de tout pour toi, Essaye de bien comprendre comment il fonctionne (exemple FAQ). Une fois que tu maîtriseras std::vector, tu auras pas mal progressé en C++

[mister3957]

Alors j'ai recopié:

Code :

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

1
2
3
4
 
_vertexBuffer->Lock(0, 0, reinterpret_cast<void**>(&pVertices), 0);
std::copy(vertexVector.begin(), vertexVector.end(), pVertices);
_vertexBuffer->Unlock();

et j'obtien cette erreur:

error C2182: 'reference' : illegal use of type 'void'
C:\Program Files\Microsoft Visual Studio 8\VC\include\xutility

sur la ligne "typedef _Ty& reference;" de xutility

[Laurent Gomila]

Tu as bien déclaré pVertices en CUSTOMVERTEX* et non en void* ?

[mister3957]

Oups ouais j'avais laissé en VOID*

Ok ça compile maintenant

J'vous remercie beaucoup beaucoup


A bientôt