C++ vecteur de vecteurs

Question

Je dispose d'un fichier d'en-tête de classe appelée Grid.h qui contient les éléments suivants 2 objet de données privées:

vector<int> column; 
vector<vector<int>> row; 

Et une méthode publique dont le prototype en Grid.h est tel:

int getElement (unsigned int& col, unsigned int& row); 

la définition de la fonction mentionnée ci-dessus est défini comme tel dans Grid.cpp:

int getElement (unsigned int& col, unsigned int& row) 
{ 
    return row[row][col] ; 
} 

Quand je lance le programme, je reçois ce erreur:

error C2109: subscript requires array or pointer type 

Qu'est-ce qui ne va pas?

Answer 1

Sur la ligne return row[row][col];, la première row correspond au int&, et non la vector. La variable déclarée dans la portée interne observe la variable dans la portée externe, de sorte que le compilateur essaie d'indexer un int plutôt qu'un vector, ce qu'il ne peut évidemment pas faire.

Vous devez corriger vos noms de variable afin qu'ils ne soient pas en conflit.

EDIT: En outre, alors que l'erreur que vous obtenez indique que le compilateur est de trouver la variable row mal, comme A. Levy souligne, vous avez également un problème avec la déclaration de votre vector, donc même Si vous corrigez les noms des variables, si vous avez effectivement déclaré le vector comme indiqué ici, il ne sera pas compilé. Les modèles imbriqués ont besoin d'espaces entre les symboles >, sinon le compilateur lira >> comme opérateur de décalage vers la droite plutôt que comme partie d'une déclaration de modèle. Il doit être

std::vector<std::vector<int> > row; 

ou

std::vector< std::vector<int> > row; 

En outre, comme vous faites ceci dans un fichier d'en-tête, vous allez avoir besoin de virer de bord l'étiquette std:: sur le devant de quoi que ce soit de l'espace de noms std - tel que vector. Si c'était dans un fichier cpp, alors vous pourriez utiliser using namespace std; mais ce serait très mauvais à faire dans un fichier d'en-tête (car cela polluerait l'espace de noms global). Sans la balise std:: ou l'instruction using, le compilateur ne reconnaîtra pas vector.

Answer 2

Ceci n'est probablement pas le problème d'index, mais vous avez également besoin d'un espace entre les chevrons imbriqués dans votre déclaration de type vecteur de vecteurs. Les compilateurs C++ ont du mal à faire la différence entre les types de modèles imbriqués et l'opérateur de décalage de bits correct.

Exemple:

vector<vector<int> > vec2d;  // Good. 

vector<vector<int>> anotherVec2d; // Bad! 

vector< vector<int> > yetAgain;  // Best IMHO. 
            // Keeps the white space balanced. 

Answer 3

Il me semble (bien que vous devrez peut-être vérifier sur votre propre, je ne me sens pas envie d'écrire une application de test) que le problème vient du fait que Votre paramètre contient un row nommé et votre classe a une variable interne row et il y a un conflit de nommage.

Vous devrez peut-être qualifier la ligne que vous utilisez.Considérez:

return Grid::row[row][col]; 

Answer 4

Je pense que vous voulez quelque chose comme ça ... (même si je ne peux pas imaginer pourquoi :-))

#include <vector> 
#include <iostream> 

using namespace std; 

typedef vector<int> row; 
typedef vector<row> matrix; 

matrix mat(2,2); 

int getElement (unsigned int ri, unsigned int ci) 
{ 
    return mat[ri][ci] ; 
} 

int main() { 

    mat[1][0] = 1234; 
    cout << getElement(1,0) << endl; 

    return 0; 
} 

Answer 5

C'est ce que vous avez besoin:

return Grid::row[row][col];