Question

在我正在构建的框架中，我正在努力使我的代码更易于测试，因为我之前已经沉迷于MVC + Singleton模式并且拥有静态类。从那时起，我开始更多地了解单元测试和TDD，因此它促使我重新考虑了很多代码。这种重新分解的部分原因促使我尝试在PHP中正确使用Extension类，即不仅抛出Exception类，而且抛出更多相关的异常。

我有以下课程：

<?php

namespace Framework;

class Uri {

    public static function new_from_http() {

        $uri = '';

        if (isset($_SERVER['REQUEST_URI'])) {
            $uri = $_SERVER['REQUEST_URI'];
        } elseif (isset($_SERVER['PATH_INFO'])) {
            $uri = $_SERVER['PATH_INFO'];
        }

        return static::new_from_string($uri);
    }

    public static function new_from_string($string) {

        return new static(explode('/', $string));
    }

    protected $uri = [];

    public function __construct(array $uri) {

        $this->uri = array_values(array_filter($uri));
    }

    public function get_segment($offset, $default = null) {

        if (!is_int($offset)) {
            throw new \InvalidArgumentException(
                sprintf('%s requires argument 1 to be an integer, %s given.',
                    __METHOD__,
                    gettype()
                )
            );
        }

        return isset($this->uri[$offset - 1])
            ? $this->uri[$offset - 1]
            : $default;
    }
}

这一切都很好，正如您所看到的，get_segment方法需要一个整数，否则会抛出InvalidArgumentException。麻烦的是，我想创建一些方法，这些方法也需要整数作为参数，我不想在任何地方剪切和粘贴代码。合并所有这些类型的参数检查的最佳选择是什么，以便我可以在不同的类和方法中使用它们，同时保持消息彼此一致。

我的一个想法是扩展框架命名空间下的异常类，并让构造函数采用不同的参数，例如：

namespace Framework;

class InvalidArgumentException extends \InvalidArgumentException {

    public function __construct($method, $argument, $value) {

        parent::__construct(
            sprintf('%s requires argument 1 to be an integer, %s given.',
                $method,
                gettype($value)
            )
        );
    }
}

将被称为：

if (!is_int($arg)) {

    throw new \Framework\InvalidArgumentException(__METHOD__, 1, $arg);
}

还可以改进\Framework\InvalidArgumentException可以通过后跟踪获得__METHOD__值。

我还有哪些其他选择？最佳选择是什么？

Answer 1

我会将/InvalidArgumentException扩展为NonIntegerException基本相同的事情。这样，如果您想要strings，arrays或任何其他类型，您就可以创建新的异常，并且您不必使用疯狂的逻辑来确定要使用的消息。

PHP DRY抛出InvalidArgumentException

1 个答案: