Версия для печати темы Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > перегрузка бинарных операторов

Автор: skyboy 16.4.2007, 22:00

есть класс. имя, правда, у него немного "неадекватное"  Код class charList {     private: int k;     public: charList(void) {                 this->k= 0;             }     public: charList(int _k) {                 this->k= _k;             }     public: int setK(const int _k) {         this->k= _k;         return this->k;     }     public: int getK(void) {         return this->k;             }     public: charList operator=(charList another) {                 this->setK(another.getK());                 return *this;             }     public: charList operator+(charList another) {                 charList result;                 result.setK(this->getK() + another.getK());                 return result;             }     public: charList operator+(charList* another) {                 charList result;                 result.setK(this->getK() + another->getK());                 return result;             }     public: ~charList()             {                 cout<<"i have been destroyed!("<<this->getK()<<")\n";             } }; создаю объект класса: Код charList* zero= new charList(0); строка  Код zero + zero; вызывает сообщение об ошибке: Цитата error C2110: cannot add two pointers Ну, ладно- думаю я,- получи. И делаю ход конем: Код zero= zero->operator +(zero); на оператор + не ругается(!), зато сообщает Цитата error C2679: binary '=' : no operator defined which takes a right-hand operand of type 'class charList' (or there is no acceptable conversion) Будьте добры, поясните, что я не понимаю в логике перегрузки, что допускаю такие ошибки? ------ VC++ 6.0

Автор: archimed7592 16.4.2007, 22:06

Цитата(skyboy @  16.4.2007,  22:00 )      zero= zero->operator +(zero); operator+ возвращает НЕуказатель, а ты пытаешься засунуть это в указатель... нужно делать *zero + *zero

Автор: powerfox 16.4.2007, 22:12

Цитата(skyboy @  16.4.2007,  23:00 ) создаю объект класса: Выделить всёкод C++ 1:      charList* zero= new charList(0); строка  Выделить всёкод C++ 1:      zero + zero; вызывает сообщение об ошибке: Цитата error C2110: cannot add two pointers Гм... А зачем складывать указатели в таком контексте? У тебя и сложение элементов и указателей одинаковы... Цитата(skyboy @  16.4.2007,  23:00 )      zero= zero->operator +(zero); zero у тебя указатель, а не твой объект. Вот и ругается. Добавлено через 6 минут и 26 секунд Цитата(powerfox @  16.4.2007,  23:12 ) Гм... А зачем складывать указатели в таком контексте? У тебя и сложение элементов и указателей одинаковы... Какой-то бред сморозил.

Автор: skyboy 16.4.2007, 22:27

Цитата(archimed7592 @  16.4.2007,  21:06 ) operator+ возвращает НЕуказатель, а ты пытаешься засунуть это в указатель... Цитата(powerfox @  16.4.2007,  21:12 ) zero у тебя указатель, а не твой объект. Вот и ругается. а как же перегрузка  оператора присвоения: Цитата(skyboy @  16.4.2007,  21:00 ) public: charList operator=(charList another) ? идет лесом? так делать нельзя?  Цитата(powerfox @  16.4.2007,  21:12 ) А зачем складывать указатели в таком контексте? У тебя и сложение элементов и указателей одинаковы... не совсем понял мысль. я перегрузил оператор +. по моей внутренней логике, после перегрузки object1+object2 должно складывать не указатели, а вызывать всё тот же object1->operator+(object2). Только неявно. Но, почему-то, не вызывает. Значит моё виденье работы перегрузки операторов неверно.

Автор: archimed7592 16.4.2007, 22:27

Цитата(skyboy @  16.4.2007,  22:00 ) на оператор + не ругается(!) и не должен - это вполне себе нормальное имя ф-ции ;) зы. в идеале, толку от *zero + *zero нету - они складываются, но результат игнорируется... пиши *a = *b + *c или *a+=*b

Автор: skyboy 16.4.2007, 22:28

Цитата(powerfox @  16.4.2007,  21:12 ) Какой-то бред сморозил.  сам сказал, сам откоммнетился   Добавлено через 2 минуты и 32 секунды Цитата(archimed7592 @  16.4.2007,  21:27 ) в идеале, толку от *zero + *zero нету это я отделил ошибку касательно перегрузки оператора присвоения от ошибки перегрузки оператора сложения  вычленил, так сказать  да и пример, если не заметно   соверщенно учебный...

Автор: archimed7592 16.4.2007, 22:31

Цитата(skyboy @  16.4.2007,  22:27 ) а как же перегрузка  оператора присвоения: Цитата(skyboy @  16.4.2007,  21:00 Найти цитируемый пост) public: charList operator=(charList another) ? идет лесом? так делать нельзя?  ну и что тебе эта перегрузка(делающая двойную работу, кстати) даёт? возможно писать Код charList a, b; a = b; не больше... можешь сделать ещё charList operator=(charList *another), тогда можно будет писать Код charList a, *zero = new charList (0); a = zero; Добавлено через 58 секунд Цитата(archimed7592 @  16.4.2007,  22:31 ) делающая двойную работу, кстати тройную!

