Переопределение операторов

@BullDogoff · Регистрация: 16.05.2009

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

Доброго времени суток..
В древние времена, когда ещё жили мамонты, ковырял я потихоньку плюсы.. Потом был долгий перерыв, и, как итог - забыл всё нафиг

Теперь вспоминается, но до того медленно и печально..
Ситуация такая - описываю класс точки (Point). Переопределил приравнивание, пытаюсь переопределить сложение - ругается..
Листинг:

Point.h

C++

//Îïðåäåëåíèå êëàññà, îïèñûâàþùåãî òî÷êó
 
class Point
{
public:
    Point operator=(Point);                    //Îïåðàòîð ïðèñâàèâàíèÿ
    Point operator+(Point);                    //Îïåðàòîð ñëîæåíèÿ
    
    Point();                                   //Êîíñòðóêòîð ïî óìîë÷àíèþ
    Point(Point&);                             //Êîíñòðóêòîð êîïèè
    Point(double, double, double);             //Êîíñòðóêòîð ïðîèçâîëüíîé òî÷êè
    double x, y, z;                            //Êîîðäèíàòû äîñòóïíû êëèåíòàì êëàññà
};

Point.cpp
Конструкторы (три штуки) опустил..

C++

//Описание функций-элементов класса Point
#include "point.h"
 
//Здесь были конструкторы
 
Point Point::operator=(Point i)
{
    x = i.x;
    y = i.y;
    z = i.z;
 
    return *this;
}
 
Point Point::operator+(Point i)
{
    Point q;
 
//Здесь собственно код сложения координат двух точек и приравнивание q результату
    
    return q;
}

Матерится на строку 17 в Point.cpp такими словами:
"error C2668: 'Point::Point' : ambiguous call to overloaded function"

Пространно, конечно, зато полная картина бедствия.. Скажите что-нибудь умное

З.Ы.: Я в курсе, что всё это уже написано и присутствует в стандартных хедерах, просто охота вручную.. ВотЪ!

Evg · 16.05.2009, 11:58

У меня съело. У тебя что за компилятор?

@~~rrrFer~~ · 16.05.2009, 11:59

C++

class Point{
    double x, y, z;   
public:
    Point & operator = (Point &i);                    //Оператор присваивания
    Point & operator + (Point &i);                    //Оператор сложения
    Point(){};                                   //Конструктор по умолчанию
    Point(Point&){};                             //Конструктор копии
    Point(double, double, double){};             //Конструктор произвольной точки
};
 
Point & Point::operator=(Point &i){
    x = i.x;
    y = i.y;
    z = i.z;
    return *this;
}
Point & Point::operator+(Point &i){
    Point *q=new Point();   
    return *q;
}

а вообще и так работает вроде

Evg · 16.05.2009, 12:02

Как вариант можешь попробовать агрументы операторов делать в виде ссылок (что более правильно)

@BullDogoff · 16.05.2009, 12:20 **[ТС]**

Компилятор MS Visual C++ 5.0

Попробую по ссылке, по указателю.. Увидим..

Evg · 16.05.2009, 12:35

5-го нет, но 6-ой тоже съел
Когда ты пишешь операторы с аргументом по значению (т.е. без ссылки), то в моменте передачи аргумента делается копия объекта (т.е. вызывается copy-конструктор). Покажи на всякий случай код конструкторов, может там ухитрился накосячить?

@BullDogoff · 16.05.2009, 12:39 **[ТС]**

Попробовал.. Матерится, скотина..
Пробовал объявлять внешнюю функцию сложения (координаты-то public) и вызывать её из тела переопределения оператора - всё равно ругается.. На то же самое объявление, только в другом месте.. Чудеса бывают? По-моему - нет..

Конструкторы:

C++

Point::Point()
{
    x = y = z = 0;
}
 
Point::Point(Point& i)
{
    x = i.x;
    y = i.y;
    z = i.z;
}
 
Point::Point(double a = 0, double b = 0, double c = 0)
{
    x = a;
    y = b;
    z = c;
}

Все три работают как надо, проверял..

З.Ы.: А если я конструктор копии не переопределял, как он аргумент по значению передавать будет?.. Или я чего-то не догнал?

Evg · 16.05.2009, 13:00

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

