Segmentation fault при итерации QGraphicsItemGroup.childItems()

[Mist1K]

Делаю игру в которой надо складывать картинку из частей. При стыковке двух элементов наследников класса QGraphicsItem объединяю их в класс наследник QGraphicsItemGroup. При перемещении группы каждому члену группы меняю координаты для отслеживания их позиции на сцене, при этом иногда получаю вылет "Segmentation fault"  и не вижу где косяк, помогите пожалуйста разобраться.

main.cpp

Код:

#include "mygi.h"
#include "mygig.h"
#include <QtWidgets>

int main(int argc, char *argv[]){
    QApplication a(argc, argv);

    QGraphicsScene scene;
    scene.setSceneRect(0, 0, 500, 500);

    MyGIG* group = new MyGIG();
    for ( int i = 0; i < 4; i++ ) {
        MyGI* item = new MyGI();
        item->setPos(i*100, 0);
        group->addToGroup(item);
        qDebug() << item;
    }
    group->setFlag(QGraphicsItem::ItemIsMovable);
    scene.addItem(group);

    QGraphicsView view(&scene);
    view.show();

    return a.exec();
}

mygi.h

Код:

#ifndef MYGI_H
#define MYGI_H

#include <QGraphicsItem>

class MyGI : public QGraphicsItem {
    public:
        MyGI();
        ~MyGI();

        QRectF boundingRect() const;
        void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
};

#endif // MYGI_H

mygi.cpp

Код:

#include "mygi.h"
#include "ui_mygi.h"
#include <QPainter>

MyGI::MyGI() {}

MyGI::~MyGI() {}

QRectF MyGI::boundingRect() const {
    return QRectF(0, 0, 100, 100);
}

void MyGI::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget) {
    painter->setPen(Qt::black);
    painter->drawRect(0, 0, 100, 100);
}

mygig.h

Код:

#ifndef MYGIG_H
#define MYGIG_H

#include <QGraphicsItemGroup>

class MyGIG : public QGraphicsItemGroup {
    public:
        MyGIG();
        void mouseReleaseEvent(QGraphicsSceneMouseEvent * event);
};

#endif // MYGIG_H

mygig.cpp

Код:

#include "mygig.h"
#include <qdebug.h>

MyGIG::MyGIG() {}

void MyGIG::mouseReleaseEvent(QGraphicsSceneMouseEvent *event) {
    QGraphicsItemGroup::mouseReleaseEvent(event);
    for ( QList<QGraphicsItem*>::iterator it = childItems().begin(); it != childItems().end(); ++it ) {
        qDebug() << (*it);
        (*it)->moveBy(pos().x(), pos().y()); // Segmentation fault
    }
    setPos(0, 0);
}

PS: Буду благодарен за любые предложения, замечания, советы.

[Bepec]

QGraphicsItemGroup::mouseReleaseEvent(event);
А вы уверены что после этой строчки event уже не удалён?

[Mist1K]

QGraphicsItemGroup::mouseReleaseEvent(event);
А вы уверены что после этой строчки event уже не удалён?

Да я на самом деле мало в чем уверен)
Среду разработки Qt использую впервые и изучаю по ходу необходимости.

Код:

void MyGIG::mouseReleaseEvent(QGraphicsSceneMouseEvent *event) {
    for ( QList<QGraphicsItem*>::iterator it = childItems().begin(); it != childItems().end(); ++it ) {
        qDebug() << (*it);
        (*it)->moveBy(pos().x(), pos().y()); // Segmentation fault
    }
    setPos(0, 0);
    QGraphicsItemGroup::mouseReleaseEvent(event);
}

перестановка мест слагаемых сумму не изменило(

[Bepec]

Вы ещё пару раз строчки поменяйте, мб поможет? Издеваюсь конечно, извините

Проверку там сделайте на NULL.

PS и вообще хватит ребячиться - breakpoint тудыть и пошагово смотрите что происходит и проверяй всё. Хотя б место вылета уточните.

[Swa]

Вообщем, я опытным путем выяснил, что список, полученный через childItems()  - одноразовый, и при вызове этого метода старые указатели на детей, полученные этим методом, инвалидируются.
Такой код работает:

Код:

QList<QGraphicsItem*> list = childItems();

for (QList<QGraphicsItem*>::iterator iter = list.begin(); iter != list.end(); iter++ ) {
	qDebug() << (*iter);
	(*iter)->moveBy(pos().x(), pos().y());
}

[Mist1K]

Вообщем, я опытным путем выяснил, что список, полученный через childItems() - одноразовый, и при вызове этого метода старые указатели на детей, полученные этим методом, инвалидируются.

Спасибище! Уж не знаю сколько я бы еще ковырялся без вашей помощи...

[GreatSnake]

Вообщем, я опытным путем выяснил, что список, полученный через childItems()  - одноразовый, и при вызове этого метода старые указатели на детей, полученные этим методом, инвалидируются.

А без опытов просто взять и посмотреть прототип функции?