c++ - Initialisere klasse med en reference - Udvikleren.dk

Tags: c++

Bruger #8249 @ 20.09.07 14:12

Hej Udviklere,
et hurtigt lille spørgsmål jeg ikke synes jeg kan finde svaret på:
Hvis jeg vil have en klassevariabel der referer til en anden klasse, uden at skulle lave en ny kopi - hvordan kan jeg så gøre det rent praktisk?
Og er det bedst at bruge en reference eller en pointer?
Og hvis jeg bruger en reference, hvordan kan jeg så initialisere den?

Et eksempel med reference kunne se sådan ud:

Kode

class Map
{
friend class Species;

private:
  std::vector<std::vector<int> > domain;
//etc..
};

class Species
{
private:
  const Map& r_map;
//etc...
};

- så har 'Species' mulighed for at tilgå 'domain' i 'Map'.

Men hvis jeg prøver at lave en constructor:

Kode

Species::Species(const Map& domain_map) : r_map(domain_map){}

får jeg en fejlmeddelelse fra compileren (måske pga min vector<vector>> i Map?). Det løser ikke problemet at give Map en copy constructor.

Jeg har prøvet at løse det ved at gøre r_map til en pointer, og så give constructoren:

Kode

Species::Species(const Map& domain_map) : r_map(new Map(domain_map)){}

men bruger det ikke hukommelse op?

Tak!

Indlæg senest redigeret d. 20.09.2007 14:13 af Bruger #8249

8 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 1 karma

Sorter efter stemmer Sorter efter dato

Bruger #4414 @ 21.09.07 14:40

113

Man bør undgå "friend" declerationen hvis det overhovedet er muligt. "Friend are evil!" som en af mine bekendte, som har mange år erfaring inden for emnet fortalte mig for nyligt. Der er andre måder at gøre det på, selv om det måske er en smule mere besværlige.

Jeg har prøvet at løse det ved at gøre r_map til en pointer, og så give constructoren:
Kode
Species::Species(const Map& domain_map) : r_map(new Map(domain_map)){}
men bruger det ikke hukommelse op?

Tak!

Jo du vil have et "memory leak" hvis ikke du definerer en destructor som tager sig af problemet:

Kode

Species::Species(const Map& domain_map) : r_map(new Map(domain_map)){}
Species::~Species() {
  delete r_map;
}

Indlæg senest redigeret d. 21.09.2007 15:26 af Bruger #4414

Bruger #1151 @ 20.09.07 17:31

421

Hvordan lyder fejlmeddelelsen?

Bruger #4996 @ 20.09.07 20:30

196

Dette kompilerer problemfrit:

Kode

#include <vector>
using namespace std;

class Map
{
   friend class Species;
private:

  std::vector<std::vector<int> > domain;
//etc..
};

class Species
{
public:
   Species(const Map& domain_map) : r_map(domain_map){}
private:
  const Map& r_map;
};

int main()
{
   Map MyMap;
   Species MySpecies(MyMap);
}

Bruger #8249 @ 21.09.07 09:58

Indlæg senest redigeret d. 21.09.2007 11:06 af Bruger #8249

Bruger #8249 @ 21.09.07 10:51

Dette kompilerer problemfrit:

Kode

#include <vector>
using namespace std;

class Map
{
   friend class Species;
private:

  std::vector<std::vector<int> > domain;
//etc..
};

class Species
{
public:
   Species(const Map& domain_map) : r_map(domain_map){}
private:
  const Map& r_map;
};

int main()
{
   Map MyMap;
   Species MySpecies(MyMap);
}

Sorry, den lille kodestump kompilerer også uden problemer på mit system. Jeg skulle selvfølgelig have sikret mig at det kunne genskabe fejlen

