在这段代码的 sn-p 中，我们键入 $inputs['user_id'] 3 次。
if (isset($inputs['user_id']) && $inputs['user_id']) { // The consumer is passing a user_id
doSomethingWith($inputs['user_id']);
}
为了避免重复并避免注意到索引 user_id 不存在，我可以做的最易读和最健壮的重构是什么？
谢谢。
【问题讨论】：
标签： php if-statement code-duplication
这里的重复没有任何问题。在检查变量是否已设置之前，不能将 $inputs['user_id'] 分配给变量，否则会产生 Notice undefined index ...。
这里唯一可以做的就是省略isset 调用并改用!empty，如下所示：
if(!empty($inputs['user_id'])) {
doSomething($inputs['user_id']);
}
现在您只需输入两次并检查
!empty($inputs['user_id'])
等于
isset($inputs['user_id']) && $inputs['user_id']
编辑： 基于一个 cmets，这里引用 documentation:
以下的东西被认为是空的：
"" (an empty string) 0 (0 as an integer) 0.0 (0 as a float) "0" (0 as a string) NULL FALSE array() (an empty array) $var; (a variable declared, but without a value)
所以empty(0) 或empty('0') 将返回true，这意味着
if(!empty('0') || !empty(0)) { echo "SCREW YOU!"; }
什么都不回应...或者，以礼貌的方式，我将重复上面的陈述：
!empty($inputs['user_id']) === (isset($inputs['user_id']) && $inputs['user_id'])
通过省略isset 并用!empty 替换变量仍然检查，索引是否已设置，请阅读documentation，其中说：
如果变量不存在，则不会生成警告。这意味着 empty() 本质上是与 !isset($var) || 的简明等效。 $var == false.
【讨论】：
empty('0') = true，应该知道这一事实。
这个呢：
// put validation check to the function body
function doSomethingWith($userId) {
if($userId === -1) {
// if this is not a valid user id -> return
return;
}
// do something ...
}
// initalize $user with proper default values.
// doing so you can be sure that the index exists
$user = array(
'id' => -1,
'name' => '',
...
);
// merge inputs with default values:
$user = array_merge($user, $request);
// now you can just pass the value:
doSomethingWith($user['id']);
【讨论】：
doSomethingWith 无法接受的“默认”无效有效值（我通常会选择null，而不是-1；-1 通常最终会是error，因为它是ID 的正确数据类型），在开头有一个保护语句，所以它会提前返回。
doSomethingWith(&$userId) 之前添加 & 以防变量未定义，因此 php 不会抱怨
下面可能不是所有情况的最佳方法，但肯定会减少重复。
您的示例代码将变成：
doSomethingWith($inputs['user_id']);
你的函数看起来像这样（注意通过引用提供的参数，以避免未定义的变量警告）：
function doSomethingWith(&$userID) {
if (empty($userID)) return;
// ... actual code here ...
}
【讨论】：
假设0 和"" 和null 不是有效的user_ids：
if ($id = $inputs['user_id']) {
doer($id);
}
你也可以用 evil @ 来避免在你的日志中被注意到，（我不喜欢这种方式）：
if ($id = @$inputs['user_id']) {
doer($id);
}
【讨论】：
user_id 始终设置 - 我从原始问题中的 isset 假设情况可能并非如此。