Fallos en la entrega - Githubissues

samuelmorenov / RI-JPA

0 stars 0 forks source link

Closed samuelmorenov closed 4 years ago

samuelmorenov commented 4 years ago

[x] Líneas de código demasiado largas
[x] Gestión de workOrders (fallos) : find unfinished no está implementado los métodos findWorkOrderBy tampoco
[x] Gestión de workOrders (fallos): Faltan comprobaciones: existe la workorder, existe mecánico, existe tipo vehículo, existe id, …
[x] Generación de certificado: TrainingHours::calculate debe dividir por 10000
[x] Gestión de workOrders (fallos): asignar usa Associations.link y no el método correspondiente de la entidad
[x] Listados: utiliza los porcentajes como horas Revision: Todo OK, excepto que olvidó dividir por 100
[x] No hay validación de argumentos en clases del modelo
[x] No has terminado las prácticas del laboratorio: CreateInvoiceFor

samuelmorenov commented 4 years ago

Concepto	Peso	Error
Líneas de código demasiado largas	-0,5	C3
Gestión de workOrders (fallos) : `find unfinished no está implementado` `los métodos findWorkOrderBy tampoco`	-1	E2
Gestión de workOrders (fallos): `Faltan comprobaciones: existe la workorder, existe mecánico, existe tipo vehículo, existe id, …`	-2	E3+
Generación de certificado: `TrainingHours::calculate debe dividir por 10000`	-2	E4+
Gestión de workOrders (fallos): `asignar usa Associations.link y no el método correspondiente de la entidad`	-2	E6+
Listados: `utiliza los porcentajes como horas` Revision: `Todo OK, excepto que olvidó dividir por 100`	-0,5	E8
No hay validación de argumentos en clases del modelo	-1	M17
No has terminado las prácticas del laboratorio: CreateInvoiceFor	-1	P6

samuelmorenov commented 4 years ago

Generación de certificado: TrainingHours::calculate debe dividir por 10000

Al calcular las horas (GenerateDertificates.java línea 42), el método TrainingHours::Calculate utiliza el porcentaje de dedicación como un valor a multiplicar de forma absoluta. Pero es un porcentaje sobre 100; multiplicas las horas por 25 (por ejemplo) en lugar de dividir entre 4.

Listados: utiliza los porcentajes como horas Revision: Todo OK, excepto que olvidó dividir por 100

Exactamente lo mismo. Utilizas los porcentajes como valores absolutos, en lugar de dividir por 100.