プロパティと一致する配列からオブジェクトを削除するにはどうすればよいですか？

Question

は、次のデータを考えてみましょう：

food = { 
    id: 1, 
    name: 'Pizza', 
    price: 16 
}; 

orders = [ 
    { food_id: 2, table_id: 5 }, 
    { food_id: 2, table_id: 5 }, 
    { food_id: 1, table_id: 5 }, 
    { food_id: 3, table_id: 5 }, 
    { food_id: 1, table_id: 5 } 
]; 

私は配列がfood_idに一致受注から単一の項目を削除したいです。ここに私が試したものです：

removeFoodOrder(food: Food): void { 
    for (let order of this.orders) { 
    let match = this.orders.filter((order) => order.food_id == food.id); 
    match ? this.orders.splice(this.orders.indexOf(order), 1) : null; 
    break; 
    } 
    console.log(this.orders); 
} 

私はremoveFoodOrder(food)を呼び出した場合、それは関係なく、私がのparamsに渡すものを食品アイテムの配列から最初の要素を削除しません。

removeFoodOrder(food) 
// removes {food_id: 2, table_id: 5} (the first element) 
// I want to remove {food_id: 1, table_id: 5}, 

私は配列から一致する要素を対象とし、それの単一インスタンスを削除します。どこで私は間違えましたか？

Answer 1

：

food = { 
    id: 1, 
    name: 'Pizza', 
    price: 16 
}; 

orders = [ 
    { food_id: 2, table_id: 5 }, 
    { food_id: 2, table_id: 5 }, 
    { food_id: 1, table_id: 5 }, 
    { food_id: 3, table_id: 5 }, 
    { food_id: 1, table_id: 5 } 
]; 

removeFoodOrder = (food: Food): void => { 
    this.orders = this.orders.filter(order => order.food_id !== food.id);   
} 

編集：

あなたの配列は要素の重複を可能にし、あなただけ削除したいので、最初の試合では、Array#findIndex方法：

const index = orders.findIndex(order => order.food_id === food.id); 
orders.splice(index, 1); 

Answer 2

私の最初のステップは、常にその三項演算子とあなたのブレークstmtのような混乱を取り除くことです。 は、ここではそれを使用する方法を100％がわからない場合、私はアレイ＃フィルタを使用しないことをお勧めします、私はそれを

let food = { 
    id: 1, 
    name: 'Pizza', 
    price: 16 
} 

let orders = [ 
    {food_id: 2, table_id: 5}, 
    {food_id: 2, table_id: 5}, 
    {food_id: 1, table_id: 5}, 
    {food_id: 3, table_id: 5}, 
    {food_id: 1, table_id: 5} 
] 

for (let order of this.orders) { 
    if (food.id === order.food_id) { 
     this.orders.splice(this.orders.indexOf(order), 1); 
     break; 
    } 
} 
console.log(this.orders); 

をやった方法です。

UPDATE私はArray＃フィルタ方法を使用しないことを言っているわけではありません。私はちょうどあなたのコードが動作していない場合は、あなたの問題の原因となる可能性のあるものを削除しようとすると簡単な構文（forループやstmtのような）を使用して段階的に行くことを試みる必要があります。

Answer 3

これを試してください：あなたはArray#filter方法を使用することができます

function removeFoodOrder(food: Food): void 
{ 
    for (let order of this.orders) { 
     if (order.food_id == food.id) { 
      this.orders.splice(this.orders.indexOf(order), 1); 
      break; 
     }  
    } 
    console.log(this.orders); 
};