Автор: skyboy 16.4.2007, 22:34

Цитата(archimed7592 @  16.4.2007,  21:06 ) нужно делать *zero + *zero  поясни, пожалуйста, почему. у меня ведь есть вариант перегрузки оператора + для правого операнда  типа charList(без указателя-звёздочки). А слева, как я понял, без разыменовывания не обойтись, да? Добавлено через 2 минуты и 24 секунды Цитата(archimed7592 @  16.4.2007,  21:31 ) делающая двойную работу, кстати вот здесь подробнее. что, неявно вызывается несколько раз? или что? Цитата(archimed7592 @  16.4.2007,  21:31 ) ну и что тебе эта перегрузка (....) даёт? Цыц! Я изучаю перегрузку на простых примерах. Сам её(перегрузку операторов), кстати, считаю злом - интерфейсы куда "прозрачнее" и ортодоксальнее. да и с С++ знаком слабо...

Автор: powerfox 16.4.2007, 22:38

skyboy, перегрузка и разыменование не есть одно и то же.

Автор: archimed7592 16.4.2007, 22:39

Цитата(skyboy @  16.4.2007,  22:34 ) А слева, как я понял, без разыменовывания не обойтись, да?  да... можно *zero + zero... про operator =: писать нужно так Код myClass &operator = (const myClass &another) как и для operator + Код charList operator+(const charList &another) const а ты из java к нам пожаловал видимо  Добавлено @ 22:41 Цитата(skyboy @  16.4.2007,  22:34 ) что, неявно вызывается несколько раз? конструктор копирования  type var - передача по значению type &var - по ссылке const type &var - по ссылке (объект изменить нельзя) Добавлено @ 22:42 Цитата(skyboy @  16.4.2007,  22:34 ) интерфейсы куда "прозрачнее" и ортодоксальнее. ну и пользуй наздоровье... они медленее правда, ну да ладно

Автор: skyboy 16.4.2007, 22:46

Цитата(powerfox @  16.4.2007,  21:38 ) skyboy, перегрузка и разыменование не есть одно и то же. ээээ.... это ты к чему? спасибо, я уже не мальчик в курсе, что не одно и то же  а идею не внял.

Автор: powerfox 16.4.2007, 22:53

Цитата(skyboy @  16.4.2007,  23:46 ) ээээ.... это ты к чему? спасибо, я уже не мальчик в курсе, что не одно и то же smile а идею не внял.  Цитата(skyboy @  16.4.2007,  23:00 )     public: charList operator+(charList* another) {                 charList result;                 result.setK(this->getK() + another->getK());                 return result; К тому, что этот код не означает, что будет возможно делать: Код charList* el1; charList summ; charList el; summ = el + el1;

Автор: skyboy 16.4.2007, 22:54

Цитата(archimed7592 @  16.4.2007,  21:39 ) а ты из java к нам пожаловал видимо льстишь  с Java только вторую неделю знакомлюсь ))) РНР, Delphi...  Цитата(archimed7592 @  16.4.2007,  21:39 ) ну и пользуй наздоровье...  я что - я бы с радостью. Брата вот выручать приходится   Цитата(archimed7592 @  16.4.2007,  21:39 ) const charList &another const charList &another и charList* another - тут имеется радикальная разница? А вот при использовании такой штуки: Цитата(archimed7592 @  16.4.2007,  21:39 ) charList operator+(const charList &another) const объект-результат создастся на стеке, да? если да, то вариант передачи правого операнда const another по ссылке меня удивляет. А как же случай: object1 + (object2 + object3)?  при выполнении object2 + object3 будет создан объект(а не указатель на него). Получается при работе object1 + result[object2 + object3] будет сложение указателя на класс(charList* object1) и объекта-значения на стеке(charList result), а перегруженного оператора + нет. Или - и не надо? я прошу даже не столько коррекции кода(мне потом сложнее код писать придется), а именно растолковывания, что я неверно разумею  Если придется растолковывать много - пожалуйста, ткните в конкретное(чтоб за ночь осилить) чтиво. Спасибо

Автор: Daevaorn 16.4.2007, 22:57

Твой оператор работает с объектами, не с указателями. Что тут не понятного? Складывать два указателя ка-то не логично в данном случае

Автор: skyboy 16.4.2007, 22:58

powerfox, почему не пройдет?  тип параметра в сигнатуре перегрузки оператора и тип правого операнда не совпадают разве? Добавлено через 1 минуту и 27 секунд Цитата(Daevaorn @  16.4.2007,  21:57 ) Твой оператор работает с объектами, не с указателями. Что тут не понятного? Складывать два указателя ка-то не логично в данном случае угу. наконец-то понял, спасибо   а ведь и раньше то же самое говорили.... )))

