str_replace 为 strpos？

发布于 2024-12-10 13:50:46 字数 418 浏览 0 评论 0原文

带 strpos 检查的 str_replace 函数可以避免额外的工作吗？

方法 1

...
if (strpos($text, $tofind) !== FALSE)
 $text = str_replace($tofind, $newreplace, $text);
...

方法 2

...
$text = str_replace($tofind, $newreplace, $text);
...

问题：这两种方法有效，但是......我想知道 strpos 检查（或其他）是好方法还是坏方法，无用（以及优化）反模式）。

原文

The str_replace function with a strpos checking can avoid extra work?

METHOD 1

...
if (strpos($text, $tofind) !== FALSE)
 $text = str_replace($tofind, $newreplace, $text);
...

METHOD 2

...
$text = str_replace($tofind, $newreplace, $text);
...

Question: This two methods works but... I want know if strpos-checking (or other) is good way or a bad, useless (and optimization antipattern).

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

孤蝉 2024-12-17 13:50:46

您可能会保存一些str_replace()调用，但您总是会得到额外的strpos()调用和!== false 比较。然而，我认为，只要这段代码不会运行大约 100000 次（或类似的次数），它就不会产生任何可衡量的影响。
因此，只要您不需要知道，如果需要进行替换，您就应该避免这种“优化”，以使事情更加简单和可读。

回复收藏 0 原文

孤蝉 2024-12-17 13:50:46

您始终可以自己计时：

$start = 0; $end = 0;

$tofind = 'brown';

// A
$start = microtime(true);
for ($a=0; $a<1000; $a++) {
    if (strpos('The quick brown fox', $tofind) !== FALSE)
        str_replace($tofind, 'red', 'The quick brown fox');

}
$end = microtime(true);
echo $end - $start . "<br />\n";

// B
$start = microtime(true);
for ($b=0; $b<1000; $b++) {
    str_replace($tofind, 'red', 'The quick brown fox');
}
$end = microtime(true);
echo $end - $start . "<br />\n";

/*
various outputs:

0.0021979808807373
0.0013730525970459

0.0020320415496826
0.00130295753479

0.002094030380249
0.0013539791107178

0.0020980834960938
0.0013020038604736

0.0020389556884766
0.0012800693511963

0.0021991729736328
0.0013909339904785

0.0021369457244873
0.0012800693511963

*/

添加 strpos 每次都会变慢，但不会慢很多。

一个好的经验法则是不要猜测你的瓶颈在哪里。功能代码和良好、简洁的设计。之后，您可以在性能测试需要时分析。

You can always time things yourself:

$start = 0; $end = 0;

$tofind = 'brown';

// A
$start = microtime(true);
for ($a=0; $a<1000; $a++) {
    if (strpos('The quick brown fox', $tofind) !== FALSE)
        str_replace($tofind, 'red', 'The quick brown fox');

}
$end = microtime(true);
echo $end - $start . "<br />\n";

// B
$start = microtime(true);
for ($b=0; $b<1000; $b++) {
    str_replace($tofind, 'red', 'The quick brown fox');
}
$end = microtime(true);
echo $end - $start . "<br />\n";

/*
various outputs:

0.0021979808807373
0.0013730525970459

0.0020320415496826
0.00130295753479

0.002094030380249
0.0013539791107178

0.0020980834960938
0.0013020038604736

0.0020389556884766
0.0012800693511963

0.0021991729736328
0.0013909339904785

0.0021369457244873
0.0012800693511963

*/

Adding strpos is slower every time, but not by much.

A good rule of thumb is don't guess where your bottlenecks will be. Code for functionality and good, clean design. After that, you can profile when performance tests warrant it.

回复收藏 0 原文

去了角落 2024-12-17 13:50:46

没有strpos的方法更好。

我们假设 strpos 和 str_replace 具有相同的最坏情况运行时间，因为它们都必须迭代整个文本。

通过使用两者，在最坏的情况下，运行时间比单独使用 str_replace 增加了一倍。

回复收藏 0 原文

用心笑 2024-12-17 13:50:46

我刚刚测试了 3 种方法来替换我的配置文件中的常量：

// No check
function replaceConstantsNoCheck($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        $value = str_replace($constant, constant($constant), $value);

    return $value;
}

// Check with strstr
function replaceConstantsStrstr($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        if (strstr($value, $constant))
            $value = str_replace($constant, constant($constant), $value);

    return $value;
}

// Check with strpos
function replaceConstantsStrpos($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        if (strpos($value, $constant) !== false)
            $value = str_replace($constant, constant($constant), $value);

    return $value;
}

一些测量：

/*
No check : 0.0078179836273193
Strstr   : 0.0034809112548828
Strpos   : 0.0034389495849609

No check : 0.0067379474639893
Strstr   : 0.0034348964691162
Strpos   : 0.0034480094909668

No check : 0.0064759254455566
Strstr   : 0.0031521320343018
Strpos   : 0.0032868385314941

No check : 0.0068850517272949
Strstr   : 0.003389835357666
Strpos   : 0.0031671524047852

No check : 0.006864070892334
Strstr   : 0.0032939910888672
Strpos   : 0.0032010078430176
*/

在我的所有测试中，没有检查方法使用的时间至少是两倍！

strstr 和 strpos 方法之间似乎没有显着差异。

I've just tested 3 ways to replace constants in my config file :

// No check
function replaceConstantsNoCheck($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        $value = str_replace($constant, constant($constant), $value);

    return $value;
}

// Check with strstr
function replaceConstantsStrstr($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        if (strstr($value, $constant))
            $value = str_replace($constant, constant($constant), $value);

    return $value;
}

// Check with strpos
function replaceConstantsStrpos($value)
{
    foreach (array_keys(get_defined_constants()) as $constant)
        if (strpos($value, $constant) !== false)
            $value = str_replace($constant, constant($constant), $value);

    return $value;
}

Some measurements :

/*
No check : 0.0078179836273193
Strstr   : 0.0034809112548828
Strpos   : 0.0034389495849609

No check : 0.0067379474639893
Strstr   : 0.0034348964691162
Strpos   : 0.0034480094909668

No check : 0.0064759254455566
Strstr   : 0.0031521320343018
Strpos   : 0.0032868385314941

No check : 0.0068850517272949
Strstr   : 0.003389835357666
Strpos   : 0.0031671524047852

No check : 0.006864070892334
Strstr   : 0.0032939910888672
Strpos   : 0.0032010078430176
*/

No check method used a least double of the time in all my tests !

It seems to not have significant difference between strstr and strpos methods.

回复收藏 0 原文