当前位置：文江博客话题详情

在 PHP 中重写子类中的函数时更改返回类型？

发布于 2024-08-13 02:52:24 字数 114 浏览 5 评论 0原文

这很糟糕吗？或者说这在PHP框架中很常见？

例如，在父类中，有一个 save() 函数，它返回数据库中受影响的行数。然后在子类中，我重写此函数来执行一些预验证，并且还想简单地返回成功/失败的布尔值。

原文

分享到QQ

分享到微博

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

发布评论

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

人事已非 2024-08-20 02:52:24

我同意这样的共识：改变返回值的类型（甚至含义）是不好的。

怎样

下面是它的糟糕情况的说明：假设您有一个接受字符串和“Writer”的函数：

function printThroughWriter(String $text, Writer $writer) {
    $count = $writer->write($text);
    print "Writer '".get_class($writer)."' printed $count characters.";
}

这是原始的“Writer”类：

class Writer {
    public function write(String $text) {
        print $text;
        return strlen($text);
    }
}

如果我们将“LazyWriter”传递给该函数

class LazyWriter extends Writer {
    public function write(String $text) {
        return true;
    }
}

会：假设Writer::write()的返回值是printThroughWriter中的字符数已经被破坏，从而导致不一致。

I agree with consensus that it's bad to change the type (or even the meaning) of the return value.

Here is an illustration how it will be bad:

Let say you have a function that accepts a string and a 'Writer':

function printThroughWriter(String $text, Writer $writer) {
    $count = $writer->write($text);
    print "Writer '".get_class($writer)."' printed $count characters.";
}

And here is the original 'Writer' class:

class Writer {
    public function write(String $text) {
        print $text;
        return strlen($text);
    }
}

What if we pass a 'LazyWriter' instead to the function:

class LazyWriter extends Writer {
    public function write(String $text) {
        return true;
    }
}

The assumption that the return value of the Writer::write() is the number of characters in printThroughWriter has been broken, thus causing inconsistency.

回复收藏 0 原文

乖乖哒 2024-08-20 02:52:24

您应该覆盖底层代码，而不是合约。

所以，不，改变返回值（这是合同的一部分）是不好的。

回复收藏 0 原文

萌无敌 2024-08-20 02:52:24

继承的思想是允许客户端可互换地使用类的子类型。如果您正在更改子类型的返回类型，那么您就会破坏继承的规范实用性，并且最终只会造成混乱：
“这里我采用了一种 Collection 类型，但是 myCollection.add() 是否会返回一个布尔值或添加的元素或其他内容？我不知道！我不能再用这种类型做任何事情了！我”我必须为每个子类型制定一个方法才能解决这个问题！”

回复收藏 0 原文

渡你暖光 2024-08-20 02:52:24

是的，这很糟糕。该方法的客户很难知道这一点。除非，该方法旨在处理它（在本例中，它似乎并非如此）。

我会强烈建议反对。

回复收藏 0 原文

前事休说 2024-08-20 02:52:24

虽然 PHP 中没有强制返回类型，但与重写方法保持一致是一个好主意，因为否则意味着会增加耦合并减少封装。

从插座和设备的角度考虑公共方法及其返回类型。任何电器都应该能够插入任何插座，并且每个插座应该向每个电器输出相同量的能量，除非插座是专门为特定类型的电器设计的。如果有人决定绿色插座应该输出一种能量，红色插座应该输出另一种能量，会发生什么。突然之间，您必须始终关注您的设备正在处理的特定类型的插座。

设计公共方法和属性是完全相同的，并且重写的方法和属性应该始终表现一致，无论访问它们的上下文如何。

回复收藏 0 原文

作业与我同在 2024-08-20 02:52:24

我同意，这通常是一个坏主意。

如果您迫切需要从函数中传递除返回值之外的另一个值，则可以添加按引用传递参数。或者，如果出现错误消息，请使用异常。

回复收藏 0 原文

睫毛溺水了 2024-08-20 02:52:24

您可以返回更指定的类型，例如来自重写方法的返回类型的后代的实例，而无需违反约定。

如果被重写的方法返回布尔值，则如果 0 返回值符合导致被重写的方法返回 FALSE 且非零值对应于 TRUE 返回值。如果有人在返回值上使用相同的比较运算符 (===)，这仍然可能会导致问题，但您通常仅在（例如）方法可能同时返回 FALSE< 时才会看到这种情况。不同条件下的 /code> 和 0。

在您的示例中，重写返回类型并不比重写返回类型更具体，因此您不应该这样做。