QPoint

[GoogleCodeExporter]

Se trata de una clase estática de utilidades para trabajo con puntos. 
Actualemnte, proveerá funcionalidad para aplicar transformaciones a una lista 
de puntos representados por vectores. Los métodos creados para tal efecto 
serán utilizados por todas las clases geométricas.

Más información en:

http://quimeraengine.googlecode.com/files/QPoint.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 6 Aug 2011 at 8:07

[GoogleCodeExporter]

El constructor debe ser privado.

Original comment by Lince3D@gmail.com on 9 Aug 2011 at 7:12

[GoogleCodeExporter]

Comienzo.

Original comment by txan...@gmail.com on 31 Aug 2011 at 6:59

• Changed state: Started

[GoogleCodeExporter]

Ready for review.

Original comment by txan...@gmail.com on 11 Sep 2011 at 6:15

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Documentación de la clase: "thet" = "that". "related with" = "related to". No 
necesitas decir dónde deben ser usados los métodos de la clase, la 
documentación es para el usuario final, no para nosotros.

-Documentación del constructor: Ponla sin summary. No debe aparecer en la 
documentación generada.

-Nombres de parámetros: El prefijo para arrays es ar-. En lugar de por ejemplo 
vPointArray sería más arPoints. Cuida de actualizarlo también en la 
documentación.

-Documentación en general: Procura decir "2D points" en lugar de "2D vector 
type points", queda muy engorroso.

-Parámetros en general: Los parámetros se pasan por referencia.

-Documentación de parámetros en general: Los parámetros cuya información 
vaya a usarse en un cálculo interno (son entrada) y que además guardarán el 
resultado (son también de salida) se marcan con [IN/OUT].

-Por favor, pon X, Y y Z como sufijos en los nombres de los parámetros en 
lugar de ponerlo delante, se ven mucho mejor. Cuida de actualizarlo también en 
la documentación.

-Aunque la optimización se hará más adelante, te adelanto que habría una 
forma más eficiente de recorrer los arrays en este caso: 

unsigned int i = nElements; 
while(--i > 0)
{
    ...
}

-Documentación de Rotate que recibe un ángulo: Lo de que es Right Handed se 
va a comprobar. Debería ser Left Handed. Este tipo de cosas no se ponen jamás 
en la documentación, sino en un comentario normal con exclamaciones o con una 
etiqueta SMELL o como prefieras.

-Documentación de Rotate y Scale: "translated" = "scaled". Cuidado con los 
copypastes ;)

-La palabra "Scalation" no existe, se utiliza "Scale".

-Todos los métodos deberían ser inline, o de lo contrario ir en el CPP.

-El cartel METHODS sólo debe estar una vez.

-Creo que no es necesario especificar los tipos de plantilla al llamar a los 
métodos de plantilla, se supone que se deducen de las variables que pasas como 
parámetro.

-Transform que recibe un QTransformationMatrix y un VectorType: Sería 
"vPoint.Transform(matrix);" a secas o 
"vPoint.Transform<QTransformationMatrix<MatrixType>>(matrix);".
Es similar lo que ocurre en Translate que recibe QTranslationMatrix y 
VectorType.

Original comment by Lince3D@gmail.com on 22 Sep 2011 at 8:45

• Changed state: Started

[GoogleCodeExporter]

Ready for review

Original comment by txan...@gmail.com on 25 Sep 2011 at 10:16

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Translate que recibe un QBaseVector2: El parámetro vTranslation debería 
pasarse por referencia.

-Scale que recibe un QBaseVector2: El parámetro vScale debería pasarse por 
referencia.

-Documentación de Rotate y Scale: "translated" = "scaled" y "translated" = 
"rotated". Cuidado con los copypastes ;)

-La palabra "Scalation" no existe, se utiliza "Scale".

-Scale que recibe 3 float_q (privado): Los parámetros deben ser pasados por 
referencia.

Original comment by Lince3D@gmail.com on 25 Sep 2011 at 9:35

• Changed state: Started

[GoogleCodeExporter]

Ready for review. Corregido lo indicado.

Original comment by txan...@gmail.com on 26 Sep 2011 at 9:55

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de Rotate: "translated" = "rotated". Cuidado con los copypastes 
;)

-Cambia el método GetNormal como hablamos en el foro antes de subir.

Original comment by Lince3D@gmail.com on 26 Sep 2011 at 9:39

• Changed state: Done

[GoogleCodeExporter]

Thund, creo que aqui ha habido un desliz. Creo que la tarea que quer�as dar
por Done es QTriangle3D, �verdad?

Original comment by txan...@gmail.com on 27 Sep 2011 at 9:21

[GoogleCodeExporter]

Se me ha ido la olla con el comentario de GetNormal, demasiadas cosas en la 
cabeza. Pero sí, esta tarea está DONE, la puedes subir cuando corrijas la 
documentación.

Original comment by Lince3D@gmail.com on 27 Sep 2011 at 10:14

[GoogleCodeExporter]

Subido en la revisión 807, con la documentación corregida (si queda algún 
translated donde no deba estar, prometo gritar "esto es ESPARTA" y dar una 
patada a la mesa, con las botas de puntera dura).

Original comment by txan...@gmail.com on 27 Sep 2011 at 10:38