方法属性输入验证

Question

我应该如何处理方法的输入验证？
这两种方式，哪一种更正确呢？或者有更好的方法
该方法由构造函数调用，$prodID 可以是用户输入或来自数据库。

private function fill_data1($prodID)
{
    //Way 1
    filter_var($prodID, FILTER_VALIDATE_INT, array('options'=>array('min_range'=>1, 'max_range'=>1000000)));
    if (is_null($prodID)) {
        return FALSE;
    } elseif ($prodID === FALSE) {
        return FALSE;
    }
    $prod = getArtData($prodID);
    $this->set_id($prod['artID']);
    $this->set_name($prod['artName']);
    $this->set_price($prod['precio']);
}

private function fill_data(2$prodID)
{
    //Way 2
    filter_var($prodID, FILTER_VALIDATE_INT, array('options'=>array('min_range'=>1, 'max_range'=>1000000)));
    if (is_null($prodID) || $prodID === FALSE)
    {
        die('invalid input for prodID (' . $prodID . '). It has to be an integer > 0');
    }
    $prod = getArtData($prodID);
    $this->set_id($prod['artID']);
    $this->set_name($prod['artName']);
    $this->set_price($prod['precio']);
}

Answer 1

选项 3：使用异常并将 ID 验证尽可能靠近数据。

public function getArtData($id) {
    if (!is_int($id) || $id <= 0) {
        throw new InvalidIdentifierException(
                "Article ID $id must be a valid, positive integer.");
    }
    ...
}

返回 false 的问题是您必须检查返回值来处理并跳过它。如果处理它的位置（表示层）是从验证位置（数据层）删除的多个函数调用，则必须在每个级别检查 false。异常将传播到捕获它的第一个函数，并绕过每个函数中的剩余代码。

function displayProductAction(...) {
    $prodID = $request->getParam('prod');
    $form = ...
    try {
        $form->fill_data($prodID)
        $view->form = $form;
    }
    catch (InvalidIdentifierException $e) {
        $view->error = $e->getMessage();
    }
    $view->render();
}

调用 die() 会导致其自身的困难。单元测试更加困难，并且它迫使您将错误显示代码放在故障点。当您想使用网络服务中的代码时会发生什么？

Answer 2

验证输入时不应使用die()。
您的用户应该看到正确的错误消息以及如何提供正确输入的提示。

所以我建议你使用第一种方式。
根据您的架构，抛出异常而不是返回 false 可能更有意义。