Borni Borni - 1 year ago 106
AngularJS Question

AngularJS - ForEach Loop

I'm currently having a small issue with a specific loop for showing a button.

I'd like to show a 'suggestBirdname' button depending on two statements:
1. That the user isn't owner of the thread.
2. That the user has not suggested a bird yet on the thread.

This is what I currently have:

// Globelvariable
$scope.showSuggestionButton = false;

var currentSuggestions =;
currentSuggestions.forEach(function(suggestions) {
if(suggestions.userId === auth.profile.user_id
&& $scope.capture.birdname==='Unknown') {
$scope.showSuggestionButton = false;
} else {
$scope.showSuggestionButton = true;

As you can see in the above code, it goes through all my data of the current thread we are on searching through the suggestions.
As you can see, I set the default value to
$scope.showSuggestionButton = false;
seeing that it does not have to be visible unless the above statements are passed.

The problem when going through the loop is: If the current logged in user isn't the last one that posted a suggestion, the loop simply keeps going on, hence setting the statement to true.
I need a way to exit the loop once the following statements are triggered:
suggestions.userId === auth.profile.user_id && $scope.capture.birdname==='Unknown'

It is probably and easy fix but I just can't seem to wrap my head around it at the moment..

Any help is much appreciated.

For further clarification:

Below I have a button to open a form, to post a suggestion.
Though I only want it to be visible if the statements posted above are valid.
Going through the data of the post (which has the suggestions linked to it), I want to use the loop above to show the button.

<span class="suggest-birdname-button" ng-if="showSuggestionButton">
<span popover-title="{{capture.birdname}}" popover-placement="bottom-left" popover-trigger="mouseenter"
popover-append-to-body uib-popover-template="birdInfoPopover.templateUrl" class="suggestion-button-text clickable"
ng-bootbox-title="Suggest Birdname"
ng-bootbox-custom-dialog-template="/partials/model/suggestBirdname.html">Suggest Birdname <span class="glyphicon glyphicon-pencil"></span></span>

Answer Source

I think what you need to do is stop looping as soon as you find out that the user has already suggested a bird.

So, instead of looping on all the suggestions, even if you find that user has already suggested a bird, you need to break out of the loop.

I'd tweak your logic to do opposite of what you've done, i.e show suggestion by default, and hide it when you find out that the user has suggested a bird already.

$scope.showSuggestionButton = true;
$scope.checkedAllSuggestions = false;
var currentSuggestions =;
for(var i=0; i<currentSuggestions.length;i++) {
    if(currentSuggestions[i].userId ===  auth.profile.user_id && $scope.capture.birdname==='Unknown') { 
   //is $scope.capture.birdname==='Unknown' check needed here, cause it has nothing to do with loop variables?
        $scope.showSuggestionButton = false;
$scope.checkedAllSuggestions = true;

EDIT: You can alter your ng-if condition to show only when all the suggestions are processed. This will solve your initial display of button case. It may seem like a ugly hack, will update this answer when I find a better way to do it.

<span class="suggest-birdname-button" ng-if="showSuggestionButton && checkedAllSuggestions">

If you don't want to use for loop, there are other clean ways of doing this. You can use underscore for such functions.

Recommended from our users: Dynamic Network Monitoring from WhatsUp Gold from IPSwitch. Free Download