リファクタリング - 電子メールアドレスを具体化する-jsチュートリアル-php.cn

リファクタリング - 電子メールアドレスを具体化する

Barbara Streisand

リリース： 2024-12-25 16:50:15

オリジナル

960 人が閲覧しました

一度だけ言ってください

TL;DR: 重複した電子メール検証を避けてください。

対処された問題

複数の場所で電子メール検証ロジックが繰り返されます。
検証ルールが矛盾するリスクがあります。
検証ルールを維持するのが難しい。
全単射違反
原始的な強迫観念
時期尚早な最適化

コードの匂い 46 - 繰り返されるコード

Maxi Contieri ・ 2020年12月8日

#おっと #コード初心者＃チュートリアル #webdev

コードの匂い 122 - 原始的な強迫観念

Maxi Contieri ・ 2022 年 3 月 17 日

#おっと #webdev ＃チュートリアル #初心者

コード・スメル 66 - ショットガン手術

Maxi Contieri ・ 2021年4月5日

#コード初心者＃チュートリアル #おっと #webdev

コードの匂い 177 - 小さな物体が欠けている

Maxi Contieri ・ 2022 年 11 月 5 日

#webdev #javascript #初心者＃プログラミング

コードの匂い 20 - 時期尚早な最適化

Maxi Contieri ・ 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;
    }
}