[C++] В чём тут ошибка? (делаю конструктор-копировщик)

@#pragma · Регистрация: 12.04.2009

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

Пытаюсь написать класс с конструктором-копировщиком.В чём тут ошибка?

C++

  class Subroutine {
     public:
        Subroutine (){};
        Subroutine (const Subroutine& copy) : _name (copy._name),
                                 params (new vector <expr_Node_t *> (copy.params->size()))
        {
           std::copy (copy.params->begin(),
                      copy.params->end() + copy.params->size(), params);
        }
       ~Subroutine () { delete params;};
 
        string name() const {return _name;};
        void setName (const string  *const s) {_name = *s;};
 
        void pushParam (expr_Node_t *expr) {
           params->push_back(expr);
        }
     private:
        string _name;
        vector <expr_Node_t *> *params;
  };

Получаю ошибки:

Ошибки

Код

/usr/include/c++/4.3/bits/stl_algobase.h|396|instantiated from ‘_OI std::__copy_move_a(_II, _II, _OI) [with bool _IsMove = false, _II = expr_Node_t**, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >*]’|
/usr/include/c++/4.3/bits/stl_algobase.h|435|instantiated from ‘_OI std::__copy_move_a2(_II, _II, _OI) [with bool _IsMove = false, _II = __gnu_cxx::__normal_iterator<expr_Node_t**, std::vector<expr_Node_t*, std::allocator<expr_Node_t*> > >, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >*]’|
/usr/include/c++/4.3/bits/stl_algobase.h|466|instantiated from ‘_OI std::copy(_II, _II, _OI) [with _II = __gnu_cxx::__normal_iterator<expr_Node_t**, std::vector<expr_Node_t*, std::allocator<expr_Node_t*> > >, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >*]’|
/home/user/Interpreter/function.h|50|instantiated from here|
/usr/include/c++/4.3/bits/stl_algobase.h|342|error: no match for ‘operator=’ in ‘* __result = * __first’|
||=== Build finished: 1 errors, 0 warnings ===|

и среда указывает сюда

кусок из stl_algobase.h

C++

  template<>
    struct __copy_move<false, false, random_access_iterator_tag>
    {
      template<typename _II, typename _OI>
        static _OI
        __copy_m(_II __first, _II __last, _OI __result)
        { 
      typedef typename iterator_traits<_II>::difference_type _Distance;
      for(_Distance __n = __last - __first; __n > 0; --__n)
        {
          *__result = *__first; // Вот на эту строку.
          ++__first;
          ++__result;
        }
      return __result;
    }
    };

Мне нужно перегрузить оператор '=' для класса? Я же копирую контейнер,размер элементов известен в памяти,вроде бы какая разница для STL,что там копируется.. Объясните,отчего ошибка?

@R0mm · 16.12.2009, 06:42

мб params = new ...

@#pragma · 16.12.2009, 06:57 **[ТС]**

Так ведь эта строка и есть

C++

1	params (new vector <expr_Node_t *> (copy.params->size()))

вызов конструктора для params.Или я не прав?

@R0mm · 16.12.2009, 07:03

#pragma, это первое что в голову пришло..
ошибка при компиляции? или при исполнении?

@#pragma · 16.12.2009, 07:33 **[ТС]**

Ой,а ведь params это не экземпляр класса,а вектор,тогда какой конструктор? Хотя.. такое ведь возможно

C++

int a(2);

Там ведь,наверное,внутреннее представление типов в компиляторе и всё такое.Ох,что-то я не пойму ...

@R0mm · 16.12.2009, 09:05

конструктор при создании же вызывается.
а у тебя я так понимаю указатель params уже создан..

@#pragma · 16.12.2009, 16:32 **[ТС]**

C++

1	params = new expr_List_t (copy.params->size());

Так тоже не работает - та же ошибка.

Код

stl_algobase.h|342|error: no match for ‘operator=’ in ‘* __result = * __first’|

Может,надо определить оператор '=' для элемента вектора?

@Nick Alte · 16.12.2009, 19:59

Сразу видна ошибка в вызове std::copy: передавай ей не params, а *params. Кроме того, попытка передать ей в качестве итератора для последнего элемента не просто end, а end + size вызывает крайнее недоумение по поводу выхода за границы массива.

@#pragma · 16.12.2009, 21:52 **[ТС]**

Вот,я переделал таким образом:

C++

  class Subroutine {
     public:
        Subroutine (){};
        Subroutine (const Subroutine& copy) : _name (copy._name),
                                 params (new vector <expr_Node_t *> (copy.params->size()))
        {
           std::copy (copy.params->begin(),
                      copy.params->end(), *params);
        }
       ~Subroutine () { delete params;};
 
        string name() const {return _name;};
        void setName (const string  *const s) {_name = *s;};
 
        void pushParam (expr_Node_t *expr) {
           params->push_back(expr);
        }
     private:
        string _name;
        vector <expr_Node_t *> *params;
  };

Теперь выдаётся такая ошибка(и):

ошибки

Код

/usr/include/c++/4.3/bits/stl_algobase.h|388|instantiated from ‘_OI std::__copy_move_a(_II, _II, _OI) [with bool _IsMove = false, _II = expr_Node_t**, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >]’|
/usr/include/c++/4.3/bits/stl_algobase.h|435|instantiated from ‘_OI std::__copy_move_a2(_II, _II, _OI) [with bool _IsMove = false, _II = __gnu_cxx::__normal_iterator<expr_Node_t**, std::vector<expr_Node_t*, std::allocator<expr_Node_t*> > >, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >]’|
/usr/include/c++/4.3/bits/stl_algobase.h|466|instantiated from ‘_OI std::copy(_II, _II, _OI) [with _II = __gnu_cxx::__normal_iterator<expr_Node_t**, std::vector<expr_Node_t*, std::allocator<expr_Node_t*> > >, _OI = std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >]’|
/home/user/Interpreter/function.h|50|instantiated from here|
/usr/include/c++/4.3/bits/stl_iterator_base_types.h|133|error: no type named ‘iterator_category’ in ‘class std::vector<expr_Node_t*, std::allocator<expr_Node_t*> >’|
||=== Build finished: 1 errors, 0 warnings ===|

И указывает на этот кусок кода:

stl_iterator_base_types.h

C++

  template<typename _Iterator>
    struct iterator_traits
    {
      typedef typename _Iterator::iterator_category iterator_category; // На эту строку
      typedef typename _Iterator::value_type        value_type;
      typedef typename _Iterator::difference_type   difference_type;
      typedef typename _Iterator::pointer           pointer;
      typedef typename _Iterator::reference         reference;
    };

Я понимаю,что не знаю каких-то элементарных вещей,просто хотя бы понять суть ошибки,там уже я сам докопаюсь.

@Nick Alte · 17.12.2009, 18:42

Третьим параметром в std::copy надо передавать не сам контейнер, а итератор элемента, начиная с которого будет произведено копирование. То есть,

C++

1 2	std::copy (copy.params->begin(), copy.params->end() + copy.params->size(), params->begin());

Блоги программистов
Как перейти с Options API на Composition API в Vue.js BasicMan 06.01.2025 Почему переход на Composition API актуален В мире современной веб-разработки фреймворк Vue. js продолжает эволюционировать, предлагая разработчикам все более совершенные инструменты для создания. . .	Архитектура современных процессоров inter-admin 06.01.2025 Процессор (центральный процессор, ЦП) является основным вычислительным устройством компьютера, которое выполняет обработку данных и управляет работой всех остальных компонентов системы. Архитектура. . .	История создания реляционной модели баз данных, правила Кодда Programming 06.01.2025 Предпосылки создания реляционной модели В конце 1960-х годов компьютерная индустрия столкнулась с серьезными проблемами в области управления данными. Существовавшие на тот момент модели данных -. . .	Полезные поделки на Arduino, которые можно сделать самому raxper 06.01.2025 Arduino как платформа для творчества Arduino представляет собой удивительную платформу для технического творчества, которая открывает безграничные возможности для создания уникальных проектов. Эта. . .	Подборка решений задач на Python IT_Exp 06.01.2025 Целью данной подборки является предоставление возможности ознакомиться с различными задачами и их решениями на Python, что может быть полезно как для начинающих, так и для опытных программистов. . . .	С чего начать программировать микроконтроллеры raxper 06.01.2025 Введение в мир микроконтроллеров Микроконтроллеры стали неотъемлемой частью современного мира, окружая нас повсюду: от простых бытовых приборов до сложных промышленных систем. Эти маленькие. . .
Из чего собрать игровой компьютер inter-admin 06.01.2025 Сборка игрового компьютера требует особого внимания к выбору комплектующих и их совместимости. Правильно собранный игровой ПК не только обеспечивает комфортный геймплей в современных играх, но и. . .	Обновление сайта www.historian.by Reglage 05.01.2025 Обещал подвести итоги 2024 года для сайта. Однако начну с того, что изменилось за неделю. Добавил краткий урок по последовательности действий при анализе вредоносных файлов и значительно улучшил урок. . .	Как использовать GraphQL в C# с HotChocolate Programming 05.01.2025 GraphQL — это современный подход к разработке API, который позволяет клиентам запрашивать только те данные, которые им необходимы. Это делает взаимодействие с API более гибким и эффективным по. . .	Модель полного двоичного сумматора с помощью логических операций (python) AlexSky-coder 04.01.2025 def binSum(x:list, y:list): s=^y] p=x and y for i in range(1,len(x)): s. append((x^y)^p) p=(x and y)or(p and (x or y)) return s x=list() y=list()	Это мы не проходили, это нам не задавали...(асихронный счётчик с управляющим сигналом зад Hrethgir 04.01.2025 Асинхронный счётчик на сумматорах (шестиразрядный по числу диодов на плате, но наверное разрядов будет больше - восемь или шестнадцать, а диоды на старшие), так как триггеры прошли тестирование и. . .	Руководство по созданию бота для Телеграм на Python IT_Exp 04.01.2025 Боты для Телеграм представляют собой автоматизированные программы, которые выполняют различные задачи, взаимодействуя с пользователями через интерфейс мессенджера. В данной статье мы рассмотрим,. . .

@R0mm Псевдо программист 192 / 113 / 37 Регистрация: 19.09.2009 Сообщений: 303
	16.12.2009, 06:42	2
	мб params = new ... 0

@R0mm Псевдо программист 192 / 113 / 37 Регистрация: 19.09.2009 Сообщений: 303
	16.12.2009, 07:03	4
	#pragma, это первое что в голову пришло.. ошибка при компиляции? или при исполнении? 1

@R0mm Псевдо программист 192 / 113 / 37 Регистрация: 19.09.2009 Сообщений: 303
	16.12.2009, 09:05	6
	конструктор при создании же вызывается. а у тебя я так понимаю указатель params уже создан.. 0

@Nick Alte 1675 / 1047 / 174 Регистрация: 27.09.2009 Сообщений: 1,945
	16.12.2009, 19:59	8
	Сразу видна ошибка в вызове std::copy: передавай ей не params, а *params. Кроме того, попытка передать ей в качестве итератора для последнего элемента не просто end, а end + size вызывает крайнее недоумение по поводу выхода за границы массива. 1

	17.12.2009, 18:42