Форум программистов, компьютерный форум CyberForum.ru

Ошибка в перегрузке конструктора копирования - C++

Восстановить пароль Регистрация
 
Corvit
0 / 0 / 0
Регистрация: 14.10.2013
Сообщений: 13
14.11.2013, 15:33     Ошибка в перегрузке конструктора копирования #1
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
// strimem.cpp
// Класс String с экономией памяти
// Перегружаемая операция присваивания и конструктор          // копирования
#include <iostream>
#include <cstring>              // для strcpy() и т. д.
using namespace std;
///////////////////////////////////////////////////////////
class strCount                  // Класс-счетчик уникальных строк
  {                    private:
    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                    // класс String
  {
  private:
    strCount* psc;              // указатель на strCount
  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;             // удалить strCount
      else                      // иначе
        (psc->count)--;         // уменьшить счетчик
      }
//---------------------------------------------------------
    void display()              // вывод String
      {
      cout << psc->str;         // вывести строку
      cout << " (addr=" << psc << ")";  // вывести адрес
      }
//---------------------------------------------------------
    void operator = (String& S) // присвоение String
      {
      if(psc->count==1)         // если последний
                                // пользователь,
        delete psc;             // удалить strCount
      else                      // иначе
        (psc->count)--;         // уменьшить счетчик
      psc = S.psc;              //использовать strCount
                                //аргумента
      (psc->count)++;           //увеличить счетчик
      }
  };
///////////////////////////////////////////////////////////
int main()
  {
  String s3 = "Муха по полю пошла, муха денежку нашла";
  cout << "\ns3="; s3.display(); //вывести s3
 
  String s1;                     //определить объект String
  s1 = s3;                       //присвоить его другому объекту
  cout << "\ns1="; s1.display(); //вывести его
 
  String s2(s3);                 //инициализация
  cout << "\ns2="; s2.display(); //вывести
                                 //инициализированное
  cout << endl;
  return 0;
  }
Все конструкторы есть. В чем причина ошибки??
Миниатюры
Ошибка в перегрузке конструктора копирования  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.11.2013, 15:33     Ошибка в перегрузке конструктора копирования
Посмотрите здесь:

Вызов конструктора для this в констукторе копирования C++
Перегрузка конструктора копирования и оператора присвоения C++
C++ Касательно конструктора копирования
C++ По поводу ссылок и конструктора копирования
C++ Подскажите реализацию конструктора копирования
C++ Неявный вызов конструктора копирования
Не понятна работа конструктора копирования C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Alexxx7
97 / 33 / 5
Регистрация: 24.04.2012
Сообщений: 151
14.11.2013, 15:42     Ошибка в перегрузке конструктора копирования #2
У меня на VS 2008 всё работает, конструктор в классе strCount объяви с спецификаторам паблик если не поможет создай конструктор в strCount без параметров т.е. по умолчанию.
Corvit
0 / 0 / 0
Регистрация: 14.10.2013
Сообщений: 13
14.11.2013, 16:01  [ТС]     Ошибка в перегрузке конструктора копирования #3
Паблик не помог.
Strcount без параметров? а как тогда адрес на строку передавать?

Добавлено через 9 минут
Да в студии и у меня запустилось. А вот почему Qt бастует не ясно
Alexxx7
97 / 33 / 5
Регистрация: 24.04.2012
Сообщений: 151
14.11.2013, 16:27     Ошибка в перегрузке конструктора копирования #4
иногда компиляторы помимо конструктора с параметрами требуют конструктор по умолчанию, и если не ошибаюсь конструкторы всегда должны быть пабликами
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
14.11.2013, 16:46     Ошибка в перегрузке конструктора копирования #5
Нужно больше констов
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
#include <iostream>
#include <cstring>              // для strcpy() и т. д.
using namespace std;
///////////////////////////////////////////////////////////
class strCount                  // Класс-счетчик уникальных строк
  {                    private:
    int count;                  // собственно счетчик
    char* str;                  // указатель на строку
    friend class String;        // сделаем себя доступными
    //методы скрыты
//---------------------------------------------------------
    strCount(const char* s)           // конструктор с одним аргументом
      {
      int length = strlen(s);   // длина строкового
                                // аргумента
      str = new char[length+1]; // занять память
                                // под строку
      strcpy(str, s);           // копировать в нее аргументы
      count=1;                  // считать с единицы
      }
//---------------------------------------------------------
    ~strCount()                 // деструктор
      { delete[] str; }         // удалить строку
  };
///////////////////////////////////////////////////////////
class String                    // класс String
  {
  private:
    strCount* psc;              // указатель на strCount
  public:
    String()                    // конструктор без аргументов
    {psc = new strCount("NULL");}
 
//---------------------------------------------------------
    String(const char *s)             // конструктор с одним аргументом
    { psc = new strCount(s);}
//---------------------------------------------------------
    String ( const String& S)         // конструктор копирования
    {
        psc = S.psc;
        (psc->count)++;
    }
 
//---------------------------------------------------------
    ~String()                   // деструктор
      {
      if(psc->count==1)         // если последний
                                // пользователь,
        delete psc;             // удалить strCount
      else                      // иначе
        (psc->count)--;         // уменьшить счетчик
      }
//---------------------------------------------------------
    void display()              // вывод String
      {
      cout << psc->str;         // вывести строку
      cout << " (addr=" << psc << ")";  // вывести адрес
      }
//---------------------------------------------------------
    void operator = (const String& S) // присвоение String
      {
      if(psc->count==1)         // если последний
                                // пользователь,
        delete psc;             // удалить strCount
      else                      // иначе
        (psc->count)--;         // уменьшить счетчик
      psc = S.psc;              //использовать strCount
                                //аргумента
      (psc->count)++;           //увеличить счетчик
      }
  };
///////////////////////////////////////////////////////////
int main()
  {
  String s3 = "Муха по полю пошла, муха денежку нашла";
  cout << "\ns3="; s3.display(); //вывести s3
 
  String s1;                     //определить объект String
  s1 = s3;                       //присвоить его другому объекту
  cout << "\ns1="; s1.display(); //вывести его
 
  String s2(s3);                 //инициализация
  cout << "\ns2="; s2.display(); //вывести
                                 //инициализированное
  cout << endl;
  return 0;
  }
Добавлено через 1 минуту
Цитата Сообщение от Alexxx7 Посмотреть сообщение
иногда компиляторы помимо конструктора с параметрами требуют конструктор по умолчанию, и если не ошибаюсь конструкторы всегда должны быть пабликами
компиляторы требуют конструктор по умолчанию, если он нужен. Если он не вызывается (явно/неявно), то ошибки не будет. Конструкторы не обязательно должны быть пабликами
Corvit
0 / 0 / 0
Регистрация: 14.10.2013
Сообщений: 13
15.11.2013, 10:23  [ТС]     Ошибка в перегрузке конструктора копирования #6
А в чем фишка. Где можно почитать ?

Добавлено через 17 часов 12 минут
Ну в любом случае спасибо. Впредь буду учитывать магию const))
programina
 Аватар для programina
1912 / 597 / 37
Регистрация: 23.10.2011
Сообщений: 4,468
Записей в блоге: 2
15.11.2013, 10:30     Ошибка в перегрузке конструктора копирования #7
А конструктор разве можно перегружать?

Добавлено через 3 минуты
можно!
Corvit
0 / 0 / 0
Регистрация: 14.10.2013
Сообщений: 13
15.11.2013, 10:31  [ТС]     Ошибка в перегрузке конструктора копирования #8
Куда интересней - почему без const'ов в объявлении не работает
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
15.11.2013, 12:33     Ошибка в перегрузке конструктора копирования #9
Немного запоздало, но
конструктор копирования имеет сигнатуру
C++
1
class_name(const class_name&);
аналогично дела обстоят с оператором присваивания
в некоторых местах тип входного параметра я поменял на const char*
это потому, что строка вида "text" - это и есть const char*, а не просто char*
Corvit
0 / 0 / 0
Регистрация: 14.10.2013
Сообщений: 13
15.11.2013, 12:38  [ТС]     Ошибка в перегрузке конструктора копирования #10
Ну да кажется понял. Не поставить const в данном примере с точки зрения компилятора равнозначно попытке изменить значение указателя на константу. Это я решил почитать Страуструпа. Его сухой стиль как-то способствует шевелению извилин. Но в любом случае спасибо за ответ.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.11.2013, 14:24     Ошибка в перегрузке конструктора копирования
Еще ссылки по теме:

C++ Пример конструктора копирования не понятен
C++ Реализация конструктора копирования для класса
C++ В чём отличия конструктора копирования и конструктора перемещения? Где и как их нужно использовать?
C++ Грамотное использование конструктора копирования

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Tulosba
:)
Эксперт С++
4382 / 3225 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
15.11.2013, 14:24     Ошибка в перегрузке конструктора копирования #11
Цитата Сообщение от I.M. Посмотреть сообщение
конструктор копирования имеет сигнатуру
Вы не совсем правы.
12.8.2 A non-template constructor for class X is a copy constructor if its first parameter is of type X&, const X&,
volatile X& or const volatile X&
, and either there are no other parameters or else all other parameters
have default arguments (8.3.6).
Yandex
Объявления
15.11.2013, 14:24     Ошибка в перегрузке конструктора копирования
Ответ Создать тему
Опции темы

Текущее время: 21:31. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru