Question

我在PHP中有对象比较的问题。看起来像一个简单的代码实际上按照我喜欢的方式运行太慢，因为我不是那种先进的语言，我想要一些关于以下代码的反馈和建议：

class TestTokenGroup {
    private $tokens;
    ...

    public static function create($tokens) {
        $instance = new static();
        $instance->tokens = $tokens;
        ...
        return $instance;
    }

    public function getTokens() {
        return $this->tokens;
    }

    public static function compare($tokenGroup1, $tokenGroup2) {
        $i = 0;
        $minLength = min(array(count($tokenGroup1->getTokens()), count($tokenGroup2->getTokens())));
        $equalLengths = (count($tokenGroup1->getTokens()) == count($tokenGroup2->getTokens()));
        $comparison = strcmp($tokenGroup1->getTokens()[$i], $tokenGroup2->getTokens()[$i]);
        while ($comparison == 0) {
            $i++;
            if (($i == $minLength) && ($equalLengths == true)) {
                return 0;
            }
            $comparison = strcmp($tokenGroup1->getTokens()[$i], $tokenGroup2->getTokens()[$i]);
        }
        $result = $comparison;
        if ($result < 0)
            return -1;
        elseif ($result > 0)
            return 1;
        else
            return 0;
    }
    ...

}

在上面的代码中，$tokens只是一个简单的字符串数组。

通过usort()使用上述方法获取由大约40k个对象组成的TestTokenGroup数组，需要大约2秒。

有没有明智的方法可以加快速度？这里的瓶颈在哪里？

编辑：添加了我最初忘记包含的getTokens（）方法。

Answer 1

你知道对象是“通过引用传递”，而数组是“按值传递”吗？

如果getTokens()返回$this->tokens，则每次调用该方法时都会复制该数组。

尝试直接通过$tokens访问$tokenGroup1->tokens。您也可以使用引用（&），尽管返回引用并不适用于所有PHP版本。

或者，只制作一份副本：

$tokens1 = $tokenGroup1->getTokens();
$tokens2 = $tokenGroup2->getTokens();

即使每个令牌组相对较小，它也将至少保存40000 * ( 6 + $average_token_group_length * 2)个数组副本。

<强>更新

我使用以下方法对OP的代码（删除...行）进行了基准测试：

function gentokens() {
        $ret = [];
        for ( $i=0; $i< 3; $i++)
        {
                $str = "";
                for ( $x = rand(0,3); $x < 10; $x ++ )
                        $str .= chr( rand(0,25) + ord('a') );
                $ret[] = $str;
        }
        return $ret;
}


$start = microtime(true);

$array = [];    // this will hold the TestTokenGroup instances
$dummy = "";    // this will hold the tokens, space-separated and newline-separated
$dummy2= [];    // this will hold the space-concatenated strings

for ( $i=0; $i < 40000; $i++)
{
        $array[] = TestTokenGroup::create( $t = gentokens() );

        $dummy   .= implode(' ', $t ) . "\n";
        $dummy2[] = implode(' ', $t );
}

// write a test file to benchmark GNU sort:
file_put_contents("sort-data.txt", $dummy);

$inited = microtime(true);
printf("init: %f s\n", ($inited-$start));

usort( $array, [ 'TestTokenGroup', 'compare'] );

$sorted = microtime(true);
printf("sort: %f s\n", ($sorted-$inited));

usort( $dummy2, 'strcmp' );

$sorted2 = microtime(true);
printf("sort: %f s\n", ($sorted2-$sorted));

得到以下结果：

init: 0.359329 s    // for generating 40000 * 3 random strings and setup
sort: 1.012096 s    // for the TestTokenGroup::compare
sort: 0.120583 s    // for the 'strcmp' compare

并且，运行time sort sort-data.txt > /dev/null会产生

.052 u  (user-time, in seconds).

优化1：删除数组副本

将->getTokens()替换为->tokens收益率（我只会列出TestTokenGroup::compare个结果）：

sort: 0.832794 s

优化2：删除array()中的多余min

将$minlength行更改为：

$minLength = min(count($tokenGroup1->tokens), count($tokenGroup2->tokens));

给出

sort: 0.779134 s

优化3：仅针对每个count 致电tokenGroup

    $count1 = count($tokenGroup1->tokens);
    $count2 = count($tokenGroup2->tokens);
    $minLength = min($count1, $count2);
    $equalLengths = ($count1 == $count2);

给出

sort: 0.679649 s

替代方法

到目前为止最快的排序是strcmp( $stringarray, 'strcmp' )：0.12s - 仍然是GNU排序的两倍，但后者只做了一件事，做得很好。

因此，为了有效地对TokenGroups进行排序，我们需要构造一个由简单字符串组成的排序键。我们可以使用\0作为标记的分隔符，我们不必担心它们的长度相等，因为只要一个字符不同，比较就会中止。

以下是实施：

$arr2 = [];
foreach ( $array as $o )
  $arr2[ implode("\0", $o->getTokens() ) ] = $o;

$init2 = microtime(true);
printf("init2: %f s\n", ($init2-$sorted2));

uksort( $arr2, 'strcmp' );

$sorted3 = microtime(true);
printf("sort: %f s\n", ($sorted3-$init2));

，结果如下：

init2: 0.125939 s
sort: 0.104717 s

PHP中的对象比较和数组排序

1 个答案: