一度だけ言ってください TL;DR: 重複した電子メール検証を避けてください。 対処された問題 複数の場所で 。 電子メール検証ロジックが繰り返されます 検証ルールが矛盾するリスク。 違反 一対一変換 原始的な強迫観念 時期尚早な最適化 関連するコードスメル 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 コード臭 20 - 時期尚早な最適化 手順 電子メール検証ロジックが重複している場所を特定します。 検証ルールをカプセル化する クラスを作成します。 Email Address 生の文字列の代わりに クラスを使用するようにコードをリファクタリングします。 Email Address サンプルコード 前に 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; } } タイプ セミオートマチック [x] 安全性 このリファクタリングは、生の電子メール文字列のすべての出現を 'EmailAddress' クラスに置き換え、すべてのテストに合格することを保証すれば安全です。 なぜコードの方が優れているのでしょうか? アプリケーション全体で電子メールの検証を一貫させます。 検証ルールが 1 か所に集中しているため、コードの保守が容易になります。 また、一貫性のないロジックによって発生するバグのリスクも軽減されます。 現実の世界では、  存在する であり、文字列ではありません。 Email Addresses 小さなオブジェクト リファクタリングされたコードは、実際の に近くなります。 MAPPER バイジェクション名が必須であることに注意してください。 Email は実際のメッセージに 必要があるため、  ではなく を作成すると役立ちます。 マップされる Email EmailAddress このソリューションにはパフォーマンスの低下があると言われないようにしてください。 早すぎる最適化者から、 彼らは現実世界のデータを使って実際のベンチマークを行うことは決してありません。  AIによるリファクタリング 適切な指示がなければ 具体的な指示 チャットGPT   チャットGPT   クロード クロード 困惑 困惑 副操縦士 副操縦士 ジェミニ ジェミニ タグ カプセル化 関連するリファクタリング 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 クレジット による の画像 Gerd Altmann Pixabay この記事はリファクタリング シリーズの一部です。 簡単なリファクタリングでコードを改善する方法

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

このオーディオは、ストーリーの元の言語で制作されています。

長すぎる; 読むには

Make resilience your competitive advantage

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

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

About Author

コメント

ラベル

この記事は

Related Stories

クラウド移行を成功させるための完全ガイド: 戦略とベストプラクティス

海を航海する: データレイクを使用した本番環境レベルの RAG アプリケーションの開発

フォーラムからフィードへ: ソーシャルメディアアルゴリズムがデジタルインタラクションを形作る仕組み

Telegram: クリプト島と本土を結ぶ橋

クラウド移行を成功させるための完全ガイド: 戦略とベストプラクティス

海を航海する: データレイクを使用した本番環境レベルの RAG アプリケーションの開発

フォーラムからフィードへ: ソーシャルメディアアルゴリズムがデジタルインタラクションを形作る仕組み

Telegram: クリプト島と本土を結ぶ橋

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps