Внутри класса объявил вектор, а при попытке что-то сделать с этим вектором программа вылетает

А можно код хотя бы?

Вот:

где создание обьекта ?

А где 


создается?

Да создается все, чесслово:


Это сообщение отредактировал(а) Чупакабро - 27.9.2012, 09:38

Крайне сомнительная строчка. При dwVertexCount, равном 1, будет деление на ноль. 

Это сообщение отредактировал(а) feodorv - 27.9.2012, 09:39

Скорее всего выход за границы массивов.

feodorv это оффтоп... там не может быть значение меньше 3. а по теме подсказать можете?

где создается ?! чесслово, я вижу только выделение памяти..   

Это юмор?

Вы проверяли?


Вот. Понавставляйте проверок разных, будет Вам счастье.

А точку кода, на которой программа рушится, Вы определили?

блин, вот

могу всю функцию запостить, там 100500 строк)
да создется mesh, там со всеми другими мемберами все в порядке, значения присваиваются.

Добавлено @ 09:49
 
feodorv, вылет происходит вот именно на этой строчке:
mesh->separatingAxis.resize(1);
а по поводу dwVertexCount - это число вершин в многоугольнике. которое не может быть меньше 3. ну, по хорошему надо проверку поставить...

Добавлено @ 09:52
косяк именно с std::vector, только вот в чем он заключается, не пойму...

Это сообщение отредактировал(а) Чупакабро - 27.9.2012, 09:54

Нет, ну а там инициализации разные, в том числе и separatingAxis?

malloc, realloc не создают С++ обьектов, коим является стд::вектор.. и даже 100500 маллоков его не создатут.. new в помощь..

Вот же подсказывают:

Ну или массив этих мешей...

Вот! тут, я думаю, корень зол! Но как инициализировать, не знаю!
Пробовал так:

но вылетает...

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

Это сообщение отредактировал(а) feodorv - 27.9.2012, 10:00

Вот - mesh у меня нормально инициализирован. Но если я нажму F10, то произойдет вылет


Добавлено через 3 минуты и 37 секунд
Вот ошипка:


Добавлено через 14 минут и 57 секунд
Друзья!
Методом научного втыка я решил проблему:

Всем спасибо за советы!
Видимо я и создатели STL друг друга немного недопоняли))

нелегально, если separatingAxis не создан.. см. выше.. 

Оператор присваивания не есть инициализация (это не C). Конструктор для separatingAxis должен быть вызван до оператора =, который здесь означает copy vector content:
А дропать у Вас можно только мусор, присутствующий в separatingAxis...

да конструктор вот он: VertexVector ()
Но в общем, я решил проблему, см. выше

Это вызов конструктор другого экземпляра VertexVector, не separatingAxis.


STL тут совсем не причём. А вот ООП - очень даже)))

VertexVector это имя типа
typedef std::vector<XMFLOAT33> VertexVector;
с таким же успехом можно написать 
 mesh->separatingAxis= std::vector<XMFLOAT33>();

Это сообщение отредактировал(а) Чупакабро - 27.9.2012, 10:33

успехов в многочасовой отладке неуловимых багов   раз мимо ушей пропускаете то, что Вам советуют.. 

вроде же прислушался к совету))
Ну и собственно, у меня все заработало.

Чупакабро, ты видимо так и не понимаешь. Основная проблема в том, что ты выделяешь память с помощью realloc для массива объектов типа Mesh. А Mesh с вектором не является POD (plain old data) типом. Т.е. его, например, нельзя тупо копировать. Поэтому, забудь о реаллок и маллок. Тебе необходимо использовать new[], а лучше векторы и списки.
В целом, прежде чем что-то писать рекомендую почитать книжки.

Mesh это просто структура, не класс.
Потом вектор в меше я заменил на указатель на вектор.
Соответсвенно, Mesh так и осталась структурой фиксированного размера.
А маллоки я заменю на new, да.


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

Это сообщение отредактировал(а) Чупакабро - 27.9.2012, 11:27

И в итоге у тебя структура стала не POD

Вот сначала у меня было, грубо говоря, так:
struct S1
{
  int a;
  std::vector<int> b;
};

Потом стало так:
struct S1
{
  int a;
  void* b;
}; - разве это не POD?

В общем, есть неочевидные моменты, которые не во всех книгах расписаны. Поэтому и спрашиваю здесь))



Это сообщение отредактировал(а) Чупакабро - 27.9.2012, 11:46

  Если стандартный метод выделения памяти не подходит, то перегрузите оператор new для своего класса и выделяйте в нем память своим способом, при этом будет правильно работать инициализация объектов.

Надо до этого дорасти)) Начать использовать new вместо malloc. Пойду читать книжки)
И с вектором все ж таки разберусь до конца.

  По моему худший вариант смешивать С и С++ . Это чревато появлением кучи трудноуловимых ошибок. Лучше уж писать на чистом С и проводить все инициализации и перераспределения памяти руками. Т.е. всегда сам следишь за всем. Сам контролируешь время жизни структур данных и последовательность инициализации. Или же совсем забыть про С как про страшный сон и даже перестать использовать указатели в явном виде (Использовать классы, которые сами следят за своей памятью, которую они выделили в куче). 

лучше заменить ручное распределение памяти на контейнеры:


Это сообщение отредактировал(а) baldina - 27.9.2012, 13:10

ну вот я начал потихоньку заменять все на контейнеры, поэтому собственно тема и образовалась

Чупакабро, начинать надо не изнутри, а снаружи. А по хорошему, почитай хорошую литературу, список приведен в "Ответах на часто задаваемые вопросы" (тема закреплена в разделе).

В С++ структура от класса отличается только тем, что в структуре члены класса по умолчанию публичные, а в классе приватные (если в новом, стандарте, конечно, этого не понимали)

Есть такое правило выделять и освобождать память определенным методом. Если в С мы могли выделить память malloc, то и очищать мы его должны родственной функцией (free). Создание классов в С++ намного сложнее. Каждый член класса должен быть проинициализирован (иначе, он будет содержать просто мусор). В итоге, все члены класса (которые тоже может быть классами) запускают свои конструкторы (по просту говоря функции инициализации), которые будут запускать конструкторы и для встроенных членов. и так далее.. Для этого служит оператор new, который помимо выделения памяти еще и производит действия инициализации.. Оператор delete выполняет противоположные обязанности. Сначала производит определенные действия при удалении объекта, а потом очищает память.

Все операторы, фактически, являются функциями, начинающимися с [b]operator[b]. в итоге 


преобразуется к виду (как примерный показательный вариант)



где функция operator= просто копирует определенным образом объект из второго параметра в объект из первого параметра. Но, если первый параметр был выделене без инициализации (malloc), он будет скопирован в мусор.

Поэтому все объекты в С++ надо выделять через оператор new, а удалять через delete.

P.S. Для инициализации массивов объектов в С++ существует оператор new[] и обратный оператор delete []

  

// удаляем все

bsa, baldina, т.е. это
> Mech a* = new Mech[15]
вас не смутило?

Это сообщение отредактировал(а) borisbn - 29.9.2012, 00:50

  


Ну очипятка же.. что вы, право слово!

Это сообщение отредактировал(а) NoviceF - 29.9.2012, 13:08