Discussion :

[poulet123]

Bonsoir à tous,

J'ai un problème d'exécution avec mon vector 2D. Je cherche à faire un tableau en 2 dimensions dont la taille n'est connue qu'à la compilation (mais après ne change pas)


Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void init(vector<vector<double> > u)
{ 
	vector<double> u_intermediate ;
	for (int i=0 ; i<2 ; i++) 
	{			
	     for (int j=0; j<= nbColonnes ; j++) 
	     {	
		u[j][i]=0; 
	     }
                u_intermediate.push_back(j);
		u_intermediate.clear();
        }
}

J'ai pourtant repris ce que j'ai trouvé sur le forum mais j'ai toujours l'erreur suivante :

error: attempt to subscript container with out-of-bounds index 0, but
container only holds 0 elements.

Je ne savais pas s'il fallait mettre "u_intermediate.push_back(j);" juste au début de la boucle sur les j ou à la fin mais j'ai tout essayé et rien ne marche ...



J'ai également essayé de lire la doc de boost mais je ne comprends pas comment on l'utilise si on ne connait pas la dimension à l'avance ... (car ça a l'air de faire partie des prérequis lors de la création)

Si quelqu'un sait d'où vient ce problème, ce serait super

Bonne soirée

[bertry]

Salut,

Pour les <vector> c'est là et là

[bertry]

ReSalut,

1) Si tu veux un vecteur de deux 'doubles' initalisés à 0 il faut écrire :

Code :

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

vector<double> u_intermediate ( 2, 0.0 );

2) Tu veux manifestement créer un 'vector' de 'nbColonnes' 'vector's de type 'u_intermediate', il faut alors :

Code :

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

1
2
for ( int i = 0; i < nbColonnes; i++ )
     u.push_back( u_intermediate );

3) Le problème, c'est que tout ça est effectué sur la copie de u qui est créée au lors de l'appel de ta fonction : ça ne sert donc à rien, puisque la copie est détruite à la fin de celle-ci!!
Il faut faire une transmission d'argument ( ici 'u' ) par référence ou par adresse : ça permet de travailler sur l'original et non pas sur une copie! Ça donnerai :
Exemple de transmission par référence :

Code :

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

void init(vector<vector<double> > &u)

On obtient finalement quelque chose comme ça :

Code :

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

1
2
3
4
5
6
void init(vector<vector<double> > &u)
{
     vector<double> u_intermediate ( 2, 0.0 );
     for ( int i = 0; i < nbColonnes; i++ )
          u.push_back( u_intermediate );
}

Ça dois marcher... enfin je crois...
Bye

[poulet123]

Merci beaucoup de m'aider. En fait, je veux initialiser mais j'utilise le tableau ensuite pour faire d'autres opérations comme par exemple :

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
void init(vector<vector<double> > &u, vector<vector<double> > &F, vector<vector<double> > &y, int nbColonnes) 
{ 
	vector<double> u_intermediate ;
	for (int i=0 ; i<2 ; i++) //boucle temporelle
	{	u.push_back(u_intermediate) ; 		
 
	for (int j=0; j<= nbColonnes ; j++) //boucle spatiale  nbColonnes
	{	u_intermediate.push_back(j);
		u[j][i]=i+j; 
 
 
	}
		u_intermediate.clear();
    }
}

J'ai passé en référence comme tu m'as dit et ça me met toujours la même erreur

[AF_2.8]

Est ce que tu ne te serais pas trompé dans tes indices ?
quand tu accédes à "u" tu définis "j" comme la première variable mais comme tu n'as ajouté q'un seul "vector" a "u", tu ne peux pas accéder à

Code :

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

u[1]

car la taille de "u" vaut 1 donc seul

Code :

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

u[0]

est accessible
Pour moi tu ne remplis pas correctement tes vecteurs. Décrit nous ce que tu veux faire exactement.

Je t'ai mis ton bout de code avec des commentaires qui doivent expliquer pourquoi cela ne marche pas

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
#include <vector>
#include <iostream>
 
using namespace std;
 
void init(vector< vector<double> > &u, vector< vector<double> > &F, vector< vector<double> > &y, int nbColonnes) 
{ 
  vector<double> u_intermediate ;
  for (int i=0 ; i<2 ; i++) //boucle temporelle
  {	
    u.push_back(u_intermediate);                                          // Tu ajoutes un élément à ta liste 
    cout << "u.size() = " << u.size() << endl;                            // Donc au premier coup la taille de u vaut 1,  
    cout << "u_intermediate.size() = " << u_intermediate.size() << endl;  // de même que pour u_intermediate.
 
    for (int j=0; j<= nbColonnes ; j++)                                   //boucle spatiale  nbColonnes
    {	
      u_intermediate.push_back(j);                                        // Tu ajoutes un élément à u_intermediate,
      cout << "u_intermediate.size() = " << u_intermediate.size() << endl;// sa taille passe à 1 au premier coup
      u[j][i]=i+j;                                                        // au premier coup u[0][0] existe car u contient un élement de dimension 1
                                                                          // mais pour les coups suivants u[1][0] u[1] n'existe pas tu n'as ajouté q'un
                                                                          // seul élément, de même u[0][1] n'existe pas car le seul élément de u est de dimension 1
    }
    u_intermediate.clear();
  }
}
 
int main(int argc, char* argv[])
{
  vector < vector<double> > u, F,y;
  int nbColonnes =2;
  try {
    init(u,F,y,nbColonnes);
  } catch (exception &e)
  {
    cout << e.what() << endl;
  }
 
  return 0;
}