Uncaught TypeError: X is not a function. While using javascript closure












0















I'm playing with JavaScript and see following error in console:




Uncaught TypeError: div is not a function.







function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div();





I left the following for simplicity:



function createDiv(interval) {

    let timerId = window.setTimeout(function tick()

    {

        setTimeout(tick, 10);

    }, 10)

}



let div = createDiv(10);

div();


This is essentially an infinite loop. And error occurs when calling div() but loop is working.



What is wrong? And how it fix?






javascript closures settimeout infinite-loop







share|improve this question




















  • 1





    createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

    – pmkro
    Nov 20 '18 at 21:56











  • Thanks. Yes in console div is undefined. How it fix?

    – Seva
    Nov 20 '18 at 22:01











  • What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

    – pmkro
    Nov 20 '18 at 22:03











  • What is div() supposed to do?

    – Barmar
    Nov 20 '18 at 22:07
















0















I'm playing with JavaScript and see following error in console:




Uncaught TypeError: div is not a function.







function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div();





I left the following for simplicity:



function createDiv(interval) {

    let timerId = window.setTimeout(function tick()

    {

        setTimeout(tick, 10);

    }, 10)

}



let div = createDiv(10);

div();


This is essentially an infinite loop. And error occurs when calling div() but loop is working.



What is wrong? And how it fix?






javascript closures settimeout infinite-loop







share|improve this question




















  • 1





    createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

    – pmkro
    Nov 20 '18 at 21:56











  • Thanks. Yes in console div is undefined. How it fix?

    – Seva
    Nov 20 '18 at 22:01











  • What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

    – pmkro
    Nov 20 '18 at 22:03











  • What is div() supposed to do?

    – Barmar
    Nov 20 '18 at 22:07














0












0








0








I'm playing with JavaScript and see following error in console:




Uncaught TypeError: div is not a function.







function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div();





I left the following for simplicity:



function createDiv(interval) {

    let timerId = window.setTimeout(function tick()

    {

        setTimeout(tick, 10);

    }, 10)

}



let div = createDiv(10);

div();


This is essentially an infinite loop. And error occurs when calling div() but loop is working.



What is wrong? And how it fix?






javascript closures settimeout infinite-loop







share|improve this question
















I'm playing with JavaScript and see following error in console:




Uncaught TypeError: div is not a function.







function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div();





I left the following for simplicity:



function createDiv(interval) {

    let timerId = window.setTimeout(function tick()

    {

        setTimeout(tick, 10);

    }, 10)

}



let div = createDiv(10);

div();


This is essentially an infinite loop. And error occurs when calling div() but loop is working.



What is wrong? And how it fix?






function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div(); 





function getRandomColor() {

    var letters = '0123456789ABCDEF';

    var color = '#';

    for (var i = 0; i < 6; i++) {

        color += letters[Math.floor(Math.random() * 16)];

    }

    return color;

}



function createDiv(interval) {

    let radius = 300;

    let int = interval;

    var f = 0;

    var s = 2 * Math.PI / 180;

    let timerId = window.setTimeout(function tick()

    {

        var div = document.createElement('div');

        div.className = 'circle';

        div.style.backgroundColor = getRandomColor();

        div.style.left = (radius + radius * Math.cos(f)) + 'px';

        div.style.top = (radius + radius * Math.sin(f)) + 'px';

        document.body.appendChild(div);

        f += s;

        setTimeout(tick, int);

    }, int)



}



let div = createDiv(10);

div();





javascript closures settimeout infinite-loop




javascript closures settimeout infinite-loop






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 20 '18 at 21:59









ggorlen

7,3883826




7,3883826










asked Nov 20 '18 at 21:51









SevaSeva

3461422




3461422








  • 1





    createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

    – pmkro
    Nov 20 '18 at 21:56











  • Thanks. Yes in console div is undefined. How it fix?

    – Seva
    Nov 20 '18 at 22:01











  • What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

    – pmkro
    Nov 20 '18 at 22:03











  • What is div() supposed to do?

    – Barmar
    Nov 20 '18 at 22:07














  • 1





    createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

    – pmkro
    Nov 20 '18 at 21:56











  • Thanks. Yes in console div is undefined. How it fix?

    – Seva
    Nov 20 '18 at 22:01











  • What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

    – pmkro
    Nov 20 '18 at 22:03











  • What is div() supposed to do?

    – Barmar
    Nov 20 '18 at 22:07








1




1





createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

– pmkro
Nov 20 '18 at 21:56





createDiv(10) invokes the function, and it will return void as your have no return statement in the function. Therefore div is not a function but undefined. Did you try console logging div? after it was set?

– pmkro
Nov 20 '18 at 21:56













Thanks. Yes in console div is undefined. How it fix?

– Seva
Nov 20 '18 at 22:01





Thanks. Yes in console div is undefined. How it fix?

– Seva
Nov 20 '18 at 22:01













What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

– pmkro
Nov 20 '18 at 22:03





What do you think its supposed to be? As soon as you use createDiv it will run the setTimeout. Look at answer response.

– pmkro
Nov 20 '18 at 22:03













What is div() supposed to do?

– Barmar
Nov 20 '18 at 22:07





What is div() supposed to do?

– Barmar
Nov 20 '18 at 22:07












2 Answers
2






active

oldest

votes


















2














There is no closure here: you're calling a function that returns undefined and attempting to call undefined() as if it were a function. Simply call your function like createDiv(10);. Note that I've added a height and width to your div so the result is visible.






function getRandomColor() {

  var letters = '0123456789ABCDEF';

  var color = '#';

  for (var i = 0; i < 6; i++) {

    color += letters[Math.floor(Math.random() * 16)];

  }

  return color;

}



function createDiv(interval) {

  let radius = 300;

  let int = interval;

  var f = 0;

  var s = 2 * Math.PI / 180;

  let timerId = window.setTimeout(function tick() {

    var div = document.createElement('div');

    div.className = 'circle';

    div.style.backgroundColor = getRandomColor();

    div.style.height = div.style.width = "50px";

    document.body.appendChild(div);

    f += s;

    setTimeout(tick, int);

  }, int)

}



createDiv(1000);








share|improve this answer































    1

















    function getRandomColor() {
    
    var letters = '0123456789ABCDEF';
    
    var color = '#';
    
    for (var i = 0; i < 6; i++) {
    
        color += letters[Math.floor(Math.random() * 16)];
    
    }
    
    return color;
    
}
    

    
function createDiv(interval) {
    
    let radius = 300;
    
    let int = interval;
    
    var f = 0;
    
    var s = 2 * Math.PI / 180;
    
    window.setTimeout(function tick()
    
    {
    
        var div = document.createElement('div');
    
        div.className = 'circle';
    
        div.style.backgroundColor = getRandomColor();
    
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
    
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
    
        document.body.appendChild(div);
    
        f += s;
    
        setTimeout(tick, int);
    
    }, int)
    

    
}
    

    
let div = createDiv(10);
    
console.log(div);
    
try {
    
  div();
    
} catch (err) {
    
  console.log('error')
    
}
    

    

    
function createDivSmall(interval) {
    
   window.setTimeout(function tick()
    
    {
    
        setTimeout(tick, interval);
    
    }, 10)
    
}
    

    
let divSmall = createDivSmall(10);
    
console.log(divSmall);
    
