VPF::Использование Clone( ) в конструкторе копирования

ANTON_AL

Дата 30.4.2007, 22:30 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 292
Регистрация: 27.1.2007

Репутация: нет
Всего: нет

Добрый день всем !!

Имею вот такой код...

Код


#include <conio.h>
#include <iostream>

using namespace std;




class Base
{
    public:
        Base( ){ }
        Base( const Base &rhs )
        {
            *this = rhs;
        }
        Base operator= ( const Base &rhs )
        {
            CleanUp( );
            this = rhs.Clone( );    //Error =(
            return rhs;
        }
        Base* Clone( )
        {
            //Cloning
        }


    protected:
        void CleanUp( ){/*Cleaning all Data in this object*/ }


    private:
        //Some data
};


void main( )
{
    Base Obj;


    getch( );
}

Он не компилруется, ругается так:
Error 1 error C2662: 'Base::Clone' : cannot convert 'this' pointer from 'const Base' to 'Base &'

И что самое непонятное, он ругается на this даже в том случае, если this не ставить, а просто написать что то типа этого

Код


rhs.Clone( );

В чем тут дело ??

mr.Anderson

Дата 30.4.2007, 22:33 (ссылка)

(нет голосов)

Загрузка ...

iOS Lead Developer

Профиль
Группа: Участник Клуба
Сообщений: 3374
Регистрация: 20.12.2004
Где: далеко

Репутация: нет
Всего: 128

ANTON_AL, скорее всего, так:

Код


(*this) = rhs.Clone();

--------------------

ANTON_AL

Дата 30.4.2007, 22:38 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 292
Регистрация: 27.1.2007

Репутация: нет
Всего: нет

