多少限制?

时间:2012-10-18 00:47:52

标签: php error-handling user-input validation

我有一个功能可以设置一首歌用于搜索查询的键。它来自c..b0..11,如果给定的"键"落在允许的范围之外,它只是恢复到NULL。我应该抛出异常而不是警告所提供的$ key超出允许的范围吗?

以下是代码:

<?php
function between($x, $y, $z) {
    return (($x <= $y) && ($y <= $z));
}

function setKey($key) {
    if (is_string($key)) {
        $key = array_search(strtolower($key), array("c", "c-sharp", "d", "e-flat", "e", "f", "f-sharp", "g", "a-flat", "a", "b-flat", "b"));
    } else {
        $key = (!between(0, $key, 11)) ? false : $key;
    }
    return (!$key) ? null : $key;
}

var_dump(setKey("g"))."\n";
var_dump(setKey("r"))."\n";
var_dump(setKey(2))."\n";
var_dump(setKey(12))."\n";

输出:

int(7)
NULL
int(2)
NULL

2 个答案:

答案 0 :(得分:1)

这里有两个不同的问题。

第一种方法是验证输入是否在允许的范围内,并在用户输入超出范围的值时向用户提供反馈。你绝对应该执行此验证。在可能的情况下,在用户开始输入之前将这些值显而易见,或者以消除错误可能性的方式显示选择。

第二个问题是是否应该抛出异常。理想情况下,只有在程序无法在当前状态下执行或者继续执行可能导致数据损坏时才会以这种方式引发异常。我不认为你的场景就是这种情况,因为你只是在进行阅读。

对于您的特定场景,我会使用验证,但不是例外。我已经包含了额外的细节,因此具有微妙不同场景的人在遇到问题时可以做出明智的决定。

答案 1 :(得分:0)

是的,让用户知道他们的输入超出范围总是一个好主意。这很可能是用户乐于了解和纠正的拼写错误。