サービス層とクラスの簡略化

Question

私のコードの構造とクラスを簡単に保つ方法について質問があります。私は、C＃プロジェクトのサービスレイヤを単純化することに取り組んでいます。コードの多くはOOPの実践を考慮しておらず、200行以上のメソッドを持つクラスはほとんどありません。私は小さなメソッドを抽出し始めましたが、これを行う方法に関するクイッククエリがあります。

たとえば、顧客に固有のファイルディレクトリを取得し、存在するかどうかを確認し、見つからない場合は作成し、最後にこれらのディレクトリのリストを持つオブジェクトを返すメソッドがあります。私は、プライベートメソッドを持たないという原則に固執し、新しいクラスに抽出したいと思いますが、ディレクトリが存在するかどうかをチェックするためのプライベートメソッドを作成し、それらを作成するためのプライベートメソッドを作成し、フォルダ名を取得し、 publicメソッドを使用して、これらのすべてを1つのメソッドと関連付けられたインターフェイスで順番に呼び出します。

これらのプライベートメソッドのそれぞれに対して新しいクラスを作成する必要がありますか？その場合はすべてインターフェイスが必要ですか？あるいはそれらをすべて公開し、他の場所から呼び出すことができますか？

ありがとうございます！

Answer 1

短い答え：あなたはこれらのことをしないでください。

オブジェクト指向の観点から問題にアプローチしたい場合は、メソッドが何をしているのかしばらく忘れてください。コードが約であることを考えてください。可能性のある「ビジネス」関連のものとして「顧客」としか言及していませんでした。他のビジネス関連のものを思いついてみてください。これらのファイルは何ですか？レポート？ ActivityLogs？メッセージ？ CreditReports :)？

オブジェクト指向は、異なるクラスのメソッドを持つだけではありません。クラスとメソッドは、ビジネス上の意味があります。彼らが何かを意味するわけではないなら、最初にそれらを持つ本当の理由はありません！

「StorageManager」、「StorageUtil」などは、ビジネス上の意味が全くないため存在してはいけないことも明らかです。

アプリケーションが約（もの）であることを見つけ出すことから始め、特定の責任を適切なものに移すことができます。