中断for循环

Question

假设您有以下代码：

function doSomething($array)
{
  for($i = 0; $i < sizeof($array); $i++)
  {
    if ($array[$i] == "ok")
      return true;
  }

  return false;
}

请注意，我不是在谈论 PHP 特定的（这适用于所有语言）或这个特定的示例。这是关于中断 for 循环（在本例中，返回 true；停止循环）。

据我的一位老师说，这是非常非常糟糕的做法，而且还没有完成。

打破循环真的是一种未完成的练习吗？

谢谢

Answer 1

从循环中中断或返回是完全可以的。

你的老师可能指的是经典的函数应该只有一个返回点扩展到循环。其背后的基本原理是，您的控制流程应始终尽可能简单且易于理解。这并不是一个你必须不假思索地遵守的严格规则。

要在不使用 break 和 return 的情况下重写示例：

function doSomething($array)
{
  $ret = false;
  for($i = 0; $i < sizeof($array) && !$ret; $i++)
  {
    if ($array[$i] == "ok")
      $ret = true;
  }

  return $ret;
}

这很难阅读和维护。你的更简洁。

Answer 2

如果可能的话，我会远离循环中断。如果你的循环变得越来越大，它就会变得越来越难以阅读。任何不熟悉您的代码或特定函数的人都会认为您的循环只需查看第一行即可迭代整个数组。做任何其他事情都是“令人惊讶的”，因此打破了 CleanCode 哲学中的“最小惊讶原则”。如果您有多个退出循环的条件，那么 for 循环不是您应该寻找的。这就是 while 循环的用途。

function doSomething($array) {
    $found = false;
    $i = 0;

    while ($i < sizeof($array) && !$found) {
        if ($array[$i] == "ok") {
            $found = true;
        }
        $i++;
    }

    return $found;
}

Answer 3

我认为这不是一个坏习惯。这些类型的事情，比如打破循环，或者在循环中使用 return 都是不应该粗心地做的事情。

当我还是个初学者时，我总是听到这些话。不要使用这个，也不要使用那个。但后来我意识到他们这么说只是为了防止新人不小心使用这些东西而犯下严重错误。那些没有意识到这一点，后来自己成为老师的人，会让你远离这些，因为它们是邪恶的东西。

所以是的，使用它，有时它非常方便。正如其他人提到的那样，请注意使用这些时需要注意的所有事项。了解何时不应使用它们。

（还有一件事：在循环条件中获取数组的维度确实被认为是不好的做法。您不想在每次迭代中获取大小。

而不是：

for($i = 0; $i < sizeof($array); $i++)

使用

$size=sizeof($array);
for($i = 0; $i < $size; $i++)

：）

Answer 4

一些程序设计方法，例如 Jackson 并不是真正擅长打破循环。这是一个相当学术性的论点，在现实生活中并没有得到真正遵循。

Answer 5

打破循环并没有什么不好。它就像一个非常有限的转到。不要关心你的老师:-)

Answer 6

我不认为打破循环是一个坏习惯。不过，我通常根据情况使用 break 或 continue 来完成此操作。

Answer 7

人们通常说使用 break 或 continue （或它们的等效项）是不好的。无论如何我都会使用它们:-P

另一种方法是将所有代码包装在 if 或 else 中。

function doSomething($array)
{
  for($i = 0; $i < sizeof($array); $i++)
  {
    if ($array[$i] != "ok"){
       // do stuff...
    }
    else{
      // do something else
      // or omit this else block to do nothing
    }
  }
}

在从循环返回的情况下。我建议您在循环外设置一个变量，在需要时中断循环，然后返回该变量。

$ret = false;
function doSomething($array)
{
  for($i = 0; $i < sizeof($array); $i++)
  {
    if ($array[$i] == "ok"){
      $ret = true;
      break;
    }
  }

  return $ret;
}