QLineSegment2D Extension 002

[GoogleCodeExporter]

Añade funcionalidad relacionada con el cálculo de puntos de intersección.

Más información en:

http://quimeraengine.googlecode.com/files/QLineSegment2D%20Extension%20002.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 24 Mar 2011 at 5:39

[GoogleCodeExporter]

Original comment by chrisal...@gmail.com on 29 Jun 2011 at 12:41

[GoogleCodeExporter]

Original comment by chrisal...@gmail.com on 29 Jun 2011 at 4:15

• Changed state: Started

[GoogleCodeExporter]

Original comment by chrisal...@gmail.com on 7 Jul 2011 at 8:22

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Documentación de Intersection que recibe un QBaseLineSegment: "wheter" = 
"whether". Sobra la coma. Creo que no es necesario indicar que la 
implementación se basa en la de la clase base.

-Documentación de IntersectionPoint que recibe un QBaseLineSegment: No es 
necesario indicar que la implementación se basa en la de la clase base.

-IntersectionPoint que recibe QBaseTriangle: Correcciones de estilo: El prefijo 
arr- debe ser sustituido por ar-. Cuando el * se refiere al tipo y no al 
objeto, es preferible que esté pegado al tipo, por ejemplo "QVector2* 
firstInterPoint".

-IntersectionPoint que recibe QBaseTriangle: No es necesario utilizar el tipo 
u16_q. Esto no lo he explicado pero tales tipos son sólo obligatorios en 
partes del motor donde la longitud de los mismos debe ser siempre la misma, 
independientemente del compilador y el SO.

-IntersectionPoint que recibe QBaseTriangle: Si currentIntersections es E_None, 
¿devuelve infinito? Creo que ahí iría un else if en lugar de un else.

-IntersectionPoint que recibe QBaseTriangle: ¿Podrías poner un comentario en 
el último bloque else, indicando que ahí se asignan los parámetros de salida?

-IntersectionPoint que recibe QBaseQuadrilateral: Lo mismo que para la que 
recibe QBaseTriangle.

-TriangleContainsPoint: Se deben usar las cosntantes de QFloat en lugar de 
números cuando se trate con operaciones en coma flotante. En este caso, 1 debe 
ser sustituido por QFloat::_1 y 0 por QFloat::_0.

-TriangleContainsPoint: Las comparaciones entre floats, salvo casos concretos, 
deben realizarse mediante los métodos de QFloat, como por ejemplo 
IsGreaterThan, para aplicar una tolerancia de precisión.

-TriangleContainsPoint: Una posible optimización es el uso de referencias en 
lugar de declaraciones de variables nuevas. O sea, usar float_q& en lugar de 
float_q para guardar los resultados de las operaciones.

-IntersectionPoint: ¿Y si se encuentran ambas intersecciones cuando se han 
comprobado 2 aristas? ¿Por qué calcular el resto? Podrías modificarlo para 
evitar cálculos innecesarios. Quizá sea mejor usar ahí un bucle while en 
lugar del for, por no usar break dentro y tal.

Original comment by Lince3D@gmail.com on 21 Jul 2011 at 7:03

• Changed state: Started

[GoogleCodeExporter]

Si la persona que continúa esta tarea necesita el parche al que hace 
referencia la anterior revisión, se le enviará.

Original comment by Lince3D@gmail.com on 25 Sep 2011 at 10:42

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 26 Sep 2011 at 9:19

[GoogleCodeExporter]

Enviame el parche, plis, Thund

Original comment by jwl...@gmail.com on 27 Sep 2011 at 10:20

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 3 Oct 2011 at 3:22

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de Intersection que recibe un QBaseLineSegment: "wheter" = 
"whether". Sobra la coma. Creo que no es necesario indicar que la 
implementación se basa en la de la clase base.

Original comment by Lince3D@gmail.com on 5 Oct 2011 at 6:55

• Changed state: Done