ローカル変数の取り扱い about ec-cube HOT 4 CLOSED

初めてコメントさせていただきます。
クラス変数とはメンバ変数のことでしょうか。

全てをメンバ変数とするとメモリがオブジェクトのdestroyまで開放されず、オブジェクトのシリアライズなどにも影響する気がしますが、いかがでしょうか。
OOPの中で、本来の意味でメンバ変数とするべきものはメンバ変数で、それ以外はローカル変数にしてもよいのではないでしょうか。

あるいは、メンバ変数は必ずphpdoc形式のコメントつきでclassの先頭で宣言を行うなどはいかがでしょうか。

from ec-cube.

issue内には記載がないのですが、
開発方針にはaction()内の規約として規定されているように読み取れます。
action()以外の関数でローカル変数を使用することに関しては問題ないのではないでしょうか。
action()からビジネスロジックを排除すれば可読性が上がりますし、
SC_FormParamのインスタンス等をメンバ変数に移動すれば
プラグインの連携もしやすくなるように思います。

from ec-cube.

@ttsuru @izayoi256

コメントありがとうございます。
プラグインでローカル変数の書き換えができないという問題を解決するために、この方法でどうかとissueを立てました。
なので

開発方針にはaction()内の規約として規定されているように読み取れます。
action()以外の関数でローカル変数を使用することに関しては問題ないのではないでしょうか。

上記で@izayoi256 のおっしゃる通り、action()以外では問題ございません。
混乱を招かぬようissueの表題を変えておきますね。

また、「action()からビジネスロジックを排除すれば可読性が上が」る件に関しては、 #7 と関連が深くなってきそうなので、別でissueをたてたほうがよいかもしれません。

from ec-cube.

Silexの採用に伴い、action()がそもそもなくなったのでCloseさせていただきます。

from ec-cube.