Перегрузка операторов, указатели

@Antnemo · Регистрация: 27.04.2017

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

Добрый день! Помогите пожалуйста разобраться с перегрузкой постинкремента. Не могу понять, почему при присваивании объекта заданного класса объекту другого класса с постинкрементом получается непонятное значение...

C++

    #include <iostream>
    using namespace std;
    typedef unsigned short int USH;
    
    class SimpleCircle
    {
        public:
            SimpleCircle(){itsRadius = new USH;*itsRadius = 0;
        }
            SimpleCircle(int); 
            ~SimpleCircle() { delete itsRadius;itsRadius = 0; }
            SimpleCircle(SimpleCircle &) ;
            
            const SimpleCircle &operator ++ ();
            const SimpleCircle operator ++ (int x);
            
            void SetRad(USH rad);
            USH GetRad();
    
        private:
            USH *itsRadius; 
            
        
    };
    
    SimpleCircle :: SimpleCircle(int x) //Перегрузка конструктора
    {
        itsRadius = new USH;
        *itsRadius = x;
        
    }
    
    SimpleCircle :: SimpleCircle(SimpleCircle &rhs) //Конструктор - копировщик
    {
        itsRadius = new USH;
        *itsRadius = rhs.GetRad();
        cout << "\ncopy constructor\n ";
    }
    
    const SimpleCircle &SimpleCircle :: operator ++ () //Преинкремент
    {
        ++ (*itsRadius);
        return (*this);
    }
    
     const SimpleCircle SimpleCircle :: operator ++ (int x) //Проблема с данной функцией
    {
        SimpleCircle temp(*this);
        ++(*itsRadius);
        return temp;
        
    }
    
    void SimpleCircle :: SetRad(USH rad)
    {
        *itsRadius = rad;
    }
    
    USH SimpleCircle :: GetRad()
    {
        return (*itsRadius);
    }
    
    int main()
    {
        SimpleCircle Circ;
        Circ.SetRad(3);
        ++ Circ;
        ++ Circ;
        cout << "\n Circ = " << Circ.GetRad();
        Circ ++ ;
        cout << "\n Circ = " << Circ.GetRad();
                
        SimpleCircle A;
        Circ ++; //Значение Circ = 7
        A = Circ++;//Присваивается значение А = 3840..Почему??, а Circ = 8
        A++;
        cout << "\n Circ = " << Circ.GetRad() << "\nAnd A = " << A.GetRad();
        
        return 0;
    }

@Dani · 27.04.2017, 15:49

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

A = Circ++;

Вызывается operator=, который просто скопирует указатель на USH из Circ++ в A. Потом объект Circ++ удалится и указатель будет не валидным.

@Antnemo · 27.04.2017, 15:59 **[ТС]**

По задумке необходимо, чтобы значение разыменованного указателя *itsRadius в объекте Circ копировалось в объект А без приращения, а сам *itsRadius в Circ увеличивался на 1. Как это реализовать?

MrGluck · 27.04.2017, 16:32

У вас в классе идёт работа с памятью, но при этом не объявлен явно конструктор копий и оператор копирующего присваивания.

@DemolitionMan · 27.04.2017, 16:36

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

typedef unsigned short int USH;

- что за тип такой? Либо int, либо short.

@Mournful Max · 27.04.2017, 16:46

DemolitionMan, short -- всего лишь модификатор целого типа (int). Сам тип может быть опущен, если есть хоть один из его модификаторов. То бишь, если написать unsigned short, то подразумевается unsigned short int.

@DemolitionMan · 27.04.2017, 16:51

Ну я не знаю, это наверное в другом компиляторе так. У меня в C++Builder 6: short - 16-битный тип данных;
int - 32-битный тип данных.

@Antnemo · 27.04.2017, 16:53 **[ТС]**

MrGluck, Извините за глупые вопросы, а вот это разве не явное объявление конструктора копий...

SimpleCircle(SimpleCircle &) ;//вот прототип конструктора копий

SimpleCircle :: SimpleCircle(SimpleCircle &rhs) //Конструктор - копий
{
itsRadius = new USH;
*itsRadius = rhs.GetRad();
cout << "\ncopy constructor\n ";
}

@Mournful Max · 27.04.2017, 16:54

DemolitionMan, short -- это не тип, а его модификатор. int не всегда имеет 32 бита. Он имеет ширину не меньше 16 бит.
Скачайте более современную IDE, и почитайте вотэтавот