З.Ы.: А если я конструктор копии не переопределял, как он аргумент по значению передавать будет?.. Или я чего-то не догнал?

Если ты не создал, то компилятор сам создаст default copy constructor, в котором создаст код для копирования всех полей

Что-то у тебя какое-то шаманство творится. Выложи пожалуйста полные версии твоих исходников. Может что-то за кадром осталось?

Добавлено через 1 минуту 55 секунд
И напиши, как ты проект создавал

@BullDogoff · 16.05.2009, 13:24 **[ТС]**

Вот исходники полностью - три файла..

point.h

C++

//Îïðåäåëåíèå êëàññà, îïèñûâàþùåãî òî÷êó
 
class Point
{
public:
    Point operator=(Point);                    //Îïåðàòîð ïðèñâàèâàíèÿ
    Point operator+(Point);                    //Îïåðàòîð ñëîæåíèÿ
    
    Point();                                   //Êîíñòðóêòîð ïî óìîë÷àíèþ
    Point(Point&);                             //Êîíñòðóêòîð êîïèè
    Point(double, double, double);             //Êîíñòðóêòîð ïðîèçâîëüíîé òî÷êè
    double x, y, z;                            //Êîîðäèíàòû äîñòóïíû êëèåíòàì êëàññà
};

point.cpp

C++

//Îïèñàíèå ôóíêöèé-ýëåìåíòîâ êëàññà Point
#include "point.h"
 
///////////////////////////////////////////////////////////////////////////////////
//Êîíñòðóêòîðû
///////////////////////////////////////////////////////////////////////////////////
 
Point::Point()
{
    x = y = z = 0;
}
 
Point::Point(Point& i)
{
    x = i.x;
    y = i.y;
    z = i.z;
}
 
Point::Point(double a = 0, double b = 0, double c = 0)
{
    x = a;
    y = b;
    z = c;
}
 
///////////////////////////////////////////////////////////////////////////////////
//Îïåðàòîðû
///////////////////////////////////////////////////////////////////////////////////
 
Point Point::operator=(Point i)
{
    x = i.x;
    y = i.y;
    z = i.z;
 
    return *this;
}
 
Point Point::operator+(Point i)
{
    Point* q = new Point();
 
//Собственно здесь ничего нет, но оно обязательно там появится, как только объявлю экземпляр класса
    
    return *q;
}

main.cpp

C++

#include <iostream.h>
#include <iomanip.h>
#include "point.h"
 
 
main()
{
    Point i, k, l;
 
    k.x = 1;
    k.y = 2;
    k.z = 3;
 
    cout<<"i("<<i.x<<", "<<i.y<<", "<<i.z<<");\n";
    cout<<"k("<<k.x<<", "<<k.y<<", "<<k.z<<");\n\n";
 
    i = k;
 
    cout<<"i("<<i.x<<", "<<i.y<<", "<<i.z<<");\n";
    cout<<"k("<<k.x<<", "<<k.y<<", "<<k.z<<");\n\n";
 
    return 0;
}

Проект создавал как полагается, New->Project->Win32 Console Application.
В функции переопределения передавал значение, ссылку, указатель - ноль эмоций..
Ругается на любое объявление экземпляра класса в функциях-элементах, в функциях, вызываемых из оных..
Статическое выделение памяти, динамическое - пофигу.. Всё чудесатее и чудесатее, блин..

Evg · 16.05.2009, 13:44

У тебя возникает неопределённость между первым и третьим конструкторами. В третьем конструкторе у первого параметра убери дефолтное значение

Добавлено через 11 минут 7 секунд
И вообще, дефолтные значения параметров нужно описывать внутри класса. В противном случае подцепив один лишь point.h, у тебя не будет работать конструктор с одним или двумя параметрами (т.к. внутри класса описан конструктор с тремя параметрами). Ну и сами x,y,z надо упрятать в private (или protected) часть

@BullDogoff · 16.05.2009, 13:59 **[ТС]**

Не допёр, спасибо

Всё робит..

Координаты в private не стал прятать вполне осознанно, просто каждый раз вызывать функцию для доступа к ним как-то.. Муторно

Насчёт дефолтных значений внутри класса поясни, пожалуйста..
Конструкцию типа:

C++

