Форум программистов, компьютерный форум CyberForum.ru

Ошибка в копирующем конструкторе - C++

Восстановить пароль Регистрация
 
kowandser
0 / 0 / 0
Регистрация: 01.10.2010
Сообщений: 50
07.10.2010, 23:10     Ошибка в копирующем конструкторе #1
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
#include "stdafx.h"
//#include "Person.h"
#include <conio.h>
#include <string.h>
 
class Person
{
private:
    char Firstname [15];
    char Surname [15];
    char Patrname [15];
public:
    //Функции записи (сеттеры)
    void setFirstname (char s[]) {strcpy_s(Firstname, sizeof(Firstname), s);};
    void setSurname (char s[]) {strcpy_s(Surname, sizeof(Surname), s);};
    void setPatrname (char s[]) {strcpy_s(Patrname, sizeof(Patrname), s);};
    //Функции чтения (геттеры)
    char * getFirstname () {return Firstname;};
    char * getSurname () {return Surname;};
    char * getPatrname () {return Patrname;};
    //Конструкторы: по умолчанию, с параметрами, копирующий соответственно
    Person ();
    Person (char Fn[], char Sn[], char Pn[]);
    Person (const Person &obj);
};
 
Person::Person() {Firstname[0] = Surname [0] = Patrname [0] = '\x00';}
 
Person::Person(char Fn[], char Sn[], char Pn[])
    {
        setFirstname(Fn);
        setSurname(Sn);
        setPatrname(Pn);
    }
 
Person::Person(const Person &obj)
    {
        setFirstname(obj.Firstname);
        setSurname(obj.Surname);
        setPatrname(obj.Patrname);
    }
 
int _tmain(int argc, _TCHAR* argv[])
{
    Person p1;
    Person *pp2 = new Person();
    int size1 = sizeof (p1);
    int size2 = sizeof (*pp2);
    printf("Memory for object p1: %d.\nMemory for object pp2: %d.",size1, size2);
    p1.setFirstname("A");
    p1.setSurname("K");
    p1.setPatrname("S");
    pp2->setFirstname("E");
    pp2->setSurname("K");
    pp2->setPatrname("G");
    getch();
    return 0;
}
Сообщение компилятора:
1>------ Build started: Project: oop, Configuration: Debug Win32 ------
1>Compiling...
1>oop.cpp
1>d:\cpp\oop\oop\oop.cpp(38) : error C2664: 'Person::setFirstname' : cannot convert parameter 1 from 'const char [15]' to 'char []'
1> Conversion loses qualifiers
1>d:\cpp\oop\oop\oop.cpp(39) : error C2664: 'Person::setSurname' : cannot convert parameter 1 from 'const char [15]' to 'char []'
1> Conversion loses qualifiers
1>d:\cpp\oop\oop\oop.cpp(40) : error C2664: 'Person::setPatrname' : cannot convert parameter 1 from 'const char [15]' to 'char []'
1> Conversion loses qualifiers
1>Build log was saved at "file://d:\CPP\oop\oop\Debug\BuildLog.htm"
1>oop - 3 error(s), 0 warning(s)
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.10.2010, 23:10     Ошибка в копирующем конструкторе
Посмотрите здесь:

Исключения в конструкторе C++
Шаблон в конструкторе C++
C++ Ошибка в конструкторе копирования класса
Ошибка в конструкторе копирования C++
Ошибка в унаследованном конструкторе C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
ForEveR
Модератор
Эксперт C++
 Аватар для ForEveR
7927 / 4709 / 318
Регистрация: 24.06.2010
Сообщений: 10,524
Завершенные тесты: 3
07.10.2010, 23:35     Ошибка в копирующем конструкторе #2
C++
1
2
3
void setFirstname (char* s) {strcpy_s(Firstname, sizeof(Firstname), s);};
void setSurname (char* s) {strcpy_s(Surname, sizeof(Surname), s);};
void setPatrname (char* s) {strcpy_s(Patrname, sizeof(Patrname), s);};
Попробуйте так, вместо посылки char s[]

Добавлено через 5 минут
А если целиком то так. Вы подаете константный объект. Следовательно либо так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
#include <conio.h>
#include <string.h>
#include <stdio.h>
 
class Person
{
private:
char Firstname [15];
char Surname [15];
char Patrname [15];
public:
//Функции записи (сеттеры)
void setFirstname (char* s) {strcpy_s(Firstname, sizeof(Firstname), s);};
void setSurname (char* s) {strcpy_s(Surname, sizeof(Surname), s);};
void setPatrname (char* s) {strcpy_s(Patrname, sizeof(Patrname), s);};
//Функции чтения (геттеры)
char * getFirstname () {return Firstname;};
char * getSurname () {return Surname;};
char * getPatrname () {return Patrname;};
//Конструкторы: по умолчанию, с параметрами, копирующий соответственно
Person ();
Person (char Fn[], char Sn[], char Pn[]);
Person (const Person &obj);
};
 
Person::Person() {Firstname[0] = Surname [0] = Patrname [0] = '\x00';}
 
Person::Person(char Fn[], char Sn[], char Pn[])
{
setFirstname(Fn);
setSurname(Sn);
setPatrname(Pn);
}
 
Person::Person(const Person &obj)
{
setFirstname(const_cast<char*>(obj.Firstname));
setSurname(const_cast<char*>(obj.Surname));
setPatrname(const_cast<char*>(obj.Patrname));
}
 
int main()
{
Person p1;
Person *pp2 = new Person();
int size1 = sizeof (p1);
int size2 = sizeof (*pp2);
printf("Memory for object p1: %d.\nMemory for object pp2: %d.",size1, size2);
p1.setFirstname("A");
p1.setSurname("K");
p1.setPatrname("S");
pp2->setFirstname("E");
pp2->setSurname("K");
pp2->setPatrname("G");
getch();
return 0;
}
Либо меняем в самом классе параметр:
C++
1
2
3
void setFirstname (const char* s) {strcpy_s(Firstname, sizeof(Firstname), s);};
void setSurname (const char* s) {strcpy_s(Surname, sizeof(Surname), s);};
void setPatrname (const char* s) {strcpy_s(Patrname, sizeof(Patrname), s);};
Второй вариант имхо логичнее.
kowandser
0 / 0 / 0
Регистрация: 01.10.2010
Сообщений: 50
08.10.2010, 00:05  [ТС]     Ошибка в копирующем конструкторе #3
void setFirstname (const char* s) {strcpy_s(Firstname, sizeof(Firstname), s);};

Какой смысл (const char* s)???
ForEveR
Модератор
Эксперт C++
 Аватар для ForEveR
7927 / 4709 / 318
Регистрация: 24.06.2010
Сообщений: 10,524
Завершенные тесты: 3
08.10.2010, 00:16     Ошибка в копирующем конструкторе #4
kowandser, А какой смысл не const char*? Вы изменяете строку которую подаете в этой функции? Нет. Следовательно почему бы не константной?
П.С.
А еще потому как у вас передача в копи-конструктор обязана быть const ссылкой на объект. Следовательно, все поля объекта этого класса константные.
Yandex
Объявления
08.10.2010, 00:16     Ошибка в копирующем конструкторе
Ответ Создать тему
Опции темы

Текущее время: 12:32. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru