Конструктор копии в Классах

@123c · Регистрация: 17.11.2015

Author24 — интернет-сервис помощи студентам

Подскажите пожалуйста правильно ли я написала и вызвала конструктор копии и деструктор, а то мне выдает 2 ошибки.

C++

#include <iostream.h>
#include <conio.h>
#include <string.h>
 
class Employee{
public:
   //Konstruktor bez parametrov
   Employee();
 
   //Konstruktor s parametrami
   void setEmployee (char *, int, int);
 
   //Konstruktor kopii
   Employee ( const Employee &);
 
   //Destruktor
   ~Employee ();
 
   void printEmployee ();
private:
   char name[25];
   int age;
   int gaffer;
};
 
Employee::Employee ()  {
age = gaffer = 0;
char *k="pustoi";
int length = strlen(k);
length = length < 25 ? length : 24;
strncpy (name, k, length);
name[length] = '\0';
 ; }
void Employee::setEmployee (char *n, int a, int g)
{
int length = strlen(n);
length = length < 25 ? length : 24;
strncpy (name, n, length);
name[length] = '\0';
age=a;
gaffer=g;
}
 
void Employee::printEmployee ()
{
 
cout <<"Imja slujashego: " << endl << name << endl <<
"Vozrast: " << endl << age << endl <<
"Stag: " <<endl << gaffer << endl;
}
 
// Konstruktor kopii
Employee ( const Employee & obj)
    {
        memcpy(name,obj.name,NAMESIZE);
        age = obj.age;
        gaffer = obj.gaffer;
    }
 
 //Destruktor
    ~Employee (){};
 
main ()
{
clrscr ();
Employee e;
e.printEmployee ();
cout << endl;
e.setEmployee ("Ivan", 21, 1);
Employee e2 ("Egor", 21, 1);
e2.printEmployee();
 
Employee e3(e2);
e3.printEmployee ();
getch ();
return 0;
}

@nmcf · 06.04.2016, 22:16

strcpy() используй, а не memcpy().
В первом конструкторе нет смысла проверять длину, если и так известно, что копируется - там тоже strcpy() хватит.

@123c · 06.04.2016, 22:21 **[ТС]**

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

strcpy() используй, а не memcpy().

Всё равно 2 ошибки

@nmcf · 06.04.2016, 22:24

Актуальный вариант покажи и текст ошибок.

@TheCalligrapher · 06.04.2016, 22:34

Сообщение от 123c

Employee ( const Employee & obj)

Определения членов класса за пределами класса делаются при помощи квалифицированного имени, т.е. имени с префиксом Employee::. Вы об этом знаете, суда по другим определениям (и конструктор по умолчанию вы определили с правильным именем). Почему тогда определение конструктора копирования у вас сделано без этого префикса???

Та же проблема в определении деструктора.

Добавлено через 1 минуту

Сообщение от 123c

main ()

int main(). int.

Добавлено через 1 минуту
Зачем вы написали в этом классе явный деструктор, когда деструктировать там совершенно нечего - не понятно.

@hoggy · 06.04.2016, 22:38

Сообщение от 123c

Подскажите пожалуйста правильно ли я написала и вызвала конструктор копии и деструктор

не правильно.

Сообщение от 123c

а то мне выдает 2 ошибки.

привести которые конечно же не обязательно.

а зачем? это же такой тест на внимательность:
местные телепаты сами, глядя на неумелые закорючки
должны догадаться, что с ними не так.

это:

C++

// Konstruktor kopii
Employee ( const Employee & obj)
    {
        memcpy(name,obj.name,NAMESIZE);
        age = obj.age;
        gaffer = obj.gaffer;
    }
 
 //Destruktor
    ~Employee (){};

заменить на это:

C++

// Konstruktor kopii
Employee::Employee ( const Employee & obj)
    {
        memcpy(name,obj.name,24);
        age = obj.age;
        gaffer = obj.gaffer;
    }
 
 //Destruktor
    Employee::~Employee (){};

@123c · 06.04.2016, 22:40 **[ТС]**

1.Error LABA1-1.CPP 53: Declaration terminated incorrectly
2.Error LABA1-1.CPP 70:Could not find a match for "Employee::Employee(char *,int,int)"

@hoggy · 06.04.2016, 22:44

Сообщение от 123c

2.Error LABA1-1.CPP 70:Could not find a match for "Employee::Employee(char *,int,int)"

рядом с этим:

C++

void Employee::setEmployee (char *n, int a, int g)
{
int length = strlen(n);
length = length < 25 ? length : 24;
strncpy (name, n, length);
name[length] = '\0';
age=a;
gaffer=g;
}

покласть это:

C++

Employee::Employee(char *n, int a, int g)
{
    setEmployee (n,a,g);
}

@123c · 06.04.2016, 22:52 **[ТС]**

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

заменить на это:

всё равно эта ошибка осталась: Error LABA1-1.CPP 70:Could not find a match for "Employee::Employee(char *,int,int)"

Добавлено через 5 минут

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

рядом с этим:

правильно?

C++

#include <iostream.h>
#include <conio.h>
#include <string.h>
 
class Employee{
public:
   //Konstruktor bez parametrov
   Employee();
 
   //Konstruktor s parametrami
   void setEmployee (char *, int, int);
 
   //Konstruktor kopii
   Employee ( const Employee &);
 
   //Destruktor
   ~Employee ();
 
   void printEmployee ();
private:
   char name[25];
   int age;
   int gaffer;
};
 
Employee::Employee ()  {
age = gaffer = 0;
char *k="pustoi";
int length = strlen(k);
length = length < 25 ? length : 24;
strncpy (name, k, length);
name[length] = '\0';
 ; }
void Employee::setEmployee (char *n, int a, int g)
{
int length = strlen(n);
length = length < 25 ? length : 24;
strncpy (name, n, length);
name[length] = '\0';
age=a;
gaffer=g;
}
Employee::Employee (char *n, int a, int g)
{
   setEmployee (n,a,g);
}
 
void Employee::printEmployee ()
{
 
cout <<"Imja slujashego: " << endl << name << endl <<
"Vozrast: " << endl << age << endl <<
"Stag: " <<endl << gaffer << endl;
}
 
// Konstruktor kopii
Employee::Employee ( const Employee & obj)
    {
    memcpy(name,obj.name,24);
        age = obj.age;
        gaffer = obj.gaffer;
    }
 
 //Destruktor
   Employee:: ~Employee (){};
 
main ()
{
clrscr ();
Employee e;
e.printEmployee ();
cout << endl;
e.setEmployee ("Ivan", 21, 1);
Employee e2 ("Ivan", 21, 1);
e2.printEmployee();
 
Employee e3(e2);
e3.printEmployee ();
cout <<"Destruktor rabotaet";
getch ();
return 0;
}

@123c · 06.04.2016, 22:58 **[ТС]**

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

делделделдел

не поняла?

@hoggy · 06.04.2016, 23:00

http://rextester.com/ORP12246

C++

#include <iostream>    //<--------- исправленно 
#include <string.h>
 
class Employee{
public:
   //Konstruktor bez parametrov
   Employee();
    
   Employee (const char *n, int a, int g);     //<--------- исправленно 
 
   //Konstruktor s parametrami
   void setEmployee (const char *, int, int); //<--------- исправленно 
 
   //Konstruktor kopii
   Employee ( const Employee &);
 
   //Destruktor
   ~Employee ();
 
   void printEmployee ();
private:
   char name[25];
   int age;
   int gaffer;
};
 
Employee::Employee ()  {
age = gaffer = 0;
const char *k="pustoi";    //<--------- исправленно 
int length = strlen(k);
length = length < 25 ? length : 24;
strncpy (name, k, length);
name[length] = '\0';
 ; }
 
 
void Employee::setEmployee (const char *n, int a, int g) //<--------- исправленно 
{
int length = strlen(n);
length = length < 25 ? length : 24;
strncpy (name, n, length);
name[length] = '\0';
age=a;
gaffer=g;
}
 
 
Employee::Employee (const char *n, int a, int g)     //<--------- исправленно 
{
   setEmployee (n,a,g);
}
 
void Employee::printEmployee ()
{
 
std::cout <<"Imja slujashego: \n"  << name <<          //<--------- исправленно 
"\nVozrast: " << '\n' << age << 
"\nStag: " <<'\n' << gaffer << '\n';
}
 
// Konstruktor kopii
Employee::Employee ( const Employee & obj)
    {
    memcpy(name,obj.name,24);
        age = obj.age;
        gaffer = obj.gaffer;
    }
 
 //Destruktor
   Employee:: ~Employee (){};
 
int main ()                         //<--------- исправленно 
{
// clrscr ();                    //<--------- исправленно 
Employee e;
e.printEmployee ();
std::cout << std::endl;
e.setEmployee ("Ivan", 21, 1);
Employee e2 ("Ivan", 21, 1);
e2.printEmployee();
 
Employee e3(e2);
e3.printEmployee ();
std::cout <<"Destruktor rabotaet";    //<----- исправленно
//getch ();                           //<----- исправленно
return 0;
}

@123c 0 / 0 / 0 Регистрация: 17.11.2015 Сообщений: 46
	06.04.2016, 22:58 [ТС]	10
	Сообщение от hoggy делделделдел не поняла? 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	06.04.2016, 22:16	2
	strcpy() используй, а не memcpy(). В первом конструкторе нет смысла проверять длину, если и так известно, что копируется - там тоже strcpy() хватит. 0

@123c 0 / 0 / 0 Регистрация: 17.11.2015 Сообщений: 46
	06.04.2016, 22:21 [ТС]	3
	Сообщение от nmcf strcpy() используй, а не memcpy(). Всё равно 2 ошибки 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	06.04.2016, 22:24	4
	Актуальный вариант покажи и текст ошибок. 0

@TheCalligrapher Вездепух 12792 / 6669 / 1795 Регистрация: 18.10.2014 Сообщений: 16,875
	06.04.2016, 22:34	5
	Сообщение от 123c Employee ( const Employee & obj) Определения членов класса за пределами класса делаются при помощи квалифицированного имени, т.е. имени с префиксом `Employee::`. Вы об этом знаете, суда по другим определениям (и конструктор по умолчанию вы определили с правильным именем). Почему тогда определение конструктора копирования у вас сделано без этого префикса??? Та же проблема в определении деструктора. Добавлено через 1 минуту Сообщение от 123c main () `int main()`. `int`. Добавлено через 1 минуту Зачем вы написали в этом классе явный деструктор, когда деструктировать там совершенно нечего - не понятно. 0

@123c 0 / 0 / 0 Регистрация: 17.11.2015 Сообщений: 46
	06.04.2016, 22:40 [ТС]	7
	1.Error LABA1-1.CPP 53: Declaration terminated incorrectly 2.Error LABA1-1.CPP 70:Could not find a match for "Employee::Employee(char *,int,int)" 0

	06.04.2016, 23:00