리팩토링 - 이메일 주소 구체화-JS 튜토리얼-php.cn

리팩토링 - 이메일 주소 구체화

Barbara Streisand

풀어 주다： 2024-12-25 16:50:15

원래의

844명이 탐색했습니다.

한 번만 말해 보세요

TL;DR: 중복 이메일 확인을 피하세요.

해결된 문제

여러 곳에서 이메일 검증 논리가 반복됩니다.
일관되지 않은 유효성 검사 규칙의 위험.
유효성 검사 규칙을 유지하기가 어렵습니다.
전단사 위반
원시적 집착
성급한 최적화

코드 냄새 46 - 반복되는 코드

맥시 콘티에리 ・ 2020년 12월 8일

#앗 #코드뉴비 #지도 시간 #웹개발

코드 스멜 122 - 원시적 집착

맥시 콘티에리 ・ 2022년 3월 17일

#앗 #웹개발 #지도 시간 #초보자

코드 스멜 66 - 샷건 수술

맥시 콘티에리 ・ 2021년 4월 5일

#코드뉴비 #지도 시간 #앗 #웹개발

코드 냄새 177 - 작은 물체 누락

맥시 콘티에리 ・ 2022년 11월 5일

#웹개발 #자바스크립트 #초보자 #프로그램 작성

코드 냄새 20 - 조기 최적화

맥시 콘티에리 ・ 2020년 11월 8일

#앗 #개발 중 #코딩 #코드냄새

단계

이메일 확인 로직이 중복되는 위치를 식별하세요.
유효성 검사 규칙을 캡슐화하기 위해 이메일 주소 클래스를 만듭니다.
원시 문자열 대신 이메일 주소 클래스를 사용하도록 코드를 리팩터링합니다.

샘플 코드

전에

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;
    }
}

로그인 후 복사

후에

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;
    }
}

로그인 후 복사