未认证用户通过构造函数跳转登录页出现ERR_TOO_MANY_REDIRECTS问题

这个问题我做CMS后台开发时也踩过一模一样的坑，核心原因特别明确：你的Admin控制器构造函数会拦截所有请求，包括login方法对应的登录页请求。当未登录用户访问admin/login时，构造函数检测到用户未登录，又重定向到admin/login，直接形成了无限循环，浏览器自然就抛出重定向次数过多的错误。

快速修复方案：在构造函数中排除登录方法 #

最简单的解决方式是在构造函数的登录检查逻辑里，跳过login方法的请求。以你使用的MVC框架（看起来像CodeIgniter）为例，可以通过路由获取当前请求的方法名，然后针对性地跳过检查：

class Admin extends Controller {
    public function __construct() {
        parent::__construct(); // 务必调用父类构造函数，框架核心初始化依赖它
        // 获取当前正在执行的控制器方法名
        $currentMethod = $this->router->fetch_method();
        
        // 只有当当前方法不是login时，才执行登录检查和跳转
        if(!isLoggedIn() && $currentMethod !== 'login') {
            redirect('admin/login');
        }
        $this->AuthModel = $this->model('Auth');
    }

    // 你的login方法代码...
}

修改后，当用户访问admin/login时，构造函数会识别到当前执行的是login方法，跳过登录检查，用户就能正常加载登录页面完成登录流程；登录成功后会话建立，后续访问Admin控制器的其他方法时，就会通过检查，不会触发跳转。

更优架构方案：拆分登录逻辑到独立控制器 #

如果想让代码结构更清晰，从根源避免这类控制器内的逻辑冲突，建议把登录、登出这类不需要登录验证的操作，拆分到一个独立的控制器（比如AuthController）中：

// AuthController.php 负责处理登录、登出等无需登录的操作
class Auth extends Controller {
    public function __construct() {
        parent::__construct();
        $this->AuthModel = $this->model('Auth');
    }

    public function login() {
        // 你的登录逻辑代码...
    }

    public function logout() {
        // 登出逻辑...
    }
}

// AdminController.php 仅处理后台管理功能，强制要求登录
class Admin extends Controller {
    public function __construct() {
        parent::__construct();
        if(!isLoggedIn()) {
            redirect('auth/login'); // 跳转到Auth控制器的login方法
        }
        $this->AuthModel = $this->model('Auth');
    }

    // 后台仪表盘、用户管理等需要登录的方法...
}

这种方式不仅彻底解决了循环重定向的问题，还让代码职责划分更明确，后续维护和扩展都会更方便。

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