php - else 永远不是必需的，您可以简化代码以在没有 else 的情况下工作

Question

我收到 PHPMD 的消息告诉我:

else is never necessary and you can simplify the code to work without else on this portion of code:

if ($settings == null) {
        $settings = new self($arrSettings);
} else {
        $settings->fill($arrSettings);
}
$settings->save();

return $settings;

我的问题是:我应该如何避免 else()。我看到的唯一方法是复制 $setting->save() 并返回。

有什么想法吗？

Answer 1

可能是因为它可以重写为

if ($settings === null) {
    $settings = new self; // or new self([]);
}
$settings->fill($arrSettings);
$settings->save();

return $settings;

但是，说实话，整件事看起来像是对 SRP 的重大违反。 ，因为类实例不应该能够创建自己的新实例。这根本没有任何意义..但话又说回来，我不是一个“artisan ”。