Foreach-loop for Array, which got via AJAX-query

Question

I have the JS-code:

$("#select_bank").change(function () {
  selected_bank = $("#select_bank option:selected").text();

  $.ajax({
    type: 'POST',
    dataType: 'html',
    data: {
      selectedBank: selected_bank
    },
    url: '<?= base_url() . 'atm/select_region'; ?>',
    success: function (list_regions) {
      foreach(keyVar in list_regions) {
        alert(list_regions[keyVar]);
      }
    }
  });
});

On callback "succes"s I get the array from server's script - in alert I see the "Array" - so I want to iterate via this array on client-side like I coded above, but when doing I get the error in console - "var keyVar is not defined". As I understand I need to typecast the list_regions param as array or some another way to fix it. Please, how to make it better? Thanks!

upd: enter image description here

Try changing 'foreach' to 'for'

timidboy
– timidboy

2012-09-01 06:15:52 +00:00
Commented Sep 1, 2012 at 6:15 — timidboy
– timidboy, Commented Sep 1, 2012 at 6:15

Florian Bauer · Accepted Answer · 2015-11-10 04:16:11Z

6

If I am right you cannot transform the foreach loop into jquery in that way.

You should use .each to iterate the values

$.each(list_regions, function(index, value) {
  alert(value);
});

You can find more information here.

edited Nov 10, 2015 at 4:16

answered Sep 1, 2012 at 6:17

Florian Bauer

6373 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Eugene Shmorgun Over a year ago

Thanks for propose, but it works as I've mentioned at my first comment - "Seems to be I have the trouble on my server's script: I get the sequense of alerts with the letters "A","r","r","a","y" and not the value's of array ?"

Daniil Ryzhkov · Accepted Answer · 2012-09-01 06:14:19Z

4

Javascript doesn't have foreach construction. Use $.each method of jQuery

answered Sep 1, 2012 at 6:14

Daniil Ryzhkov

7,6264 gold badges44 silver badges59 bronze badges

6 Comments

Lucas Green Over a year ago

Eugene is trying to use a JavaScript construct, and you jumped to telling him the jQuery solution. Just because Eugene is already using jQuery is not a good reason to advocate that they overlook the language's built-in construct.

Daniil Ryzhkov Over a year ago

@Mythril, but $.each() is much more userful in this case

Lucas Green Over a year ago

Exactly how is it more useful?

Eugene Shmorgun Over a year ago

Please, don't argue. Just I'd like to stream to use the pure JavaScript instead of Jquery to make some simple actions, when it's possible in rather simple way)

Lucas Green Over a year ago

@Daniil, both of which are available in a for..in looop

|

Lucas Green · Accepted Answer · 2012-09-01 06:13:16Z

0

Try this:

$("#select_bank").change(function(){
    // The following line was missing a var declaration,
    // Making it an implicit global
    var selected_bank = $("#select_bank option:selected").text();

    $.ajax({
        type:'POST',
        dataType:'html',
        data: { selectedBank: selected_bank },
        url:'<?=base_url().'atm/select_region'; ?>',
        success:function(list_regions){
            var keyVar;
            for(keyVar in list_regions) {
                if (list_regions.hasOwnProperty(keyVar)) {
                    alert(list_regions[keyVar]);
                }
            }
        }
    });

});

answered Sep 1, 2012 at 6:13

Lucas Green

3,95924 silver badges27 bronze badges

3 Comments

Eugene Shmorgun Over a year ago

Seems to be I have the trouble on my server's script: I get the sequense of alerts with the letters "A","r","r","a","y" and not the value's of array ?

Lucas Green Over a year ago

My guess would be that base_url() is returning a PHP array which is being cast to a string as "Array", which is then being iterated over letter by letter in JavaScript

Eugene Shmorgun Over a year ago

I've updated the post to show what I get on JS-script. But to do it - I've made the dirty trick in CodeIgnater, but I really want to avoid such bad way...

Oleg Barabanov · Accepted Answer · 2022-09-02 17:45:34Z

0

Use $.each method of jQuery, not used foreach.

$.each(res.keyVar, function(index, row) {
   alert(row);
});

edited Sep 2, 2022 at 17:45

Oleg Barabanov

2,7622 gold badges12 silver badges21 bronze badges

answered Aug 28, 2022 at 7:43

sagar shrestha

53 bronze badges

Collectives™ on Stack Overflow

Foreach-loop for Array, which got via AJAX-query

4 Answers 4

1 Comment

6 Comments

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

6 Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related