Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uso de expresión compleja en un condicional #54

Open
svhurtado opened this issue May 27, 2023 · 0 comments
Open

Uso de expresión compleja en un condicional #54

svhurtado opened this issue May 27, 2023 · 0 comments
Labels

Comments

@svhurtado
Copy link
Collaborator

svhurtado commented May 27, 2023

Comportamiento esperado

El código debe ser claro (fácil de entender) para permitir realizar pruebas, realizar depuraciones y cambios.

Comportamiento actual

En una condición, en el archivo "core\client\app\controllers\members.js: line 274, col 48", se encuentra una expresión compleja, que tiene varios operadores lógicos y matemáticos y además usa negación. Esto puede ser difícil de entender y es riesgoso para cambios. Se podría aplicar alguna refactorización.

Pasos a reproducir

Este defecto se encontró gracias a un analizador estático de código (linter). Los pasos son:

  1. Ejecutar las instrucciones que hay en ReadMe de este repositorio, en la sección de "Instrucciones para ejecutar análisis de código estático con JSHint.
  2. En los resultados que se muestran por consola se observa el error identificado (ver imagen en evidencias).

Evidencias

Mensaje del analizador:
image

Expresión encontrada en el código:
image

Contexto (Ambiente de ejecución)

Herramienta probada: Ghost versión 3.41.1
Herramientas usadas para las pruebas:
Node JS versión 12.22.1
JSHint versión 2.13.6
Dispositivo: portátil HP Intel Core i5-7200U CPU @ 2.50GHz 2.71 GHz, 12 GB de RAM
S.O.: Windows 10 Pro 21H2 (64 bits)
Navegador: Google Chrome Versión 111.0.5563.147

Información adicional

Ninguna

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant