QList <internal error>, помогите Опции

[BRE]

kemiisto,

Chemistry::Molecule::Molecule(const Chemistry::Molecule& chemistryMolecule)
{
  chemistryAtoms_ = QList<Chemistry::Atom*>(chemistryMolecule.chemistryAtoms_);
  chemistryBonds_ = QList<Chemistry::Bond*>(chemistryMolecule.chemistryBonds_);
  incidenceMatrix_ = IncidenceMatrix(chemistryMolecule.incidenceMatrix_);
  charge_ = chemistryMolecule.charge_;
}

вот так не стоит писать..
в этом конструкторе каждый лист создается по 2 раза(если лист имеет конструктор по умолчанию без аргументов).

Причем копируются только указатели, а не сами объекты атомов и связей. Если бы был деструктор корректно уничтожающий все объекты атомов и связей, проблемы начались бы раньше.
Кстати в матрице тоже список указателей.

Сообщение отредактировал BRE - 5.9.2009, 19:42

[kwisp]

ну да по идее у 2 разных молекул должны быть разные атомы...

[BRE]

Жаль под linux собрать не могу, по исходникам не легко разбираться.

[kemiisto]

BRE, kwisp, я учитываю ваши комментарии. Спасибо. Избавлюсь от ссылок в списках, вспомню об инициализаторах в конструкторах, ... Потихоньку перепишу, а там видно будет.

Тему не закрываю пока.

[Litkevich Yuriy]

Жаль под linux собрать не могу, по исходникам не легко разбираться.

тебе наверно надо QGLViewer собрать сначала (проект тут)

[BRE]

Жаль под linux собрать не могу, по исходникам не легко разбираться.

тебе наверно надо QGLViewer собрать сначала (проект тут)

Сейчас гляну, но у меня еще на этапе компиляции проекта kemiisto ошибку выдает. Разбираться придется.

[kemiisto]

ну попробуй
так

Не помогает.

[kwisp]

kemiisto,
значит надо досканалнее. глянуть.
расскажи где ты удаляешь атомы?
подумай где можешь наступать на память?

[kemiisto]

kwisp, хорошо. Напишу explicit деструкторы для классов, погоняю код. А там видно будет.

[kemiisto]

Ну вот, потихоньку изменил код. Убрал пока указатели где только возможно. Нашлись ошибки!

Во-первых, вот тут был бред:

Chemistry::Bond::Bond(const Chemistry::Bond& chemistryBond)
{
  chemistryMolecule_ = chemistryMolecule_;
  beginIndex_ = chemistryBond.beginIndex_;
  endIndex_ = chemistryBond.endIndex_;
}

В первой строчке в теле. Ну это ляп, конечно же. Когда начал переписывать со списками инициализаторов - заметил.

Ну и там ещё по мелочи. Но толку нет. Симптомы теже. Портятся мои списки.

Архивчик обновил. Может посмотрите?