クラスプロパティではなく複数のメソッドで使用される変数はどのように扱うべきですか？

Question

私は時々、同じデータ（クエリオブジェクトなど）を必要とするクラス内のいくつかのメソッドを見つけることがあります。一般的に、順番にデリゲートは、いくつかのプライベートメソッドにうまく、最終的に完成品を返すparseReport()ような一般的な名前の1つのパブリックメソッドが存在します。

public function parseReport(queryObject) { 
    queryObject = correctDatesAndTimes(queryObject); 
    queryObject = sortByCusomter(queryObject); 
    queryObject = buildHierarchy(queryObject); 

    return queryObject; 
} 

private function correctDatesAndTimes(queryObject) { 
    // do some stuff 

    return queryObject; 
} 

private function sortByCusomter(queryObject) { 
    // do some stuff 

    return queryObject; 
} 

private function buildHierarchy(queryObject) { 
    // do some stuff 

    return queryObject; 
} 

だから私の質問をされ、私のqueryObjectはクラス - する必要があります私のメソッドのすべては、呼び出されるたびにメソッドの引数として渡すのではなく、すべてのメソッドが参照するレベルプロパティです。

Answer 1

このような場合、queryObjectはクラスプロパティであってはなりません。それを見ると、いくつかの小さな関数に分割された大きな関数が実際にあります。 1つの大きな関数であれば、そのクラスのプロパティを作成しません。

データが実際にクラスの一部である場合、データはクラスプロパティに属します。 クラス定義は、データと振る舞いの両方をカプセル化することを覚えておいてください。

Answer 2

この例では、プライベート関数内で変更されるため、クエリオブジェクトをパラメータとして渡す必要があります。

また、プライベートプロパティに入れておくと、マルチスレッド化すると頭痛がかかります。