最新消息:雨落星辰是一个专注网站SEO优化、网站SEO诊断、搜索引擎研究、网络营销推广、网站策划运营及站长类的自媒体原创博客

Function not defined - jQueryJavascript - Stack Overflow

programmeradmin3浏览0评论

Another hapless noob... Okay, so I've looked everywhere to try and find out how to make this jQuery/JavaScript function, I hope it's just a mistake in my syntax, but here's my code:

<script type="text/javascript">

$(function()
{
        function loadNext(choice) {
            $("#thequestion").load("question.php", { ans: choice, id: "<?php echo $row['id']; ?>", nextId: "<?php echo $nextQuestionId; ?>" } );

            $("#graph").load("graph.php", { id:"<?php echo $row['id']; ?>", nextId: "<?php echo $nextQuestionId; ?>" } );
            alert("loadNext activated");
        }

});
</script>

I then call the loadNext(choice) function using an onclick event like so: onclick="loadNext(YorN);".

Following this I get a 'loadNext is not defined' error.

Amongst the answers that I would accept are: "Y U NO LEARN!" and "You are silly."

Here is a similar problem.

Thanks.

Another hapless noob... Okay, so I've looked everywhere to try and find out how to make this jQuery/JavaScript function, I hope it's just a mistake in my syntax, but here's my code:

<script type="text/javascript">

$(function()
{
        function loadNext(choice) {
            $("#thequestion").load("question.php", { ans: choice, id: "<?php echo $row['id']; ?>", nextId: "<?php echo $nextQuestionId; ?>" } );

            $("#graph").load("graph.php", { id:"<?php echo $row['id']; ?>", nextId: "<?php echo $nextQuestionId; ?>" } );
            alert("loadNext activated");
        }

});
</script>

I then call the loadNext(choice) function using an onclick event like so: onclick="loadNext(YorN);".

Following this I get a 'loadNext is not defined' error.

Amongst the answers that I would accept are: "Y U NO LEARN!" and "You are silly."

Here is a similar problem.

Thanks.

Share Improve this question edited May 23, 2017 at 12:33 CommunityBot 11 silver badge asked Jan 18, 2012 at 1:24 user1104147user1104147 1053 silver badges9 bronze badges 0
Add a ment  | 

5 Answers 5

Reset to default 4

The loadNext function you just defined is local to the scope of your anonymous function. If you want to access it globally, then define it globally (outside of your onready handler).

The outer "$(function().." you have on your first line limits your scope - the loadNext function is only available within this block and will be undefined outside of it.

The easy way to get around this is to just define the "loadnext" function outside the $(function()..) block but this is wrong for two reasons:

  1. This makes the function global across your entire application, and polluting the global namespace is a no-no.
  2. Inline javascript like "onclick" is bad in general - this blurs the line between presentation and logic and makes debugging a pain.

To get around this, use the jquery bind events to attach your event handlers; i.e.,

$("#myfancybutton", click(function(){ loadNext(choice) }); within your $(function()..) bloc

because you have a function inside a function

 (function()
{
        function loadNext(choice) {
            this is bad
        }

function loadNext(choice) {
            this is good
        }

onclick="loadNext()"

no probs

why dont you just put your loadNext function outside of the $(function(){}) ?

Because loadNext is not in global scope. You can define it in global scope, or even better solution is to use .click() to bind your click event.

发布评论

评论列表(0)

  1. 暂无评论