try {
    
  divSmall();
    
} catch (err) {
    
  console.log('Small error')
    
}





    In your createDiv function you are not returning anything, so the default behavior is to return undefined. This sets div = undefined. At this point div is not a function so trying to invoke it will throw an error. createDiv is starting the loop for you and running it, there is not need to set a variable to the return value.






    share|improve this answer

























      Your Answer






      StackExchange.ifUsing("editor", function () {
      StackExchange.using("externalEditor", function () {
      StackExchange.using("snippets", function () {
      StackExchange.snippets.init();
      });
      });
      }, "code-snippets");

      StackExchange.ready(function() {
      var channelOptions = {
      tags: "".split(" "),
      id: "1"
      };
      initTagRenderer("".split(" "), "".split(" "), channelOptions);

      StackExchange.using("externalEditor", function() {
      // Have to fire editor after snippets, if snippets enabled
      if (StackExchange.settings.snippets.snippetsEnabled) {
      StackExchange.using("snippets", function() {
      createEditor();
      });
      }
      else {
      createEditor();
      }
      });

      function createEditor() {
      StackExchange.prepareEditor({
      heartbeatType: 'answer',
      autoActivateHeartbeat: false,
      convertImagesToLinks: true,
      noModals: true,
      showLowRepImageUploadWarning: true,
      reputationToPostImages: 10,
      bindNavPrevention: true,
      postfix: "",
      imageUploader: {
      brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
      contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
      allowUrls: true
      },
      onDemand: true,
      discardSelector: ".discard-answer"
      ,immediatelyShowMarkdownHelp:true
      });


      }
      });














      draft saved

      draft discarded


















      StackExchange.ready(
      function () {
      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53402127%2funcaught-typeerror-x-is-not-a-function-while-using-javascript-closure%23new-answer', 'question_page');
      }
      );

      Post as a guest















      Required, but never shown

























      2 Answers
      2






      active

      oldest

      votes








      2 Answers
      2






      active

      oldest

      votes









      active

      oldest

      votes






      active

      oldest

      votes









      2














      There is no closure here: you're calling a function that returns undefined and attempting to call undefined() as if it were a function. Simply call your function like createDiv(10);. Note that I've added a height and width to your div so the result is visible.






      function getRandomColor() {
      
  var letters = '0123456789ABCDEF';
      
  var color = '#';
      
  for (var i = 0; i < 6; i++) {
      
    color += letters[Math.floor(Math.random() * 16)];
      
  }
      
  return color;
      
}
      

      
function createDiv(interval) {
      
  let radius = 300;
      
  let int = interval;
      
  var f = 0;
      
  var s = 2 * Math.PI / 180;
      
  let timerId = window.setTimeout(function tick() {
      
    var div = document.createElement('div');
      
    div.className = 'circle';
      
    div.style.backgroundColor = getRandomColor();
      
    div.style.height = div.style.width = "50px";
      
    document.body.appendChild(div);
      
    f += s;
      
    setTimeout(tick, int);
      
  }, int)
      
}
      

      
createDiv(1000);








      share|improve this answer




























        2














        There is no closure here: you're calling a function that returns undefined and attempting to call undefined() as if it were a function. Simply call your function like createDiv(10);. Note that I've added a height and width to your div so the result is visible.






        function getRandomColor() {
        
  var letters = '0123456789ABCDEF';
        
  var color = '#';
        
  for (var i = 0; i < 6; i++) {
        
    color += letters[Math.floor(Math.random() * 16)];
        
  }
        
  return color;
        
}
        

        
function createDiv(interval) {
        
  let radius = 300;
        
  let int = interval;
        
  var f = 0;
        
  var s = 2 * Math.PI / 180;
        
  let timerId = window.setTimeout(function tick() {
        
    var div = document.createElement('div');
        
    div.className = 'circle';
        
    div.style.backgroundColor = getRandomColor();
        
    div.style.height = div.style.width = "50px";
        
    document.body.appendChild(div);
        
    f += s;
        
    setTimeout(tick, int);
        
  }, int)
        
}
        

        
createDiv(1000);








        share|improve this answer


























          2












          2








          2







          There is no closure here: you're calling a function that returns undefined and attempting to call undefined() as if it were a function. Simply call your function like createDiv(10);. Note that I've added a height and width to your div so the result is visible.






          function getRandomColor() {
          
  var letters = '0123456789ABCDEF';
          
  var color = '#';
          
  for (var i = 0; i < 6; i++) {
          
    color += letters[Math.floor(Math.random() * 16)];
          
  }
          
  return color;
          
}
          

          
function createDiv(interval) {
          
  let radius = 300;
          
  let int = interval;
          
  var f = 0;
          
  var s = 2 * Math.PI / 180;
          
  let timerId = window.setTimeout(function tick() {
          
    var div = document.createElement('div');
          
    div.className = 'circle';
          
    div.style.backgroundColor = getRandomColor();
          
    div.style.height = div.style.width = "50px";
          
    document.body.appendChild(div);
          
    f += s;
          
    setTimeout(tick, int);
          
  }, int)
          
}
          

          
createDiv(1000);








          share|improve this answer













          There is no closure here: you're calling a function that returns undefined and attempting to call undefined() as if it were a function. Simply call your function like createDiv(10);. Note that I've added a height and width to your div so the result is visible.






          function getRandomColor() {
          
  var letters = '0123456789ABCDEF';
          
  var color = '#';
          
  for (var i = 0; i < 6; i++) {
          
    color += letters[Math.floor(Math.random() * 16)];
          
  }
          
  return color;
          
}
          

          
function createDiv(interval) {
          
  let radius = 300;
          
  let int = interval;
          
  var f = 0;
          
  var s = 2 * Math.PI / 180;
          
  let timerId = window.setTimeout(function tick() {
          
    var div = document.createElement('div');
          
    div.className = 'circle';
          
    div.style.backgroundColor = getRandomColor();
          
    div.style.height = div.style.width = "50px";
          
    document.body.appendChild(div);
          
    f += s;
          
    setTimeout(tick, int);
          
  }, int)
          
}
          

          
createDiv(1000);








          function getRandomColor() {
          
  var letters = '0123456789ABCDEF';
          
  var color = '#';
          
  for (var i = 0; i < 6; i++) {
          
    color += letters[Math.floor(Math.random() * 16)];
          
  }
          
  return color;
          
}
          

          
function createDiv(interval) {
          
  let radius = 300;
          
  let int = interval;
          
  var f = 0;
          
  var s = 2 * Math.PI / 180;
          
  let timerId = window.setTimeout(function tick() {
          
    var div = document.createElement('div');
          
    div.className = 'circle';
          
    div.style.backgroundColor = getRandomColor();
          
    div.style.height = div.style.width = "50px";
          
    document.body.appendChild(div);
          
    f += s;
          
    setTimeout(tick, int);
          
  }, int)
          
}
          

          
createDiv(1000);





          function getRandomColor() {
          
  var letters = '0123456789ABCDEF';
          
  var color = '#';
          
  for (var i = 0; i < 6; i++) {
          
    color += letters[Math.floor(Math.random() * 16)];
          
  }
          
  return color;
          
}
          

          
function createDiv(interval) {
          
  let radius = 300;
          
  let int = interval;
          
  var f = 0;
          
  var s = 2 * Math.PI / 180;
          
  let timerId = window.setTimeout(function tick() {
          
    var div = document.createElement('div');
          
    div.className = 'circle';
          
    div.style.backgroundColor = getRandomColor();
          
    div.style.height = div.style.width = "50px";
          
    document.body.appendChild(div);
          
    f += s;
          
    setTimeout(tick, int);
          
  }, int)
          
}
          

          
createDiv(1000);






          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Nov 20 '18 at 22:10









          ggorlenggorlen

          7,3883826




          7,3883826

























              1

















              function getRandomColor() {
              
    var letters = '0123456789ABCDEF';
              
    var color = '#';
              
    for (var i = 0; i < 6; i++) {
              
        color += letters[Math.floor(Math.random() * 16)];
              
    }
              
    return color;
              
}
              

              
function createDiv(interval) {
              
    let radius = 300;
              
    let int = interval;
              
    var f = 0;
              
    var s = 2 * Math.PI / 180;
              
    window.setTimeout(function tick()
              
    {
              
        var div = document.createElement('div');
              
        div.className = 'circle';
              
        div.style.backgroundColor = getRandomColor();
              
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
              
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
              
        document.body.appendChild(div);
              
        f += s;
              
        setTimeout(tick, int);
              
    }, int)
              

              
}
              

              
let div = createDiv(10);
              
