複数のメソッドで非反復コードを破る方が良いですか？

Question

例：レポートを生成するとき、特定のコードブロックが将来他の場所で使用されることはないが、何百もの行があることがわかっています。 10個の小さなメソッドでそのコードを壊して、1000個のアイテムのレポートをエクスポートしているときに、10個のメソッドを1000回呼び出すか、メソッドに侵入することなく単純なままにしておくといいですか？

Answer 1

100行のコードを持つメソッドは、保守性と複雑さに悪いものです。私の推奨は、各機能を最大20行にすることです。

Answer 2

常に保守性を念頭に置いてください。

MainMethod(){ 
    step1(); 
    step2(); 
    step3(); 
    ... 
} 

あなたはそれを見て、それがでやっていることを一目で得ることができる。この方法は：私はサブステップでMainメソッドを分割し、mainメソッドがどのように見えるしてもらいたい。このような I例上級。 必要に応じて、ステップを実行して実行の詳細を取得することができます。 同じ手順をステップに適用することができます。サブステップで分割して同じ結果を得ることができ、読者にステップの概要を説明し、必要に応じて詳細を確認することができます。

このアプローチの理想的な状況は、ステップ間のデータの受け渡しを心配する必要がないため、ステップ間の相互作用が制限されている場合です。

Answer 3

あなたはそのようにそれを書くことができた場合：

GenerateReport(){ 
    var myReport = new ReportHelper(); 
    myReport.addPieChart(getSalesByCategories()); 
    myReport.addTable(getTopProducts(10)); 
    ... 
    return myReport; 
} 

行くそれのために。

あなたの人生をより楽にする（または見つけ出す）ヘルパークラスを作成します。このヘルパークラスのメソッドはaddTable（）、addPieChart（）です。すべてがオブジェクトを取ります。シンプルな関数を使用して、その名前が（いつものように）自明であることを確認し、実行順に記述すると、コードが読みやすくなります。 main関数はすべてのサブ関数への呼び出しを含み、最初に置いてください。将来の開発者が後で全プロセスを理解するのに良い入り口になります。それは、何かを変えるために適切な場所にジャンプすることを容易にする。また、小さな関数ごとにテストを書くこともできますが、大きなマンモスメソッドではできません。レポート生成を容易にするヘルパークラスを作成する価値があります。

実生活：ここではレポート作成ツールについて話しています。これは、アプリケーション内のコードの別の部分です。コードの他の部分とはやり取りしません。私はテストが計画されていないと思う（これは悪い）。各関数は、前のステップからのデータを必要とする可能性があり、非常に複雑です。 1つのフローでそれを書く方が簡単かもしれません。それは、グローバル変数とのジャグリングを避け、12のパラメータを渡します...しかし、上記の関数宣言と同様に、10〜20行ごとにコメントを記述してください。最初にプロセスの説明を追加することをお勧めします（他のケースの主な機能のように見えます）。ここでの問題は、将来のメンテナーがコードを変更し、メインプロセスの説明の変更を反映しない可能性がありますが、小さな機能でそれをやるように強制されることです。

私にとっては、使用する言語に依存しますが、私は小さな機能（理論）を好みますが、スパゲッティコード（現実）を書くことがよくあります。私は、コードの一部を再利用する必要があるときに関数を作成します。また、使用される言語にも依存します。私はこれを書く必要はありません。すべてのvarsの前に、グローバル変数を参照してください。

パフォーマンスに関して、各関数呼び出しには少し時間がかかりますが、このケースでは違いはありません。