Segmentation fault

@YR17 · Регистрация: 07.02.2015

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

Доброго времени суток.
Есть 2 класса.
object

C++

class object{
    void (*f)(object*);
public:
    void setFunc(void (*func)(object*)){
        f = func;
    }
    void callFunc(){
        f(this);
    }
};

Имеет 2 метода. Первый записывает указатель на функцию, второй вызывает функцию по указателю.
Objects

C++

class Objects{
    object **mass;
public:
    int size;
    Objects(){
        size = 0;
        mass = new object*[size];
    }
    void addObject(object *ob){
        mass[size++] = ob;
    }
    void callFuncs(){
        for(int c=0;c<size;c++){
            mass[c]->callFunc();
        }
    }
};

Имеет тоже 2 метода. Первый записывает адресс обекта в массив, второй вызывает метод call у всех обектов адресса которых записаны в массив.
Код целиком:

C++

#include <iostream>
using namespace std;
 
class object{
    void (*f)(object*);
public:
    void setFunc(void (*func)(object*)){
        f = func;
    }
    void callFunc(){
        f(this);
    }
};
 
class Objects{
    object **mass;
public:
    int size;
    Objects(){
        size = 0;
        mass = new object*[size];
    }
    void addObject(object *ob){
        mass[size++] = ob;
    }
    void callFuncs(){
        for(int c=0;c<size;c++){
            mass[c]->callFunc();
        }
    }
};
 
void out(object *o){
    cout<<o<<endl;
}
 
int main(){
    Objects Main;
    object *n;
    for(int c=0;c<5;c++){
        n = new object;
        n->setFunc(out);
        Main.addObject(n);
    }
    cout<<Main.size<<endl;
    Main.callFuncs();
    return 0;
}

Если в цикле поставить число меньше 5(4 например) то ошибка не возникает.
Вопрос:
Почему возникает ошибка?
Как исправить?
Спасибо.

@Phoenix_Max · 07.02.2015, 21:46

Для начала. Создали массив размером ноль элементов.

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

size = 0;
mass = new object*[size];

@castaway · 07.02.2015, 21:46

C++

1 2	size = 0; mass = new object*[size];

Ошибка тут.

@YR17 · 07.02.2015, 21:54 **[ТС]**

А как правильно?
size инкремируется при добавлении елемента

@Phoenix_Max · 07.02.2015, 22:01

но массив не увеличивается. Или реаллокацию делайте или на вектор замените массив.
или статический массив указателей используйте.

C++

1	object *mass[100];

@YR17 · 07.02.2015, 22:05 **[ТС]**

Можно пример реаллокации?

@DU · 07.02.2015, 22:12

да вы замучаетесь, доводя это до рабочего варианта. конструкторы копий еще надо, деструктор, оператор =. и все это есть в std::vector. найдите список его методов и пользуйтесь. он уже все реаллоцирует что надо и удаляет и копирует.

C++

#include <iostream>
#include <vector>
 
using namespace std;
 
class object
{
  void (*f)(object*);
public:
  void setFunc(void (*func)(object*))
  {
    f = func;
  }
 
  void callFunc()
  {
    f(this);
  }
};
 
class Objects
{
  std::vector<object*> mass;
 
public:
  unsigned size() const
  {
    return mass.size();
  }
 
  void addObject(object *obj)
  {
    mass.push_back(obj);
  }
 
  void callFuncs() const
  {
    for (auto obj : mass)
    {
      obj->callFunc();
    }
  }
};
 
void out(object *o)
{
  cout<<o<<endl;
}
 
int main(){
    Objects Main;
    object *n;
    for(int c=0;c<5;c++){
        n = new object;
        n->setFunc(out);
        Main.addObject(n);
    }
    cout<<Main.size()<<endl;
    Main.callFuncs();
    return 0;
}

@~~rocknrolla1~~ · 07.02.2015, 22:14

YR17, лучше добавьте поле capacity, которое будет отвечать за максимальное число элементов в контейнере.

C++

1
2
3

size = 0;
capacity = 100;
mass = new object* [ capacity ];

@Phoenix_Max · 07.02.2015, 22:25

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

Можно пример реаллокации?

что то типа этого, но лучше так не делать

C++

    void Realloc(size_t newsize)
    {
        object **newmass = new object *[newsize];
        for (size_t i = 0; i < size; i++)
        {
            newmass[i] = mass[i];
        }
        size = newsize;
        delete[] mass;
        mass = newmass;
    }
    void addObject(object *ob)
    {
        Realloc(size + 1);
        mass[size - 1] = ob;
    }

@Phoenix_Max 102 / 44 / 18 Регистрация: 08.01.2015 Сообщений: 97
	07.02.2015, 21:46	2
	Для начала. Создали массив размером ноль элементов. Сообщение от YR17 size = 0; mass = new object*[size]; 1

@YR17 0 / 0 / 0 Регистрация: 07.02.2015 Сообщений: 4
	07.02.2015, 21:54 [ТС]	4
	А как правильно? size инкремируется при добавлении елемента 0

@YR17 0 / 0 / 0 Регистрация: 07.02.2015 Сообщений: 4
	07.02.2015, 22:05 [ТС]	6
	Можно пример реаллокации? 0

	07.02.2015, 22:25

Segmentation fault

Решение