QTriangle

[GoogleCodeExporter]

Representa un triángulo en el espacio y aporta funcionalidad independiente del 
número de dimensiones de éste.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 31 Mar 2011 at 6:43

[GoogleCodeExporter]

Comienzo con esta tarea.

Original comment by txan...@gmail.com on 26 Apr 2011 at 9:03

• Changed state: Started

[GoogleCodeExporter]

Al fin -> ready for review.

Original comment by txan...@gmail.com on 4 Jun 2011 at 9:45

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Elimina los comentarios: Forward declarations, usings, friends, typedefs, 
constants, enumerations, PROPERTIES y ATTRIBUTES. Aquellas secciones que no se 
usen no deben aparecer (y usings tampoco lo solemos poner).

-Eliminar el destructor, no es necesario definirlo.

-Elimina los includes de QVF32.h, QLineSegment.h y EQIntersections.h, no son 
necesarios.

-Constructor por defecto: No necesita llamar al constructor base (ya se llama 
automáticamente).

-Documentación de Operator==: Incompleta.

-Detalle de estilo: los operadores && no deben pegarse a los operandos.

-Documentación de GetSurface: Con la primera frase es suficiente.

-GetSurface: Usar QFloat::_2 en lugar de 2.

-GetAngleA, B, C: Podrías usar un método protected al que le pases por 
referencia 2 vectores y no repetir código 3 veces.

-GetCircumcenter y GetOrthocenter: Elimina ambas declaraciones y deja sólo 1 
línea de comentario con un TODO hablando del o que falta.

-Documentación GetIncenter y GetCentroid: El parámetro es sólo [OUT]. Borra 
la parte <returns>.

-GetCentroid: Usar QFloat::_3 en lugar de 3.

-ToString: Los ángulos < > son simbólicos, no deben aparecer.

Original comment by Lince3D@gmail.com on 4 Jun 2011 at 12:16

• Changed state: Started

[GoogleCodeExporter]

Corregido.

Original comment by txan...@gmail.com on 4 Jun 2011 at 6:17

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de Operator==: Incompleta. Falta descripción mínima de 
parámetro y que en lugar de empezar con Operator==, empiece, por ejemplo, con 
Equality operator.

-GetCircumcenter y GetOrthocenter: Yo los dejaría como comentarios normales 
(sin <summary>) para que no haya el más mínimo problema con el generador de 
documentación.

-Has repetido METHODS cuando empiezas con los métodos protected.

-TODO: Quedarán pendientes para el futuro la implementación independiente de 
dimensiones de GetOrthocenter y GetCircumcenter.

Original comment by Lince3D@gmail.com on 7 Jun 2011 at 6:00

• Changed state: Done-TODO

[GoogleCodeExporter]

Update y commit realizado. Revisión 724.

Original comment by txan...@gmail.com on 8 Jun 2011 at 10:17

[GoogleCodeExporter]

Métodos GetOrthocenter y GetCircumcenter traspasados a sus respectivas clases 
derivadas.

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

• Changed state: Done