php - 方法内的无效参数抛出异常

Question

我正在学习异常处理。我已经知道如何使用它们，但是我不知道何时使用它们，因为很少有教程会告诉您任何有关这方面的见解。我的代码:

// 0-index part of the url
public function part($Part)
  {
  if (!is_numeric($Part))
    throw new Exception('The argument for $Url->part() should be numeric');

  $Part = (int) $Part;

  if ($Part < 0)
    throw new Exception('The argument for $Url->part() should be positive');

  if ($Part > count($this->parts))
    return false;

  return $this->parts[$Part];
  }

我觉得我的代码异常太多了。它是用于检索当前 url 及其某些部分的代码的一部分。例如，/this/is/a/test/ 将保存在 $this->parts 中作为 array('this', 'is', 'a ', '测试').

我是不是在该方法中使用了太多异常，影响了可读性？如果出现任何问题，我是否应该只使用一个异常，使调试稍微困难一些但更容易阅读源代码？

这是问题中命名的更通用的异常:

// 0-index part of the url
public function part($Part)
  {
  if (!is_numeric($Part) || intval($Part) < 0 || intval($Part) > count($this->parts))
    throw new Exception('The argument for $Url->part() is not correct');

  return $this->parts[(int) $Part];
  }

Answer 1

实际上，您的问题非常接近基于意见的问题(因此，如果没有非建设性的讨论，可能很难正确回答)。但是，有些事情需要记住。

首先，引发just exception 可能会有所改善 - 因为有标准的 exceptions在 PHP 中。抛出它们肯定会提高可读性。例如，您的代码接受一些应该是数字的参数。如果不是 - 那么它是无效参数 - 因此，可能会触发相应的异常。接下来，您的 parts 是一个数组 - 您想要检查传递的偏移量是否存在。如果不是，则为越界 异常。所以你的代码可能看起来像:

public function getPart($part)
{
   if(!is_numeric($part))
   {
      throw new InvalidArgumentException('Invalid part number passed');
   }
   if(!array_key_exists($part, $this->parts))
   {
      throw new OutOfBoundsException('Offset '.$part.' not found in parts');
   }
   return $this->parts[$part];
}

- 这对于数字偏移量似乎是合理的。但是，如果您的结构包含更复杂的数据，那么您可能希望引发逻辑异常(指出传递参数的逻辑结构或当前结构存在错误)。

在一般情况下 - 这一切都取决于情况和逻辑。没有没有 Elixir - 它与设计有关，所有解决方案都是相对的，并且受特定情况的约束。

同样，这非常基于意见(例如，我开始将您的方法重命名为 getPart() - 因为对我来说，方法是一个实体应命名为 Action 名称，而不仅仅是事物名称)。而且，更多 - 尽量避免在返回某些内容时混合不同的类型(例如示例中的 false )。它会导致不可靠的行为。最好抛出异常，但保持你的函数/方法返回类型相同。