156文字列を複数の関数に渡す

Question

私は156文字のメッセージを受け取るc＃winforms .net 4アプリケーションを持っていますが、このメッセージを変更せずに複数の関数に順番に渡します。

私の質問は、パラメータと同じ値を渡し続けるのが効率的でないか、より効率的な方法ですか？

ので、現在私が持っている：コードの

string code = getTheCode(); 
\\decode first part 
string result1 = getResult1(code); 
string result2 = getResult2(code); 
... 

値は、その初期割り当て後に変更することはありません。

Answer 1

答えはノーです。同じ文字列をパラメータとして渡し続けるのは非効率的ではありません。文字列への参照を渡すだけなので、非常に効率的です。

Answer 2

同じコードをいくつかのメソッドに渡し続けるのは非効率的な場合があります。これを何度もやらなくてはならない場合は、「結果を得る」責任のあるクラスを作成することができます。この新しいクラスのコンストラクタで 'code'を渡します。この方法でクラスの生存期間中に 'コード'を再利用することができ、パラメータと同じ値を渡す必要はありません

Answer 3

引数として文字列を渡す必要のあるコンストラクタを使用してクラスを作成できますプライベートプロパティとして設定します。次に、このプライベートプロパティを使用して結果を計算するメソッドを使用してデータを取得できます。

しかし、これはあなたが好むコーディングスタイル（もちろん、1つ以上の場所でこれらのメソッドを使用するかどうか）の問題です。私にとっては読みやすく、ResultGetterクラスのインスタンスでコード変数が変更されないようにする必要があります。

public class ResultGetter 
{ 
    private readonly string _code; 

    public ResultGetter(string code) 
    { 
     _code = code; 
    } 

    public string GetResult1() 
    { 
     var returnValue = // do something with _code property  
     return returnValue; 
    } 

    public string GetResult2() 
    { 
     var returnValue = // do something with _code property 
     return returnValue; 
    } 

    // et cetera ad nauseam 
} 

そしてメインのファイルに：

var code = getTheCode(); 
var rg = new ResultGetter(code); 
string result1 = rg.GetResult1(); 
string result2 = rg.GetResult2();