Discussion :

[sylvain.cool]

Bonjour,
je voudrais faire un tableau en 2 dimensions en c++.
voici l'equivalent en C de ce que je voudrais faire:

Code :

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

1
2
3
4
5
6
7
8
 
        int ** m_pHoleMap;
	m_pHoleMap=(int **)malloc(sizeof(int*)*nbLines);
	for(int i=0;i<nbLines;i++){
		m_pHoleMap[i]=(int*)malloc(sizeof(int)*nbColumns);
		for(int j=0;j<nbColumns;j++)
			m_pHoleMap[i][j]=-1;
	}

mais le malloc ca fait pas trop C++.
J'ai pense a un truc du style:

Code :

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

1
2
 
        int ** m_pHoleMap=new int[nbLines][nbColumns];

mais ca ne marche pas.
Si vous avez des suggestions.
Merci d'avance

[Woufeil]

Salut,

Inspire toi de ça :

Code :

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

1
2
3
4
5
6
7
8
int** m_pHoleMap=new int*;
for (int i = 0; i< nbLignes; i++) 
{
   for (int j= 0; j<nbColumns; j++)
   {
       m_pHoleMap[i][j] = -1;
   }
}

Ce code créé une matrice et remplit toutes ses cases avec -1 comme tu t'en doutais

[Bakura]

Berk. Pourquoi ne pas utiliser des vectors de vectors ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
std::vector <std::vector <int> > tab2D;
const size_t nbLignes = 5;
const size_t nbColonnes = 6;
tab2D.resize (nbLignes); // On crée un tableau de 5 lignes
 
for (size_t i = 0 ; i < nbLignes ; ++i)
{
   tab2D.resize (nbColonnes); // Dans chaque ligne, 6 colonnes
      for (size_t j = 0 ; j < nbColonnes ; ++j)
      {
         tab2D[i][j] = -1;
      }
}

EDIT : ou plus simplement :

Code :

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

1
2
3
4
5
6
7
8
9
const size_t nbLignes = 5;
const size_t nbColonnes = 6;
std::vector <std::vector <int> > tab2D (nbLignes, nbColonnes);
 
   for (size_t i = 0 ; i < nbLignes ; ++i)
      for (size_t j = 0 ; j < nbColonnes ; ++j)
      {
         tab2D[i][j] = -1;
      }

J'avais un doute mais ça marche aussi.

[yan]

salut

Code :

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

1
2
3
4
5
6
7
8
int** m_pHoleMap=new int*;
for (int i = 0; i< nbLignes; i++) 
{
   for (int j= 0; j<nbColumns; j++)
   {
       m_pHoleMap[i][j] = -1;
   }
}

tu t'es trompé dans les new

Code :

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

1
2
3
4
5
6
7
8
9
int** m_pHoleMap=new int*[nbLignes];
for (int i = 0; i< nbLignes; i++) 
{
   m_pHoleMap[i] = new int[nbColumns];
   for (int j= 0; j<nbColumns; j++)
   {
       m_pHoleMap[i][j] = -1;
   }
}

et pour desallouer

Code :

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

1
2
3
4
5
6
7
8
9
int** m_pHoleMap=new int*;
for (int i = 0; i< nbLignes; i++) 
   {
      delete [] m_pHoleMap[i];
   }
delete [] m_pHoleMap;
m_pHoleMap = NULL;// ne sert pas vraiment
 
}

[Luc Hermitte]

Pour ceux qui veulent faire des new[] (N +1), la réponse correcte est dans la FAQ.

Sinon rien ne vaut la réutilisation de ce qui a déjà été intensément validé : vector<vector<>>, boost::multi_array, stlsoft::arraynd (?), Blitz++, boost.ublas, Newmat, ...

[r0d]

En effet, boos::multi_array est trés pratique.
Sinon, pour rester dans la STL, tu peux jeter un coup d'oeil à ce post

[smashy]

Sinon rien ne vaut la réutilisation de ce qui a déjà été intensément validé : vector<vector<>>, ...

Le vector< vector<> > n engendre t il pas des reallocation en cascade ? :

que le premier vecteur doit etre realloue , il implique une recopie de sont contenu donc allocation pour le nouveau vecteur + copie + liberation de l ancien vecteur

[coyotte507]

sinon si tu veux un statique tu peux faire

Code :

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

int mp_holemap[nblignes][nbcolonnes]

Je crois que par contre si tu mets "strict ISO c++" en option de compilation ca marche pas