[SOLUCIONADO] C++ error al acceder a miembro privado desde clase anidada y friend

[Lifka]

El problema es el siguiente, tengo una clase llamada diccionario, en la que uso templates y tengo anidada la clase iterator. La clase diccionario es un tipo de objeto (no el de la STL pero parecido) con un vector (STL) de pair (STL), que contiene en el first un tipo key que no debe ser modificado, y en second que se corresponde con key y cuyos tipos varían gracias a los templates.

Clase diccionario en el fichero diccionario.h:

Código:

class diccionario;

Que contiene la clase anidada:

Código:

class iterator;

La cual está establecida como friend:

Código:

friend class iterator;

Dentro de la parte privada de la clase diccionario:

Código:

vector<entrada> dic;

Entrada se corresponde con:

Código:

typedef pair<Key, Def> entrada;

Ahora en el archivo diccionario.hxx (el cual está incluido correctamente en el .h y todo lo necesario...), tengo el siguiente método (el cual tiene su correspondiente cabecera en la parte pública de la clase anidada):

Código:

template <typename Key, typename Def> 
diccionario<Key,Def>::iterator::iterator( );

Este método debería crear un iterador que apunta al primer dato del vector privado de la clase padre.
Este puntero se guarda de la siguiente forma en la parte privada de la clase iterator:

Código:

pair<const Key, Def> * it_v;

Lo que quiero hacer en el método es algo como esto:

Código:

vector<pair<Key, Def>> dic2;
pair<const Key, Def> * it_v2 = (pair<const Key,Def> *) &dic2[0];

dic2 correspondería con el dato privado de la clase diccionario (el vector con la clave y el dato).
pair<const Key, Def> * it_v2 es it_v, de la parte privada de la clase anidada.
(pair<const Key,Def> *) es un casting ya que lo hemos declarado como constante en el iterador, pero el tipo de dato no estaba definido así.
&dic2[0] es &dic[0] la referencia que queremos devolver del dato mencionado anteriormente.
(Todo lo anterior funciona bien)
De modo que lo que quiero hacer es:

Código:

it_v = (pair<const Key,Def> *) &dic[0];

Peeeeero....

g++ -o ejecV2 -D DICC_V2 -std=c++0x ejemplo.cpp -Iinclude
In file included from ejemplo.cpp:4:0:
diccionarioV2.hxx: In instantiation of ‘diccionario<Key, Def>::iterator::iterator() [with Key = std::basic_string<char>; Def = int]’:
ejemplo.cpp:112:35: required from here
diccionario.h:198:18: error: invalid use of non-static data member ‘diccionario<std::basic_string<char>, int>::dic’
In file included from diccionario.h:213:0,
from ejemplo.cpp:4:
diccionarioV2.hxx:296:7: error: from this location
make: *** [dic2] Error 1

El caso es que desde ese método no puedo acceder a dic (el vector) de ningún modo, esto también devuelve el mismo error:

Código:

Def a = dic.at(0).second;

No consigo acceder a los datos privados desde la clase amiga anidada, ¿alguien me ayuda?

Espero haberme expresado bien xD Gracias y un saludo.

[eferion]

¿No puedes poner una parte mínima de código que permita reproducir tu problema?

[amchacon]

Has intentado resumirlo tanto que no se entiende nada.

¿Puedes poner el código de la función? Y la definición de la clase no iría mal.

[Lifka]

Disculpad, es que es larguillo y tengo muchas cosas... aquí lo dejo más claro:

diccionario.h:

Código C++:

template <typename Key, typename Def>
class diccionario{  
public:
 
  typedef pair<Key, Def> entrada;
  typedef pair<const Key, Def> value_type;
  typedef unsigned int size_type;
  class iterator;
 
  diccionario( );
 
 
  class iterator {
  public:
    iterator();
 
  private:
    iterator (entrada * p);
    iterator (typename vector<entrada>::iterator it);
    friend class diccionario;
    value_type * it_v;
  };
 
private:
        
  friend class iterator;    
  vector<entrada> dic; // vector que almacena los elementos del diccionario
};
 

diccionarioV2.hxx:

Código C++:

template <typename Key, typename Def> 
diccionario<Key,Def>::iterator::iterator( ){
     /*
      Esto es lo mismo que lo de abajo, esto funciona y lo otro no
      vector<pair<Key, Def>> dic2;
      pair<const Key, Def> * it_v2 = (pair<const Key,Def> *) &dic2[0];
      */
 
      it_v = (pair<const Key,Def> *) &dic[0];
 
      //Esto tampoco funciona...
     // Def a = dic[0].second;
}
 

[lareto]

En tu mensaje original dices:

diccionario.h:198:18: error: invalid use of non-static data member ‘diccionario<std::basic_string<char>, int>::dic’

Así que si pusieras código que pudiera compilarse y que reprodujera ese error sería mejor.

Por otro lado, te comento con flechitas en el código alguna cosa que veo que me parece que no está bien, o que no entiendo para qué está...

Código:

    template <typename Key, typename Def>
    class diccionario{  
    public:
     
      typedef pair<Key, Def> entrada;
      typedef pair<const Key, Def> value_type;
      typedef unsigned int size_type;
      class iterator;    // -> ¿Para qué está esto?
     
      diccionario( );
     
     
      class iterator {   // ----> declaras el tipo iterator, pero no veo que tengas una variable de ese tipo en la clase
      public:
        iterator();
     
      private:
        iterator (entrada * p);
        iterator (typename vector<entrada>::iterator it);
        friend class diccionario;
        value_type * it_v;
      };
     
    private:
           
      friend class iterator;  // --> ¿por qué tiene que ser friend una clase incluída?  
      vector<entrada> dic; // vector que almacena los elementos del diccionario
    };

Pero bueno, si pusieras algo para compilar y ver dónde pasa lo que pasa, por ahí me llueve una idea.

[kutcher]

El error no tiene nada que ver con el miembro de ser privado. Todo el problema se puede simplificar a esta:

Código C++:

struct dicconario {
  struct iterator{
    iterator();
  };
  int dic;
};
 
dicconario::iterator::iterator() {
  int i = dic;
}
 

dic no es miembro estático de diccionario así que tienes que tener un objeto diccionario para acceder a ella. El objeto iterador probablemente debería almacenar un miembro de diccionario*, pasado a cada uno de lo constructores

[eferion]

El ejemplo que has pasado no sirve porque:

Código C++:

pair<const Key, Def> * it_v2 = (pair<const Key,Def> *) &dic2[0];
 

Aunque sea una clase anidada, "iterator" no sabe sobre qué instancia de "diccionario" está trabajando. Otra cosa es que "dic2" sea un método estático de "diccionario", pero salvo que a "iterator" le facilites un puntero o una referencia a una instancia de "diccionario" me temo que no vas a ser capaz de acceder de forma satisfactoria a los miembros de "diccionario" desde "iterator"

[Lifka]

Lareto, hay cosas a las que yo tampoco le veo mucha lógica, pero aclaro que las definiciones no son mías, y en teoría no debo modificarlas. Forman parte de un ejercicio. Aquí dejo parte -> https://mega.co.nz/#!l8MggTIS!p8O8SONVDQxwdVzHJpaRTiHNjIj0RZsftR5rW01 kWcs

kutcher, no puedo pasarle un parámetro a ese constructor, el ejercicio me obliga a hacerlo así.

Andaba bastante perdido ya que no comprendía que no tuviese acceso a la instancia... Gracias a vuestra explicación creo que ya se me ocurre como continuar! Muchas gracias.

[Lifka]

Se envió duplicado