タイムアウトを引き起こす文字列連結

Question

以下私は最近、整数（$ naEUS）の配列をとり、それを繰り返しますが、その間にカンマで数字を追加します開始と終了。最終結果はこのようになります文字列でなければならない：（###、###、###、###）

$num = count($naEUS[$f]); 
    $resultsFields_values = "("; 
    for($b = 0; $b < $num; $b++) 
    { 
     if($b = 0) 
     { 
      $resultsFields_values = substr_replace($resultsFields_values, " {$naEUS[$b]} " , (strlen($resultsFields_values)), 0); 
     } 
     $resultsFields_values = substr_replace($resultsFields_values, ", {$naEUS[$b]} " , (strlen($resultsFields_values)), 0); 
    } 
    $resultsFields_values = substr_replace($resultsFields_values, ")" , (strlen($resultsFields_values)), 0); 

私は文字列の連結を扱うスレッドの多くがあります実現が、彼らだけをアドレス私の問題。私はこれがひどく非効率的なやり方であることを知っています。彼らはそれを行う良い方法を示していますが、それは簡単に見つけることができます。それは30秒のタイムアウトだとが、それは私の5秒、実行時のPHPを取った理由は、私が本当に知りたい何

はです。

もちろん、より良い解決策も歓迎します。

Answer 1

まあ、それは、配列のサイズに依存しますが、何をここでやっていることは、むしろ非効率的な機能を使用して、すべての反復でその文字列を再割り当てされます。あなたの配列が小さい限り、これはおそらく問題なく動作しますが、数千もの項目が含まれていると、実行に時間がかかることは不思議ではありません。 $b = 0で

$resultFields_values = '(' . implode(' , ', $naEUS) . ')'; 

Answer 2

使用比較演算子==の代わりに、$ B = 0

Answer 3

for($b = 0; $b < $num; $b++) 
    if($b = 0) 

、あなたはすべての繰り返しでゼロに戻るループをリセットしている：

より良い解決策は、このように、破機能を使用することです。 =は割り当てのためのもので、==は同等性のテスト用です。