Проблема с массивом экземпляров класса переменной длины

[Xuch]

Всем привет!
Прошу помощи с массивом экземпляров классов переменной длины. Опишу проблему.

Итак заголовочный файл:

class SubClass
{
  public:
  SubClass();
  string string1;
  string string2;
  string string3;
  string string4;
  string string5;
  
  int func1(void);
  gboolean func2(void);

  private:
  int Callback(void);
}

class MainClass
{
  public:
  MainClass();
  int Init(int);
  SubClass *Class;
  string str1;
  string param2;
}

Далее в файле main.cpp:

int MainClass::Init(int q)
{
  Class = malloc(q * sizeof(SubClass));
  .
  .
  .
  return 0;
}

From Stilet: Код попрошу оформлять по правилам!

Далее при попытке записи в поле нулевого элемента массива экземпляра класса программа вываливается с ошибкой SegmentationFault:

MainClass.Class[0].string4 = "test string"; --> вылетает с ошибкой доступа к памяти

Если писать в string1 то иногда проходит нормально, а иногда также вылетает ошибка.

Явно тут проблема с выделением или доступом к памяти. С выделением вроде все нормально. В чем проблема не могу понять.

Если объявляю как SubClass Class[MAX_ARRAY_SIZE], то никаких проблем вообще не возникает. Но мне нужно динамически задавать размер массива...

Большое спасибо всем откликнувшимся!

[Abstraction]

0) Оформляйте программный код тегом форума Code.

1) Не называйте типы и переменные одинаково. Запись Class = malloc(q * sizeof(SubClass)) мне пришлось осознавать секунд двадцать.

2) MainClass.Class - бессмысленное выражение. Оператор доступа к члену '.' неприменим к имени типа. Компилятор должен был выдать ошибку. Раз он у Вас её не выдал, что-то сильно не так. Приведите контекст, в котором используете строку MainClass.Class[0].string4.

3) Есть malloc - должен быть free. Железное правило. Даже теоретически не вижу в Вашем коде, где он мог бы быть.

[waleri]

В догонку - не используйте malloc/free для классов - для этого есть new/delete.
Я не знаю, что у вас string, но если это класс и память выделяете через malloc, то тогда у string не срабатывают конструкторы, ну и далее при доступе к string переменным все ломается ибо в них "мусор".

[Xuch]

Спасибо за ваши замечания!
Насчет доступа к члену класса я конечно ошибся когда упрощал код - в main.cpp определяется экземпляр и уже к его членам осуществляется доступ:

MainClass Test;
Test.Class[0].string4 = "test string";

free() есть отдельно, просто я когда упрощал код его выкинул как не относящегося к сути вопроса.

А почему при malloc() для string не срабатывают конструкторы? Это точно так? Где про это можно почитать?

[Abstraction]

А почему при malloc() для string не срабатывают конструкторы? Это точно так? Где про это можно почитать?

Там же, где и про то, что при вызове malloc() не происходит форматирования жёсткого диска: нигде. malloc(), как указано в её описании и положено по стандарту, делает ровно одну вещь: выделяет память. Она не обнуляет эту память, не вызывает конструкторов - ничего сверх этого. Обратите внимание, что "с точки зрения" malloc ей на вход подали одно целое число - количество байт требуемой памяти; у неё в принципе нет информации о том, как Вы этой памятью собрались в дальнейшем воспользоваться.

[Xuch]

Понятно
Спасибо большое!