console.log(div);
              
try {
              
  div();
              
} catch (err) {
              
  console.log('error')
              
}
              

              

              
function createDivSmall(interval) {
              
   window.setTimeout(function tick()
              
    {
              
        setTimeout(tick, interval);
              
    }, 10)
              
}
              

              
let divSmall = createDivSmall(10);
              
console.log(divSmall);
              
try {
              
  divSmall();
              
} catch (err) {
              
  console.log('Small error')
              
}





              In your createDiv function you are not returning anything, so the default behavior is to return undefined. This sets div = undefined. At this point div is not a function so trying to invoke it will throw an error. createDiv is starting the loop for you and running it, there is not need to set a variable to the return value.






              share|improve this answer






























                1

















                function getRandomColor() {
                
    var letters = '0123456789ABCDEF';
                
    var color = '#';
                
    for (var i = 0; i < 6; i++) {
                
        color += letters[Math.floor(Math.random() * 16)];
                
    }
                
    return color;
                
}
                

                
function createDiv(interval) {
                
    let radius = 300;
                
    let int = interval;
                
    var f = 0;
                
    var s = 2 * Math.PI / 180;
                
    window.setTimeout(function tick()
                
    {
                
        var div = document.createElement('div');
                
        div.className = 'circle';
                
        div.style.backgroundColor = getRandomColor();
                
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
                
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
                
        document.body.appendChild(div);
                
        f += s;
                
        setTimeout(tick, int);
                
    }, int)
                

                
}
                

                
let div = createDiv(10);
                
console.log(div);
                
try {
                
  div();
                
} catch (err) {
                
  console.log('error')
                
}
                

                

                
function createDivSmall(interval) {
                
   window.setTimeout(function tick()
                
    {
                
        setTimeout(tick, interval);
                
    }, 10)
                
}
                

                
let divSmall = createDivSmall(10);
                
console.log(divSmall);
                
try {
                
  divSmall();
                
} catch (err) {
                
  console.log('Small error')
                
}





                In your createDiv function you are not returning anything, so the default behavior is to return undefined. This sets div = undefined. At this point div is not a function so trying to invoke it will throw an error. createDiv is starting the loop for you and running it, there is not need to set a variable to the return value.






                share|improve this answer




























                  1












                  1








                  1










                  function getRandomColor() {
                  
    var letters = '0123456789ABCDEF';
                  
    var color = '#';
                  
    for (var i = 0; i < 6; i++) {
                  
        color += letters[Math.floor(Math.random() * 16)];
                  
    }
                  
    return color;
                  
}
                  

                  
function createDiv(interval) {
                  
    let radius = 300;
                  
    let int = interval;
                  
    var f = 0;
                  
    var s = 2 * Math.PI / 180;
                  
    window.setTimeout(function tick()
                  
    {
                  
        var div = document.createElement('div');
                  
        div.className = 'circle';
                  
        div.style.backgroundColor = getRandomColor();
                  
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
                  
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
                  
        document.body.appendChild(div);
                  
        f += s;
                  
        setTimeout(tick, int);
                  
    }, int)
                  

                  
}
                  

                  
let div = createDiv(10);
                  
