配列をループすると正しい結果が返されない

Question

変数を配列と比較すると、forループを使用した後にifステートメントを持つ$scope.object.idと$scope.groepen.idが表示されます。 $scope.object.idが$scope.groepen.idのIDのいずれかとまったく同じ場合は、$scope.overlapのインデックスをtrueにする必要があります。

$scope.overlapのいずれかがtrueであるかどうかを確認するために別のものを使用しています。 $scope.overlapのいずれかの要素がtrueの場合、$scope.bestaandがtrueになります。そうでなければ、それは間違っているはずです。

for (var i = 0; i < $scope.groepen.length; i++) { 
    if ($scope.object.id === $scope.groepen[i].id) { 
    $scope.overlap[i] = true; 
    if ($scope.overlap[i]) { 
     $scope.bestaand = true; 
    } 
    else { 
     $scope.bestaand = false; 
    } 
    } 
    else { 
    $scope.overlap[i] = false; 
    } 
} 

マイコンソールログは$scope.overlapが実際に正しい値を（何も同じではありませんので、もし、すべてのインデックスが偽である）を示していていることを私に示しています。 $scope.bestaandは、何かが同じであればtrueになりますが、falseには戻っていません。

私はチェックはここに示す作業されているかどうかを示すために、角度フォーム検証を使用しています：私はここで間違って

<div class="col-md-3" ng-class="{ 'has-error' : bestaand }"> 
    <label class="control-label" for="textinput">Groepsnaam</label> 
    <input id="groepen" name="groepen" type="text" class="form-control input-md" ng-model="object.id" ng-minlength="4" ng-maxlength="16" ng-change="checkOverlap()" required> 
    <p ng-show="bestaand" class="help-block">Deze groepsnaam bestaat al!</p>        
</div> 

何をしているのですか？

編集：

私はif文の場所を変更しました。私は、同じ値の入力を持っている（

真になるんようだが、それは上書きされます：

for (var i = 0; i < $scope.groepen.length; i++) { 
    if ($scope.object.id === $scope.groepen[i].id) { 
    $scope.overlap[i] = true; 
    } 
    else { 
    $scope.overlap[i] = false; 

    } 
    if ($scope.overlap[i]) { 
     $scope.bestaand = true; 
     console.log("works") 
    } 
    else { 
    $scope.bestaand = false; 
    console.log("doesnt work") 
    } 
} 

コンソールログは、この私を示しています。更新されたコードは、ここに示されています配列の2番目の値）。配列の最後の値と同じ値を入力した場合、それは機能します。

Answer 1

あなたの問題はif ($scope.overlap[i]) {をif ($scope.object.id === $scope.groepen[i].id) {の中に入れたので、$scope.overlapは常に真実です。つまり、$scope.bestaandはtrueに設定するか、undefinedとします。あなたが実際にしたいことはある -

for (var i = 0; i < $scope.groepen.length; i++) { 
    if ($scope.object.id === $scope.groepen[i].id) { 
    $scope.overlap[i] = true; 
    } 
    else { 
    $scope.overlap[i] = false; 
    } 
    if ($scope.overlap[i]) { 
    $scope.bestaand = true; 
    } 
    else { 
    $scope.bestaand = false; 
    } 
} 

編集 あなた$scope.overlap要素のいずれかに該当する場合はtrueに$scope.bestaandを設定したい場合、あなたは少し違ったものになるでしょう - この

$scope.bestaand = false; 
for (var i = 0; i < $scope.groepen.length; i++) { 
    if ($scope.object.id === $scope.groepen[i].id) { 
    $scope.overlap[i] = true; 
    } 
    else { 
    $scope.overlap[i] = false; 
    } 
    if(!$scope.bestaand) { 
    if ($scope.overlap[i]) { 
     $scope.bestaand = true; 
    } 
    } 
} 

Answer 2

をは、それ以外の場合はが到達不能です：

$scope.overlap[i] = true; 

if ($scope.overlap[i]) { 
    $scope.bestaand = true; 
} 
else { 
    console.log('UNREACHABLE'); 
    $scope.bestaand = false; 
} 

あなたの編集した質問について：

$scope.bestaandはエラーを示していますので、それが一度も真でない場合は、エラーであると判断します。

だから、次のようにループを書き直す必要があります。

$scope.bestaand = false; 
for (var i = 0; i < $scope.groepen.length; i++) { 
    if ($scope.object.id === $scope.groepen[i].id) { 
    $scope.overlap[i] = true; 
    } else { 
    $scope.overlap[i] = false; 
    } 

    if ($scope.overlap[i] && !$scope.bestaand) { 
    $scope.bestaand = true; 
    } 
} 

Answer 3

をfalseに設定する一切ケースはありませんので。 if ($scope.object.id === $scope.groepen[i].id)は、すでにoverlap = trueそれを設定し、オーバーラップがfalseの場合は、その後、あなたはどのような場合