Оптимизация отрисовки QGraphicsEllipseItem Опции

[Kagami]

Я не совсем это имел в виду. Тебе надо переопределить функцию, рассчитывающую столкновения. Она будет сама вызываться при необходимости. В последнем обновлении документации пример Colliding Mice был переведен на русский. Посмотри его внимательней (заодно ошибки поищешь )

[Litkevich Yuriy]

Petr0vi4, щёлкай по имени автора поста, вместо цитаты. Его имя сразу добавится в форму быстрого ответа.

[Petr0vi4]

Kagami, пришлось перелопатить все программу. Сделал теперь класс Control наследником QGraphicsEllipseItem, наследство от QObject убрал за ненадобностью (сначала хотел использовать сигналы/слоты, но потом отпало). Переписал функцию collidesWithItem() для молекул, работает, но как теперь отличить молекулу от стены?

[Kagami]

Посмотри в документации по QGraphicsItem::UserType и QGraphicsItem::type(). При определении столкновения смотришь какого типа исследуемый объект. Если это молекула или стена, то ты уже знаешь что с ними делать. В противном случае можно вызывать метод родителя.

[Petr0vi4]

Что-то плохо все работает, может все-таки решить проблему как-то по-другому? Вроде программа простая.

[Kagami]

Может выложишь исходники программы? Если она, конечно, не секретная

[Petr0vi4]

Да, конечно, вот проект.

Прикрепленные файлы

 PhysX.zip ( 11.45 килобайт ) Кол-во скачиваний: 116

 

[Kagami]

Мда.. в коде пока особо не копался, но нашел одно узкое место, устранение которого резко повысило скорость. Проблема была в функции Control::BallCollision(). Ты постоянно вызывал функцию collidingItems(), что приводило к постоянному созданию и удалению списка элементов. Правильнее было бы так:

Раскрывающийся текст

void Control::BallCollision() // столкновение
{
    QList<QGraphicsItem *> list = collidingItems();
    for(int i=0;i<list.count();i++)
    {
        if(list.at(i)->type() == 4)// с шаром
        {
            int Num = list.at(i)->data(1).toInt();
            SetVectorToBuffer(pos().x(),pos().y(),AllObjects[Num]->dx,AllObjects[Num]->dy);
            Changed = true;
        }
        else
        {
            if(list.at(i)->type() == 3)// со стеной
            {
                int Num = list.at(i)->data(1).toInt();
                if(Num==0)
                {
                    qreal Y_k = pos().y();
                    qreal X_k = pos().x();
                    SetVector(X_k,Y_k,X_k+dx,Y_k-dy);
                }
                //if(Num==1)
                else
                {
                    qreal X_k = pos().x();
                    qreal Y_k = pos().y();
                    SetVector(X_k,Y_k,X_k-dx,Y_k+dy);
                }
            }
        }
    }
}

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

P.S. Теперь узким местом стала фунция Module(). Поэтому в Control::MoveBall() я заменил ее на qAbs().

Сообщение отредактировал Kagami - 16.7.2009, 22:24

[Petr0vi4]

Ну вообще-то это и есть правильно, но медленно.

[Kagami]

После исправлений у меня стало работать со 100 элементами довольно шустро