Los problemas de rendimiento pueden venir por varias vías:
Bounding boxes: No has indicado cómo calculas el tamaño del QGraphicsItem
. Si el tamaño es excesivo se lanzarán eventos de repintado más a menudo y los mismos involucrarán más pixeles... hecha cuentas. Siempre y cuando su coste no sea totalmente privativo es preferible reducir el bounding box al mínimo imprescindible.
Uso de cliping: Ya lo estás aplicando... así que esta parte me la ahorro. Lo único que la llamada a setClipping
es redundante e innecesaria.
Antialiasing: Vale que los gráficos quedan más chulos... pero esta capa de procesamiento tiene un coste que puede resultar excesivo. Al igual que sucede con el primer punto, no indicas nada al respecto, así que ignoro si lo estás aplicando o no.
No preguntes por el tamaño del vector en cada iteración. Aunque los vectores pueden estar bastante optimizados... no tiene por qué ser así. Si en cada iteración tienes que preguntar por el tamaño del vector estás malgastando unos ciclos que pueden resultar preciosos:
for( int x=0, max=polys.count(); x<max; x++ )
No hagas copias innecesarias. En cada iteración del bucle estás creando una copia de un objeto de tipo PolyTrack
y esto es algo no solo innecesario sino además contraproducente. Considera usar referencias (constantes a poder ser):
PolyTrack const& pl = polys.at(x);
Conjugando los dos últimos puntos, si usas C++11 o superior plantéate usar el for
basado en rangos:
for( PolyTrack const& pl : polys )
No uses Q_UNUSED
. Esta macro está pensada, principalmente, para evitar warnings al usar RAII. Si no vas a usar un parámetro de la función es preferible comentar el propio parámetro. Con esto puedes conseguir liberar un registro del procesador y reducir la necesidad de tirar de la pila (más lenta):
void TrackFarmGraph::paint(QPainter *painter, const QStyleOptionGraphicsItem *item, QWidget * /*widget*/)
Dicho esto:
Es correcto lo que estoy haciendo?
El escalado es algo que realiza el siguiente nivel. Es ajeno a QGraphicsItem
, luego el debe dar siempre las mismas coordenadas.
Por otro lado, definir variables en el paint
, ¿también es motivo para que el rendimiento sea malo?
Depende... Si se acaban usando registros para gestionar esas variables y su construcción es gratuita (como sucede con los tipos nativos)... el rendimiento no se va a ver penalizado.
En caso de ser inevitable la definición de variables en el paint
, es conveniente que sean punteros (son mas veloces)?
Eso de que los punteros son más veloces que las variables es mentira, al menos si se toma como una respuesta canónica.
Por supuesto que es más rápido copiar un puntero que un objeto pesado:
struct POO
{
int array[100];
};
POO var1, var2, *ptr1, *ptr2;
POO var2 = var1; // Se copia el array -> pesado
ptr1 = ptr2; // Se copian direcciones de memoria -> ligero
Pero los tiempos son los mismos cuando se trabaja con tipos más pequeños:
int var1, var2;
int *ptr1, *ptr2;
// Ambas operaciones consumen los mismos ciclos de reloj
var1 = var2;
ptr1 = ptr2;
Y, por otro lado, los punteros añaden una indirección que debe ser resuelta cada vez que se pretenda acceder a información del objeto apuntado. Dicho de otra forma, los punteros penalizan el acceso al objeto:
POO objeto;
POO* ptr;
objeto.funcion(); // Llamada directa -> más rápido
ptr->funcion(); // Añade indirección -> más lento
Una posible excepción a este caso serían las funciones virtuales... pero no creo que sea un tema que se vaya a aplicar en tu caso.