如何避免PHP嵌套if...else？CodeIgniter登录方法代码优化问询

我正在CodeIgniter里写登录方法，流程是：验证用户输入→若有效则获取数据库用户详情→若查询成功则验证密码→若通过则登录系统。我知道用嵌套if...else能实现，但代码繁琐难读，所以试了收集错误数组的写法：

// 用于存储处理错误的空数组
$processing_errors = array();
// 检查输入是否有效
if(!$is_valid_input) array_push($processing_errors, "Invalid inputs");
// 获取用户详情
if($get_user_details) {
    // 验证用户
    if(!$is_correct_user) array_push($processing_errors, "Incorrect username or password.");
} else array_push($processing_errors, "Database query error");
// 检查是否存在处理错误
if(empty($processing_errors)) echo "Successfully logged in."; // 登录成功
else foreach($processing_errors as $errors) echo $errors; // 输出错误信息

但这个写法有问题：就算输入验证失败了，后面的代码还是会执行，造成不必要的性能浪费。想请教下，写整洁、可读性强的代码的最优方案是什么？

好问题！你已经意识到了错误收集写法的性能问题，同时也不想用嵌套if把代码写成“意大利面条”。下面几个方案可以帮你写出既整洁又高效的登录逻辑：

方案1：快速失败（提前返回） #

这是最直接的优化方式，一旦发现错误就立即终止当前流程，避免执行后续不必要的代码。这种写法逻辑清晰，每一步验证不通过就及时止损，完全不会做无用功：

// 第一步：验证输入有效性
if (!$is_valid_input) {
    echo "Invalid inputs";
    return; // 实际项目里可以用CodeIgniter的redirect、加载错误视图等方式终止流程
}

// 第二步：从数据库获取用户详情
$user = $this->user_model->get_user_by_username($this->input->post('username'));
if (!$user) {
    echo "Incorrect username or password."; // 登录场景建议统一提示，避免泄露用户存在性
    return;
}

// 第三步：验证密码（注意用password_verify，不要自己写加密逻辑）
if (!password_verify($this->input->post('password'), $user['password_hash'])) {
    echo "Incorrect username or password.";
    return;
}

// 所有验证通过，执行登录逻辑
$this->session->set_userdata('user_id', $user['id']);
echo "Successfully logged in.";

这种写法的优势：

• 每一步逻辑独立，没有嵌套，可读性极强
• 错误发生时立即终止，不会执行后续的数据库查询等操作，性能更优
• 可以针对每一步的错误返回精准提示（登录场景为了安全，用户名/密码错误建议统一提示）

方案2：利用CodeIgniter内置的表单验证类 #

CodeIgniter本身提供了强大的表单验证库，既能帮你完成输入验证，还能自动收集错误信息，完全不用自己写重复的输入校验逻辑，代码会更简洁规范：

public function login() {
    // 加载表单验证库
    $this->load->library('form_validation');

    // 设置验证规则：必填、去除首尾空格
    $this->form_validation->set_rules('username', 'Username', 'required|trim');
    $this->form_validation->set_rules('password', 'Password', 'required|trim');

    // 执行输入验证
    if (!$this->form_validation->run()) {
        // 验证失败，加载登录页面并传递错误信息
        $this->load->view('login', ['errors' => $this->form_validation->error_array()]);
        return;
    }

    // 输入验证通过，获取用户详情
    $username = $this->input->post('username');
    $user = $this->user_model->get_user_by_username($username);

    // 验证用户存在性和密码正确性
    if (!$user || !password_verify($this->input->post('password'), $user['password_hash'])) {
        // 添加自定义错误提示
        $this->form_validation->set_message('login_failed', 'Incorrect username or password.');
        $this->load->view('login', ['errors' => ['login_failed' => $this->form_validation->message('login_failed')]]);
        return;
    }

    // 登录成功，跳转到后台首页
    $this->session->set_userdata('user_id', $user['id']);
    redirect('dashboard');
}

这种方式的好处：

• 复用框架内置的验证能力，减少重复代码
• 错误信息自动收集，方便在视图中展示
• 符合CodeIgniter的最佳实践，其他维护者更容易理解你的代码

方案3：封装成验证服务类（适合复杂场景） #

如果你的登录逻辑后续会扩展（比如加上验证码、多因素认证等），可以把验证逻辑封装成独立的服务类，让控制器只负责处理请求和响应，实现代码解耦：

比如创建application/services/AuthService.php：

class AuthService {
    protected $user_model;

    public function __construct() {
        $this->user_model = new User_model();
    }

    // 验证输入合法性
    public function validate_login_input($username, $password) {
        if (empty(trim($username)) || empty(trim($password))) {
            return ['status' => false, 'message' => 'Invalid inputs'];
        }
        return ['status' => true];
    }

    // 验证用户身份
    public function verify_user($username, $password) {
        $user = $this->user_model->get_user_by_username($username);
        if (!$user) {
            return ['status' => false, 'message' => 'Incorrect username or password.'];
        }
        if (!password_verify($password, $user['password_hash'])) {
            return ['status' => false, 'message' => 'Incorrect username or password.'];
        }
        return ['status' => true, 'user' => $user];
    }
}

然后在控制器里使用：

public function login() {
    $auth_service = new AuthService();

    // 验证输入
    $input_validation = $auth_service->validate_login_input(
        $this->input->post('username'),
        $this->input->post('password')
    );
    if (!$input_validation['status']) {
        $this->load->view('login', ['errors' => [$input_validation['message']]]);
        return;
    }

    // 验证用户身份
    $user_verification = $auth_service->verify_user(
        $this->input->post('username'),
        $this->input->post('password')
    );
    if (!$user_verification['status']) {
        $this->load->view('login', ['errors' => [$user_verification['message']]]);
        return;
    }

    // 登录成功，设置会话并跳转
    $this->session->set_userdata('user_id', $user_verification['user']['id']);
    redirect('dashboard');
}

这种方案的优势：

• 把验证逻辑从控制器中剥离，控制器代码更简洁
• 验证逻辑可以复用在其他场景（比如API登录）
• 后续扩展验证规则时，只需要修改服务类，不会影响控制器代码

内容的提问来源于stack exchange，提问作者Muhammad Tarique