品牌+年份搜索筛选逻辑问题及代码修正需求咨询

问题根源 #

当前代码存在三个关键问题导致年份筛选失效：

1. 年份筛选代码块存在冗余大括号，破坏代码结构，导致年份条件无法被正确追加到查询语句中。
2. 车身类型（body_types）和年份（mfg_start/mfg_end）的筛选逻辑位于isset($_GET['search'])判断外部，既可能引发$where_clause未定义的错误，也不符合“仅在搜索触发时应用筛选”的逻辑。
3. 年份筛选被错误嵌套在品牌判断中，限制了年份条件的应用场景，实际上只要指定了年份范围就应该应用该筛选。

修正后的代码 #

if (isset($_GET['search'])) {                    
    $where_clause = " WHERE 1=1";
    $params = []; // 初始化参数数组，避免未定义问题
    
    // 处理品牌筛选
    if (!empty($_GET['brand_name']) && $_GET['brand_name'] != 'Choose Brand Name...') {
        $brand_id = $_GET['brand_name'];
        $where_clause .= " AND brand_id = ?";
        $params[] = $brand_id;
    }
    
    // 处理车身类型筛选
    if (!empty($_GET['body_types']) && $_GET['body_types'] != 'Choose Body Type...') {
        $body_type = $_GET['body_types'];
        $where_clause .= " AND body_type = ?";
        $params[] = $body_type;
    }
    
    // 处理年份范围筛选
    if (!empty($_GET['mfg_start']) && !empty($_GET['mfg_end']) 
        && $_GET['mfg_start'] != 'start_year' && $_GET['mfg_end'] != 'end_year') {
        $start_year = $_GET['mfg_start'];
        $end_year = $_GET['mfg_end'];
        $where_clause .= " AND mfg_year BETWEEN ? AND ?";
        $params[] = $start_year;
        $params[] = $end_year;
    }
    
    $sql .= $where_clause;
}

修正说明 #

• 将所有筛选逻辑统一放入isset($_GET['search'])块内，确保仅在触发搜索时生成筛选条件，同时避免$where_clause未定义的问题。
• 移除年份筛选代码块中的冗余大括号，修复代码结构错误。
• 取消年份筛选的品牌嵌套判断：只要用户指定了有效的年份范围，就将该条件追加到查询中。当同时选择品牌和年份时，会自动结合两个条件，返回该品牌下符合年份范围的数据；若该品牌无对应年份数据，查询结果为空，完全符合需求。
• 显式初始化$params数组，避免因未初始化导致的参数绑定错误。

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