Problemet opstår når jeg prøver at lave en vector med flere Species (jeg holder styr på alle mine speciesobjekter i en member vector i en anden klasse (class Dataset).
Denne lille modifikation replicerer min compile fejl:

Kode

#include <vector>
using namespace std;

class Map
{
   friend class Species;
private:

  std::vector<std::vector<int> > domain;
//etc..
};

class Species
{
public:
   Species(const Map& domain_map) : r_map(domain_map){}
private:
  const Map& r_map;
};

int main()
{
   Map MyMap;

   std::vector<Species> list;       //NB

   for (int i=0;i!=3;++i)           //NB
   {
        Species MySpecies(MyMap);
        list.push_back(MySpecies);  //NB
   }
}

Fejlmeddelsen fra MinGW:

Kode

C:\\MinGW\\bin\\..\\lib\\gcc\\mingw32\\3.4.5\\..\\..\\..\\..\\include\\c++\\3.4.5\\bits\\vector.tcc:: In member function `Species& Species::operator=(const Species&)':
C:\\MinGW\\bin\\..\\lib\\gcc\\mingw32\\3.4.5\\..\\..\\..\\..\\include\\c++\\3.4.5\\bits\\vector.tcc:238: instantiated from `void std::vector<_Tp, _Alloc>::_M_insert_aux(__gnu_cxx::__normal_iterator<typename _Alloc::pointer, std::vector<_Tp, _Alloc> >, const _Tp&) [with _Tp = Species, _Alloc = std::allocator<Species>]'
C:\\MinGW\\bin\\..\\lib\\gcc\\mingw32\\3.4.5\\..\\..\\..\\..\\include\\c++\\3.4.5\\bits\\stl_vector.h:564: instantiated from `void std::vector<_Tp, _Alloc>::push_back(const _Tp&) [with _Tp = Species, _Alloc = std::allocator<Species>]'
H:\\Dokumenter\\C++\\Worldmap to GIS\\Dataset.cpp:53: instantiated from here
C:\\MinGW\\bin\\..\\lib\\gcc\\mingw32\\3.4.5\\..\\..\\..\\..\\include\\c++\\3.4.5\\bits\\vector.tcc:238: error: non-static reference member `const Map&Species::r_map', can't use default assignment operator
:: === Build finished: 1 errors, 1 warnings ===

Indlæg senest redigeret d. 21.09.2007 11:09 af Bruger #8249

Bruger #4996 @ 21.09.07 20:18

196

Kompileren fortæller at den skal bruge en assignment operator, at den ikke kan lave en og at du ikke har givet din en.

Det bliver du nødt til for at kunne kopiere Species objekter, hvilket sker i "list.push_back(MySpecies);"

For at kunne kopiere dine Species objekter skal r_map gøres ikke const. Med assignment operatoren bliver det til:

Kode

class Species
{
public:
   Species(Map& domain_map) : r_map(domain_map){}

   Species& operator = (const Species& rhs) {r_map = rhs.r_map; return *this; }
private:
   Map& r_map;
};

Bruger #4996 @ 21.09.07 20:20

196

Man bør undgå "friend" declerationen hvis det overhovedet er muligt.

Nej, brugt rigtigt kan "friend" være med til at lave et bedre design.

Bruger #8249 @ 22.09.07 08:32

Tak Bertel,
det lyder som om jeg var lidt på det rigtige spor, bare det forkerte sted, med at lave en copy constructor til Map. Det er lidt synd at jeg må gøre r_map non-const, idet Species for guds skyld ikke må ændre i Map - men jeg må bare være forsigtig.
Tak igen!

Felix >> hvis der er nogle gode måder at forbedre et design der ikke bruger friend vil jeg gerne høre dem. I dette tilfælde bruger jeg den da Map indeholder en 145x165 celler 2dvector - da jeg har 3000 Species objekter sparer friend deklarationen mig for 71.775.000 kald til

Kode

bool Map present(int x, int y) {return (bool) domain[x][y];}

hver eneste gang jeg vil lave en beregning. Det er også på grund at det store antal objekter at jeg bekymrer mig om ikke at lave for mange unødvendige kopier...

Indlæg senest redigeret d. 22.09.2007 08:35 af Bruger #8249

Initialisere klasse med en reference

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler