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

Tener un code smell para comparaciones redundantes #230

Open
Gustrucco opened this issue Apr 6, 2019 · 3 comments
Open

Tener un code smell para comparaciones redundantes #230

Gustrucco opened this issue Apr 6, 2019 · 3 comments

Comments

@Gustrucco
Copy link
Contributor

Por ejemplo:

elemento < otro_elemento && elemento != otro_elemento

@flbulgarelli
Copy link
Member

¿Cuales serían todas las comparaciones redundantes @Gustrucco?

Anoto algunas que se me ocurren:

  • elemento < otro_elemento && elemento != otro_elemento => sobra el distinto
  • elemento > otro_elemento && elemento != otro_elemento => sobra el distinto
  • elemento < otro_elemento || elemento != otro_elemento => sobra el menor
  • elemento > otro_elemento || elemento != otro_elemento => sobra el mayor
  • elemento <= otro_elemento || elemento == otro_elemento => sobra el igual
  • elemento >= otro_elemento || elemento == otro_elemento => sobra el igual
  • elemento <= otro_elemento && elemento == otro_elemento => sobra el menor o igual
  • elemento >= otro_elemento && elemento == otro_elemento => sobra el mayor o igual

@flbulgarelli
Copy link
Member

flbulgarelli commented Apr 22, 2020

Generalización: salvo los casos imposibles (elemento >/< otro_elemento && elemento == otro_elemento) sería:

(X COMPARACION1 Y) OP_LOGICA (X COMPARACION2 Y)

¿No? @Gustrucco @julian-berbel

@Gustrucco
Copy link
Contributor Author

Gustrucco commented Apr 22, 2020

Correcto. En los casos de la conjunción te quedas con el que conjunto más acotado y de la disyunción con el más amplio, ¿no?

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

No branches or pull requests

2 participants