PHP应该首先检查所有函数的参数类型吗？

我已经构建了一系列字符串函数,它们可以执行各种操作,我注意到我实际上没有任何内部函数检查,以确保变量是一个字符串,然后才能完成它.

因此,在开发过程中的几次,我偶然传递了一些字符串以外的其他内容,从而导致错误.

现在,我想知道这是否是我应该一直这样做的事情.首先检查以确保已发送正确类型的数据/检查可能首先出现问题的事情,以某种方式记录它们,然后如果一切正常,则使用它做一些事情.

这是我应该坚持的吗？

你可以看到这是一个有点争议的话题.这是我的看法：

类型提示

尽可能使用类型提示.对于原始类型,PHP中不能使用类型提示,所以是的,您应该检查以确保您已收到有效参数.如果还没有,您的函数可以抛出异常或返回一些默认值,如null或false.

防守编程

编写可测试代码的想法是失败不是沉默或神秘的.没有理由避免显式参数验证：冗长,代码更清晰,更实用.

除了验证参数之外,您还可以实现错误处理程序来捕获边缘情况.但是你应该验证大多数参数,特别是如果它们对持久数据(如数据库)有影响.

墨菲定律完全有效,因此你必须尽可能多地应对可预测的错误.无效参数是一个容易预测的错误 – 无法验证它是代码中的定时炸弹.例如,调用is_string很容易并且扩散炸弹.

拳击

另一个考虑因素是“装箱”你的变量.这导致了非常冗长的代码,但它确实具有允许基元类型提示的优点.

我从来没有见过任何人通过他们的整个代码库实际做到这一点,但它就在那里.有一些SPL类可用于主要类型,所以你最终会这样：

function stringThing (\SplString $myString) { ... }

stringThing(new \SplString('This is my string'));

Spltypes强制执行基本类型,并在滥用时抛出异常.从文档：

$string = new SplString("Testing");
try {
    $string = array(); // <----------this will throw an exception
} catch (UnexpectedValueException $uve) {
    echo $uve->getMessage() . PHP_EOL;
}

Spltypes是PECL扩展,并不总是标准PHP安装的一部分,因此请在使用之前检查扩展.该扩展也被认为是实验性的,尽管它已经存在了一段时间.

你也可以简单地创建自己的盒子：

class myStringBox {
  private $string = '';
  public function __construct($string=null) {
      if ($string)
          $this->set($string);
  }
  public function set($val) {
    if (!is_string($string)) throw new \invalidargumentexception();
    $this->string= $val;
  }
  public function __toString() {
    return $this->string;
  }
  public function trim() { return trim($this->string); } // extend with functions?
}

…但是这有一个主要的功能差异,你不能直接设置一个新的字符串值,如下所示：

$stringBox = new myStringBox('Hello World! ');
echo $stringBox; // "Hello World![space]"
echo $stringBox->trim(); // "Hello World!"

$stringBox = 'A new string';
echo $stringBox->trim(); // Error: Call to a member function trim() on a non-object

相反,您必须使用setter方法：

$stringBox = new myStringBox('Hello World! ');
echo $stringBox; // "Hello World![space]"
echo $stringBox->trim(); // "Hello World!"

$stringBox->set('A new world');
echo $stringBox->trim(); // "A new world"

这一切都引导我们回到类型提示,这可能是不必验证你的论点的最有效方式.

PHP应该首先检查所有函数的参数类型吗？

相关推荐