<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()函数被称为未定义。有什么想法吗?
答案 0 :(得分:6)
该功能正在使用
删除自身vote = 100
它取代了外部范围中vote
变量的值(是函数,现在是一个数字)。
您可以使用var
关键字阻止它,但不应使用两次相同的名称,因为它会破坏可读性。