检查引用者是否为空或者是否在数组中

Question

我正在尝试编写一个 if 语句，该语句基本上检查用户引荐来源网址是否在允许的引荐来源网址列表中，如果不是则失败。

我有两个变量控制这个 $this->allowAllReferer 和 $this->allowEmptyReferer，根据它们的名称决定是否每个引用者都应该被允许访问以及是否为空分别允许推荐人。以及 $this->allowedReferers ，它是允许的引用者数组。

我有这个功能，我很确定它不能正常工作，但我已经盯着它调整了半个小时，我已经到了我无法判断它是否正常工作的地步。

//If the referee is empty and allow empty referrer is false
//or
//If it is not in the allowed list and allow all referer is false 
if(!(empty($_SERVER['HTTP_REFERER']) && $this->allowEmptyReferer)
    &&
   !(!$this->allowAllReferer && in_array(
      strtolower(empty($_SERVER['HTTP_REFERER']) ? null : $_SERVER['HTTP_REFERER']), //Silly php access null variable
      $this->allowedReferers)
    )) {
    throw new sfException("Internal server error. Please contact system administrator. File download disabled.");
} 

您知道正确的或更好的方法吗/您能确认上述工作吗？

---

案例，希望这能让大家更清楚

empty_referrer | allowEmpty | in_array | allReferer | result
----------------------------------------------------------------
true           | true       | false    | false      | false - no error - empty allowed
false          | true       | false    | false      | true - error - not in array
false          | true       | false    | true       | false - no error - not in array but allowed
false          | false      | false    | false      | true - error - empty and now allowed

Answer 1

如果您想将逻辑保留在一个巨大的 if 块中，请尝试以下操作：

if (
    // throw an error if it's empty and it's not allowed to be
    (empty($_SERVER['HTTP_REFERER']) && !$this->allowEmptyReferer)
    || (
      // don't bother throwing an error if all are allowed or empty is allowed
      (!empty($_SERVER['HTTP_REFERER']) && !$this->allowAllReferer)
      // throw an error if it's not in the array
      && !in_array((empty($_SERVER['HTTP_REFERER']) ? null : strtolower($_SERVER['HTTP_REFERER'])), $this->allowedReferers)
    )
)
{
  throw new sfException("Internal server error. Please contact system administrator. File download disabled.");
}

如果 in_array 为空，则第二次检查现在将跳过 in_array。

Answer 2

怎么样：

$ref = &$_SERVER['HTTP_REFERER'];
if($allowAll) {
  // allowed
} else if($allowEmpty && empty($ref)) {
  // allowed
} else if(!empty($ref) && in_array($ref, $allowedReferers)) {
  // allowed
} else {
  // fail
}

如果您想在一个 if 中进行所有检查，您可以简单地使用 or/|| 将条件链接在一起。短路评估可确保正确的变量值并立即终止条件检查：

$ref = &$_SERVER['HTTP_REFERER'];
if($allowAll
    || ($allowEmpty && empty($ref))
    || (!empty($ref) && in_array($ref, $allowedReferers))) {
  // allowed
} else {
  // fail
}

Answer 3

如果我正确理解了您的要求，那么这最符合您的原始代码

        if( (!$this->allowEmptyReferer && empty($_SERVER['HTTP_REFERER'])
           || (!$this->allowAllReferer && !in_array(
          strtolower(empty($_SERVER['HTTP_REFERER']) ? null : $_SERVER['HTTP_REFERER']),
          $this->allowedReferers)
        ) { // throw your exception } 

Answer 4

我会简化你的逻辑，像这样：

if (!$this->allowAllReferer)
{
    if (empty($_SERVER['HTTP_REFERER']) && !$this->allowEmptyReferer)
    {
        // emtpy referer - not allowed. handle as you wish (throw exception?)
    }

    else if (!empty($_SERVER['HTTP_REFERER']) &&
        !in_array(strtolower($_SERVER['HTTP_REFERER'])), $this->allowedReferers)
    {
        // referer supplied is not approved/allowed. - handle appropriately.
    }

    else
    {
        // referer should be ok if we get here.
    }   

}

即。首先，如果您允许所有引用者，那么您不需要进行任何处理 - 只需跳过此操作（if (!this->allowAllReferer)）。
其次，将逻辑检查分解为管理块，这使得编写、读取和维护变得更容易。

Answer 5

if(isset($_SERVER['HTTP_REFERER'])) {
    echo $_SERVER['HTTP_REFERER'];
}