0

我有一个功能可以设置搜索查询中歌曲的键。如果给定的“键”超出允许的范围，它会从c..bor我得到它，它只是恢复到. 我是否应该抛出异常来警告提供的 $key 超出允许范围？0..11NULL

这是代码：

<?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
4

2 回答 2

1

这里有两个不同的问题。

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

第二个问题是是否应该抛出异常。理想情况下，仅当您的程序在当前状态下无法执行或继续可能导致数据损坏时，才应以这种方式引发异常。我认为您的情况并非如此，因为您只是在执行读取。

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

于 2012-10-18T01:18:23.547 回答
0

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

于 2012-10-18T00:49:49.463 回答