mr.Anderson, метод Clone( ) возвращает указатель, так что так не выйдет. =(

Xenon

Дата 30.4.2007, 22:40 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Участник Клуба
Сообщений: 1529
Регистрация: 12.4.2006

Репутация: 11
Всего: 50

const нужно брать, чтобы просто Clone() работал, но вообще чего ты хочешь сделать?
У тебя вообще по-моему, должен бесконечный вызов operator = начаться

Это сообщение отредактировал(а) Xenon - 30.4.2007, 22:41

--------------------

Fazil6

Дата 30.4.2007, 22:42 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1653
Регистрация: 3.5.2006
Где: Минск

Репутация: 35
Всего: 60

Код


 Base* Clone( )

эта функция не может вызываться константным объектом типа Base , а в оператор = передается именно константа

archimed7592

Дата 30.4.2007, 22:45 (ссылка)

(нет голосов)

Загрузка ...

Архимед

Профиль
Группа: Завсегдатай
Сообщений: 2531
Регистрация: 12.6.2004
Где: Moscow

Репутация: 58
Всего: 93

ANTON_AL, Base* Clone( ) const...
само тело Clone в общем случае такое: return new myClass (*this);

--------------------

If you have an apple and I have an apple and we exchange apples then you and I will still each have one apple. But if you have an idea and I have an idea and we exchange these ideas, then each of us will have two ideas.
© George Bernard Shaw

ANTON_AL

Дата 30.4.2007, 22:46 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 292
Регистрация: 27.1.2007

Репутация: нет
Всего: нет

Метод Clone( ) будет использоваться в дальнейшем, вне этого класса, просто я вставил его в operator=, чтобы не повторять код два раза ( для Clone, и для Copy-Constructor/operator= )

А бесконечного вызова не будет, т.к. Clone возвращает указатель, а не объект. А указатели присваиваются безо всяких конструкторов... smile

Вот.

Xenon

Дата 30.4.2007, 22:46 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Участник Клуба
Сообщений: 1529
Регистрация: 12.4.2006

Репутация: 11
Всего: 50

А разве строчка

Код


this = rhs.Clone( );    //Error =(

Не будет ли в последствии вызывать Base operator= ( const Base &rhs ), а потом еще и еще?..

PS. Не заметил насчет указателя. Const тогда нужно переставить просто.

Это сообщение отредактировал(а) Xenon - 30.4.2007, 22:47

--------------------

Daevaorn

Дата 30.4.2007, 22:49 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2155
Регистрация: 29.11.2004
Где: Москва

Репутация: 51
Всего: 70

Цитата(Xenon @ 30.4.2007, 23:46

)

Не будет ли в последствии вызывать Base operator= ( const Base &rhs ), а потом еще и еще?..

не должна

Mayk

Дата 30.4.2007, 22:50 (ссылка)

(нет голосов)

Загрузка ...

^аВаТаР^ сообщение>>

Профиль
Группа: Участник
Сообщений: 2616
Регистрация: 22.5.2005
Где: за границей разум а

Репутация: 45
Всего: 134

Цитата(Xenon @ 1.5.2007, 02:46

)

Не будет ли в последствии вызывать Base operator= ( const Base &rhs ), а потом еще и еще?..

нет. this нельзя приравнивать.

--------------------

Здесь был кролик. Но его убили.
Человеки < кроликов, йа считаю.

ANTON_AL

Дата 30.4.2007, 22:52 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 292
Регистрация: 27.1.2007

Репутация: нет
Всего: нет

archimed7592,

Цитата

само тело Clone в общем случае такое: return new myClass (*this);

Для Clone( ) так не пойдет. Там надо хитро копировать, т.к. class Base - это просто для демонстрации проблемы, а вообще это у меня дерево-( class BoundingAreasTree ), которое нужно клонировать ... ну сами знаете как smile

const пробовал в конец добавить, не помогает, вместо старой ошибки выдаёт такую: Error 1 error C2106: '=' : left operand must be l-value

Добавлено через 3 минуты и 47 секунд
Mayk,

Цитата

нет. this нельзя приравнивать.

А почему ??

Это сообщение отредактировал(а) ANTON_AL - 30.4.2007, 22:54

Любитель

Дата 30.4.2007, 22:56 (ссылка)

(нет голосов)

Загрузка ...

Программист-романтик

Профиль
Группа: Комодератор
Сообщений: 3645
Регистрация: 21.5.2005
Где: Воронеж

Репутация: 24
Всего: 92

this изначально константный указатель. Подобный код в принципе невозможен. Напротив реализуем опреатор присваивания. При большрй лени -копи-конструктор можно релизить через него. Этот Clone тем более.

--------------------

http://oryolalex.mybrute.com

ANTON_AL

Дата 30.4.2007, 23:03 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 292
Регистрация: 27.1.2007

Репутация: нет
Всего: нет

Любитель, Согласен, сам только что так сделал уже. Спасибо.

Mayk

Дата 30.4.2007, 23:11 (ссылка)

(нет голосов)

Загрузка ...

^аВаТаР^ сообщение>>

Профиль
Группа: Участник
Сообщений: 2616
Регистрация: 22.5.2005
Где: за границей разум а

Репутация: 45
Всего: 134

Цитата(Любитель @ 1.5.2007, 02:56

)

this изначально константный указатель

[нудит]

Цитата(class.this (N2135))

In the body of a non-static (9.3) member function, the keyword this is a non-lvalue expression whose value is the
address of the object for which the function is called. The type of this in a member function of a class X is X*. If
the member function is declared const, the type of this is const X*, if the member function is declared volatile,
the type of this is volatile X*, and if the member function is declared const volatile, the type of this is const
volatile X*.

то есть он не является константным указателем. this просто не является lvalue.
[/нудит]

Это сообщение отредактировал(а) Mayk - 30.4.2007, 23:11

--------------------

Здесь был кролик. Но его убили.
Человеки < кроликов, йа считаю.

Любитель

Дата 30.4.2007, 23:15 (ссылка)

(нет голосов)

Загрузка ...

Программист-романтик

Профиль
Группа: Комодератор
Сообщений: 3645
Регистрация: 21.5.2005
Где: Воронеж

Репутация: 24
Всего: 92

Mayk, хватит придираться! Понятно, что не lavalue - this подставляется как значение в вызов методов. Нигде, конечно же, не хранится.

К слову - есть у нас препод (точнее преподша - вообще неплохая) в универе. Про дельфийский self объясняла как про специальное поле...

--------------------

http://oryolalex.mybrute.com

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »