Перегрузка оператора (C++)

[Sergey S]

Пытаюсь откомпилировать следующий код с помощью mingw32 :

#include <iostream>
#include <cstring>
#include <cstdlib>

using namespace std;

class sample {
  char *s;
public:
  sample(); // normal constructor
  sample(const sample &ob); // copy constructor
  ~sample( ) { if(s) delete [] s; cout << "Freeing s\n"; }
  void show() { cout << s << "\n"; }
  void set(char *str);
  sample operator=(sample &ob); // overload assignment
};

// Normal constructor.
sample::sample()
{
  cout << "making class \n";
  s = new char('\0'); // s points to a null string.
}

// Copy constructor.
sample::sample(const sample &ob)
{
  cout << "making copy of class \n";
  s = new char[strlen(ob.s)+1];

  strcpy(s, ob.s);
}

// Load a string.
void sample::set(char *str)
{
  s = new char[strlen(str)+1];

  strcpy(s, str);
}

// Overload assignment operator.
sample sample::operator=(sample &ob)
{
  /* If the target string is not large enough
     then allocate a new string. */
  if(strlen(ob.s) > strlen(s)) {
    delete [] s;
    s = new char[strlen(ob.s)+1];
  }
  strcpy(s, ob.s);
  return *this;
}

// Return an object of type sample.
sample input()
{
  char instr[80];
  sample str;

  cout << "Enter a string: ";
  cin >> instr;

  str.set(instr);
  return str;
}

int main()
{
  sample ob;

  // assign returned object to ob
  ob = input();  // This is now OK
  ob.show();

  return 0;
}

на что компилятор выдаёт:

\overloading\main.cpp||In function 'int main()':|
\overloading\main.cpp|73|error: no match for 'operator=' in 'ob = input()()'|
\overloading\main.cpp|43|note: candidates are: sample sample::operator=(sample&)|
||=== Build finished: 1 errors, 0 warnings ===|

взял из книги Г.Шилдта, добавил пару строк всего.

[MooNDeaR]

sample& operator=(sample &ob);

Возвращайте ссылку на объект, а не сам объект.

sample& sample::operator=(sample &ob)
{
  /* If the target string is not large enough
     then allocate a new string. */
  if(strlen(ob.s) > strlen(s)) {
    delete [] s;
    s = new char[strlen(ob.s)+1];
  }
  strcpy(s, ob.s);
  return this;
}

[ERAGORN]

Определите фактический параметр operator= как константную ссылку.

Я магу ошибатся. Инструкция "ob = input();" выполняется в следующей последовательности:
1)вызывается функция input(), возвращенное значение сохраняется во временной переменной созданой компилятором - ob = input(); ==> ob = tempSample;
2)Вызывается operator= объекта который находится слева от знака =, с аргументом который справа от знака.
т.е operator= получает в качестве аргумента временную переменную, а определение фактического параметра просто ссылкой позволяет функции изменять значение временной переменой, чего не допускается.

[Sergey S]

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

sample& operator=(sample &ob);

Возвращайте ссылку на объект, а не сам объект.

sample& sample::operator=(sample &ob)
{
  /* If the target string is not large enough
     then allocate a new string. */
  if(strlen(ob.s) > strlen(s)) {
    delete [] s;
    s = new char[strlen(ob.s)+1];
  }
  strcpy(s, ob.s);
  return this;
}

так не работает, выдаёт

\overloading\main.cpp||In member function 'sample& sample:perator=(sample&)':|
\overloading\main.cpp|52|error: invalid initialization of non-const reference of type 'sample&' from a temporary of type 'sample* const'|
\overloading\main.cpp||In function 'int main()':|
\overloading\main.cpp|73|error: no match for 'operator=' in 'ob = input()()'|
\overloading\main.cpp|43|note: candidates are: sample& sample:perator=(sample&)|
||=== Build finished: 2 errors, 0 warnings ===|

[Sergey S]

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

Определите фактический параметр operator= как константную ссылку.

Я магу ошибатся. Инструкция "ob = input();" выполняется в следующей последовательности:
1)вызывается функция input(), возвращенное значение сохраняется во временной переменной созданой компилятором - ob = input(); ==> ob = tempSample;
2)Вызывается operator= объекта который находится слева от знака =, с аргументом который справа от знака.
т.е operator= получает в качестве аргумента временную переменную, а определение фактического параметра просто ссылкой позволяет функции изменять значение временной переменой, чего не допускается.

так он уже по ссылке передаётся, параметр то этот, разве нет ?

[Sergey S]

сделал вот так,

sample sample::operator=(const sample *ob)
{
  /* If the target string is not large enough
     then allocate a new string. */
  if(strlen(ob->s) > strlen(s)) {
    delete [] s;
    s = new char[strlen(ob->s)+1];
  }
  strcpy(s, ob->s);
  return *this;
}

ERAGORN, если Вы это имели ввиду, спасибо

[Sergey S]

А в общеобразовательных целях, как понимать, что код из книжек не работает ? Что тому виной ? Новые стандарты какие-то ?