sanyaade-g2g-repos / quimeraengine

Automatically exported from code.google.com/p/quimeraengine
0 stars 1 forks source link

QSphere #186

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Representa una esfera en el espacio tridimensional, representada por su centro 
y su radio.
Incluye funcionalidad de transformaciones.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 24 Aug 2011 at 10:43

GoogleCodeExporter commented 8 years ago
Empiezo con esta tarea.

Original comment by txan...@gmail.com on 29 Aug 2011 at 12:12

GoogleCodeExporter commented 8 years ago
Me pongo de owner, siempre se me olvida!

Original comment by txan...@gmail.com on 29 Aug 2011 at 12:15

GoogleCodeExporter commented 8 years ago
Ready for review

Original comment by txan...@gmail.com on 17 Nov 2011 at 4:40

GoogleCodeExporter commented 8 years ago
Resultado de la revisión:

-Documentación de constructor que recibe el centro y el radio: Podrías decir 
directamente que es un constructor que recibe el centro y el radio de la 
esfera, en lugar de un valor en coma flotante que define el radio de la esfera, 
lo digo porque la forma de decirlo es compleja sin necesidad.

-Documentación de SpaceRelation: "recieved" = "received". No pongas Space 
Relation en mayúsculas, no es el nombre de algo. Especifica que el lado 
positivo es aquel hacia donde apunta la normal del plano. Si no vas a devolver 
Contained, mejor no mencionarlo.

-SpaceRelation: ¿No podrías usar QFloat::Abs(fDistP) en lugar de 
fRealDistance? Creo que estás calculando 2 veces la distancia.

-Documentación de ProjectToPlane: Especifica que sólo se proyecta el punto, 
el radio permanece intacto.

Original comment by Lince3D@gmail.com on 17 Nov 2011 at 11:39

GoogleCodeExporter commented 8 years ago

Original comment by txan...@gmail.com on 21 Nov 2011 at 8:13

GoogleCodeExporter commented 8 years ago
Resultado de la revisión: Correcta.

-Documentación de SpaceRelation: "recieved" = "received".

Original comment by Lince3D@gmail.com on 21 Nov 2011 at 9:41