長いメソッドのメソッド名

Question

良いスタイル（クリーンコードブック）では、メソッドの名前がメソッドの機能を記述する必要があることを示しています。だから、例えば私は名前が何かする必要があり、そのデータベースに格納し、アドレスを確認する方法があり、電子メールを送信する場合などverifyAddressAndStoreToDatabaseAndSendEmail(address);

または

verifyAddress_StoreToDatabase_SendEmail(address);

私は分割することができますが、その3つのメソッドの機能は、私はまだこれらの3つのメソッドを呼び出すメソッドが必要です。したがって、大きなメソッド名は避けられません。

名前付きのメソッドは確かにそのメソッドが何をしているのかを記述しますが、IMOは名前が非常に大きくなる可能性があるためあまり読み込めません。どのようにそれを解決しますか？

編集：たぶん私のようなメソッド名を分解するために流暢なスタイルを使用することができます。

verifyAddress(address).storeToDatabase().sendEmail();

が、私は、呼び出しの順序を確保するための方法が必要です。たぶん、状態パターンを使用して、しかし、これはコードが成長する原因となります。

Answer 1

3つの方法で1つの方法ではいけません。

1. verifyAddress
私は、これはあなたが言及し、その高い方法のように3つの小さなメソッドを作ることですアプローチ方法
2. storeAddress
3. のsendEmail

Answer 2

：このように3つの方法の中に作業を分割私は3つの小さなものを呼び出し、私は "なぜ"私はそれらの3つのことを行う必要がある後に名前を付けます。

これらの手順を実行する必要がある理由を定義し、それをメソッド名の基礎として使用してください。

Answer 3

私は以前のコメントをフォローアップしていますが、コメントに合っているというよりも私はここにいます。私は答えています。

詳細はメソッドの名前ではなくドキュメントに記載されています（私の意見では）。このように考える... SendEmailをメソッドの名前に入れると、実装の詳細がメソッド名にコミットされます。電子メールではなくSMSやTwitterなどで通知を送信するという決定が下された場合、どうなりますか？メソッドの名前を変更してAPIを壊すか、APIのコンシューマーを誤解させるメソッド名がありますか？考慮すべき何か。

名前にメソッドの機能を維持することを強くお勧めする場合は、より一般的なものを見つけることをお勧めします。おそらくVerifySaveAndNotify(Address address)の行に沿った何か。このように、メソッド名は、を指定することなく、何をしているのかを示します。それを行います。 Addressのパラメータはをご存知でしょうかが確認されており、保存されています。そのすべてが協力して、メソッド名を有益で柔軟性があり、簡潔にします。

Answer 4

EDIT：

verifyAddress（アドレス）.storeToDatabase（）：たぶん私のようなメソッド名を分解するために流暢なスタイルを使用することができます。メールを送る（）;

しかし、私は呼び出しの順序を保証する方法が必要です。たぶん、状態パターンを使用して、しかし、これはコードが成長する原因となります。

流暢なスタイルのコマンドの順序を保証するために、各結果は次のステップで必要な機能だけを公開するオブジェクトになります。たとえば：

public class Verifier 
{ 
    public DataStorer VerifyAddress(string address) 
    { 
     ... 
     return new DataStorer(address); 
    } 
} 

public class DataStorer 
{ 
    public Emailer StoreToDataBase() 
    { 
     ... 
     return new Emailer(...); 
    } 
} 

public class Emailer 
{ 
    public void SendEmail() 
    { 
     ... 
    } 
} 

これは非常に顆粒状のデザインを作成する必要があるとreuseabilityのためにあなたのクラスを最適化したい場合に便利ですが、ほとんどの状況下で設計やり過ぎである可能性が高いです。おそらく他の人が、プロセス全体が何を表しているのかを表す名前を選んだと言われています。単に「StoreAndEmail」と呼ぶことができます。これは、データがどの宛先にもコミットされる前に定期的に確認が行われていることを前提にしています。あなたが長い名前を気にしないならば、それを単に完全に記述し、長い名前が必要であることを受け入れることです。結局のところ、実際には何もコストはかかりませんが、確かにあなたのコードをより具体的なものにすることができます。