Using each function

Question

Multiple products per page. I want to display message based on quantity.

$j(document).ready(function () {
    $j('#stockqty').text((parseInt($j('#stockqty').text()) > 0) ? "In Stock" : "Out of Stock");
});

I've tried this:

$j(document).ready(function () {
    $j('#stockqty').each(function () {.text((parseInt($j('#stockqty').text()) > 0) ? "In Stock" : "Out of Stock");
    });
});

Just not working.

.text in your .each callback needs to be called on something. — Blazemonger
– Blazemonger, Commented Jan 3, 2014 at 20:25
$("#someid").each doesn't make sense, there can be only one $("#someid") — user400654
– user400654, Commented Jan 3, 2014 at 20:26
$('#stockqty') will only return one element. What are you trying to do? — gen_Eric
– gen_Eric, Commented Jan 3, 2014 at 20:27

Josh Beam · Accepted Answer · 2014-01-03 20:26:01Z

1

$(function () {
    $('.stockqty').each(function () {
        parseInt($(this).text()) > 0) ? return "In Stock" : return "Out of Stock");
    });
});

Also, there should not be multiple elements with the ID of "stockqty". Consider changing that to class="stockqty" (subsequently using the corresponding jQuery selector).

Additionally, although I have used a return statement in the above code, it is possible that you might want to instead append a value to the user interface. You can do this through something like:

$('#resultArea').html("In Stock") : $('#resultArea').html("Out of Stock");

Lastly, it's perfectly acceptable to just do

$(function() {

instead of

$(document).ready(function() {

(Note: the commonly used jQuery symbol is "$", unless you previously passed in "$j" as a parameter.)

answered Jan 3, 2014 at 20:26

Josh Beam

19.9k5 gold badges48 silver badges69 bronze badges

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

3 Comments

SteveAustin Over a year ago

Thanks, that all makes perfect sense. I should of realized the ID thing, but still no joy. Fiddler seems to have a problem with it also.

Josh Beam Over a year ago

Can you post a link to your fiddle?

Josh Beam Over a year ago

I still saw a few syntax issues with your code (in addition, there was no HTML). I chose to go with jQuery 1.10, as well, instead of what you were using (1.7). See this updated jsfiddle

Collectives™ on Stack Overflow

Using each function

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related