QRay2D

[GoogleCodeExporter]

Representa un "rayo" en el espacio bidimensional, esto es, un punto del cual 
parte una normal o dirección que traza una línea infinita imaginaria. Aporta 
funcionalidad relacionada intersecciones y transformaciones en el espacio 
bidimensional.

Más información en:

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

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

[GoogleCodeExporter]

Original comment by marcial....@gmail.com on 4 Apr 2011 at 10:55

• Changed state: Started

[GoogleCodeExporter]

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

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 9 Oct 2011 at 7:00

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 13 Oct 2011 at 8:13

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Documentación de las constantes: "growing direction" ¿es una expresión 
utilizada normalmente? Lo pregunto desde la ignorancia.

-Documentación del parámetro vDirection en el constructor: Podríamos añadir 
que el vector debería estar normalizado.

-Documentación de IntersectionPoint que recibe un rayo: "When rays are 
parallel, one ray containing the other ray end point is checked." Esto no lo 
entiendo.

-En general, debería hablarse más del "origen del rayo" en lugar del "extremo 
del rayo".

-Transform: Supongo que ya lo habrás hecho en extensiones posteriores, pero 
este en realidad ahora se llama Rotate y usa QPoint.

-Contains: Esto es solo un detalle, pero tenlo en cuenta, en esta línea por 
ejemplo, "QFloat::IsNegative(vPoint.y - this->Point.y) == 
QFloat::IsNegative(this->Direction.y)", dado que ambas expresiones son 
booleanas, debe usarse siempre un doble ampersand (&&) en lugar de una 
comparación. De lo contrario el código se hace menos legible a cambio de nada.

-Intersection e IntersectionPoint (protected): ¿Cuál es el problema con que 
vAux o this->Direction sean vector nulo? Me refiero a los asserts.

-[TODO] jwladi: Replace by the QTriangle2D or QTriangle Contains method, when 
it exists.

-[TODO] jwladi: Replace by the QQuadrilateral Contains method, when it exists.

Original comment by Lince3D@gmail.com on 21 Oct 2011 at 10:43

• Changed state: Done-TODO

[GoogleCodeExporter]

-[TODO] jwladi: Esto debería ir en la clase base, es independiente de 
coordenadas.

-Por favor, avisa allá donde creas necesario que el rayo debe estar 
normalizado. Por ejemplo hay métodos IntersectionPoint que lo presuponen para 
un correcto funcionamiento.

Original comment by Lince3D@gmail.com on 21 Oct 2011 at 10:48

[GoogleCodeExporter]

LA pongo en RFR ya que te he mandado las correcciones para que las veas.

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

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

Original comment by Lince3D@gmail.com on 23 Oct 2011 at 12:12

• Changed state: Done-TODO