値が存在しない場合にのみ、lodash pushを配列に使用しますか？

Question

値が存在しない場合は追加されますが、値がある場合は配列からその値も削除したいという配列を作成しようとしています。

Lodashのような感じは、このようなことができるはずです。

ベストプラクティスの提案に興味があります。

また、それは私がその項目をチェックするためにAngular.js

*更新*

if (!_.includes(scope.index, val)) { 
    scope.index.push(val); 
} else { 
    _.remove(scope.index, val); 
} 

Answer 1

ES6によって導入Set機能は、まさにそれを行うだろう。

var s = new Set(); 

// Adding alues 
s.add('hello'); 
s.add('world'); 
s.add('hello'); // already exists 

// Removing values 
s.delete('world'); 

var array = Array.from(s); 

それとも、Lodashの上にバニラJSを使用して、通常の配列

function add(array, value) { 
    if (array.indexOf(value) === -1) { 
    array.push(value); 
    } 
} 

function remove(array, value) { 
    var index = array.indexOf(value); 
    if (index !== -1) { 
    array.splice(index, 1); 
    } 
} 

を使用して保存しておきたい場合は良い習慣です。それは依存関係を取り除き、コードを理解させ、しばしばより効果的です。

Answer 2

使用includes機能をされて使用していますが、既存の削除に配列し、removeに存在することを指摘する価値があります項目。

function addOrRemove(arr, val) { 
 
    if (!_.includes(arr, val)) { 
 
    arr.push(val); 
 
    } else { 
 
    _.remove(arr, item => item === val); 
 
    } 
 
    console.log(arr); 
 
} 
 

 
var arr = [1, 2, 3]; 
 
addOrRemove(arr, 1); // arr = [2, 3] 
 
addOrRemove(arr, 4); // arr = [2, 3, 4] 
 
addOrRemove(arr, 2); // arr = [3, 4]

<script src="https://raw.githubusercontent.com/lodash/lodash/4.11.2/dist/lodash.min.js"></script>

Answer 3

おそらく_.pull()は助けることができる：

var _ = require('lodash'); 

function knock(arr,val){ 
    if(arr.length === _.pull(arr,val).length){ 
     arr.push(val); 
    } 
    return arr; 
} 

は、既存の配列を変異させるだけでなく、重複を削除します。

> var arr = [1,2,3,4,4,5]; 

> knock(arr,4); 
[ 1, 2, 3, 5 ] 

> knock(arr,6); 
[ 1, 2, 3, 5, 6 ] 

> knock(arr,6); 
[ 1, 2, 3, 5 ] 

Answer 4

この単一のライナーは仕事をする必要があります。挿入する要素が存在しない場合は、要素を挿入し、結果の配列の長さを返します。要素が配列内に存在する場合、要素を削除し、削除された要素を別の配列に返します。

var arr = [1,2,3,4,5], 
 
    aod = (a,e,i=0) => !!~(i = a.indexOf(e)) ? a.splice(i,1) : a.push(e); 
 
    
 
document.write("<pre>" + JSON.stringify(aod(arr,6)) + JSON.stringify(arr) + "</pre>"); 
 
document.write("<pre>" + JSON.stringify(aod(arr,6)) + JSON.stringify(arr) + "</pre>");

それは無駄な時間のほとんどで得られた配列の長さの値を返すので、まあ、実際に私はプッシュを嫌います。関数を連結できるように、返される結果の配列への参照を持つことをお勧めします。したがって、それを達成するための簡単な方法は、

var arr = [1,2,3,4,5], 
 
    aod = (a,e,i=0) => !!~(i = a.indexOf(e)) ? a.splice(i,1) : (a.push(e),a); 
 
     
 
document.write("<pre>" + JSON.stringify(aod(arr,6)) + JSON.stringify(arr) + "</pre>"); 
 
document.write("<pre>" + JSON.stringify(aod(arr,6)) + JSON.stringify(arr) + "</pre>");

だから今、これは合理的に連鎖可能です。

Answer 5

あなたは、入力と出力が役立つだろう_.union

_.union(scope.index, [val]);