MrGluck · 27.04.2017, 16:56

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

что за тип такой? Либо int, либо short.

На самом деле, short == short int. Просто полный тип можно не писать.

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

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

а вот это разве не явное объявление конструктора копий...

Да, это конструктор копий. Я его не приметил среди вашей простыни. Используйте теги форматирования кода.
Оператора присваивания всё равно нет, и именно это вызывает ошибку в вашем коде.

@Antnemo · 27.04.2017, 17:17 **[ТС]**

MrGluck, Спасибо за критику, первый вопрос на этом форуме..В следующий раз постараюсь с табуляцией.

Добавлено через 16 минут
MrGluck, Вы имеете ввиду добавить это?

C++

SimpleCircle &SimpleCircle :: operator = ( SimpleCircle &rhs )  
    {
        if (this == &rhs)
        return(*this);
        delete itsRadius;
        itsRadius = new USH;
        *itsRadius = rhs.GetRad();
        return (*this);
        
    }

MrGluck · 27.04.2017, 17:36

Antnemo, да, что-то типа того.

@DemolitionMan · 27.04.2017, 17:40

Мне, кстати, профессионалы тут не советовали работать с консолью в конструкторе, хотя и видно когда какой конструктор запскается.

@Antnemo · 27.04.2017, 18:00 **[ТС]**

После добавления оператора присваивания не работает присваивание с приращением..Что-то я запутался

C++

    #include <iostream>
    using namespace std;
    typedef unsigned short USH;
    
    class SimpleCircle
    {
        public:
            SimpleCircle(){itsRadius = new USH;*itsRadius = 0;
        }
            SimpleCircle(int); 
            ~SimpleCircle() { delete itsRadius;itsRadius = 0; }
            SimpleCircle(SimpleCircle &) ;
            
            SimpleCircle &operator = ( SimpleCircle &rhs ); 
            const SimpleCircle &operator ++ ();
            const SimpleCircle operator ++ (int x);
            
            
            void SetRad(USH rad);
            USH GetRad();
    
        private:
            USH *itsRadius; 
            
        
    };
    
    SimpleCircle :: SimpleCircle(int x) 
    {
        itsRadius = new USH;
        *itsRadius = x;
        
    }
    
    SimpleCircle :: SimpleCircle(SimpleCircle &rhs) 
    {
        itsRadius = new USH;
        *itsRadius = rhs.GetRad();
        cout << "\ncopy constructor\n ";
    }
    
    SimpleCircle &SimpleCircle :: operator = ( SimpleCircle &rhs )  
    {
        if (this == &rhs)
        return(*this);
        delete itsRadius;
        itsRadius = new USH;
        *itsRadius = rhs.GetRad();
        return (*this);
        
    }
    
    const SimpleCircle &SimpleCircle :: operator ++ () 
    {
        ++ (*itsRadius);
        return (*this);
    }
    
     const SimpleCircle SimpleCircle :: operator ++ (int x) 
    {
        SimpleCircle temp(*this);
        ++(*itsRadius);
        return temp;
        
    }
    
 
    void SimpleCircle :: SetRad(USH rad)
    {
        *itsRadius = rad;
    }
    
    USH SimpleCircle :: GetRad()
    {
        return (*itsRadius);
    }
    
    int main()
    {
        SimpleCircle Circ;
        Circ.SetRad(3);
        ++ Circ;
        ++ Circ;
        cout << "\n Circ = " << Circ.GetRad();
        Circ ++ ;
        cout << "\n Circ = " << Circ.GetRad();
                
        SimpleCircle A;
        Circ ++; //Значение  7
        //A = Circ++;//При попытке присваивания с приращением - ошибка..
        A = Circ;
        //A = ++Circ ;
        cout << "\n Circ = " << Circ.GetRad() << "\nAnd A = " << A.GetRad();
        
        return 0;
    }

@Dani · 27.04.2017, 18:03

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

: operator = ( SimpleCircle &rhs )

const SimpleCirle& rhs

@Antnemo · 27.04.2017, 18:44 **[ТС]**

Dani, спасибо, разобрался.
Косяк был где Вы сказали и тут:

USH GetRad();

Заработало с

C++

1	USH GetRad() const;

Еще вопрос, а если const стоит после функции, на что он указывает?

@Dani · 27.04.2017, 18:53

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

