jQuery拼接数组删除错误

Question

I'm making a hybrid app with cordova and I have two problems

1. When I double tap an item in my list it removes the correct item, however when I close and re-open the app it has removed the last one in the list instead, so basically, it removes the one I double tap on screen but in the array it always removes the last item in the list.

2. I can't remove a item i just added from the list, I need to close and re-open the app to remove it.

The problems occur in the bottom function

Here's my javascript file(JQuery):

    var taskListArray = new Array();

    $(document).ready(function()
    {
        var addInput = $("#addInput");
        var taskList = $("#taskList");

        if(window.localStorage)
        {
            taskListArray = JSON.parse(window.localStorage.getItem("taskList"))
        }
        else
        {
            window.plugins.toast.showLongCenter("LocalStorage not found, saving unavailable!");
        }

        if(taskListArray != null)
        {
            for(i = 0; i < taskListArray.length; i++)
            {
                var task = "<li>" + taskListArray[i].task + "</li>";
                taskList.append(task);
            }
        }
        else
        {
            taskListArray = new Array();
        }

        $("#addButton").on("click", function()
        {
            if($("#addInput").val() != 0)
            {
                var task = "<li>" + addInput.val() + "</li>";
                taskList.append(task);
                taskListArray.push({task:addInput.val()});

                if(window.localStorage)
                {
                    window.localStorage.setItem("taskList", JSON.stringify(taskListArray));
                }

                addInput.val("");
                window.plugins.toast.showShortCenter("Task added!");
            }
        });

        $("li").dblclick(function()
        {
            //Removes last task instead of the task I double tapped on
            //and I can't remove newly added tasks
            taskListArray.splice($.inArray($(this), taskListArray), 1);

            $(this).remove();
            window.plugins.toast.showShortCenter("Task removed!");

            if(window.localStorage)
            {
                window.localStorage.setItem("taskList", JSON.stringify(taskListArray));
            }
        });
    });

Edit: Still no answer to question #2? I would really appreciate answers to both questions or one at least, thanks!

Answer 1

The $.inArray needs two arguments, but you are only passing one. With only one parameter the function returns -1 and the splice function removes the last element when the argument is -1. It should be something like this taskListArray.splice($.inArray($(this), taskListArray), 1); . JQuery inArray Doc

Answer 2

Can you please replace your code taskListArray.splice($.inArray($(this)), 1); with taskListArray.splice($.inArray($(this),taskListArray), 1); and try.

In the jQuery documentation its stated that you need to provide the array in which u need to search.

jQuery.inArray( value, array [, fromIndex ] )