Análisis de la calidad con SonarCloud - UniExtremadura/proyecto-gps-25-26-gb05 GitHub Wiki

Servicio de Compras

Análisis y corrección realizadas por: Iván Ruiz López.

Análisis inicial

Incidencias resueltas

Función asíncrona fuera del constructor

  • Resuelto en la rama: hotfix/async-operation-outside-constructor.

Análisis tras la integración de la rama:

Marcar parámetros no reasignados como de sólo lectura

  • Resuelto en la rama: hotfix/auth-readonly-parameters

Análisis tras la integración de la rama:

Dejar de manejar una excepción lanzando otra excepción

  • Resuelto en la rama: hotfix/handle-exception-or-dont-catch-it-at-all

Análisis tras la integración de la rama:

Eliminar importación no usada

Esta incidencia surgió tras la resolución de la anterior.

  • Resuelto en la rama: hotfix/remove-unused-import

Análisis tras la integración de la rama:

Mejora final del servicio

En la siguiente gráfica se muestra la mejora en el número de incidencias del servicio, pasando de 22 incidencias iniciales a 18 (sin contar la introducida en la rama hotfix/handle-exception-or-dont-catch-it-at-all).

Servicio Usuarios (Alejandro Paniagua García)

Incidencias resueltas

Rama: hotfix/reflector-as-readonly

  • Marcar parámetros no reasignados como de sólo lectura

Rama: hotfix/remove-unused-imports

  • Eliminar imports sin utilizar

Rama: hotfix/rename-class-remove-unused-postBashDto

  • Eliminar import sin utilizar

  • Cambiar el nombre de una clase para que se ajuste a su expresión regular

Mejora final del servicio

En la siguiente gráfica se muestra la mejora en el número de incidencias del servicio de Usuarios, pasando de 28 incidencias iniciales a 23.

Servicio de Contenidos

Analisis realizado por Daniel Barrantes Pulido

Análisis inicial

Imagen de analisis inicial

Incidencias resueltas

Función asíncrona fuera del constructor

  • Resuelto en la nueva rama hotfix/async-operation-outside-constructor

Análisis tras el hotfix:

AnalisisTrasFix

Reflector leído pero nunca reasignado

Reflector no reasignado
  • Resuelto en la nueva rama hotfix/mark-reflector-as-readonly

Análisis tras el hotfix:

Analisis tras hotfix

Elemento importado pero no empleado (CacheModule)

CacheModule no usado
  • Resuelto en la nueva rama hotfix/remove-unused-import

Análisis tras el hotfix:

image

Preferencia de Number.parseInt antes que parseInt

image
  • Resuelto en la nueva rama hotfix/number-parse-int

Análisis tras el hotfix:

image

Mejora final del servicio

En la siguiente gráfica se muestra la mejora en el número de incidencias, pasando de 96 incidencias a 93 (pues el arreglo del constructor llevo a lugar a un nuevo issue debido a un signo ! mal puesto).

image

Servicio de estadísticas (Pablo Rebanales Álvarez)

¡¡¡ NOTA: Se ha tenido que pasar todo lo de la rama develop a la rama main, para que SonarQube detectase los ficheros correctamente !!!

Incidencias resueltas

Renombrar una variable local para cumplir una expresión regular

  • Resuelto en la rama: hotfix/rename-local-variable.

Análisis tras la integración de la rama:

NOTA: Se han realizado 2 commits y 2 merges debido a que se me olvidó cambiar el nombre de la variable al final del fichero.

Renombrar una variable local para cumplir una expresión regular 2

  • Resuelto en la rama: hotfix/rename-local-variable2.

Análisis tras la integración de la rama:

Eliminar un parámetro que no se usa

  • Resuelto en la rama: hotfix/remove-unused-parameter.

Análisis tras la integración de la rama:

Renombrar una función para cumplir una expresión regular

  • Resuelto en la rama: hotfix/rename-function.

Análisis tras la integración de la rama:

Mejora final del servicio

En la siguiente gráfica se muestra la mejora en el número de incidencias del servicio, pasando de 44 incidencias iniciales a 40.

Servicio Front-end (Miguel Alejandro García Sevilla)

Incidencias resueltas

Rama: hotfix/nested-ternary-operation

  • Reemplazar operador ternario por sentencia independiente.
numero-errores-inicial problema numero-errores-ej1

Rama: hotfix/use-input-button

  • Usar etiquetas o en lugar del rol "button".
problema-2 numero-errores-ej2

Rama: hotfix/number-parseint

  • Preferible usar Number.parseInt sobre parseInt.
problema-3 numero-errores-ej3

Rama: hotfix/anchor-button

  • Anchor usado como button.
problema-4-completo numero-errores-final

Mejora final del servicio

En la siguiente gráfica se muestra la mejora en el número de incidencias del servicio de Front-end, pasando de 524 incidencias iniciales a 508. numero-problemas-iniciales numero-problemas-finales

⚠️ **GitHub.com Fallback** ⚠️