QList <internal error>, помогите Опции Подписка на тему Сообщить другу Версия для печати Скачать тему Подписка на этот форум Режимы отображения Переключить на: Древовидный Переключить на: Стандартный Линейный

[kemiisto]

Здравствуйте!

Если у кого-нибудь найдётся минутка-другая, посмотрите проектик. Исходники Link. Я не стал убирать *.dll и *.a от QGLViewer'а и инклюды от QGLViewer'а и Eigen'а. Так чтоб под Windows собиралось "из коробки"...

Я первый раз пишу на ++ и проект только-только начинается. Так что там много методов-пустышек и "говнокода".

Собственно, беда такая. Есть у нас класс (молекула)

namespace Chemistry
{
  class Molecule
  {
  public:
  ...

  private:
    typedef QList<Chemistry::Bond*> IncidenceList;
    typedef QList<IncidenceList> IncidenceMatrix;

    IncidenceMatrix incidenceMatrix_;
    QList<Chemistry::Bond*> chemistryBonds_;
    QList<Chemistry::Atom*> chemistryAtoms_;    
    ...
  };
}

В Render::Viewer попытка использовать экземпляр этого класса

namespace Render
{
  class Viewer : public QGLViewer
  {
    Q_OBJECT

  public:
  ...

  protected:
  ...

  private:
    Chemistry::Molecule chemistryMolecule_;
    ...

  };
}

Попытка не совсем успешная. Когда отрабатывает конструктор Chemistry::Molecule, все 3 списка инициализируются.



При попытке добавить атом в Render::Viewer::mousePressEvent в самом начале ставим точку останова:



Вот. Куда то пропал incidenceMatrix_. Куда?

А самое интересное. Если в объявлении поменять местами скажем так:

  private:
    typedef QList<Chemistry::Bond*> IncidenceList;
    typedef QList<IncidenceList> IncidenceMatrix;

    QList<Chemistry::Bond*> chemistryBonds_;
    QList<Chemistry::Atom*> chemistryAtoms_;
    IncidenceMatrix incidenceMatrix_;

то на точке останова не будет уже chemistryBonds_.

То есть получается: кто первым объявлен (первым инициализируется в конструкторе), тот куда-то пропадает.

Немного побегал дебагом. Список портится внутри void Render::Viewer::updateMolecule().

До вызова updateGL() всё отлично, после <internal error>. А это метод (точнее слот) QGLViewer'а.

Мыслей больше нет.

[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 собрать сначала (Link)

[BRE]

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

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

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

[kemiisto]

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

Не помогает.

[kwisp]

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

[kemiisto]

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

[kemiisto]

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

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

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

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

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

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