QHexahedron

[GoogleCodeExporter]

Representa un poliedro de 6 caras que no tienen por qué ser iguales. Sus 8 
vértices no están sujetos a ninguna restricción espacial.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 1 Sep 2011 at 7:50

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 14 Oct 2011 at 7:06

• Changed state: Started

[GoogleCodeExporter]

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

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Documentación de la clase: Tengo mis dudas sobre si deberíamos emplear la 
palabra box. Generalmente, tiene connotaciones que dan a entender que es un 
ortoedro, es decir, con las caras paralelas 2 a 2 y las aristas formando 
ángulos rectos. Generalmente en la documentación inglesa se utilizan para 
temas de colisiones y tal, donde se habla de boxes en el sentido que digo. Es 
algo sutil, pero si se te ocurre algo distinto así a volapié, tanto mejor.
"vertex" = "vertices".

-Documentación de parámetros de constructor que recibe 2 vértices: No me 
llega a gustar el paréntesis "We'll call it A / G", la verdad. Si se 
corresponde con A o G, se dice claramente. Si es por llamarlo así por 
entenderse, quedaría mejor "let's call it A". La primera persona no debe 
usarse directamente en documentación oficial. ¿Podrías cambiarlo por favor 
tanto aquí como en QBaseHexahedron?

-Documentación de Rotate, que recibe un cuaternión y devuelve un hexaedro: 
"-The segment is NOT modified, it stays the same.". Sería reemplazar segment 
por hexahedron o quitarlo.

-Documentación de GetPlanes: ¿No sería más útil decir qué cara 
corresponde con qué índice?

-GetPlanes: Falta el modificador const al final de la firma de la función. El 
parámetro planes debe tener el sufijo ar-, por ser un array.

-Documentación de Contains: "Check" = "Checks". "hexahedron is considered" = 
"hexahedron are considered". El parámetro es `[`IN`]`.

-Documentación de SpaceRelation: No entiendo si un punto contenido en el plano 
pertenecerá a una cara o si se considerará que se está cruzando el plano.

-SpaceRelation: Corrígeme si me equivoco, ¿no existe funcionalidad para 
hallar la distancia de un punto a un plano? ¿Lo haces por optimizar? (hay que 
investigar una cosa referente a QPlane::PointDistance, lo pongo en el foro).

-SpaceRelation y ToString: Falta el modificador const al final de la firma de 
la función.

Original comment by Lince3D@gmail.com on 20 Oct 2011 at 8:32

• Changed state: Started

[GoogleCodeExporter]

-Utiliza además el tipo QBaseVector3 en lugar de QVector3 en los parámetros 
de traslación y escala.

Original comment by Lince3D@gmail.com on 20 Oct 2011 at 8:53

[GoogleCodeExporter]

Resultado del a revisión (II): Correcta.

-Falta sustituir un box por un hexahedron en la documentación de clase.

Original comment by Lince3D@gmail.com on 20 Oct 2011 at 10:50

• Changed state: Done