PHP - 如果出现这种情况，则不执行任何操作

Question

“如果出现这种情况，就什么也不做”这样的说法是否正确？

if ( ($hostNameInfo == $hostNameInput) && ($hostAddressInfo == $hostAddressInput) )
{
  return;
}

更新： 我不在函数内部。 :( 所以返回只是废话。

这是更多代码：

//if the input fields are equal to database values, no need to update and waste resources,hence, do nothing:
if ( ($hostNameInfo == $hostNameInput) && ($hostAddressInfo == $hostAddressInput) )
{
  //do nothing
}
//If, however, (they are NOT equal and) input fields are not empty:
elseif (!empty($hostNameInput) && (!empty($hostAddressInput)))
{
 //do something.
}

提前致谢， MEM

Answer 1

如果不执行任何操作，您只需输入：

function relax() {
    ;
}

if (($hostNameInfo == $hostNameInput) && ($hostAddressInfo == $hostAddressInput)) {
    relax();
}

Answer 2

也许你应该做相反的事情，如果你的情况没有得到验证，就做一些事情

if($hostNameInfo != $hostNameInput || $hostAddressInfo != $hostAddressInput) {
   // do something
}

Answer 3

我假设您位于一个函数内部，在这种情况下，它会按照您的预期执行操作，尽管函数内的多个 return 语句可能导致混乱和缺乏可读性.（显然我错了。）

相反，我更喜欢让所有条件块（我对 if 的 {...} 块）包含相关代码，即，在需要额外处理（子流程）时，以总条件计算为 true 的方式编写条件检查：

if ($hostNameInfo != $hostNameInput || $hostAddressInfo != $hostAddressInput) {
    // do stuff, else skip
}

此外，您可以提取条件语句，以提高控制流的可读性和简单性：

$hostInfoEqualsInput = ($hostNameInfo == $hostNameInput && $hostAddressInfo == $hostAddressInput);
if (!$hostInfoEqualsInput) {
    ...
}

---

更新（基于更新的问题）。考虑一下这个：

$fieldsAreFilled = (!empty($hostNameInput) && !empty($hostAddressInput));
$hostInfoEqualsInput = ($hostNameInfo == $hostNameInput && $hostAddressInfo == $hostAddressInput);

if ($fieldsAreFilled && !$hostInfoEqualsInput) {
    ...
}

---

ERGO
通过编写您想要满足的条件（而不是您想要忽略的所有异常（主观））来最小化分支率并避免空块。

Answer 4

您在这里谈论的是最佳实践..
最佳实践之一是例程应具有 单一退出点，尽管它广泛 已讨论，由开发人员/风格决定。

更新：

新答案，因为问题已更改：

如果代码只应在某些情况下运行，则没有任何理由添加额外的检查。为了使代码更具可读性，您应该坚持您认为易于维护的任何内容，如下所示（或类似的内容）：

// Do something only if required
if (($hostNameInfo != $hostNameInput) || ($hostAddressInfo != $hostAddressInput)) &&
    !empty($hostNameInput) && !empty($hostAddressInput))
{
    echo 'place some code here';
}

Answer 5

有时，本机 do_nothing() 函数会非常好且可读。

为了避免来自语法检查器和语法检查器的压力警报linter，当你有一个空的 if 块时会变得疯狂，我使用：

echo(null);

Answer 6

另一种可能性是抛出一个新的异常，您稍后可以在应用程序中捕获该异常。

更新：不在函数内部，这可能是一个坏主意。