Jquery ReferenceError: (function name) is not defined

Question

I have this code that got me stuck for days I don't understand why does it say in my console that my function is not defined here's the code. by the way, i am super new to jQuery I really don't have much knowledge in it so any help? can somebody tell me where I went wrong that it's giving me this error

(ReferenceError: createGraph is not defined)

<!DOCTYPE html>
<html>
<head>
  <script src="http://code.jquery.com/jquery-1.9.1.js"></script>

        <script src="bin/js/raphael.js"></script>
        <script src="bin/js/popup.js"></script>
        <script src="bin/js/analytics.js"></script>
</head>
<body>

 <div class="action-button">
    <button id="1" value="1074">Button 1</button>
    <button id="2" value="1074">Button 2</button>
 </div>

  <div id="output"></div>

<script>
/////////////////////////////////////////////////////
// this script will check if the document is ready and will display data for button 1
     $(document).ready(function () {
    $("#1").trigger("click");
    });

    ///////////////////////////////////////////////////////////////////////////////////////
        $("button").click(function () {
          var attr = $(this).attr("id");
          var val = $(this).val();

          $.ajax({
          type: "POST",
          url: "bin/some.php",
          data: { lookbookID: val, type:attr  }
    }).done(function( html ) {
      $("#output").html(attr + ' - ' +  val + ' - ' + html );
        createGraph();

    });
});
</script>
</body>
</html>

and the JS code

window.onload = function () {
    function getAnchors(p1x, p1y, p2x, p2y, p3x, p3y) {
        var l1 = (p2x - p1x) / 2,
            l2 = (p3x - p2x) / 2,
            a = Math.atan((p2x - p1x) / Math.abs(p2y - p1y)),
            b = Math.atan((p3x - p2x) / Math.abs(p2y - p3y));
        a = p1y < p2y ? Math.PI - a : a;
        b = p3y < p2y ? Math.PI - b : b;
        var alpha = Math.PI / 2 - ((a + b) % (Math.PI * 2)) / 2,
            dx1 = l1 * Math.sin(alpha + a),
            dy1 = l1 * Math.cos(alpha + a),
            dx2 = l2 * Math.sin(alpha + b),
            dy2 = l2 * Math.cos(alpha + b);
        return {
            x1: p2x - dx1,
            y1: p2y + dy1,
            x2: p2x + dx2,
            y2: p2y + dy2
    };
}

    function createGraph() {
       // Grab the data
       alert("i made it!");
        var labels = [],
            data = [];
        $("#data tfoot th").each(function () {
        labels.push($(this).html());
        });
        $("#data tbody td").each(function () {
            data.push($(this).html());
        });

     }
    };

I updated this sorry I excluded that window.onload I still get that reference error but if I put the function outside window.onload = function then my function would work is there any way I can access my function? I don't know what will happen if I remove the said code from windows.onload I need help again :)

the link for the function just in case it would be of any help https://github.com/ksylvest/raphael-analytics

That's not how functions are defined. You need to learn basic Javascript syntax. — SLaks
– SLaks, Commented May 10, 2013 at 17:06
$(function createGraph()() { is a pretty blatant syntax error. — user229044
– user229044 ♦, Commented May 10, 2013 at 17:07
thanks guys i didn't realize i had that $() lol epic fail anyway thank you very much! — Louie Jay Jayme
– Louie Jay Jayme, Commented May 10, 2013 at 17:11

Johnny000 · Accepted Answer · 2013-05-10 17:07:38Z

2

function createGraph() {
// Grab the data
    var labels = [],
        data = [];
    $("#data tfoot th").each(function () {
        labels.push($(this).html());
    });
    $("#data tbody td").each(function () {
        data.push($(this).html());
    });

   //code here i erased so it wont be too long and i just added this function so i could call it anyway this code is actually the code that creates the graph in raphael JS
 }

answered May 10, 2013 at 17:07

Johnny000

2,1006 gold badges31 silver badges61 bronze badges

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

7 Comments

user229044 Over a year ago

You've got all kinds of dangling braces and parenthesis.

Louie Jay Jayme Over a year ago

Thanks bro this one helped a lot

Johnny000 Over a year ago

make a new question if it's not related to this specific problem

Louie Jay Jayme Over a year ago

well it's the same i still get that reference error anyway i updated the JS above can you check if that's a good idea? thanks

Johnny000 Over a year ago

try to wrap it in $(document).ready(function(){ // YOURCODE });

|

moe · Accepted Answer · 2013-05-10 17:13:55Z

1

createGraph is not a global function , Because $() create a new context,createGraph became a local variables,so if you try to call it ,you get an error.

so let it out from $().

Code like Upstairs..

answered May 10, 2013 at 17:13

moe

214 bronze badges

Collectives™ on Stack Overflow

Jquery ReferenceError: (function name) is not defined

2 Answers 2

7 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

7 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related