функция не возвращает результат=(

[Chica Bond]

Добрый вечер. у меня такая проблема. мне нужно перегрузить функцию для моего класса:

numb numb::operator +(const numb &p)const
{
        numb res;
        res.add(4);
        return res;
}

но эта функция не возвращает результат. я понимаю, что это локальная переменная, но не очень разбираюсь со всем этим... пробывала объявлять res как глобальную и как статическую, но компилятор всё-равно выдаёт ошибку. работат только в том случае если убрать деструктор, но он мне нужен... вот уже и не знаю что делать. помогите пжлс) заранее спасибо!))

[X-Zef]

Как я понял res - это не переменная а экземпляр класса - numb. И если можно полностью выложи объявление класса.

[p51x]

numb numb::operator +(const numb &p)const
{
        numb res;
        res.add(4);
        return res;
}

Что за бред? Зачем здесь еще локальная переменная? У вас оператор как вызывается фактически?

[c++ learner]

Я не пойму а зачем тогда р – по ссылке передается в тело если оно не используется, и потом кто вас столько const’ов заставил написать?)

	object operator+(object &p) 
	{
		object temp;
		temp.x=(x+p.x);
		temp.y=(y+p.y);
		temp.z=(z+p.z);
		return temp;
	}

[Гром]

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

res.add(4);

только для отладки годится, а в реальности надо использовать p.
Так, и еще секунду - а как насчет

numb numb::operator +(const numb &p)const
{
        numb res = *this;
        res.add(4);
        return res;
}

?

и потом кто вас столько const’ов заставил написать?

А вот как раз на const'ы наезжать не надо. Их по-хорошему надо ставить везде, где только это возможно и не противоречит логике кода. Спросите, к примеру, у Скотта Мейерса.
Поясняю на конкретном примере.
const numb& res
гарантирует, что правый операнд оператора + не будет изменен внутри тела этого оператора (вы же наверняка удивитесь, если после выполнения операции 2+2 у вас то 2, что справа стало не 2, а 123,5?)
Объявление функции оператора как const гарантирует, что и левый операнд не будет изменен внутри тела оператора (левым операндом выступает сам объект класса).

[Chica Bond]

res.add(4) я действительно использую просто для проверки как работает функция. на самом деле этот опрератор предназначен для сложения двух объектов класса, т.е.

int main()
{
numb a, b, c;
cin>>a>>b;
c=a+b;
return 0;
}

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

#include <iostream>
using namespace std;
struct list2
{
	int info;
	list2 *next;
	list2 *prev;
};

class numb
{
	friend class list2_it;
protected:
	list2 *first;
public:
	numb();
	~numb();
	numb(const numb &p);
	void add(int x);
	friend list2 *end_list(const numb &p);//поиск конца списка
	//арифметические операции:
	numb &operator =(const numb &p);
	numb operator +(const numb &p)const;
	numb &operator +=(const numb &p);
//и т.д.
};

если в res скопировать *this то всё равно программа не работает, она вместо "4" печатает "-57098"(какое-то такое число....)
операторы присваивания и копирования работаю правильно. если убрать деструктор в классе, то всё работает замечательно, вот поэтому я и решила что функция не может вернуть результат, так как после её работы он удаляется... а как исправить не знаю

[Chica Bond]

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

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

вовсе нет....

[AJlekceu]

А нельзя ли увидеть описание конструктора копирования и функции add?

[Chica Bond]

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

А нельзя ли увидеть описание конструктора копирования и функции add?

конструктор копирования:

numb::numb(const numb &p)
{
	first=p.first;
}

функция add здесь вапще просто для виду, чтобы как-то рез поменять))
вот её код:

void numb::add(int x)
{
	list2 *q=new list2;
	q->info=x;
	q->next=first;
	q->prev=NULL;
	if (first)
		first->prev=q;
	first=q;
}

[Гром]

Ну тогда все ясно. Вы же просто копируете указатель на область памяти, в которой располагаются данные из другого экземпляра класса. А что у нас происходит здесь?

numb numb::operator +(const numb &p)const
{
        numb res;
        res.add(4);
        return res;
}

Создается экземпляр класса numb, в него добавляется один узел, а затем происходит копирование с помощью operator= в экземпляр c. И затем память, выделенная под локальный объект освобождается. И теперь указатель c.first ссылается на невалидный мусор.
Сделайте конструктор копирования и operator= с полным копированием всей информации, примерно так:

numb::numb(const numb &p)
{
first = new list2;
first -> prev = 0;
first -> info = p.first -> info;
list2 *curr = first, *currp = p.first;
while (currp -> next)
 {
 list2* node = new list2;
 node -> prev = curr;
 node -> next = 0;
 node -> info = currp -> next -> info;
 curr -> next = node;
 curr = node;
 currp = currp -> next;
 }
}

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