console.log(div);
                  
try {
                  
  div();
                  
} catch (err) {
                  
  console.log('error')
                  
}
                  

                  

                  
function createDivSmall(interval) {
                  
   window.setTimeout(function tick()
                  
    {
                  
        setTimeout(tick, interval);
                  
    }, 10)
                  
}
                  

                  
let divSmall = createDivSmall(10);
                  
console.log(divSmall);
                  
try {
                  
  divSmall();
                  
} catch (err) {
                  
  console.log('Small error')
                  
}





                  In your createDiv function you are not returning anything, so the default behavior is to return undefined. This sets div = undefined. At this point div is not a function so trying to invoke it will throw an error. createDiv is starting the loop for you and running it, there is not need to set a variable to the return value.






                  share|improve this answer


















                  function getRandomColor() {
                  
    var letters = '0123456789ABCDEF';
                  
    var color = '#';
                  
    for (var i = 0; i < 6; i++) {
                  
        color += letters[Math.floor(Math.random() * 16)];
                  
    }
                  
    return color;
                  
}
                  

                  
function createDiv(interval) {
                  
    let radius = 300;
                  
    let int = interval;
                  
    var f = 0;
                  
    var s = 2 * Math.PI / 180;
                  
    window.setTimeout(function tick()
                  
    {
                  
        var div = document.createElement('div');
                  
        div.className = 'circle';
                  
        div.style.backgroundColor = getRandomColor();
                  
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
                  
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
                  
        document.body.appendChild(div);
                  
        f += s;
                  
        setTimeout(tick, int);
                  
    }, int)
                  

                  
}
                  

                  
let div = createDiv(10);
                  
console.log(div);
                  
try {
                  
  div();
                  
} catch (err) {
                  
  console.log('error')
                  
}
                  

                  

                  
function createDivSmall(interval) {
                  
   window.setTimeout(function tick()
                  
    {
                  
        setTimeout(tick, interval);
                  
    }, 10)
                  
}
                  

                  
let divSmall = createDivSmall(10);
                  
console.log(divSmall);
                  
try {
                  
  divSmall();
                  
} catch (err) {
                  
  console.log('Small error')
                  
}





                  In your createDiv function you are not returning anything, so the default behavior is to return undefined. This sets div = undefined. At this point div is not a function so trying to invoke it will throw an error. createDiv is starting the loop for you and running it, there is not need to set a variable to the return value. 






                  function getRandomColor() {
                  
    var letters = '0123456789ABCDEF';
                  
    var color = '#';
                  
    for (var i = 0; i < 6; i++) {
                  
        color += letters[Math.floor(Math.random() * 16)];
                  
    }
                  
    return color;
                  
}
                  

                  
function createDiv(interval) {
                  
    let radius = 300;
                  
    let int = interval;
                  
    var f = 0;
                  
    var s = 2 * Math.PI / 180;
                  
    window.setTimeout(function tick()
                  
    {
                  
        var div = document.createElement('div');
                  
        div.className = 'circle';
                  
        div.style.backgroundColor = getRandomColor();
                  
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
                  
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
                  
        document.body.appendChild(div);
                  
        f += s;
                  
        setTimeout(tick, int);
                  
    }, int)
                  

                  
}
                  

                  
let div = createDiv(10);
                  
console.log(div);
                  
try {
                  
  div();
                  
} catch (err) {
                  
  console.log('error')
                  
}
                  

                  

                  
function createDivSmall(interval) {
                  
   window.setTimeout(function tick()
                  
    {
                  
        setTimeout(tick, interval);
                  
    }, 10)
                  
}
                  

                  
let divSmall = createDivSmall(10);
                  
console.log(divSmall);
                  
try {
                  
  divSmall();
                  
} catch (err) {
                  
  console.log('Small error')
                  
}





                  function getRandomColor() {
                  
    var letters = '0123456789ABCDEF';
                  
    var color = '#';
                  
    for (var i = 0; i < 6; i++) {
                  
        color += letters[Math.floor(Math.random() * 16)];
                  
    }
                  
    return color;
                  
}
                  

                  
function createDiv(interval) {
                  
    let radius = 300;
                  
    let int = interval;
                  
    var f = 0;
                  
    var s = 2 * Math.PI / 180;
                  
    window.setTimeout(function tick()
                  
    {
                  
        var div = document.createElement('div');
                  
        div.className = 'circle';
                  
        div.style.backgroundColor = getRandomColor();
                  
        div.style.left = (radius + radius * Math.cos(f)) + 'px';
                  
        div.style.top = (radius + radius * Math.sin(f)) + 'px';
                  
        document.body.appendChild(div);
                  
        f += s;
                  
        setTimeout(tick, int);
                  
    }, int)
                  

                  
}
                  

                  
let div = createDiv(10);
                  