class T
{
    int x = 0;
};

использовать вроде нельзя..

Evg · 16.05.2009, 14:15

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

Координаты в private не стал прятать вполне осознанно, просто каждый раз вызывать функцию для доступа к ним как-то.. Муторно

Как бы это сказать... Не по пацански это

К тому же плагается, что если ты завёл объект типа point, то ты и всё остальное будешь делать через этот объект. Т.е. получается у тебя некая абстракция, с которой координаты по отдельности ты берёшь только в крайних случаях. Идеальным вариантом было бы разрешить читать напрямую, но записывать только через метод, только хз как это делается

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

Насчёт дефолтных значений внутри класса поясни, пожалуйста..
Конструкцию типа:

C++

class T
{
    int x = 0;
};

использовать вроде нельзя..

Внутри класса нельзя писать никаких дефолтных значений. Все дефолтные значения должны инициализироваться внутри конструкторов

@BullDogoff · 16.05.2009, 15:02 **[ТС]**

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

Идеальным вариантом было бы разрешить читать напрямую, но записывать только через метод, только хз как это делается

Вот-вот, это было бы как раз то, что мне надо.. Мож где и накопаю, как это сделать..
Просто когда я доберусь до стадии Объекта (class Object), а он состоит из Массы (class Mass) и двух Векторов (class Vector)..

А каждый Вектор из двух Точек..

Иной раз на улицу выходишь и думаешь - вот, блин, и ведь без всяких 3D-ускорителей

Ладно, флужу уже..

З.Ы.: А может, возьмусь как-нибудь - и Life3D напишу.. ВооотЪ..

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	16.05.2009, 13:44	10
	У тебя возникает неопределённость между первым и третьим конструкторами. В третьем конструкторе у первого параметра убери дефолтное значение Добавлено через 11 минут 7 секунд И вообще, дефолтные значения параметров нужно описывать внутри класса. В противном случае подцепив один лишь point.h, у тебя не будет работать конструктор с одним или двумя параметрами (т.к. внутри класса описан конструктор с тремя параметрами). Ну и сами x,y,z надо упрятать в private (или protected) часть 1

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	16.05.2009, 11:58	2
	У меня съело. У тебя что за компилятор? 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	16.05.2009, 12:02	4
	Как вариант можешь попробовать агрументы операторов делать в виде ссылок (что более правильно) 0

@BullDogoff 10 / 10 / 1 Регистрация: 16.05.2009 Сообщений: 30
	16.05.2009, 12:20 [ТС]	5
	Компилятор MS Visual C++ 5.0 Попробую по ссылке, по указателю.. Увидим.. 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	16.05.2009, 12:35	6
	5-го нет, но 6-ой тоже съел Когда ты пишешь операторы с аргументом по значению (т.е. без ссылки), то в моменте передачи аргумента делается копия объекта (т.е. вызывается copy-конструктор). Покажи на всякий случай код конструкторов, может там ухитрился накосячить? 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	16.05.2009, 13:00	8
	Сообщение от BullDogoff З.Ы.: А если я конструктор копии не переопределял, как он аргумент по значению передавать будет?.. Или я чего-то не догнал? Если ты не создал, то компилятор сам создаст default copy constructor, в котором создаст код для копирования всех полей Что-то у тебя какое-то шаманство творится. Выложи пожалуйста полные версии твоих исходников. Может что-то за кадром осталось? Добавлено через 1 минуту 55 секунд И напиши, как ты проект создавал 1

@BullDogoff 10 / 10 / 1 Регистрация: 16.05.2009 Сообщений: 30
	16.05.2009, 15:02 [ТС]	13
	Сообщение от Evg Идеальным вариантом было бы разрешить читать напрямую, но записывать только через метод, только хз как это делается Вот-вот, это было бы как раз то, что мне надо.. Мож где и накопаю, как это сделать.. Просто когда я доберусь до стадии Объекта (class Object), а он состоит из Массы (class Mass) и двух Векторов (class Vector).. А каждый Вектор из двух Точек.. Иной раз на улицу выходишь и думаешь - вот, блин, и ведь без всяких 3D-ускорителей Ладно, флужу уже.. З.Ы.: А может, возьмусь как-нибудь - и Life3D напишу.. ВооотЪ.. 0