关于含嵌套IF/ELSEIF结构的PHP代码的技术咨询

先帮你梳理下这段代码的核心逻辑：

• 获取当前请求的URI、服务器域名，以及Cloudflare提供的访客国家代码（不过这段代码里暂时没用到$country_code变量）
• 检测请求是否来自Facebook爬虫（通过判断User-Agent是否包含facebookexternalhit/或Facebot标识）
• 如果确认是Facebook爬虫，就根据当前域名做对应跳转（比如针对www.example.co.uk跳转到自身域名加URI的地址）

接下来我从技术层面给你几个关键点的分析和实用建议：

1. 跳转逻辑的合理性优化 #

你当前针对www.example.co.uk的跳转是直接跳转到自身，这看起来有点多余——除非你是想强制指定HTTP协议（代码里用了http://，如果原请求是HTTPS的话，这样会导致协议降级）。如果是想让Facebook爬虫抓取特定协议的内容，建议先判断原请求的协议，再动态拼接前缀：

$protocol = isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] === 'on' ? 'https://' : 'http://';
$link = $protocol . $activepage . $URI;

2. User-Agent检测的鲁棒性提升 #

Facebook官方的爬虫User-Agent不止你写的这两个，比如还有facebookcatalog/等标识。另外，直接用strpos判断时，建议先把User-Agent转成小写，避免因大小写差异漏判：

$userAgent = strtolower($_SERVER["HTTP_USER_AGENT"] ?? '');
if (strpos($userAgent, "facebookexternalhit/") !== false || strpos($userAgent, "facebot") !== false) {
    // 爬虫处理逻辑
}

3. 完善未完成的分支逻辑 #

你的代码里elseif ($activepage=="...")没写完，建议补充完整所有需要处理的域名分支，同时最好加一个默认的else分支，避免出现没有匹配到域名时无处理逻辑的情况：

if ($activepage=="www.example.co.uk") {
    $link = 'https://www.example.co.uk' . $URI;
    header("Location: $link", true, 302);
    exit;
} elseif ($activepage=="www.example.com") {
    $link = 'https://www.example.com' . $URI;
    header("Location: $link", true, 302);
    exit;
} else {
    // 默认处理逻辑，比如跳转到主域名
    $link = 'https://www.example.com' . $URI;
    header("Location: $link", true, 302);
    exit;
}

4. HTTP跳转的规范处理 #

使用header("Location: ...")时，建议明确指定HTTP状态码：如果是临时跳转需求用302，永久跳转用301。另外，header调用后一定要用exit或die()终止后续代码执行：

header("Location: $link", true, 302); // 302临时跳转适合动态内容场景
exit;

5. 变量的安全性与兼容性处理 #

• 直接使用$_SERVER['REQUEST_URI']时，建议做简单的过滤，避免潜在的非法路径问题：

  $URI = filter_var($_SERVER['REQUEST_URI'] ?? '', FILTER_SANITIZE_URL);
  

• $_SERVER["HTTP_CF_IPCOUNTRY"]只有在服务器使用Cloudflare作为CDN时才会存在，否则会报错，建议加上存在性判断：

  $country_code = isset($_SERVER["HTTP_CF_IPCOUNTRY"]) ? $_SERVER["HTTP_CF_IPCOUNTRY"] : 'Unknown';
  

6. 代码可读性优化 #

可以把判断Facebook爬虫的逻辑封装成独立函数，让代码结构更清晰：

function isFacebookCrawler() {
    $userAgent = strtolower($_SERVER["HTTP_USER_AGENT"] ?? '');
    $facebookAgents = [
        'facebookexternalhit/',
        'facebot',
        'facebookcatalog/'
    ];
    foreach ($facebookAgents as $agent) {
        if (strpos($userAgent, $agent) !== false) {
            return true;
        }
    }
    return false;
}

// 调用函数处理逻辑
if (isFacebookCrawler()) {
    // 域名分支判断
}

如果有具体的问题（比如跳转异常、爬虫识别不准等），可以再细化描述哦！

内容的提问来源于stack exchange，提问作者Henry Aspden