I use the Array map() Method to check each element in an Array and if the condition in the if clausel is true i call another function.

$.map(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
   };
});

Now my problem is that i want to stop the map method if a element fullfills the if condition. Because i noticed that when i have for example 6 elements that fullfill the condition the function Diagnose.single(field.id); is called 6 times instead of once!

I tried:

$.map(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
      return true;
   };
});

But this didnt worked! What can i do instead? Thanks

I use the Array map() Method to check each element in an Array and if the condition in the if clausel is true i call another function.

$.map(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
   };
});

Now my problem is that i want to stop the map method if a element fullfills the if condition. Because i noticed that when i have for example 6 elements that fullfill the condition the function Diagnose.single(field.id); is called 6 times instead of once!

I tried:

$.map(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
      return true;
   };
});

But this didnt worked! What can i do instead? Thanks

• javascript
• jquery

Share Follow asked May 10, 2014 at 15:32 John SmithJohn Smith 6,2791919 gold badges6161 silver badges113113 bronze badges 4

• have you tried break; – Ehsan Sajjad Commented May 10, 2014 at 15:33
• this didnt worked ... please elaborate. – OMGtechy Commented May 10, 2014 at 15:34
• 1 that defeats the purpose of map, use $.each function instead stackoverflow./q/1799284/1117720 – Amine Hajyoussef Commented May 10, 2014 at 15:38
• Just read the specification of map. And do a for loop by yourself. Is it that difficult to write a function with a for loop ???? – GameAlchemist Commented May 10, 2014 at 15:39

Add a ment  | 

4 Answers 4

Sorted by: Reset to default 6

Use simple for-loop with break statement:

for (var i = 0; i < data['icd'].length; i++) {
   if (data['icd'][i].nummer == search){
      Diagnose.single(data['icd'][i].id);
      break;
   };    
}

map is used for different tasks.

If you do not need IE8 support, you can use Array.prototype.some

data['icd'].some(function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
      return true;
   };
});

Return false and replace .map with .each

$.each(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
      // Need to return false, return true will go for a success and still continue.
      return false;
   };
});

I think in your case you want to use the jQuery each function. You can then return false to break out of the loop

$.each(data['icd'], function (field, i) {
   if(field.nummer == search){
      Diagnose.single(field.id);
      return false;
   };
});

or even better, use a simple for-loop as @disq shared.