Автор: powerfox 16.4.2007, 23:00

Кстати, public писать каждый раз необязательно (public - метка секции, а не функции). Вот, что будет работать с твоим кодом: Код charList el1; charList summ; charList el2; charList* el_ptr = new charList(0); summ = el1 + el_ptr; // Ошибка! summ = el1.operator+(el_ptr); //charList operator+(charList* another)

Автор: skyboy 16.4.2007, 23:05

powerfox, ок. Спасибо. понял  втроем доносили свет знаний... спасибо  Сейчас попробую подкорректировать исходный код

Автор: powerfox 16.4.2007, 23:12

Цитата(skyboy @  17.4.2007,  00:05 ) powerfox, ок. Спасибо. понял smile втроем доносили свет знаний... спасибо smile Не, стой. Извини, я опять бред написал...

Автор: Xenon 16.4.2007, 23:13

Ну и жесть, вообще, если нормально, то так: Код class charList { private:     int m_k; public:      int setK(const int k)      {         m_k = k;         return m_k;     }     int getK()     {         return m_k;     }     charList& operator= (const charList& another)     {         m_k = another.m_k;         return *this;     }     charList operator+ (const charList& another)      {         return (m_k + another.m_k);     }     charList():m_k(0) {}     charList(int k):m_k(k) {}     charList(const charList& another):m_k(another.m_k) {}     ~charList()     {         std::cout << "i have been destroyed!(" << m_k << ")\n";     } };

Автор: powerfox 16.4.2007, 23:13

skyboy, про неодназначность я ошибся. Остальное написал верно.

Автор: skyboy 16.4.2007, 23:26

Цитата(Xenon @  16.4.2007,  22:13 ) charList operator+ (const charList& another) а как быть с случаем Код (charList*)object1 + ((charList*)object2 + (charList*)object3) ? Ведь результат последнего сложения - разве "const charList&"? а не без ссылки разве? усиленно путаю указатели и ссылки )))

Автор: Xenon 17.4.2007, 00:01

Без ссылки. Указатели рекдо складывают и вообще лучше не переопределять арифметики указателей (вычитание указателей необходимо для вычисления количества элементов в том же массива). Если у тебя будут указатели на объекты, то просто разыменуй и все - это лучше чем перегрузить операцию сложения для указателей. Ссылка - грубо говорю синтаксическое упрощение указателя - звездочка сама пририсовывается (разыменовывание) и при передаче фактических параметров не надо писать &.

Автор: archimed7592 17.4.2007, 01:16

Цитата(skyboy @  16.4.2007,  22:54 ) const charList &another и charList* another - тут имеется радикальная разница? в синтаксисе разница радикальная. в сущности: иногда ссылка есть указатель, иногда - синоним имени переменной. когда какую сущность выбрать решает компилятор из соображений эффективности. маленький пример: Код #include <cstddef> #include <cstdlib> #include <ctime> #include <iostream> #include <ostream> void recursive_function (int step, const int &maxStep, int &someOtherReference) {     if (step < maxStep)     {         for (int i = 0; i < 5; ++i)             if (std::rand () % 2)             {                 int localVar = rand ();                 recursive_function(step + 1, maxStep, localVar);                 std::cout << localVar << std::endl;             }             else                 someOtherReference = rand ();     } } int main( int argc, char *argv[] ) {     std::srand ( std::time(NULL) );;     int max_step;     std::cin >> max_step;     int mainVar = 0;     recursive_function(0, max_step, mainVar); // maxStep скорее всего будет синонимом max_step т. к. во всей рекурсии maxStep имеет одно значение // mainVar передастся как ссылка-указатель т. к. на некоторых путях рекурсии ссылка someOtherReference принимает иное значение     int mainRef = &mainVar; // скорее всего будет являться синонимом т. к. смысла большого делать указатель нету.     std::cout << mainRef << std::endl;          return 0; }

Автор: Xenon 17.4.2007, 01:39

Да, как подметил archimed7592 не обязательно физически будет существовать объект "ссылка", которая ссылается на объект. И еще разница в констатности - у констнатной ссылки типа T, в отличии от констатного указателя, в роли иницилизатора может выступать lvalue типа T, а также не типа T (будет неявное преобразование) и временная переменная, то есть не lvalue (также может и T, а может и не T) Код void func(const int& var) {     std::cout << var << std::endl; } void func2(int& var) {     std::cout << var << std::endl; } int main(int argc, char* argv[]) {         int var = 10;     double var2 = 12.2;     func(var); //Гуд     func(var2); //Гуд - неявное преобразование     func2(var); //Гуд     func2(var2);//НеГуд     func(10); //Гуд - временная переменная     func(12.2); //Гуд - неявное преобразование + временная переменная     func2(10);//Негуд     func2(110.4);//НЕгуд     getchar();     return 0; }

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)