Huele porque es probable que haya muchos casos en los que podría editarse o mejorarse.  La mayoría de estos olores son solo indicios de algo que podría estar mal. Por lo tanto, no es necesario que se arreglen per se... (Sin embargo, debe investigarlo).   Olores de código anteriores  Puedes encontrar todos los olores del código anterior (Parte i - XXXI)   . aquí  Continuemos...  Code Smell 156 - Implícito Else    Aprendemos if/else en nuestro primer día de programación. Entonces nos olvidamos de lo demás.  TL; DR: Sea explícito. Incluso con Else.  Problemas  Legibilidad  Carga cognitiva  Condiciones imprevistas   Violación del principio Fail Fast  Soluciones  Escribe el else explícito  Contexto  Si regresamos temprano a una oración IF, podemos omitir la parte else.  Luego,   y usamos polimorfismo. eliminamos el IF  Ahí es cuando nos perdemos los casos reales.  Código de muestra  Equivocado   function carBrandImplicit(model) { if (model === 'A4') { return 'audi'; } return 'Mercedes-Benz'; }  Bien   function carBrandExplicit(model) { if (model === 'A4') { return 'audi'; } if (model === 'AMG') { return 'Mercedes-Benz'; } // Fail Fast throw new Exception('Model not found); }  Detección    Automático [x]  Podemos verificar los árboles de sintaxis y analizarlos y advertir si falta algo más.  También podemos reescribirlos y realizar pruebas de mutación.  Etiquetas  Condicionales  Conclusión  Este tipo de olor genera mucho debate público   . y odio  Debemos intercambiar opiniones y valorar cada pro y contra.  Relaciones   Código Olor 102 - Código de flecha   Code Smell 36 - Declaraciones Switch/case/else if/else/if  Más información   Dejar de usar Implícito Else   Cuándo usar Implícito Else   Fallar rapido   Cómo deshacerse de los IF molestos para siempre  Créditos  Foto de   en  Elena Mozhvilo Unsplash  El mayor problema de los equipos de software es asegurarse de que todos entiendan lo que hacen los demás.   Martín Cazador  Código Olor 157 - Saldo a 0    Hoy, esperaba un pago en mi billetera. El saldo era 0. Entré en pánico.  TL;DR: nulo no es 0. El error no es 0. solo 0 es 0.  Problemas   nulos  experiencia de usuario  usabilidad  Soluciones  Haga una distinción clara entre un Cero y un error.  Contexto  Leí mucho sobre temas de seguridad.  Especialmente en criptografía.  La semana pasada, leí sobre un   . hilo de piratería criptográfica  Cuando mi billetera me mostró 0 como saldo, entré en pánico.  Era solo un olor a UX.  La cadena de bloques era inalcanzable 💩  Código de muestra  Equivocado   """ Below code is automatically generated by code-davinci-002 on GTP3 Codex 1. check balance with blockchain 2. If blockchain is unreachable show 0 as the balance """ import requests import json def get_balance(address): url = "https://blockchain.info/q/addressbalance/" + address response = requests.get(url) if response.status_code == 200: return response.text else: return 0  Bien   """ Below code is automatically generated by code-davinci-002 on GTP3 Codex 1. check balance with blockchain 2. If blockchain is unreachable throw an error """ import requests import json def get_balance(address): url = "https://blockchain.info/q/addressbalance/" + address response = requests.get(url) if response.status_code == 200: return response.text else: raise BlockchainNotReachableError("Error reaching blockchain")  Detección    manuales [x]  Este es un olor de diseño.  Podemos encontrar patrones cuando se lanza una   o   y se enmascara con un 0. excepción un código de retorno  Etiquetas  experiencia de usuario  Conclusión  Siga siempre el   como guía. principio del menor asombro  Relaciones   Código Olor 12 - Nulo   Code Smell 139 - Código comercial en la interfaz de usuario   Code Smell 73 - Excepciones para casos esperados   Code Smell 72 - Códigos de retorno  Más información   Null: El error del billón de dólares  Crédito  Foto de   en  Jasmin Sessler Unsplash  Descargo de responsabilidad  Code Smells es solo mi   . opinión  Mi verdadera crítica con Null es que trae de vuelta innecesariamente toda la agonía de tener que elegir si ejecutar su programa rápido sin verificar o hacerlo lento con verificación.    (Inventor nulo) Tony Hoare   Grandes citas de ingeniería de software  Code Smell 158 - Variables no variables    Asignas un valor a una variable y lo usas pero nunca lo cambias.  TL; DR: sea declarativo sobre la mutabilidad.  Problemas  Legibilidad  Honra la mutabilidad   . de Biyección  Posibles problemas de rendimiento y memoria.  Soluciones  Cambie   y sea claro en su alcance. la variable a una constante  Refactorizaciones   Refactorización 003 - Extraer constante   Refactorización 008 - Convertir variables a constantes  Contexto  Siempre estamos aprendiendo del dominio.  A veces suponemos que un valor puede cambiar con el   . MAPPER  Más tarde, nos enteramos de que no cambiará.  Por lo tanto, necesitamos promoverlo a una constante.  Esto también evitará   . Magic Constants  Código de muestra  Equivocado   <?php function configureUser() { $password = '123456'; // Setting a password on a variable is another vulnerability // And Code Smell $user = new User($password); // Notice Variable doesn't change }  Bien   <?php define("USER_PASSWORD", '123456') function configureUser() { $user = new User(USER_PASSWORD); } // or function configureUser() { $user = new User(userPassword()); } function userPassword() : string { return '123456'; } // Case is an oversimplification as usual  Detección    Automático [x]  Muchos linters verifican si la variable tiene una sola asignación.  También podemos realizar pruebas de mutación e intentar modificar la variable para ver si las pruebas se rompen.  Etiquetas  Mutabilidad  Conclusión  Debemos desafiarnos a nosotros mismos y refactorizar cuando el alcance de la variable sea claro y aprendamos más sobre sus propiedades y   . mutabilidad  Relaciones   Code Smell 116 - Variables declaradas con 'var'   Code Smell 127 - Constantes mutables   Code Smell 107 - Reutilización de variables   Code Smell 02 - Constantes y números mágicos  Más información   El poder maligno de los mutantes  Descargo de responsabilidad  Code Smells es solo mi   . opinión  Créditos  Foto de   en  Noah Buscher Unsplash  Invariablemente se descubre que un sistema complejo que funciona ha evolucionado a partir de un sistema simple que funcionó.   Juan Gallo  CÓDIGO OLOR 159 - MIXED_CASEI    El desarrollo serio es realizado por muchas personas diferentes. Tenemos que empezar a ponernos de acuerdo.  TL; DR: no mezcle diferentes conversiones de casos  Problemas  Legibilidad  mantenibilidad  Soluciones  Elija un estándar de caja  Aferrate a ello  Contexto  Cuando diferentes personas crean software juntas, pueden tener diferencias personales o culturales.  Algunos prefieren   🐫, otros   🐍, MACRO_CASE🗣️, y   . camelCase snake_case muchos otros  El código debe ser sencillo y legible.  Código de muestra  Equivocado   { "id": 2, "userId": 666, "accountNumber": "12345-12345-12345", "UPDATED_AT": "2022-01-07T02:23:41.305Z", "created_at": "2019-01-07T02:23:41.305Z", "deleted at": "2022-01-07T02:23:41.305Z" }  Bien   { "id": 2, "userId": 666, "accountNumber": "12345-12345-12345", "updatedAt": "2022-01-07T02:23:41.305Z", "createdAt": "2019-01-07T02:23:41.305Z", "deletedAt": "2022-01-07T02:23:41.305Z" // This doesn't mean THIS standard is the right one }  Detección    Automático [x]  Podemos informar a nuestros linters sobre los amplios   de nuestra empresa y hacerlos cumplir. estándares de nombres  Cada vez que llega gente nueva a la organización, una prueba automatizada debe pedirle cortésmente que cambie el código.  Excepciones  Siempre que necesitemos interactuar con código fuera de nuestro alcance, debemos usar los estándares del cliente, no los nuestros.  Etiquetas  Denominación  Conclusión  Tratar con los estándares es fácil.  Tenemos que hacerlas cumplir.  Relaciones   Code Smell 48 - Código sin estándares  Más información   Qué es exactamente un nombre - Parte I La Búsqueda   Todas las convenciones de nomenclatura  Descargo de responsabilidad  Code Smells es solo mi   . opinión  Créditos  Foto de   en  Wolfgang Hasselmann Unsplash  Si tiene demasiados casos especiales, lo está haciendo mal.   craig cerouni  Código Olor 160 - Id no válido = 9999    Maxint es un muy buen número para una identificación inválida. Nunca lo alcanzaremos.  TL;DR: No combine identificaciones reales con identificaciones no válidas. De hecho: Evite las identificaciones.  Problemas  Violación   . de biyección  Es posible que llegue a la identificación no válida antes de lo que piensa.  Tampoco utilice   para identificaciones no válidas. valores nulos  Acoplamiento de banderas de la persona que llama a las funciones.  Soluciones  Modele casos especiales con objetos especiales.  Evite 9999, -1 y 0, ya que son objetos de dominio y acoplamiento de implementación válidos.  Introducir objeto nulo  Contexto  En los primeros días de la informática, los tipos de datos eran estrictos.  Entonces inventamos   . El error de los mil millones de dólares  Luego crecimos y modelamos escenarios especiales con valores especiales polimórficos.  Código de muestra  Equivocado   #include "stdio.h" #include "stdlib.h" #include "stdbool.h" #define INVALID_VALUE 999 int main(void) { int id = get_value(); if (id==INVALID_VALUE) { return EXIT_FAILURE; // id is a flag and also a valid domain value } return id; } int get_value() { // something bad happened return INVALID_VALUE; } // returns EXIT_FAILURE (1)  Bien   #include "stdio.h" #include "stdlib.h" #include "stdbool.h" // No INVALID_VALUE defined int main(void) { int id; id = get_value(); if (!id) { return EXIT_FAILURE; // Sadly, C Programming Language has no exceptions } return id; } get_value() { // something bad happened return false; } // returns EXIT_FAILURE (1)  Detección    Semiautomático [x]  Podemos verificar constantes especiales y valores especiales en el código.  Etiquetas  Nulo  Conclusión  Deberíamos usar números para relacionarnos con los identificadores externos.  Si no existe un identificador externo, entonces no es un número.  Relaciones   Code Smell 120 - Identificaciones secuenciales   Código Olor 12 - Nulo  Más información   Null: El error del billón de dólares   Y2K22 - El error que nos avergüenza  Descargo de responsabilidad  Code Smells es solo mi   . opinión  Créditos  Foto de   en  Markus Spiske Unsplash  Los insectos acechan en las esquinas y se congregan en los límites.   Boris Beizer  Próximamente llegarán 5 olores de código más...

Winner of Noonies 2020 for the award Hacker Noon Contributor of the Year - REFACTORING.

2020 - Hacker Noon Contributor of the Year in REFACTORING

2022 - HackerNoon Contributor of the Year - Clean Code

2022 - HackerNoon Contributor of the Year - Debugging

2022 - HackerNoon Contributor of the Year - Github

2022 - HackerNoon Contributor of the Year - Machine Learning

2022 - HackerNoon Contributor of the Year - Quality Assurance

2022 - HackerNoon Contributor of the Year - Refactoring

2022 - HackerNoon Contributor of the Year - Software Architecture

2022 - the Greatest Storyteller

Buy My Book

Share your ideas with me!

Read My Stories

Buy Me a Coffee

Software Engineer and Book Author

Este audio es producido en el idioma original de la historia!

Demasiado Largo; Para Leer

Cómo encontrar las partes apestosas de su código [Parte XXXII]

Cómo encontrar las partes apestosas de su código [Parte XXXII]

About Author

COMENTARIOS

ETIQUETAS

ESTE ARTÍCULO FUE PRESENTADO EN

Related Stories

Crecimiento de las criptomonedas: creación de perfiles de usuarios eficaces

Las capas invisibles: por qué las entrevistas con los usuarios son un activo irremplazable

De los foros a los feeds: cómo los algoritmos de las redes sociales influyen en la interacción digital

¿Quieres ganar un concurso de redacción de HackerNoon? Esto es lo que recomiendan los ganadores del concurso #crypto-api

Crecimiento de las criptomonedas: creación de perfiles de usuarios eficaces

Las capas invisibles: por qué las entrevistas con los usuarios son un activo irremplazable

De los foros a los feeds: cómo los algoritmos de las redes sociales influyen en la interacción digital

¿Quieres ganar un concurso de redacción de HackerNoon? Esto es lo que recomiendan los ganadores del concurso #crypto-api

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps