在表单提交时未调用Javascript闭包函数

时间:2013-01-03 10:14:16

标签: javascript html html5 forms javascript-events

这让我感到困惑。我有一个文本框。输入内容,然后按Enter键。 Javascript以单独的形式创建只读输入元素。文本输入框旁边是用于删除它们的按钮。该表单底部还有一个提交按钮,用于提交所有只读文本输入。

由于单击表单内的按钮将提交表单(我只想删除包含按钮及其相应文本输入的父div),因此在表单提交时会调用一个函数。此功能确定按下了哪种类型的按钮(删除或提交)并采取相应的行动。

这就是问题所在。按下删除按钮时,永远不会调用函数destinations.enter。我为解决这个问题所做的是创建一个名为submitDestinations的全局函数,它复制destinations.enter的功能。如果调用此函数,一切都会顺利进行。

有没有人知道为什么destinations.enter无法在提交时运行,但submitDestinations会?我想相信它与闭包有关,因为函数作用域似乎是两个函数之间的唯一区别。但是,这是我第一次使用闭包,我对它们的了解有限。

使用Javascript:

var destinations = (function(){
    var max_destinations = 7;
    var counter = 0;

    function increment(){
        counter += 1;
        if(counter > max_destinations){
            throw 'Too many destinations. (Max 7)'
        }
    }
    function decrement(){
        counter += 0;
        if(counter < 0){
            alert('Cannot have less than 0 destinations..')
            throw 'Too few destinations. Get out of the console!'
        }
    }
    return {
        add : function(form){
            try{
                var formInput = form.elements[0];
                var destination = formInput.value;
                // Dont do anything if the input is empty
                if(destination == ""){
                    return false;
                }else{
                    // increment the destinations counter
                    increment();
                }
            }catch(err){
                alert(err);
                return false;
            }
            // add the text value to a visual element
            var elem = document.createElement('div');
            elem.setAttribute('class','destination');
            // create the input
            var input = document.createElement('input');
            input.setAttribute('id','dest'+String(counter));
            input.setAttribute('class','destinationText');
            input.setAttribute('style','border: none');
            input.setAttribute('name','destinations');
            input.setAttribute('readonly','readonly');
            input.setAttribute('value',destination);
            // create the remove button
            var button = document.createElement('button');
            button.setAttribute('onclick','this.form.submitted=this;');//'return destinations.remove(this);');
            button.setAttribute('class','removeButton')
            button.setAttribute('id','but'+String(counter))
            var buttonText = document.createTextNode('Remove');
            button.appendChild(buttonText);
            // add the elements to the div
            elem.appendChild(input);
            elem.appendChild(button);

            var parent = document.getElementById('destinationsDiv');
            parent.appendChild(elem);
            // clear the input box
            formInput.value = '';
            return false;
        },
        enter : function(form){
            alert('hi')
            var button = form.submitted;
            if(button.id != 'submitBtn'){
                return remove(button);
            }else{
                return true;
            }
            return false;
        },
        remove : function(button){
            try{
                decrement();
            }catch(err){
                // do not allow less than 0 counter
                alert(err);
                return false;
            }
            // remove the button's parent div altogether
            var toDelete = button.parentNode;
            toDelete.parentNode.removeChild(toDelete);
            return false;
        }

    }
})();

和html:

        <div>
            <form id='hi' onsubmit="return destinations.add(this);">
                <input type="text" value="" />
            </form>
            <!--form id='submitDiv' method="post" onsubmit="alert(this.submitted);return submitDestinations(this);"-->
            <form id='submitDiv' method="post" onsubmit="alert(this.submitted);return destinations.enter(this);">
                <div id='destinationsDiv'>
                    <div>
                        <input id="dest1" class="destinationText" style="border: none" name="destinations" readonly="readonly" value="aadasd" \>
                        <button onclick="this.form.submitted=this;" class="removeButton" id="but1" \></button>
                    </div>
                    <div>
                        <input id="dest2" class="destinationText" style="border: none" name="destinations" readonly="readonly" value="Hi Stackoverflow" \>
                        <button onclick="this.form.submitted=this;" class="removeButton" id="but2" \></button>
                    </div>
                </div>
                <input type="submit" id='submitBtn' onclick="this.form.submitted=this;"/>
            </form>
        </div>

如果我将以下javascript函数添加到全局范围并调用它,一切正常。这与destinations.enter

完全相同
function submitDestinations(form){
    var button = form.submitted;
    if(button.id != 'submitBtn'){
        return destinations.remove(button);
    }else{
        return true;
    }
}

我在html中更改的是在提交时调用的方法:

        <div>
            <form id='hi' onsubmit="return destinations.add(this);">
                <input type="text" value="" />
            </form>
            <form id='submitDiv' method="post" onsubmit="alert(this.submitted);return submitDestinations(this);">
            <!--form id='submitDiv' method="post" onsubmit="alert(this.submitted);return destinations.enter(this);"-->
                <div id='destinationsDiv'>
                    <div>
                        <input id="dest1" class="destinationText" style="border: none" name="destinations" readonly="readonly" value="aadasd" \>
                        <button onclick="this.form.submitted=this;" class="removeButton" id="but1" \></button>
                    </div>
                    <div>
                        <input id="dest2" class="destinationText" style="border: none" name="destinations" readonly="readonly" value="Hi Stackoverflow" \>
                        <button onclick="this.form.submitted=this;" class="removeButton" id="but2" \></button>
                    </div>
                </div>
                <input type="submit" id='submitBtn' onclick="this.form.submitted=this;"/>
            </form>
        </div>

2 个答案:

答案 0 :(得分:2)

事实证明存在命名冲突。我正在创建的文本输入的name属性设置为“destinations”,与我在调用提交的javascript对象的名称相同。因此,“onsubmit”中的javascript尝试引用DOM元素并在其上调用enter而不是引用我的javascript函数。

答案 1 :(得分:0)

目标.enter()的实现存在错误。它调用remove()来删除按钮,但名称'remove'没有绑定在定义targets.enter()的范围内。

看起来好像在其他地方没有调用destinations.remove(),所以最简单的解决方法就是将其移动到目标构造函数中的私有函数,而不是将它作为目标对象上的方法。

修改后的版本之所以有效,是因为您已将submitDestinations()的主体更改为调用目标.remove(),该范围绑定在该范围内。