Перегруженный оператор присваивания

@IvanInanovich · Регистрация: 03.01.2013

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

Добрый вечер! Встретил пример в книге, где есть перегруженный оператор присваивания. В итоге код который написан в книге не запускается. Во время инициализации объекта с помощью знака равно выбивает ошибку. Когда пробую инициализировать объект круглыми скобками то программа запускается. Можете подсказать в чем дело?

C++

#include<cstring>
#include<iostream>
using namespace std;
////////////////////////////////////////////////////////////////
///////////////////////////////////////////////////////////////////////////////
class strCount
{
  int count;
  char* str;
  friend class String;
 
  strCount(char* s)
  {
      int length = strlen(s);
 
      str = new char[length+1];
 
      strcpy(str,s);
      count = 1;
  }
  ~strCount()
  { delete[]str; }
};
/////////////////////////////////////////////////////////////////////
class String
{
private:
    strCount* psc;
public:
    String()
    { psc = new strCount("NULL") ;}
 
    String(char* s)
{ psc = new strCount(s) ;};
 
String(String& S)
{
    psc = S.psc;
    (psc->count)++;
}
~String()
{
    if(psc->count == 1)
        delete psc;
    else
    (psc->count)--;
}
    void display()
    {
        cout << psc->str;
        cout << " (addr=" << psc << ")";
    }
////////////////////////////////// Оператор присваивания
    void operator=(String& S) 
    {
        if(psc->count == 1)
            delete psc;
 
      else
        (psc->count)--;
    psc = S.psc;
 
    (psc->count)++;
 
    }
/////////////////////////////////////////
};
 
int main()
{
    String s3 = "Myha po poly shla, myha deneshky nashla"; //// Ошибка
 
    cout << "\ns3=";s3.display();
 
    String s1;
 
    s1 = s3;
 
    cout << "\ns1="; s1.display();
 
    String s2(s3);
    cout << "\ns2="; s2.display();
    cout << endl;
 
    return 0;
}

String s3 = "Myha po poly shla, myha denejky nashla"; - Ошибка: error: no matching function for call to 'String::String(String)'|

String s3("Myha po poly shla, myha denejky nashla"); - запускается программа.

@OhMyGodSoLong · 06.05.2013, 21:15

@@ -36,1 +36,1 @@
-String(String& S)
+String(const String& S)

Определяйте конструктор копирования правильно.

Tulosba · 06.05.2013, 21:34

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

Определяйте конструктор копирования правильно.

Но ошибка-то далеко не в этом. Оператор присваивания более интересный

Подправил Ваш код. Кажется работает:

C++

#include<cstring>
#include<iostream>
using namespace std;
////////////////////////////////////////////////////////////////
///////////////////////////////////////////////////////////////////////////////
class strCount
{
  int count;
  char* str;
  friend class String;
 
  strCount(char* s)
  {
      int length = strlen(s);
 
      str = new char[length+1];
 
      strcpy_s(str,length+1,s);
      count = 1;
  }
  ~strCount()
  { 
      delete[]str; 
  }
};
/////////////////////////////////////////////////////////////////////
class String
{
private:
    strCount* psc;
public:
    String()
    { 
        psc = new strCount("NULL") ;
    }
 
    String(char* s)
    { 
        psc = new strCount(s);
    }
 
    String(const String& S)
    {
        psc = S.psc;
        (psc->count)++;
    }
 
    ~String()
    {
        if(psc->count-- == 1)
            delete psc;
    }
    
    void display()
    {
        cout << psc->str;
        cout << " (addr=" << psc << ")";
    }
////////////////////////////////// Оператор присваивания
    String& operator=(const String& S) 
    {
        if(psc->count-- == 1)
            delete psc;
 
        psc = S.psc;
 
        (psc->count)++;
        return *this;
    }
/////////////////////////////////////////
};
 
int main()
{
    String s3 = "Myha po poly shla, myha deneshky nashla"; //// Ошибка
    //String s3("Myha po poly shla, myha deneshky nashla"); //// Ошибка
 
    cout << "\ns3=";s3.display();
 
    String s1;
 
    s1 = s3;
 
    cout << "\ns1="; s1.display();
 
    String s2(s3);
    cout << "\ns2="; s2.display();
    cout << endl;
 
    system( "pause" );
 
    return 0;
}

@IvanInanovich · 06.05.2013, 21:36 **[ТС]**

OhMyGodSoLong, пример из книги. Но суть я уловил. Спасибо.

Добавлено через 19 секунд
Если что тема актуальна. Ошибка присутствует : )

Tulosba · 06.05.2013, 21:38

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

Ошибка присутствует : )

где?

@IvanInanovich · 06.05.2013, 21:50 **[ТС]**

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

где?

Ваше сообщение не увидел.

Функцию strcpy_s компилятор не видит. Посмотрел в интернете вроде как в заголовочном файле cstring должна быть.
Можно в двух словах что не так было? Попробую еще пройтись по коду, когда дойду до оператора this.

Tulosba · 06.05.2013, 22:03

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

Функцию strcpy_s компилятор не видит.

Это из разряда "безопасных" от майкрософта. Замените на ту, которая была у Вас:

C++

1	strcpy(str,s)

В данном случае, сгодится.
Ошибок было как минимум две:
1. Сигнатура копирующего конструктора и оператора присваивания.
2. Оператор присваивания должен возвращать ссылку на текущий объект.

@OhMyGodSoLong ~ Эврика! ~ 1257 / 1006 / 74 Регистрация: 24.07.2012 Сообщений: 2,002
	06.05.2013, 21:15	2
	@@ -36,1 +36,1 @@ -String(String& S) +String(const String& S) Определяйте конструктор копирования правильно. 1

@IvanInanovich 0 / 0 / 2 Регистрация: 03.01.2013 Сообщений: 113
	06.05.2013, 21:36 [ТС]	4
	OhMyGodSoLong, пример из книги. Но суть я уловил. Спасибо. Добавлено через 19 секунд Если что тема актуальна. Ошибка присутствует : ) 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	06.05.2013, 21:38	5
	Сообщение от IvanInanovich Ошибка присутствует : ) где? 0

@IvanInanovich 0 / 0 / 2 Регистрация: 03.01.2013 Сообщений: 113
	06.05.2013, 21:50 [ТС]	6
	Сообщение от Tulosba где? Ваше сообщение не увидел. Функцию strcpy_s компилятор не видит. Посмотрел в интернете вроде как в заголовочном файле cstring должна быть. Можно в двух словах что не так было? Попробую еще пройтись по коду, когда дойду до оператора this. 0

	06.05.2013, 22:03