c++ - Kopering af en vector - Udvikleren.dk - Programmering, webdesign og grafik

Tags: c++

<< < 1 2 > >>

Bruger #1151 @ 14.08.07 21:33

421

Hej udvikleren.dk

Jeg har denne operator=()

Kode

job& job::operator=(const job& j)
{
return *this;
}

Den indholder ikke ret meget, det ved, men alt hvad jeg har forsøgt mig med har resulteret i en "segmentation fault" eller andet mærkeligt.
Job har en

Kode

vector<job*> children

og når man sætter jo forskellige job objekter lig hinanden fx.

Kode


job* job1 = new job;
job* job2 = new job;

job2 = job1;

Så skal der ske en kopiering af denne vector, men hvordan?
Jeg har snart prøvet alt. Jeg har prøvet med utallige forskellige for-løkker som bruger memcpy, eller en obskur form indholdende new og en masse pointere, men lige meget virker det.
Jeg ville høre om der var en som kunne give mig den lede tråd, som peger mod oplysning.

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

Sorter efter stemmer Sorter efter dato

Bruger #4996 @ 14.08.07 21:56

196

job1 og job2 er begge pointere, så:
job2 = job1;
Sætter blot job2 til at pege på det samme som job1 peger på.

Det er i sig selv ikke farligt, men du har nu to pointere der peger på det samme (og ingen der peger på det job2 pegede på).
Men det kan give problemer når du skal frigive hukommelsen.
Du assignment operator bliver ikke kaldt.

Hvis du vil kopiere indholdet af job1 over i job2, skal du:
*job2 = *job1;

Bruger #1151 @ 14.08.07 22:23

421

Men hvordan får jeg kopieret vektoren? Jeg har også en anden member i job klassen, en jobTask data member.

Kode

task * jobTask;

Og job task skal jeg vel også kopierer med operator=() kaldet. Lige nu er jeg i tvivl på hvordan jeg skal gøre det, må jeg ærligt indrømme. Er det ikke bare at oprette en ny instants af en task. Tror jeg har mistforstået hele konceptet ved operator overloading.

Bruger #1151 @ 14.08.07 22:26

421

Min fejl

Jeg har selvfølgelig ikke oprettet job1 og job2 som pointere!! De er oprettet sådan her,

Kode

job job1,job2;

Jeg er dog stadig i tvivl om hvordan man nu skal kopierer de data members der er med (vector og task). Jeg troede bare at det kunne gøre ved at bruge memcpy. Men det har, for mig, ikke virket.

Bruger #4996 @ 14.08.07 22:32

196

Det eneste du med sikkerhed kan kopiere med memcpy er rene POD.
vector'er og class'er (hvis de indeholder andet end POD) skal kopieres "by-hand".

Bruger #1151 @ 15.08.07 11:52

421

Og med "by-hand" men du noget i stil med

Kode

&job operator=(const & job)
{
  jobTask = new task(*job.jobTask);

  return *this;
}

Her gør jeg jo brug af copy constructoren, jeg bruger ikke memcpy og jeg opretter et nyt objekt og giver den til mit job objekt.
Så mangler jeg vel bare at kopierer min vektor på samme måde. Bare med en for løkke.
At jeg så bruger en copy constructor, betyder vel at jeg også skal til at definere den for min task klasse? Eller er det lige meget så længe min klasse ikke indholder nogen pointere? Jeg kan jo godt bare lade min task's default copy constructor tage sig af den shallow kopiering der sker.

Indlæg senest redigeret d. 15.08.2007 12:00 af Bruger #1151

Bruger #1151 @ 15.08.07 12:38

421

Jeg skal vel egentlig også delete det som først bliver peget på af min jobTask. For jeg sætter den til at pege på en default task i jobbets ctor.
Jeg burde vel egentlig også passe på at der ikke sker en
job1 = job1 ?

Og jeg burde vel også fange en evt exception så jeg kan delete mine objekter ordentlig, hvis en fejl skulle ske ved kopieringen?

