2016-04-12 7 views
0

イムに基づいて、JSコードを改善するが、我々は警告を与えるESlintを使用している:は<strong>作業</strong>で次のコードを使用したES糸くず

を一切未使用-expressionは、代入や関数呼び出しを期待していないし、代わりに見た表現

どうすればよいですか?

_createNew: function(Filecontent, config) { 
    var self = this; 
    config.position ? self._insertAfter(Filecontent, config) : 
     self._addAsLast(Filecontent, config); 
    return Filecontent; 
}, 

私は最初にリターンを入れようとするとうまくいかないのですが...考えていますか?代わりにあなたの三元の

+1

ください。将来(あなたも含めて)そのコードに触れるすべての人に有利です:if/elseだけを使用してください。三項演算子を使用している点はありません。 – Prinzhorn

+0

3進演算子は値を返し、それを受信する必要があります。私はちょうどそれを考えずにそれに答えました。 –

答えて

4
_createNew: function(Filecontent, config) { 
    if(config.position) { 
     this._insertAfter(Filecontent, config) 
    } 
    else { 
     this._addAsLast(Filecontent, config); 
    } 

    return Filecontent; 
}, 
2

config.position 
    ? self._insertAfter(Filecontent, config) 
    : self._addAsLast(Filecontent, config); 

あなたがもし/ else文を使用する必要があります。私はあなたの方法が1ライナーであることを知っていますが、それはより読みやすくはなく、理由のためにeslintのルールがあります。

if (config.position) { 
    self._insertAfter(Filecontent, config) 
} else { 
    self._addAsLast(Filecontent, config); 
} 

このコードベースで作業するすべての人にとって、もう少し多くの行が読みやすくなります。

関連する問題