含AND/OR多逻辑运算符的嵌套IF条件代码优化咨询

嘿，恭喜你在Stack Overflow的第一个提问！你的代码逻辑是能正常运行的，但确实可以通过调整结构和简化条件让它更清晰、更易维护。咱们先把原逻辑拆解清楚，再一步步优化～

原逻辑梳理 #

你的代码要实现的核心规则是：

• 绝对不处理404页面
• 跳过abc+123、def+123这两种页面+slug组合
• 符合条件的页面，先引入/file.php
• 除了def页面，额外引入file_2.php

优化方案1：扁平化条件，使用守卫模式 #

最直观的优化是把所有需要跳过的条件合并到一个判断里，提前返回（也就是常说的守卫模式），这样能减少嵌套层级，让主逻辑更突出：

// 先处理所有需要直接跳过的情况
if ($pageName === '404' || (in_array($pageName, ['abc', 'def']) && $pageSlug === '123')) {
    return null;
}

// 执行到这里说明页面符合引入要求
require_once '/file.php';

// 非def页面额外引入第二个文件
if ($pageName !== 'def') {
    require_once 'file_2.php'; // 注意：原代码这里少了开头的单引号，已经补上啦
}

优化点说明：

• 减少嵌套：把原本两层的if嵌套拆成扁平化结构，读代码时不用层层缩进找逻辑
• 简化重复条件：用in_array()替代重复的$pageName == 'abc' || $pageName == 'def'，后续如果要加更多类似页面，直接修改数组即可
• 提前守卫：把异常情况（需要跳过的页面）放在最前面，主逻辑（引入文件）放在后面，逻辑更清晰
• 修复小错误：原代码中file_2.php缺少开头的单引号，这是个容易忽略的语法问题，已经修正

优化方案2：用配置数组管理规则（适合复杂场景） #

如果后续需要添加更多页面+slug的排除规则，或者规则会频繁变动，可以把规则抽成配置数组，让代码更易维护：

// 把所有排除规则统一管理
$excludeRules = [
    'singlePages' => ['404'],
    'pageSlugPairs' => [
        ['abc', '123'],
        ['def', '123'],
        // 后续加新规则直接在这里加就行
    ]
];

// 检查是否需要排除当前页面
$shouldExclude = false;

// 检查单一页面排除规则
if (in_array($pageName, $excludeRules['singlePages'])) {
    $shouldExclude = true;
} 
// 检查页面+slug组合规则
else {
    foreach ($excludeRules['pageSlugPairs'] as $pair) {
        if ($pageName === $pair[0] && $pageSlug === $pair[1]) {
            $shouldExclude = true;
            break;
        }
    }
}

if ($shouldExclude) {
    return null;
}

// 主逻辑和之前一致
require_once '/file.php';
if ($pageName !== 'def') {
    require_once 'file_2.php';
}

这种方式的好处是，后续修改规则不需要碰核心逻辑代码，只需要修改$excludeRules数组，团队协作或者长期维护时更省心。

总的来说，优化的核心思路就是让异常/排除逻辑前置，减少嵌套，把重复的逻辑抽离出来，这样代码不仅可读性更高，后续维护也更轻松～

内容的提问来源于stack exchange，提问作者Nicolas V.