Question

function parseAttach(b) 
{ 
    var h=""; 
    for(i=0;i<b.length;i++) 
    { 
     var a=b[i]; 
     switch(a['type']) 
     { 
      case "image": 
       h+='<li class="attach aImg" style="background:#000;border-bottom:2px solid #fff"><img style="max-width:425px;max-height:500px" src="http://img.xiami.com/u/phoo/'+a['file']+'"></li>'; 
      break; 
      case "video": 
       h+='<li class="attach aVideo" style="background:#f3f3f3"><span class="MxND" f="'+a['from']+'" d="'+a['id']+'"></span></li>'; 
      break; 
      case "music": 
       h+='<li class="attach aMusic"><embed src="http://www.xiami.com/widget/0_'+a['id']+'/singlePlayer.swf" type="application/x-shockwave-flash" width="257" height="33" wmode="transparent"></embed></li>'; 
      break; 
     } 
    } 
    return h; 
} 

上記の関数が実行されると、ページは相互作用できません。そのページのメモリとCPU使用率は急上昇します。

これは、この関数に渡されるパラメータbの例である：

[{"type":"video","from":"k6","id":"kyxGTZbD-vQ8Domm-eeHiQ"}] 

てb.lengthは2以下であるず、この関数はこれ以上三回以上実行されました。この関数を削除すると、メモリリークは発生しません。

UPDATE：@GarethMcCaughanさんの提案に続き

、私はそれが0に警告し続ける、ループの先頭にアラート（i）を追加、私は、呼び出しコードに向かっ：

for(i=0;i<c[0].length;i++)//the breakpoint 
{ 
    ...... 
    if(t[6].length>0) 
    { 
     //console.log(t[6].length); 
     //var a=parseAttach(t[6]); 
     var a=""; 
     h+='<ul class="attaches">'+a+'</ul>'; 
    } 
    ...... 
} 

として私がconsole.logで呼び出しを置き換えると、コメントに表示されます。ログには4回の実行しか表示されません。しかし、なぜ機能が繰り返し呼び出されるのですか？

私は、ループの先頭にコンソールレポートがブレークポイントを見つけました（私はそれをコメントアウトしました）、これが関数が呼び出され続ける理由ですか？

Answer 1

両方のループで、i変数の前にvarを入れていません。これはそれがグローバルであることを意味します。グローバルであれば、両方のループはと同じiを使用しています。

for(i=0;i<b.length;i++) 

VARを追加し、それはそれを修正する必要があります

for(var i=0;i<b.length;i++) 

はEDIT：さらに明確化：

for(i=0;i<c[0].length;i++) 
{ 
    ...... 
    if(t[6].length>0) 
    { 
     // THIS CALL WILL SET THE GLOBAL i TO t[6].length 
     var a=parseAttach(t[6]); 
    } 
    ...... 
} 

したがって、外側のループの終了条件は、すべてのに会ったことはありませんループの反復はiを返すので、iは決して​​に到達しません。

Answer 2

1つのバグを除いて、すべてうまくいくように見えます。カウンタ変数iをローカルにするのを忘れました。ループから呼び出された場合、コードは（これは最大長であるとして、2）外側のループの同じ名前のカウンタ変数をリセットして、その終了条件に到達することはありませんになるだろう：

var i; // this variable will always be referenced 
function x(number) { 
    for (i=0; i<number; i++) 
     dosomething; 
} 
for (i=0; i<5; i++) 
    x(2); // resets i to 2 
// => never-ending loop 

無限ループブラウザがハングアップし、コードが実行されるまでインターフェイスがフリーズします。タイムアウトで実装された長時間実行されるスクリプトでは、ブラウザによってはエラーが発生することがあります。