2016-09-11 11 views
1

私はこのサムネイル回転子をjavascriptで作ったので、最初の例のように短い記述ができたと思いましたが、これがパフォーマンスに役立ちますか? JavaScriptの速度を最大化するために従うべき一般的なガイドラインはありますか? (私の醜いコードを許して、私はゲームに非常に新しいです)もっと速く動くのは何ですか?

function mouseOverfirst(videoId){ 
      var num = 8; 
      var numb = 7; 
      $('#'+videoId).attr('src',videoId+'/thumb'+numb+'.png'); 
      if(numb == 7){ 
      interval = setInterval(function(){ 
      $('#'+videoId).attr('src',videoId+'/thumb'+num+'.png'); 
       if(num == 15){ 
        num = 1; 
       }  
       else 
       { 
        num++; 
       } 
      },500); 
     } 
    } 

またはこれは?

function mouseOverfirst(videoId){ 
      var numb = 7; 
      $('#'+videoId).attr('src',videoId+'/thumb'+numb+'.png'); 
      if(numb == 7){ 
       mouseOver('video1'); 
      } 
} 

function mouseOver(videoId){ 
      var num = 8; 
      interval = setInterval(function(){ 
      $('#'+videoId).attr('src',videoId+'/thumb'+num+'.png'); 
       if(num == 15){ 
        num = 1; 
       }  
       else 
       { 
        num++; 
       } 
      },500); 
     } 
+0

あなたは正確に何を達成しようとしていますか?あなたは、画像の別々のスライドを回転させましたか? – brianxautumn

+0

あなたの変数 'num'や' numb'が宣言後に変更できるかのようには思われません。なぜif文が必要なのですか? –

+0

あなたはパフォーマンスは気になるようですが、明らかに速い 'document.getElementById(videoId).src = ...の代わりに、' $( "+ videoId).attr( 'src'、...)'を使用します。 ' - あなたは間違った場所で最適化を探しています。 –

答えて

0

両方のプログラムの複雑さは同じです。

どちらを使用しても問題ありませんが、2番目の方が機能として手近にできます。mouseOverFirstmouseOverは簡単に再利用できます。

関連する問題