CODEWITHSUNDEEP
javascriptjquerygoogle-chromewebkitstack-overflow
Andy
Andy

Reputation: 11462

Chrome/jQuery Uncaught RangeError: Maximum call stack size exceeded

I am getting the error "Uncaught RangeError: Maximum call stack size exceeded" on chrome. here is my jQuery function

$('td').click(function () {
        if ($(this).context.id != null && $(this).context.id != '') {
            foo($('#docId').val(), $(this).attr('id'));
        }
        return false;
    });

Note that there are tens of thousands of cells in the page. However, I generally associate stack overflows with recursion and in this case as far as I can see there is none.

Does creating a lambda like this automatically generate a load of stuff on the stack? is there any way round it?

At the moment the only workaround I have is to generate the onclick events explicitly on each cell when rendering the HTML, which makes the HTML much larger.

Upvotes: 130

Views: 374701

Answers (10)

Pawan Kumar
Pawan Kumar

Reputation: 1

   In My case It was html element insted of value
                          |
                          |
   var uname=$("#uname");<-
    
   let data={
     uid:uid,
     .....
   };        
    
   $.post("url",data,(responseText)=>{ 
       ..
   }).fail((xhr)=>{
       ..
   });
    
  then I updated 
                              |
                              |
  var uname=$("#uname").val()<-

Upvotes: 0

user3163139
user3163139

Reputation: 27

For me, I was trying to reference $('#inputfield') when I should have instead referencing $('#inputfield')**.val();

Upvotes: 0

AdamJones
AdamJones

Reputation: 601

Make sure you are not accessing elements without obtaining their values. I've been able to easily reporduce this issue due to the omission of the val() function.

For example:

$.ajax({
    url : '/url-test',
    type : 'GET',
    data : {
        'country':$("#test_input_field")
    }

Should be

$.ajax({
    url : '/url-test',
    type : 'GET',
    data : {
        'country':$("#country__c").val()
    }

Upvotes: 0

Rosh
Rosh

Reputation: 491

I was getting this error because of my mistake that I forgot to declare one of the variable which was passed in Ajax data.Only mode was declaredat first.

data: {
     tmp_id: tmp_id,
     mode: mode
}

Declared the tmp_id variable also and it worked fine.

let tmp_id=$("#tmp_id").val();
let mode=$("#mode").val();
$.ajax({
    url: 'my-url',
    method: 'post',
    data: {
     tmp_id: tmp_id,
     mode: mode
    }
    dataType: 'json',
    success: function(response) {
       console.log(response);
    }
    });
 }

Upvotes: 1

dev4life
dev4life

Reputation: 882

I recently just ran into this issue as well. I had a very large table in the dialog div. It was >15,000 rows. When the .empty() was called on the dialog div, I was getting the error above.

I found a round-about solution where before I call cleaning the dialog box, I would remove every other row from the very large table, then call the .empty(). It seemed to have worked though. It seems that my old version of JQuery can't handle such large elements.

Upvotes: 1

demenvil
demenvil

Reputation: 1139

U can use

  $(document).on('click','p.class',function(e){
   e.preventDefault();
      //Code 
   });

Upvotes: 1

Jsonras
Jsonras

Reputation: 1160

Mine was more of a mistake, what happened was loop click(i guess) basically by clicking on the login the parent was also clicked which ended up causing Maximum call stack size exceeded. 

$('.clickhere').click(function(){
   $('.login').click();
});

<li class="clickhere">
  <a href="#" class="login">login</a>
</li>

Upvotes: 8

John Durden
John Durden

Reputation: 401

You can also get this error when you have an infinite loop. Make sure that you don't have any unending, recursive self references.

Upvotes: 40

David Tran
David Tran

Reputation: 10606

As "there are tens of thousands of cells in the page" binding the click-event to every single cell will cause a terrible performance problem. There's a better way to do this, that is binding a click event to the body & then finding out if the cell element was the target of the click. Like this:

$('body').click(function(e){
       var Elem = e.target;
       if (Elem.nodeName=='td'){
           //.... your business goes here....
           // remember to replace $(this) with $(Elem)
       }
})

This method will not only do your task with native "td" tag but also with later appended "td". I think you'll be interested in this article about event binding & delegate

Or you can simply use the ".on()" method of jQuery with the same effect:

$('body').on('click', 'td', function(){
        ...
});

Upvotes: 143

Silvio Delgado
Silvio Delgado

Reputation: 6975

This problem happened with me when I used jQUery Fancybox inside a website with many others jQuery plugins. When I used the LightBox (site here) instead of Fancybox, the problem is gone.

Upvotes: 3

Related Questions