因此,在开发过程中的几次,我偶然传递了一些字符串以外的其他内容,从而导致错误.
现在,我想知道这是否是我应该一直这样做的事情.首先检查以确保已发送正确类型的数据/检查可能首先出现问题的事情,以某种方式记录它们,然后如果一切正常,则使用它做一些事情.
这是我应该坚持的吗?
类型提示
尽可能使用类型提示.对于原始类型,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"
这一切都引导我们回到类型提示,这可能是不必验证你的论点的最有效方式.
相关阅读
> Spl类型 – http://www.php.net/manual/en/book.spl-types.php
>在PHP中输入提示 – http://php.net/manual/en/language.oop5.typehinting.php
>防守编程 – http://en.wikipedia.org/wiki/Defensive_programming