Еще вопрос, а если const стоит после функции, на что он указывает?

Если не совсем верно упростить, то это указывает на то, что метод не изменяет значений переменных-членов класса и не вызывает методов, которые могут это сделать.

MrGluck Форумчанин 8215 / 5045 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	27.04.2017, 16:56	10
	Сообщение от DemolitionMan что за тип такой? Либо int, либо short. На самом деле, `short` == `short int`. Просто полный тип можно не писать. Добавлено через 1 минуту Сообщение от Antnemo а вот это разве не явное объявление конструктора копий... Да, это конструктор копий. Я его не приметил среди вашей простыни. Используйте теги форматирования кода. Оператора присваивания всё равно нет, и именно это вызывает ошибку в вашем коде. 0

MrGluck Форумчанин 8215 / 5045 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	27.04.2017, 17:36	12
	Antnemo, да, что-то типа того. 0

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	27.04.2017, 17:40	13
	Мне, кстати, профессионалы тут не советовали работать с консолью в конструкторе, хотя и видно когда какой конструктор запскается. 0

@Dani 1405 / 647 / 135 Регистрация: 11.08.2011 Сообщений: 2,299 Записей в блоге: 2
	27.04.2017, 15:49	2
	Сообщение от Antnemo A = Circ++; Вызывается operator=, который просто скопирует указатель на USH из Circ++ в A. Потом объект Circ++ удалится и указатель будет не валидным. 0

@Antnemo 0 / 0 / 0 Регистрация: 27.04.2017 Сообщений: 6
	27.04.2017, 15:59 [ТС]	3
	По задумке необходимо, чтобы значение разыменованного указателя itsRadius в объекте Circ копировалось в объект А без приращения, а сам itsRadius в Circ увеличивался на 1. Как это реализовать? 0

MrGluck Форумчанин 8215 / 5045 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	27.04.2017, 16:32	4
	У вас в классе идёт работа с памятью, но при этом не объявлен явно конструктор копий и оператор копирующего присваивания. 0

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	27.04.2017, 16:36	5
	Сообщение от Antnemo typedef unsigned short int USH; - что за тип такой? Либо int, либо short. 0

@Mournful Max Велосипедист... 353 / 220 / 73 Регистрация: 15.12.2015 Сообщений: 785
	27.04.2017, 16:46	6
	DemolitionMan, `short` -- всего лишь модификатор целого типа (`int`). Сам тип может быть опущен, если есть хоть один из его модификаторов. То бишь, если написать `unsigned short`, то подразумевается `unsigned short int`. 1

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	27.04.2017, 16:51	7
	Ну я не знаю, это наверное в другом компиляторе так. У меня в C++Builder 6: short - 16-битный тип данных; int - 32-битный тип данных. 0

@Antnemo 0 / 0 / 0 Регистрация: 27.04.2017 Сообщений: 6
	27.04.2017, 16:53 [ТС]	8
	MrGluck, Извините за глупые вопросы, а вот это разве не явное объявление конструктора копий... SimpleCircle(SimpleCircle &) ;//вот прототип конструктора копий SimpleCircle :: SimpleCircle(SimpleCircle &rhs) //Конструктор - копий { itsRadius = new USH; *itsRadius = rhs.GetRad(); cout << "\ncopy constructor\n "; } 0

@Mournful Max Велосипедист... 353 / 220 / 73 Регистрация: 15.12.2015 Сообщений: 785
	27.04.2017, 16:54	9
	DemolitionMan, `short` -- это не тип, а его модификатор. `int` не всегда имеет 32 бита. Он имеет ширину не меньше 16 бит. Скачайте более современную IDE, и почитайте вотэтавот 0

@Dani 1405 / 647 / 135 Регистрация: 11.08.2011 Сообщений: 2,299 Записей в блоге: 2
	27.04.2017, 18:03	15
	Сообщение от Antnemo : operator = ( SimpleCircle &rhs ) const SimpleCirle& rhs 1

@Dani 1405 / 647 / 135 Регистрация: 11.08.2011 Сообщений: 2,299 Записей в блоге: 2
	27.04.2017, 18:53	17
	Сообщение от Antnemo Еще вопрос, а если const стоит после функции, на что он указывает? Если не совсем верно упростить, то это указывает на то, что метод не изменяет значений переменных-членов класса и не вызывает методов, которые могут это сделать. 0