Indlæg senest redigeret d. 15.08.2007 12:40 af Bruger #1151

Bruger #1151 @ 15.08.07 13:41

421

Jeg har prøvet at lave en test klasse som gør brug af en operator=(), uden at kontrollere efter exceptions og l value og r value er den samme.

Kode

#include <iostream>
#include <vector>
using namespace std;

class myClass
{
private:
  vector<int*> v;
  char c;
  float * f;

public:
  myClass();
  myClass(const myClass&);
  myClass& operator=(const myClass&);
  void add(int* i);
  void print_vector();
  void print_float();
  void print_char();

};

void myClass::print_float()
{
  cout << "adressen: " << f << " = " << *f << endl;
}

void myClass::print_char()
{
  cout << "char: " << c << endl;
}

void myClass::print_vector()
{
  for(unsigned int i = 0; i <= v.size()-1; i++)
    {
      cout<<"adressen: " << v[i] << " = " << *v[i] << endl;
    }
}

void myClass::add(int* i)
{
  v.push_back(i);
}

myClass::myClass()
{
  c = 'a';
  f = new float(2.2);
}

myClass::myClass(const myClass& mc)
{
  cout << "copy constructor bliver kaldt"<<endl;

  this->c = mc.c;
  delete f;
  f = new float(*mc.f);

  for(unsigned int i=0; i <= mc.v.size()-1; i++)
    {
      this->v.push_back(new int(*mc.v[i]));
    }

}


myClass& myClass::operator=(const myClass& mc)
{
  cout << "operator=() bliver kaldt"<<endl;
  this->c = mc.c;

  delete this->f;
  this->f=0;
  this->f = new float(*mc.f);

  for(unsigned int i=0; i <= mc.v.size()-1; i++)
    {
      this->v.push_back(new int(*mc.v[i]));
    }

  return *this;
}


int main()
{
  myClass one;
  myClass two;

  one.add(new int(41));
  one.add(new int(42));
  one.add(new int(43));
  one.add(new int(44));

  cout << "printer one objektets char: "<<endl;
  one.print_char();
  cout << "printer one objektets float: "<<endl;
  one.print_float();
  cout<<"printer one objektets vektor: "<<endl;
  one.print_vector();

  cout<<endl;
  cout<<endl;

  two = one;

  cout<<endl;
  cout<<endl;

  cout << "printer two objektets char: "<<endl;
  two.print_char();
  cout << "printer two objektets float: "<<endl;
  two.print_float();
  cout<<"printer two objektets vektor: "<<endl;
  two.print_vector();

  cout<<endl;
  cout<<endl;

  myClass three(one);

  cout<<endl;
  cout<<endl;

  cout << "printer three objektets char: "<<endl;
  three.print_char();
  cout << "printer three objektets float: "<<endl;
  three.print_float();
  cout<<"printer three objektets vektor: "<<endl;
  three.print_vector();
 

  return 0;
}

Ser det galt ud?

Indlæg senest redigeret d. 15.08.2007 13:49 af Bruger #1151

Bruger #4996 @ 15.08.07 20:54

196

Det ser egentlig ok ud. Der er et par småting.

I add(int *) tilføjer du én int, i så fald burde det være nok med: add(int ), så slap du også for at myClass arver det parametren peger på.

copy constructoren og assignment operatoren ligner hinanden, du burde flytte koden til én funktion.

Bruger #1151 @ 15.08.07 21:51

421

Er det ikke meningen at de skal lige hinanden? Jeg forstår ikke helt hvad du mener med add memberen

Kan du prøve at omformulere det? Vil du have at det bare skal være en regulær int og ikke en pointer til en int?

Bruger #1151 @ 15.08.07 21:56

421

Bertel << Angående følgende kodestump:

Kode

this->v.push_back(new int(*mc.v[i]));

Altså jeg new'er jo, men skal jeg så ikke også delete?

<< < 1 2 > >>

Kopering af en vector

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler