Оператор присваивания

@DmmiTrY · Регистрация: 18.11.2015

Здравствуйте!
Есть класс, в котором перегружены операции сложения и присваивания:

C++

1 2	void operator = (Matrix&); Matrix operator + (Matrix&);

При сложении с присваиванием(a,b и c - объекты этого класса):

C++

1	a = b + c;

возникает ошибка при компиляции.
Если не передавать объект по ссылке в функцию присваивания, то всё будет хорошо.
Но объясните, пожалуйста, почему возникает проблема при передаче по ссылке: вроде бы оператор присваивания возвращает объект, равный a+b, а затем этот объект должен передаться по ссылке при присваивании. Где я не прав?

@_stanislav · 26.02.2018, 19:42

DmmiTrY, по ходу у тебя в реализации операторов ошибка. скинь ее тоже.

Добавлено через 1 минуту
DmmiTrY, operator = обычно ссылку возвращает так как может стоять в цепочке операторов.

@DmmiTrY · 26.02.2018, 20:18 **[ТС]**

_stanislav,

C++

void Matrix::operator = (Matrix &m)
{
    if((N != m.N)||(M != m.M))
    {
       cout<<"Matrix sizes are not equal!"<<endl;
       exit(1);
    }
 
    for(int i = 0; i<N; ++i)
        for(int j = 0; j<M; ++j)
            Matr[i][j] = m.Matr[i][j];
 
}

C++

Matrix Matrix::operator+(Matrix m)
{
    if((N != m.N)||(M != m.M))
    {
       cout<<"Matrix sizes are not equal!"<<endl;
       exit(1);
    }
 
    Matrix x(N,M);
    for(int i = 0; i<N; ++i)
    {
        for(int j = 0; j<M; ++j)
        {
            x.Matr[i][j] = Matr[i][j];
        }
    }
 
    for(int i = 0; i<N; ++i)
    {
        for(int j = 0; j<M; ++j)
        {
            x.Matr[i][j] += m.Matr[i][j];
        }
    }
 
    return x;
}

@_stanislav · 26.02.2018, 20:41

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

x.Matr

что такое Matr? скинь весь код

@TheCalligrapher · 26.02.2018, 20:45

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

Но объясните, пожалуйста, почему возникает проблема при передаче по ссылке: вроде бы оператор присваивания возвращает объект, равный a+b, а затем этот объект должен передаться по ссылке при присваивании.

Верно. Но ваш оператор присваивания возвращает временный объект, равный a+b. Ко временному объекту нельзя привязать неконстантную lvalue-ссылку Matrix&. Отсюда и ошибка компиляции.

Без правильного использования const эту функциональность реализовать не получится. Должно быть

C++

1 2	void operator = (const Matrix&); Matrix operator + (const Matrix&) const;

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

@New man · 26.02.2018, 20:53

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

void operator = (Matrix&);

Проблема в том, что изменяемыми ссылками T& могут быть только ссылки на lvalue (значения, которые могут "стоять слева" при присваивании).
То есть, это должны быть изменяемые переменные, например.

А в выражении a+b у тебя получается вычисленное значение, то есть, rvalue. То же самое с любыми вызовами конструкторов или функций, возвращающих объекты.

У тебя должен быть оператор вида: Matrix& operator = (const Matrix&);
const T&, то есть, неизменяемая ссылка.

Вот моя тема, которую я создал буквально вчера, где разжёвывается часть.
Особенности перегрузки некоторых операторов
И да, если следовать советам оттуда, у тебя должны быть вообще операторы вида:

C++

1 2	void operator = (const Matrix&)&; Matrix operator + (const Matrix&)const;

И это защитит тебя от ошибок вида Matrix()=a+b;

@_stanislav · 27.02.2018, 08:03

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

Верно. Но ваш оператор присваивания возвращает временный объект, равный a+b.

у него оператор присваивания возвращает void

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

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

возникает ошибка при компиляции.

конкретней

@TheCalligrapher · 27.02.2018, 09:26

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

у него оператор присваивания возвращает void

Да, оговорился и он и я. Имелось в виду: оператор сложения возвращает временный объект, равный a+b.

@_stanislav · 27.02.2018, 09:36

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

оператор сложения возвращает временный объект

так он же возвращает объект по значению, так что ошибок быть не должно.

@TheCalligrapher · 27.02.2018, 16:25

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

так он же возвращает объект по значению, так что ошибок быть не должно.

Не понимаю. Наоборот, именно потому, что объект возвращается по значению (т.е. возвращается временный объект), и получилась ошибка. В исходном варианте кода временный объект не может быть правой частью оператора присваивания.

@DmmiTrY · 27.02.2018, 16:51 **[ТС]**

_stanislav, я действительно описался с оператором присваивания. Такое описание:

C++

1 2	void operator = (const Matrix&)&; Matrix operator + (const Matrix&)const;

решает эту проблему.

New man, и TheCalligrapher, спасибо за ответ! Я думал, что const просто защищает объект от изменений. А здесь получается, что const нужен, только чтобы сообщить компилятору, что можно передавать в функцию временные объекты по ссылке. Так получается?

@TheCalligrapher С чаем беда... 8525 / 4243 / 1173 Регистрация: 18.10.2014 Сообщений: 9,212
	27.02.2018, 16:25	10
	Сообщение от _stanislav так он же возвращает объект по значению, так что ошибок быть не должно. Не понимаю. Наоборот, именно потому, что объект возвращается по значению (т.е. возвращается временный объект), и получилась ошибка. В исходном варианте кода временный объект не может быть правой частью оператора присваивания. 0

@_stanislav "C with Classes" 1622 / 1245 / 474 Регистрация: 16.08.2014 Сообщений: 5,222 Записей в блоге: 1
	26.02.2018, 19:42	2
	DmmiTrY, по ходу у тебя в реализации операторов ошибка. скинь ее тоже. Добавлено через 1 минуту DmmiTrY, `operator =` обычно ссылку возвращает так как может стоять в цепочке операторов. 0

@_stanislav "C with Classes" 1622 / 1245 / 474 Регистрация: 16.08.2014 Сообщений: 5,222 Записей в блоге: 1
	26.02.2018, 20:41	4
	Сообщение от DmmiTrY x.Matr что такое Matr? скинь весь код 0

@_stanislav "C with Classes" 1622 / 1245 / 474 Регистрация: 16.08.2014 Сообщений: 5,222 Записей в блоге: 1
	27.02.2018, 08:03	7
	Сообщение от TheCalligrapher Верно. Но ваш оператор присваивания возвращает временный объект, равный a+b. у него оператор присваивания возвращает void Добавлено через 1 минуту Сообщение от DmmiTrY возникает ошибка при компиляции. конкретней 0

@TheCalligrapher С чаем беда... 8525 / 4243 / 1173 Регистрация: 18.10.2014 Сообщений: 9,212
	27.02.2018, 09:26	8
	Сообщение от _stanislav у него оператор присваивания возвращает void Да, оговорился и он и я. Имелось в виду: оператор сложения возвращает временный объект, равный a+b. 0

@_stanislav "C with Classes" 1622 / 1245 / 474 Регистрация: 16.08.2014 Сообщений: 5,222 Записей в блоге: 1
	27.02.2018, 09:36	9
	Сообщение от TheCalligrapher оператор сложения возвращает временный объект так он же возвращает объект по значению, так что ошибок быть не должно. 0

Опции темы