Declaring variable in jQuery syntax

Question

I got my jQuery working fine in jsfiddle. However, it won't work in my actual application because of the dom.

I'm new at this, i think I might be using the wrong syntax or something?

Here's my jsfiddle, note that it works if you select domready but not wrap.. I'm really confused

Here's the actual code:

$curr = $('#first');

$(document).ready(function () {
   $('.next').click(function () {
    $curr.hide();
    $curr = $curr.next();
    $curr.show();
});
});

$(document).ready(function () {
$('.previous').click(function () {
    $curr.hide();
    $curr = $curr.prev();
    $curr.show();
  });
});

Your code has to either be in a "load" or "ready" handler, or "no wrap" at the end of the <body>. — Pointy
– Pointy, Commented Apr 8, 2014 at 18:57

pj013 · Accepted Answer · 2014-04-08 18:58:27Z

0

Try this:

$(document).ready(function () {
   $curr = $('#first');

   $('.next').click(function () {
      $curr.hide();
      $curr = $curr.next();
      $curr.show();
   });

   $('.previous').click(function () {
      $curr.hide();
      $curr = $curr.prev();
     $curr.show();

   });
});

No need for multiple document.ready events in your case :)

answered Apr 8, 2014 at 18:58

pj013

1,4291 gold badge12 silver badges31 bronze badges

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

2 Comments

user3512424 Over a year ago

Thanks for the tip :) makes sense. Works perfectly in jsfiddle, in my application I get an "Uncaught ReferenceError: $ is not defined " though.. any ideas?

pj013 Over a year ago

Yes, that means that the jquery library has not been included in your page. w3schools.com/jquery/jquery_install.asp....Either download it from the CDN, else download it store it locally on your PC an give the relative path..hope it helps

MrHunter · Accepted Answer · 2014-04-08 19:19:29Z

0

Try using this code instead:

jQuery(document).ready(function ($) {
    $curr = $('#first');

    $('.next').click(function () {
        $curr.hide();
        $curr = $curr.next();
        $curr.show();
    });

    $('.previous').click(function () {
        $curr.hide();
        $curr = $curr.prev();
        $curr.show();   
    });
});

edited Apr 8, 2014 at 19:19

answered Apr 8, 2014 at 18:57

MrHunter

1,8901 gold badge15 silver badges23 bronze badges

1 Comment

user3512424 Over a year ago

Thank you, this works in jsfiddle. When I put it on my actual application I get "Uncaught ReferenceError: $ is not defined " "anonymous function" error though.. pointing to this JS. Any ideas? There are no typo

Collectives™ on Stack Overflow

Declaring variable in jQuery syntax

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related