добавление GraphicsItem на сцену

[finka]

Допустим, код примерно такой:

Код:

QGraphicsScene * scene = new QGraphicsScene;
GraphicsItem * item;
for (int i = 0; i < MAX; i++)
{
    item = new GraphicsItem;
    scene->addItem(item);
}
QList <QGraphicsItem *> :: iterator iter;
int value;
for (iter = scene->items().begin(); iter != scene->items().end(); iter++)
{
    item = (GraphicsItem*)(*iter);
    tmp = item->myMember;
}

GraphicsItem - мой класс, наследованный от QGraphicsItem. При обращении к myMember (добавленное мной поле) происходит ошибка, точнее я вообще не могу обратиться ни к одному члену класса, кроме тех, что были в QGraphicsItem. Не могу понять, в чем дело:(

[pastor]

А что за ошибку печатает? Возможно у тебя члены класса в private/protected секции. Сделай public аксессоры к твоим членам класса

Можешь показать код GraphicsItem?

[finka]

Не, члены-то private, но функции доступа к ним public, и к членам я обращаюсь как раз через них, а ошибка происходит именно при обращению к члену (то есть уже в функции доступа). Сейчас посмотрела - там вообще все члены какие-то левые, в поле "value" написано: CXX0030: Error expression cannot be evaluated. А при обращении к какому-то члену класса, дебагер говорит :"0xC0000005: Access violation reading location 0xfeeefef2."

[goer]

Вобще то для такого преобразования типов в С++ есть dynamic_cast - не стоит им пренебрегать

И на будущее - телепаты уехали заграницу, там больше платят. Пожалуйста, описывайте проблемы так чтоб не приходилось флудить на форуме пустыми словами.

[pastor]

Приведи код GraphicsItem. У тебя гдето не инициализирован указатель. Проверь item на 0 после сроки:

item = (GraphicsItem*)(*iter);

Для приведения типа здесь достаточно static_cast.

[goer]

Приведи код GraphicsItem. У тебя гдето не инициализирован указатель. Проверь item на 0 после сроки:

item = (GraphicsItem*)(*iter);

Для приведения типа здесь достаточно static_cast.

Ребят, просил же, не разводите флуд и бесполезный флейм. Форуму это не на пользу, уважайте себя и других, не превращайте форум в помойку!

"Мудрый не все говорит, что знает, а дурак не все знает, что говорит."

Знатоки С++ идут лесом(упс!... читают Страуструпа).

[pastor]

2 goer: А где флуд в моем посте? Последняя цитата в мой огород?

[vaprele07]

Знатоки С++ идут лесом(упс!... читают Страуструпа).

в раздел шаблоны 

[finka]

Заметила интересную вещь. Если в представленном мной коде последний кусок

Код:

for (iter = scene->items().begin(); iter != scene->items().end(); iter++)
{
    item = (GraphicsItem*)(*iter);
    tmp = item->myMember;
}

заменить на следующий:

Код:

for (int i = 0; i < scene->items().size(); i++)
{
    item = (GraphicsItem *)scene->items().at(i);
    tmp = item->myMember;
}

, то все пучком. Другое дело, что если использовать такой метод доступа к элементам списка нерационален по времени.
Кстати, когда вместо тупого приведения к типу (В первом варианте кода) я использовала dynamic_cast, то программа вылетала непосредственно в самом dynamic_cast. А во втором варианте dynamic_cast естесвенно работает.

[vaprele07]

finka попробуй вот это:

Код:

for (iter = scene->items().begin(); iter != scene->items().end(); iter++)
  qDebug() << "kosjk";

потом сравни кол-во косяков с кол-вом напиханных туда GraphicsItem 

такая конструкция прежде всего говорит, что передаётся не ссылка а копия возможно итэмы вообще хранятся не в QList...
QList<QGraphicsItem *> QGraphicsScene::items () const
далее в конструкции iter != scene->items().end() сравниваются указатели не на данные а на Node * (внутренняя структура класса QList ) содержащий указатель на данные! отсюдова и...
так что второй пример работает без ошибок и возможно быстрее...

[finka]

потом сравни кол-во косяков с кол-вом напиханных туда GraphicsItem 

Естественно, количество косяков совпадает с кол-вом итемов.

такая конструкция прежде всего говорит, что передаётся не ссылка а копия возможно итэмы вообще хранятся не в QList...
QList<QGraphicsItem *> QGraphicsScene::items () const

Я с этим и не спорю.

далее в конструкции iter != scene->items().end() сравниваются указатели не на данные а на Node * (внутренняя структура класса QList ) содержащий указатель на данные! отсюдова и...

А вот это непонятно. Что отсюдова?

[vaprele07]

Естественно, количество косяков совпадает с кол-вом итемов.

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

К пример у меня ваша конструкция с преобразованиями не вызывает никаких ошибок при любых раскладах.

[Tonal]

Код:

QGraphicsScene * scene = new QGraphicsScene;
GraphicsItem * item;
for (int i = 0; i < MAX; i++)
{
    item = new GraphicsItem;
    scene->addItem(item);
}
[b]QList <QGraphicsItem *> :: iterator iter;[/b]
int value;
for (iter = scene->items().begin(); iter != scene->items().end(); iter++)
{
    item = (GraphicsItem*)(*iter);
    tmp = item->myMember;
}

GraphicsItem - мой класс, наследованный от QGraphicsItem.

Если ты привела полный код, то ошибка в выделенной строке.
Функция QGraphicsScene::items возвращает QList<QGraphicsItem *> где QGraphicsItem из стандартных заголовков Qt.
А в выделенной строке ты объявляешь итератор по QList<QGraphicsItem *> где QGraphicsItem определён тобой.
Не совсем понятно какое именно преобразование происходит при таком присвоении итераторов.
Попробуй переименовать свой класс, или явно прописать пространство имён для стандартного.

P.S. Кстати, какой компилятор и версия Qt?
P.P.S. Да, если на сцене присутствует хотя бы один итем не того типа, а например стандартного, то глюки тоже обеспечены. Для того чтобы их избежать пользуйся dynamic_cast-ом. Если же ты уверена, что такого быть не может ставь static_cast.
Я, обычно, делаю примерно так:

Код:

  assert(dynamic_cast<GraphicsItem*>(*iter));
  item = static_cast<GraphicsItem*>(*iter);

Т.е. при отладке получаешь assert, а в релизе не тормозит. ;-)

[pastor]

Есть еще такая функция: qgraphicsitem_cast

2 Tonal: что-то я не понял сказанное вами:

Функция QGraphicsScene::items возвращает QList<QGraphicsItem *> где QGraphicsItem из стандартных заголовков Qt.
А в выделенной строке ты объявляешь итератор по QList<QGraphicsItem *> где QGraphicsItem определён тобой.

У неё класс называеться GraphicsItem, который наследованный от QGraphicsItem

2 finka: Можете выложить полный код GraphicsItem?

[Tonal]

Точно, протупил. ;-)