javascript/angularjsのハッシュのキーの既存の値を更新します

Question

私はフロントエンドの星をクリックするたびにquestion_idとratingという2つのキーを持つ配列にオブジェクトを挿入します。ユーザーが星のいずれかのレーティングを変更して既存のキーの値を更新する場合はそれを探し、それ以外の場合は配列にエントリをプッシュします。

$scope.yellowPages = [] // is defined

if ($scope.yellowPages.length > 1) { 
    for (var i = 0 ; i < $scope.yellowPages.length; i++) { 
     if ($scope.yellowPages[i]["question_id"] == question_id) { 
      $scope.yellowPages[i]["rating"] = rating; // here updating the existing value of key, but what is happening it's updates and as well as creates a new entry with the updated value. 
     } 
     else{ 
     $scope.yellowPages.push({rating: rating, question_id: question_id}); 
     } // if not present 
    } 
    } 
    else{ 
    $scope.yellowPages.push({rating: rating, question_id: question_id}); // for 1st time 
    } 
} 

以下

コードサンプルは私の究極の目標はquestion_id'sユニークでありrating、配列べき唯一の5つの要素を持つことです。

ありがとうございました

Answer 1

forループのif/elseと関係があります。 forloop反復における現在の要素が同じ質問である場合は、チェックされています

if ($scope.yellowPages[i]["question_id"] == question_id) { 

それがない場合は、配列に項目を推進している。

$scope.yellowPages.push({rating: rating, question_id: question_id}); 

これは、のすべての反復のために起こりますループ。たとえば、配列に3つのアイテムがあり、一致する質問IDが3番目のアイテムである場合、新しいオブジェクト（$ scope.yellowPages.push（{rating：rating、question_id：question_id}）;）を第3のインデックスの一致するオブジェクトに到達する前に2度配列し、そのレーティングを更新する。

Answer 2

なぜあなたは配列を使用していますか？代わりに、次のようなオブジェクトを使用してください：

$scope.yellowPages = { 
    "question_id1": "rating1", 
    "question_id2": "rating2" 
} 

これでループすることなく簡単に質問にアクセスできます。

Answer 3

私は5つの値が配列に存在する必要があるので、既存の値を更新するだけです。したがって、5つの初期値が挿入された後にのみチェックします。しかし、これはそれを達成するための最適な方法ではないと思うが、これは私の問題を解決し、他の助けを歓迎します

if ($scope.yellowPages.length >= 5) { 
    for (var i = 0 ; i < $scope.yellowPages.length; i++) { 
     if ($scope.yellowPages[i]["question_id"] == question_id) { 
     $scope.yellowPages[i]["rating"] = rating; 
     } 
    } 
    } 
    else{ 
    $scope.yellowPages.push({rating: rating, question_id: question_id}); 
    }