@frees2

Как укоротить код?

Друзья, как укоротить мой быдло-код, терзают смутные сомнения, можно делать гораздо красивей и избавится от повторений, но боюсь изобретать .
$(function()
{var player;function fGetScript(){ $.ajax({url:"http://www.youtube.com/player_api/",
dataType:"script",success:function(data)
{
igor("done");
window.onYouTubeIframeAPIReady=function()
 {
igor("onYouTubeIframeAPIReady"); 

$(".loadvid").click(function()
    { 
   findMovieById($("#videoid").val()); });

$(".loadvid2").click(function()
    {  
  findMovieById($("#videoid2").val());
});

function findMovieById(videoID)
 {

      if (player===null) 
{ player = new YT.Player('player',
 { 
 videoId: videoID,
    events: { 'onReady': onPlayerReady, }
 });
 } 
else {player.loadVideoById(videoID);
 }};
  • Вопрос задан
  • 2333 просмотра
Пригласить эксперта
Ответы на вопрос 2
JekFdrv
@JekFdrv
webdeveloper
Не вижу вообще повторений. Быдло кодом он перестанет быть когда начнете соблюдать правила индентация.
Ответ написан
Комментировать
mlnkv
@mlnkv
JavaScript Developer
расставьте переносы и табы, тогда можно будет о чем то говорить

$(function() {
  var player;
  function fGetScript() {
    $.getScript("http://www.youtube.com/player_api/", function(data) {
      igor("done");
        window.onYouTubeIframeAPIReady = function() {
          igor("onYouTubeIframeAPIReady");
          $(".loadvid, .loadvid2").on("click", function() {
            var videoID = $(this).val();
            if (player === null) { 
              player = new YT.Player('player', { videoId: videoID, events: { 'onReady': onPlayerReady } }); 
            } else {
              player.loadVideoById(videoID);
            }
          });
        }
    });
  }
});
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы