4

私は次の機能を持っています：

function percentToColor($percent){
    $minBrightness = 160;
    $maxBrightness = 255;

    // Remainins?
    $brightness = ((($minBrightness-$maxBrightness)/(100-0))*$percent+$maxBrightness);
    $first = (1-($percent/100))*$brightness;
    $second = ($percent/100)*$brightness;

    // Find the influence of the middle color (yellow if 1st and 2nd are red and green)
    $diff = abs($first-$second);
    $influence = ($brightness-$diff)/2;
    $first = intval($first + $influence);
    $second = intval($second + $influence);

    // Convert to HEX, format and return
    $firstHex = str_pad(dechex($first),2,0,STR_PAD_LEFT);
    $secondHex = str_pad(dechex($second),2,0,STR_PAD_LEFT);

    return $firstHex . $secondHex . '00';
}

この関数は、0 から 100 までの範囲の整数を受け入れ、その数値の色情報を返します。赤が 0 で緑が 100 のプログレス バーを想像してみてください。これが関数の動作です。

したがって、この関数が各入力に対して常に同じ色を返す場合 (つまり、色が時間/ユーザー/セッションに依存しない場合) は、結果を含む PHP 配列を作成する方がよいのではないでしょうか?

だから私は関数を書き直します：

function percentToColorNew($percent){
   $results = array(
      0 => 'ff0000',
      1 => 'fe0500',
      2 => 'fd0a00',
      3 => 'fc0f00',
      // ... continues with 4, 5, up until 100 ...
      99 => '03a000',
      100 => '00a000'
   );
   return $results[$percent];
}

そして、私はそれをテストします。そして思わぬ事態が！結果の配列からのみ結果を返す新しい関数は、呼び出されるたびに結果を計算する必要がある元の関数の2 倍の時間がかかります。

どうしてこれなの？PHP配列はこれほど遅いですか? 関数の結果を保存して関数を高速化するより高速な方法はありますか? もしかして、スイッチ？If/elseif/else 条件? 配列を操作する別の方法はありますか?

4

1 に答える 1

8

関数を呼び出すたびに配列を初期化しているため、遅いです。

関数の外で配列を設定すると、はるかに高速になるはずです。手っ取り早い例:

$results = array(
    0 => 'ff0000',
    1 => 'fe0500',
    2 => 'fd0a00',
    3 => 'fc0f00',
    // ... continues with 4, 5, up until 100 ...
    99 => '03a000',
    100 => '00a000'
);

function percentToColorNew($percent){
    global $results;
    return $results[$percent];
}

編集static：または、キーワードをより適切に使用します：

function percentToColorNew($percent){
    static $results;
    if(!is_array($results)) {
        $results = array ( ... );
    }
    return $results[$percent];
}
于 2013-08-24T09:02:37.787 に答える