2015-08-03 5 views
1

Я использую video.js искать вариант с функциями, чтобы установить время, чтобы искать:Jquery - OnClick событие - факторизация

$('#0s').click(function(e){ 
    e.preventDefault(); 
    player.currentTime(0); 
}); 

$('#10s').click(function(e){ 
    e.preventDefault(); 
    player.currentTime(10); 
}); 

$('#20s').click(function(e){ 
    e.preventDefault(); 
    player.currentTime(20); 
    }); 
}); 

С этими ссылками:

<a href="#" id="0s">0s</a> 
<a href="#" id="10s">10s</a> 
<a href="#" id="20s">20s</a> 

Как использовать только одну функцию? Моя попытка:

$(".link1").click(function(e){ 

    e.preventDefault(); 
    var time = $('this').attr('time'); 
    player.currentTime(time); 

}); 

С этой ссылке:

<a href="#" id="0s" class="link1" time="0">0s</a> 
<a href="#" id="10s" class="link1" time="10">10s</a> 
<a href="#" id="20s" class="link1" time="20">20s</a> 

Они обеспечивают этот адрес, чтобы проверить код: http://jsbin.com/ukIDoSa/4/edit?html,js,output

ответ

0

Просто избавиться от кавычек вокруг this

var time = $('this').attr('time'); 

должно быть

var time = $(this).attr('time'); 

player.currentTime(parseInt(time, 10)); // make sure you pass an integer. 
+0

Требуется ли использовать parseInt вовремя или нет? – Toki8

+0

@ Toki8. Это было бы предпочтительнее. –

0

Использовать var time = $ (this) .attr ('time'); не «этот» правильный путь - это только

Смежные вопросы