console.log(div);
                  
try {
                  
  div();
                  
} catch (err) {
                  
  console.log('error')
                  
}
                  

                  

                  
function createDivSmall(interval) {
                  
   window.setTimeout(function tick()
                  
    {
                  
        setTimeout(tick, interval);
                  
    }, 10)
                  
}
                  

                  
let divSmall = createDivSmall(10);
                  
console.log(divSmall);
                  
try {
                  
  divSmall();
                  
} catch (err) {
                  
  console.log('Small error')
                  
}






                  share|improve this answer














                  share|improve this answer



                  share|improve this answer








                  edited Nov 20 '18 at 22:05

























                  answered Nov 20 '18 at 21:59









                  pmkropmkro

                  1,465717




                  1,465717






























                      draft saved

                      draft discarded




















































                      Thanks for contributing an answer to Stack Overflow!


                      • Please be sure to answer the question. Provide details and share your research!

                      But avoid …



                      • Asking for help, clarification, or responding to other answers.

                      • Making statements based on opinion; back them up with references or personal experience.


                      To learn more, see our tips on writing great answers.




                      draft saved


                      draft discarded














                      StackExchange.ready(
                      function () {
                      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53402127%2funcaught-typeerror-x-is-not-a-function-while-using-javascript-closure%23new-answer', 'question_page');
                      }
                      );

                      Post as a guest















                      Required, but never shown





















































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown

































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown







                      -

                      Popular posts from this blog

                      鏡平學校

                      Image
                      body.skin-minerva .mw-parser-output table.infobox caption{text-align:center} 鏡平學校 地址 小學部：俾利喇街128號 中學部：馬場大馬路389號 详细信息 其它名称 葡萄牙語： Escola Keang Peng 英语： Keang Peng School 学校类别 私立 校训 誠、毅、精、博 宗教背景 鏡湖醫院慈善會 创校日期 1892年 创始人 余倩嫻 校长 黎世祺 語言 中文 電話號碼 +853 28423118 学校网址 http://www.keangpeng.edu.mo 鏡平學校小學部 鏡平學校中學部 鏡平學校 (葡萄牙語： Escola Keang Peng ; 英语： Keang Peng School )是澳門鏡湖醫院屬下的一所學校。現任校長為黎世祺。幼稚園和小學在黑沙環俾利喇街；中學部在黑沙環馬場大馬路。 目录 1 歷史 2 辦學理念 2.1 辦學宗旨 2.2 辦學目標 2.3 育人目標 3 歷任校長 4 教學相關 4.1 幼稚園(學前教育) 5 参考资料 6 外部連結 歷史 鏡平學校是鏡湖慈善會屬下的教育機構, 創辦於1892年, 為澳門辦校歷史最悠久的學校。 鏡湖平民聯合小學合校至今已六十五周年了，學校的前身是鏡湖小學與平民學校。1948年間，兩校的校董會董事決定將兩校合併。當年，鏡湖醫院慈善會董事何善衡先生捐出望廈地段作校址之用，董事會合力籌募建校經費。 合校初期 1949年中，新校舍落成，命名為鏡湖平民聯合小學。此外，還在下環區和新橋區附設分校，正校也附設有夜校，學生人數增至一千四百多人。但由於學額有限，每年招收的學生遠遠未能滿足家長的要求。1956年，正校又在操場靠近俾利喇街地段增建一座兩層高的，擁有八個課室的校舍，學生人數增至二千人。 隨著社會的發展需要，為了進一步辦好鏡平學校，1990年鏡湖慈善會、校董會和董事們，本著關心教育造福清貧學生，培養社會人才的精神，開展了籌募鉅款改建學
                      Read more

                      ꓛꓣだゔៀៅຸ໢ທຮ໕໒ ,ໂ'໥໓າ໼ឨឲ៵៭ៈゎゔit''䖳𥁄卿' ☨₤₨こゎもょの;ꜹꟚꞖꞵꟅꞛေၦေɯ,ɨɡ𛃵𛁹ޝ޳ޠ޾,ޤޒޯ޾𫝒𫠁သ𛅤チョ'サノބޘދ𛁐ᶿᶇᶀᶋᶠ㨑㽹⻮ꧬ꧹؍۩وَؠ㇕㇃㇪ ㇦㇋㇋ṜẰᵡᴠ 軌ᵕ搜۳ٰޗޮ޷ސޯ𫖾𫅀ल, ꙭ꙰ꚅꙁꚊꞻꝔ꟠Ꝭㄤﺟޱސꧨꧼ꧴ꧯꧽ꧲ꧯ'⽹⽭⾁⿞⼳⽋២៩ញណើꩯꩤ꩸ꩮᶻᶺᶧᶂ𫳲𫪭𬸄𫵰𬖩𬫣𬊉ၲ𛅬㕦䬺𫝌𫝼,,𫟖𫞽ហៅ஫㆔ాఆఅꙒꚞꙍ,Ꙟ꙱エ ,ポテ,フࢰࢯ𫟠𫞶 𫝤𫟠ﺕﹱﻜﻣ𪵕𪭸𪻆𪾩𫔷ġ,ŧآꞪ꟥,ꞔꝻ♚☹⛵𛀌ꬷꭞȄƁƪƬșƦǙǗdžƝǯǧⱦⱰꓕꓢႋ神 ဴ၀க௭எ௫ឫោ ' េㇷㇴㇼ神ㇸㇲㇽㇴㇼㇻㇸ'ㇸㇿㇸㇹㇰㆣꓚꓤ₡₧ ㄨㄟ㄂ㄖㄎ໗ﾂڒذ₶।ऩछएोञयूटक़कयँृी,冬'𛅢𛅥ㇱㇵㇶ𥄥𦒽𠣧𠊓𧢖𥞘𩔋цѰㄠſtʯʭɿʆʗʍʩɷɛ,əʏダヵㄐㄘＲ｛ｇỚṖḺờṠṫảḙḭᴮᵏᴘᵀᵷᵕᴜᴏᵾq﮲ﲿﴽﭙ軌ﰬﶚﶧ﫲Ҝжюїкӈㇴffצּ﬘﭅﬈軌'ffistfflſtffतभफɳɰʊɲʎ𛁱𛁖𛁮𛀉 𛂯𛀞నఋŀŲ 𫟲𫠖𫞺ຆຆ ໹້໕໗ๆทԊꧢꧠ꧰ꓱ⿝⼑ŎḬẃẖỐẅ ,ờỰỈỗﮊDžȩꭏꭎꬻ꭮ꬿꭖꭥꭅ㇭神 ⾈ꓵꓑ⺄㄄ㄪㄙㄅㄇstA۵䞽ॶ𫞑𫝄㇉㇇゜軌𩜛𩳠Jﻺ‚Üမ႕ႌႊၐၸဓၞၞၡ៸wyvtᶎᶪᶹစဎ꣡꣰꣢꣤ٗ؋لㇳㇾㇻㇱ㆐㆔,,㆟Ⱶﾔﾏފ޼ޝަݿݞݠݷݐ',ݘ,ݪݙݵ𬝉𬜁𫝨𫞘くせぉて¼óû×ó£…𛅑הㄙくԗԀ5606神45,神796'𪤻𫞧ꓐ㄁ㄘɥɺꓵꓲ3''7034׉ⱦⱠˆ“𫝋ȍ,ꩲ軌꩷ꩶꩧꩫఞ۔فڱێظペサ神ナᴦᵑ47 9238їﻂ䐊䔉㠸﬎ffiﬣ,לּᴷᴦᵛᵽ,ᴨᵤ ᵸᵥᴗᵈꚏꚉꚟ⻆rtǟƴ𬎎

                      Оҁ菊湮ऍ؞ږᶣᶯꝮ꟟ꜧꜨꞤ្軌,វ៫ឌ֜׸֕㇮,,㇚㇖ٞٚڭ'ػ ث ګۮك౵౟ఁ𫠒𫝵𫞲𫟸ォス・ゥヰ'ゲִֿ׿꙼꙼⾎⾌⿐ݶݿݓݩݢ⺤㆜㆓㆞㆞ⱶⱸⱮ꧸ꧻH౜య఍ఔఀꙄꙀꙇﺂﻅﻔﻜﹾ⽜⾈⾔⿗⿕嫉礶𛀜𛀙𛂭𛂝ⱱ௮ж,ҡф 'ӇҠѺْ؅ؘ𛅝𛄻𛅯𛅃𛅭 '𛅩おｶｻﾉｭᶀᶗ𛅦𛄻𛅨𛅐׵׽ט֣ץ׶׻׫׬ꓳꓷꓶꓛꓢ, ㆼㆺㆪㆦ ௭XIUㄛㄔㄫ⃁₠⃏⃏₣₡धू޹ޞޫ޿އޛޟޖވ'޺ލިぶꬱꭦぢぜ㆛㆞㆟㇮㇫㇫ꓻꓶ㇑㇗㇈㇝㆕㆗㆓໫໲꣱𫤠ﬢ𢡊調留醙難ꓪꓟލޏݼݿݥݟⱬⱦⱶࢢ8𣫽𠻙𠣼s D,,B$h០ល៵៱ឬꓟ㆜㆕㆜㆖㆐㆕㆖㆗㆟எஓ꣥꣤ឱអɖʠɒɰʗɧʘ𬳉𬹲𫡸𬄯ﾍﾅﾝ･ㆮㆲﬗﬔ𫅡𛅘𛅥ꭉꬴꙥꚊꚌꙈꙦꚁꚚꙠꚝꬾ,꭯ꭩ꭭ᴼᴥᴄᵲᵱᵽЄеޛޞ꙳Ƞ𪡶తయఠఞభ౲ఁꭏṲṙḖɻʝꩻꩼｗ＆ｓmgU"D'ꭢꭥɜﶒﳒ꣼ ZR,㆑𛀔𛃦𛃟ݡݐݦ࣮ ࣥࣽࣟ神ࣹ ࣬࣍ࢹࣧࣻޯޔްꩼꩳꩬꩵ꩷⻽⺽⻮⺾ᶀⱣⱮⱤⱬⱡ⽊ggzy𬂑𫲇𬍪𫳵'𬎡𬛞𫢆𬗕𫵎𬰙㆛ꩫꩺ神ꩢꩰꩽາ໨,‘ó×⿈⼿purkz,qg yjລᴷᵢᵋ𥲀ﯜ'﵈ﳞﶎ﷉ﱈ﷊炲郃絒钰湨轢躽龠សㄅㄐㄋ神⺼⺽⻅⺺⻪⻺,û軌ǽ֧ǾNJԭԢԚ'ԙyuj'plrh3 1055ｺゴﻦﺒﹲwD₪صڑٔ42ﵢﮝေ၀်Ｘ'ｒＦ,＂＊！Ｈㆵヿイ, ヮベ𫝑𫠝𫞝𫞹 邏꣨E,神YGYBNT޿'ޤ޵囹أٚډ៎៦៥៲៌ îㆨㆤ ꭗꬼ𛄰₮ ₰₸₺䳬ឋ៛ꓦỲởḴṵ15,',ꝵǢƖȗ៯៑ពㇳㇲฤ๫สัฬษั,ꟾ꟣ꜽꞈꞾꟛ﬜שׁ𬃘𫧶𬀏𫱞𬌀𬯒𫬸𫨋𬯤𬅶𫪎ﳪ﷘ĶŲ៷ឯၝ၌ဩိေ𛂚𛁳𛁑𛀰ʁʧʞダfifflfifiꓱꓠꓣꓻǗDzǧ'ƥชฌ,๭ฃป๛ꧾԂࣀꧡ ㇨㇙⻑⻨ᶤﹿ𣤭𦃈𨂿𣤚ឡខ៓𫞬𫞚𫝼𫗜𫎥֠כֿ﬊ႋၵဨ𛂳٫ۋ٭１㇅㇏㇚㇤㇆㇞Ȼ𡮉𠷯֒ꩤꩠ₹C'BOGZAO𫿫𫸈𫩄𫾳𬡌𬟊ះŖĦŪčŸꙓVFマアユㄑㄈފ𛂜׌׳ו𫫇𫢯𫩥C𫝠𫞪𫝹ㄔ㄁げゕ焺萣躍ẌốỬṀด๱ฦธ㇦㇯,㇃ӫ ﰒ ﴌﳜﶅ﷨ﭽﯮݺݻݞ䊉䂖䥯𫞣𫝩𫝘𫠒𫝝㮛Б㇉ҞӶДӧяоЉ꣱ꓫ,,ꓕꓞꓣ꧹، ؄ۇ۶ظꓣʁɴ𬨉,偺𤜵帨㆘㆚㆒'㆐,神𬳰𬣧𬔏ឥ⻥𛂝𛁌𛀀ݘݑ
                      Read more

                      Why https connections are so slow when debugging (stepping over) in Java?

                      Image
                      up vote 0 down vote favorite I'm trying to debug some problems with my Java code, which establishes a http/https connection. I'm wondering why the connection gets ridiculously slow (more than one minute for downloading a small web page) when debugging, even when stepping-over the method which does the network work. And if there is some remedy. Below I post an example (you can change https: to http: , and try debugging this from Eclipse pressing F11 - for stepping, place a breakpoint in the first main() statement, and press F6 when it pauses there. My results (time in milliseconds) : conn time total time http (not stepping) 60 350 http (stepping over) 1100 1500 https (not stepping) 570 1300 https (stepping over) 21000 83000
                      Read more