第二次调用时未定义javascript函数

时间:2012-09-18 16:54:45

标签: javascript jquery html

<script type="text/javascript">
        function vote(element, val){
            prev_vote=0;
            vote = 100
            can_vote = element.find('.vote-permission').attr('value')
            alert(can_vote);
            if (can_vote=='yes'){
                if (prev_vote==0){ 
                    alert("previous vote is 0");
                    // user has not cast vote for image before
                    vote_score = vote + val;
                    element.find('.vote-count-post').text(vote_score);
                }
                else if (prev_vote != 0){
                    // user has cast a vote before
                    if (prev_vote == 1 && val == -1){
                        vote_score = vote - 2;
                        element.find('.vote-count-post').text(vote_score);
                    }
                    else if (prev_vote == -1 && val == 1){
                        vote_score = vote + 2;
                        element.find('.vote-count-post').text(vote_score);
                    }
                }
            }
        };

        $(document).on('click', '.votedown', function(e){
            alert($(e.target).prev().text());
            vote($(e.target).parent(), -1);
            alert('vote down is done');
            });

        $(document).on('click', '.voteup', function(e){
            $(e.target).css('background-position', '0 -222px');
            vote($(e.target).parent(), 1);
            alert('vote up is done');
            });
    </script>

上面的javascript函数是我正在实现的上下投票系统的脚手架。当第一次点击upvote或downvote按钮时,函数按照预期执行,并调用了vote(),但是在第二次单击时,throw()函数被称为未定义。有什么想法吗?

1 个答案:

答案 0 :(得分:6)

该功能正在使用

删除自身
vote = 100

它取代了外部范围中vote变量的值(是函数,现在是一个数字)。

您可以使用var关键字阻止它,但不应使用两次相同的名称,因为它会破坏可读性。