Question

我发现我的代码有些问题，并且不明白它为什么会这样做。任何人都可以解释一下吗？

我们有：

abstract class AbstractThing
{
    public function search(...)
    {
        $ret = false;

        $data = $database->query(...);
        foreach($data as $values)
        {
            $item  = new $this;
            $item->fill_with_values($values);

            $ret []= $item;
        }

        return $ret;
    }
}

它可以按预期工作，并在成功搜索时返回对象实例：

class Thing extends AbstractThing
{
    // ...
}

$thing = new Thing;
$things = $thing->search(...); // Thing[] on success, false on failure

但是，如果我想稍微缩短代码，它会破坏：

abstract class AbstractThing
{
    public function search(...)
    {
        $ret = false;

        $data = $database->query(...);
        foreach($data as $values) {
            $ret []= (new $this)->fill_with_values($values);
        }

        return $ret;
    }
}

这个返回布尔值为true。为什么？它适用于不是从抽象类继承的类。

Answer 1

代码做了两件不同的事情：

这会将$ item添加到您的＆＃34; $ ret＆＃34;阵列：

        $item  = new $this;
        $item->fill_with_values($values);

        $ret []= $item;

这会添加＆＃34; fill_with_values＆＃34;的返回值。你的阵列：

$ret []= (new $this)->fill_with_values($values);

上述代码的等价物是：

        $item  = new $this;
        $return = $item->fill_with_values($values);
        $ret []= $return;

如果我知道你的＆＃34; fill_with_values＆＃34;方法我可以告诉你为什么它是一个布尔值，但代码不会做同样的事情。希望这是有道理的。

Answer 2

当我们分配时：

$ret []= (new $this)->fill_with_values($values);

...我们没有设置$ret[] = (new $this)。相反，此语句将fill_with_values()的返回值推送到数组中，因为它最后执行。

看起来你正试图实现与factory method pattern类似的东西。考虑一下：

abstract class AbstractThing
{ 
    ...
    public static function fill($values) 
    { 
        $instance = new static; 
        $instance->fill_with_values($values);

        return $instance; 
    }
}

然后我们可以实际做你想要在你的问题中完成的事情：

$ret[] = static::fill($values);

这是有效的，因为fill()的返回值是类的实例，而不是fill_with_values()的返回值。此上下文中的static关键字使用 late static binding 来解析执行代码的类的类型（在本例中为Thing），而不是声明它的类，所以它通过继承来工作。有关详细信息，请参阅this question。

Answer 3

好的，最后这是我自己的错误。在某些时候，确实有可能从fill_with_values（）函数返回TRUE。很抱歉所有的问题都很糟糕，感谢您的回答！

用抽象类返回new $ this

3 个答案: