DEV Community

Discussion on: Refactoring the diagnosis of CORONA VIRUS in JS.

Collapse
 
hq063 profile image
Gonzalo HQ063

Muy bueno, un par de detalles unicamente

  • En la opcion 1 usas else if (hasFever){ y en esa no definiste un hasFever
  • En la opcion 2 if ((hasFever() && difficultyBreathing)||(hasFever() and isRiskCondition()) || (hasFever() && isRiskCondition() && difficultyBreathing)){ Si hasFever() && isRiskCondition() && difficultyBreathing es true, entonces tambien son true los dos checkeos anteriores en ese or. Por tanto no tiene sentido incluir ese ultimo caso de que las 3 sean true.
  • En todas las primeras opciones mantenes el

} else if (hasFever){
history.replace(`/cuarentena/`);
} else if (!hasFever){
history.replace(`/diagnostico_bueno/`);
} else {
history.replace(`/diagnostico_bueno/`);
}

que el else final es unreachable. if(hasFever) else if (!hasFever) ya cubren todos los casos. Por lo tanto lo logico sería simplemente:

} else if (hasFever){
history.replace(`/cuarentena/`);
} else {
history.replace(`/diagnostico_bueno/`);
}

Collapse
 
damxipo profile image
Damian Cipolat

Respecto a las opciones 1 y 2 son 1ras instancias del código a refactorizar es factible q eso q mencionas venga de implementación original.

Collapse
 
damxipo profile image
Damian Cipolat

Gracias x la observación