Dilo una vez y solo una vez  TL;DR: Evite validaciones de correo electrónico duplicadas.  Problemas abordados  Lógica   en varios lugares. de validación de correo electrónico repetida  Riesgo de reglas de validación inconsistentes.  Violación  de biyección   Obsesión primitiva   Optimización prematura  Códigos relacionados con olores   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-x-i7r34uj   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xxv   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xiv   https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xxxvi   Código Smell 20: Optimización prematura  Pasos  Identifique dónde se duplica la lógica de validación de correo electrónico.  Cree una clase   para encapsular las reglas de validación. Email Address  Refactorizar el código para utilizar la clase   en lugar de cadenas sin formato. Email Address  Código de muestra  Antes   public class Person { private String emailAddress; // Primitive Obsession public void setEmailAddress(String emailAddress) { // Duplicated code if (!emailAddress.matches( "^[\\w.%+-]+@[\\w.-]+\\.[a-zA-Z]{2,}$")) { throw new IllegalArgumentException( "Invalid email address format"); } this.emailAddress = emailAddress; } } public class JobApplication { private String applicantEmailAddress; public void setApplicantEmailAddress(String emailAddress) { // Duplicated code if (!emailAddress.matches( "^[\\w.%+-]+@[\\w.-]+\\.[a-zA-Z]{2,}$")) { throw new IllegalArgumentException( "Invalid email address format"); } this.applicantEmailAddress = emailAddress; } }  Después   public class EmailAddress { // 2. Create an `EmailAddress` class to encapsulate validation rules. private final String value; public EmailAddress(String value) { // The rules are in a single place // And all objects are created valid if (!value.matches("^[\\w.%+-]+@[\\w.-]+\\.[a-zA-Z]{2,}$")) { throw new IllegalArgumentException( "Invalid email address format"); } this.value = value; } } public class Person { private final EmailAddress emailAddress; public Person(EmailAddress emailAddress) { // 1. Identify where email validation logic is duplicated. // 3. Refactor code to use the `Email Address` // class instead of raw strings. // No validation is required this.emailAddress = emailAddress; } } public class JobApplication { private EmailAddress applicantEmailAddress; public JobApplication(EmailAddress applicantEmailAddress) { this.applicantEmailAddress = applicantEmailAddress; } }  Tipo    Semiautomático [x]  Seguridad  Esta refactorización es segura si reemplaza todas las apariciones de cadenas de correo electrónico sin procesar con la clase 'EmailAddress' y se asegura de que todas las pruebas pasen.  ¿Por qué es mejor el código?  Hace que la validación de correo electrónico sea consistente en toda su aplicación.  Como las reglas de validación están centralizadas en un solo lugar, el código resulta más fácil de mantener.  También reduce el riesgo de errores causados por una lógica inconsistente.  En el mundo real,   son   que existen y no son cadenas. Email Addresses pequeños objetos  El código refactorizado es más cercano al   del mundo real. MAPPER  Tenga en cuenta que los nombres de biyección son esenciales. Sería útil crear una   , no un   , ya que el Email debería   al mensaje real. EmailAddress Email corresponder  No permita que   le digan que esta solución tiene una penalización en el rendimiento. los optimizadores prematuros  Nunca realizan evaluaciones comparativas reales con datos del mundo real.  Refactorizar con IA  Sin instrucciones adecuadas  Con instrucciones específicas   ChatGPT   ChatGPT   Claudio   Claudio   Perplejidad   Perplejidad   Copiloto   Copiloto   Géminis   Géminis  Etiquetas  Encapsulación  Refactorizaciones relacionadas   https://maximilianocontieri.com/refactoring-007-extract-class?embedable=true   https://maximilianocontieri.com/refactoring-012-reify-associative-arrays?embedable=true   https://hackernoon.com/improving-the-code-one-line-at-a-time?embedable=true  Créditos  Imagen de   en  Gerd Altmann Pixabay  Este artículo es parte de la serie Refactoring.   Cómo mejorar tu código con refactorizaciones sencillas

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

Refactorización 019: Cómo reificar direcciones de correo electrónico

Refactorización 019: Cómo reificar direcciones de correo electrónico

About Author

COMENTARIOS

ETIQUETAS

ESTE ARTÍCULO FUE PRESENTADO EN

Related Stories

Cómo mejorar su flujo de trabajo 10 veces: 17 aplicaciones esenciales

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

Toque para ganar: Telegram puede incorporar a los próximos 10 mil millones de usuarios de criptomonedas antes de Solana

Cómo mejorar su flujo de trabajo 10 veces: 17 aplicaciones esenciales

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

Toque para ganar: Telegram puede incorporar a los próximos 10 mil millones de usuarios de criptomonedas antes de Solana

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps