2017-02-16 3 views
0

ビジネスロジック内でモデル属性を複数回チェックする必要がある状況があります。ビジネスロジック内でモデル属性を複数回チェックする

eg: if(this.model.attributes.AcmeData.attributes.isEmpty){ 
    } 

このチェックは複数回発生します。このオブジェクトを複数回チェックする必要がありますか、我々はそれは次のように、複数の場所で使用するローカル変数に格納する必要があります

var acmeDataFlag = this.model.attributes.AcmeData.attributes.isEmpty; 

し、この変数を複数回確認してください。私にとって

、ローカル変数の作成は、そのオブジェクトを複数回チェックするよりも高価です。

おかげで、

+0

これは、[コードレビュースタックエクスチェンジ](http://codereview.stackexchange.com/)からより良い回答が得られる質問です。しかし、「複数回」は主観的(5を意味することもあれば、100を意味する可能性もある)であり、合理的に知るための十分な情報を提供していないこの種の変化をもたらすことになるだろう。それに加えて、他にも多くの要因があります。機能がどのように大きな、属性がどのように変化するか、多くの場合、関数が呼び出される頻度などはIMO、これは釈明問題ではありません。あなたが逆の答えを得ることができるように – Claies

答えて

2

あなたが話す名前の変数を作成する場合はIMO、はるかに読みやすく、理解しやすいだろう。 this.model.attributes.AcmeData.attributes.isEmptyの関数は読みにくくなります。

これは本当に高性能コードではない場合、別の変数を作成することの欠点は、読みやすさの向上のために無視できることです。

+1

あなたの質問は非常意見・ベースです。しかし、ベストプラクティスは通常、長いプロパティチェーンの代わりに明白な名前で変数を作成するためのアドバイス... – Antonio

関連する問題