конструктор для коллекции

[Lers]

#include <stdio.h>
#include <conio.h>
#include <windows.h>
#include "Collection.h"
 
void main()
{
  SetConsoleCP(1251);
  SetConsoleOutputCP(1251);
 
  Collection *c2=new Collection;
 
  Disc *k1=new Disc ("Математика",10);
  c2->additem(k1);
  Disc *k2=new Disc ("Физика",22);
  c2->additem(k2);
  Disc *k3=new Disc ("Экономика",17);
  c2->additem(k3);
  Disc *k4=new Disc ("Биология",39);
  c2->additem(k4);
 
  Collection t(c2);
  
 getch();
  delete c2;
 
}

#include <string.h>
#include <stdio.h>
#include "disc.h"
 
void Disc :: setnamedisc(char *s)
{
  strcpy(namedisc,s);
}
 
char* Disc :: getnamedisc()
{
  return namedisc;
}
 
void Disc :: sethours(int b)
{
  hours=b;
}
 
int Disc :: gethours()
{
  return hours;
}
 
void Disc :: getobject()
{
  fflush(stdin);
  printf("\nВведите название дисциплины -> ");
  gets(namedisc);
  printf("Введите кол-во часов -> ");
  scanf("%d",&hours);
}
 
void Disc :: putobject()
{
  printf("\nНазвание дисциплины: ");
  puts(namedisc);
  printf("Количество часов: %d\n",hours);
}
 
Disc :: Disc()
{
  namedisc[0]=0;
  hours=0;
}
 
Disc :: Disc(char *s, int h)
{
  strcpy(namedisc,s);
  hours=h;
}
 
Disc :: Disc(Disc &x)
{
  strcpy(namedisc,x.namedisc);
  hours=x.hours;
}
 
Disc :: Disc(char *s)
{
  strcpy(namedisc,s);
  hours=0;
}

#include <string.h>
#include <stdio.h>
#include "Collection.h"
 
 
Collection :: Collection (Collection *x)
{
// заготовка для конструктора
}
 
 
Collection::Collection()
{
  col=0;
  items=new Disc*[10];
}
 
Collection::~Collection()
{
  for (int i=0; i<col; i++)
   {
      delete items[i];
    }
  delete [] items;
}
 
void Collection::additem(Disc *x)
{
    items[col] = x;
    col++;
}
 
void Collection::delitem(int a)
{
    while(a<col)
    {
      items[a]=items[a+1];
      a++;
    }
    col--;
 
}
 
int Collection::getcount()
{
  return col;
}
 
Disc * Collection::getitem(int num)
{
    return items[num];
}
 
void Collection::putitems()
{
  for(int i = 0; i < col; i++)
   {
     items[i]->putobject();
   }
 
}

У меня есть коллекция объектов класса Disc, и эти объекты хранятся в **items. Мне нужно (вот вызов Collection t(c2)) в коллекцию t скопировать c2, не использую новые объекты базового класса Disc, т.е. k1,k2,k3,k4 оставить без изменений.
Я так понимаю что мне нужно что-то делать с адресами объектов которые хранятся в с2, может создать массив для t, и в этот массив занести адреса объектов из **items.
Хелп: мысли, решения, соображения)

[Lers]

#include "Disc.h"
class  Collection
{
  int col;
  Disc ** items;
  public:
    Collection(Collection *x);
    Collection();
    ~Collection();
    void additem(Disc *);
    void delitem(int);
    int getcount();     
    Disc * getitem(int);
    void putitems();
};

class Disc
{
  private:
  char namedisc[20];
  int hours;
  
  public:
  void setnamedisc (char* s);
  char* getnamedisc();
  void sethours (int b);
  int gethours();
  void getobject();
  void putobject();
  Disc();
  Disc(char *s,int h);
  Disc(Disc &x);
  Disc(char *s);
};

хидеры....

[Lers]

уже решил, тему можно закрывать.
P.S. может кому-то интересно, вот констуктор

Collection :: Collection (Collection *x)
{
    // заготовка для конструктора
    col = x->col;
    items = new Disc *[col];
    for (int i = 0; i < col; ++i)
    {
        items[i] = new Disc;
        items[i]->sethours(x->items[i]->gethours());
        items[i]->setnamedisc(x->items[i]->getnamedisc());
    }
}

[waleri]

Наверно будет лучше если
Collection :: Collection (const Collection& x)

Наверно будет лучше если
items[i] = new Disc(x->items[i]);

[Lers]

Сообщение от waleri

Наверно будет лучше если
Collection :: Collection (const Collection& x)

Почему лучше по адресу &x?

[Lers]

Сообщение от waleri

Наверно будет лучше если
items[i] = new Disc(x->items[i]);

Так не работает, конструктора нету для такой записи

[Lers]

items[i] = new Disc;
        items[i]->sethours(x->items[i]->gethours());
        items[i]->setnamedisc(x->items[i]->getnamedisc());

Вопрос, можно ли это объединить в одну строчку?
например items[i]=x->items[i]; (так работает, но вылетит если я удалю какой-нибудь i-тый объект из первой коллекции).

[MaTBeu]

Сообщение от Lers

Почему лучше по адресу &x?

Это не адрес, а константная ссылка. Константность гарантирует, что исходный объект в вашем методе/конструкторе не изменится.

[Lers]

Сообщение от MaTBeu

Это не адрес, а константная ссылка. Константность гарантирует, что исходный объект в вашем методе/конструкторе не изменится.

Спасибо, не подскажите как объединить 3 строчки из пред.поста в 1 строчку?

[Lers]

Вопрос решил, вот так

items[i]=new Disc(*x->items[i]);