jQueryjavascript object passing - Stack Overflow

admin管理员组

文章数量:1278882

Up until this point, I haven't done much in javascript. I recently started playing with jQuery and came across a tutorial on how to do Edit in Place on a form.

The resulting code works perfectly if you hard-code the id of your container div in the function, but I'm having issues generalizing it for an id passed in to the function.

The code starts like this:

$(document).ready(function() {
    setClickable($('#editInPlace'));
});

function setClickable(target)
{
    $(target).click
    (
        function()
        {
            //Do some dom manipulation

            $('.buttonA').click(function() { saveChanges(this, false); });
            $('.buttonB').click(function() { saveChanges(this, true); });
        }
    )

    .mouseover(function() { $(this).addClass("editable"); })
    .mouseout(function() { $(this).removeClass("editable"); });
}; //end of function setClickable

function saveChanges(obj, cancel)
{
    //Handle the save changes code

    //Reset the div
    $(obj).parent().after('<div id="editInPlace">' + t + '</div>').remove();

    //Call setClickable
    setClickable($('#editInPlace'));
}

The part I'm having issues with is the last call to setClickable inside of the saveChanges function. If I hard-code the value of $('#editInPlace'), it works fine. However, if I change the function signature to

function saveChanges(target, obj, cancel)

and pass in the target object from the setClickable method:

$('.buttonB').click(function() { saveChanges(target, this, true); });

calling

setClickable(target);

doesn't seem to work and I can't figure out why.

Up until this point, I haven't done much in javascript. I recently started playing with jQuery and came across a tutorial on how to do Edit in Place on a form.

The resulting code works perfectly if you hard-code the id of your container div in the function, but I'm having issues generalizing it for an id passed in to the function.

The code starts like this:

$(document).ready(function() {
    setClickable($('#editInPlace'));
});

function setClickable(target)
{
    $(target).click
    (
        function()
        {
            //Do some dom manipulation

            $('.buttonA').click(function() { saveChanges(this, false); });
            $('.buttonB').click(function() { saveChanges(this, true); });
        }
    )

    .mouseover(function() { $(this).addClass("editable"); })
    .mouseout(function() { $(this).removeClass("editable"); });
}; //end of function setClickable

function saveChanges(obj, cancel)
{
    //Handle the save changes code

    //Reset the div
    $(obj).parent().after('<div id="editInPlace">' + t + '</div>').remove();

    //Call setClickable
    setClickable($('#editInPlace'));
}

The part I'm having issues with is the last call to setClickable inside of the saveChanges function. If I hard-code the value of $('#editInPlace'), it works fine. However, if I change the function signature to

function saveChanges(target, obj, cancel)

and pass in the target object from the setClickable method:

$('.buttonB').click(function() { saveChanges(target, this, true); });

calling

setClickable(target);

doesn't seem to work and I can't figure out why.

• javascript
• jquery

Share Improve this question Follow asked Apr 1, 2009 at 13:26 jerhinesmithjerhinesmith 15.5k1818 gold badges6363 silver badges9090 bronze badges 1

• 1 You might want to look at the Jeditable plugin, appelsiini/projects/jeditable or the Editable plugin, arashkarimzadeh./index.php/jquery/…. – tvanfosson Commented Apr 1, 2009 at 13:34

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 6

Change:

function setClickable(target)
{
    $(target).click
    (

to:

function setClickable(target)
{
    target.click
    (

You're passing in a wrapped set (also called a jQuery object) and then wrapped it in another wrapped set. The above fixes that. The alternate way to do this is:

$(document).ready(function() {
    setClickable('editInPlace');
});

function setClickable(id)
{
    $("#" + id).click
    (

You see what I'm getting at?

You can always try using setClickable($(target)); to make sure you've got a jQuery object.

本文标签： jQueryjavascript object passingStack Overflow