按姓名查询客户ID问题排查：SQL结构导致返回首条记录

你遇到的问题主要来自两个方面：SQL语句缺少表关联导致笛卡尔积，以及直接拼接变量带来的SQL注入风险，同时查询逻辑里直接取结果集第一条的写法也放大了这个问题，导致不管输入什么姓名都可能返回错误的首条记录。

核心问题拆解 #

1. 无关联的笛卡尔积查询：你的Model里的查询语句SELECT c.id_cadastro, pf.nome... FROM cadastro c, pessoa_fisica pf WHERE pf.nome = '$nome'没有关联cadastro和pessoa_fisica的外键，这会生成两张表的笛卡尔积——也就是说，所有cadastro记录都会和匹配姓名的pessoa_fisica记录关联，最终结果集的第一条自然不是你要的对应记录。
2. SQL注入风险：直接把用户输入的$nome拼接到SQL语句里，非常容易被注入攻击，这是严重的安全问题。
3. 字段匹配错误：后续查询pessoa_fisica时用了id_pessoa_fisica= "' . $cliente[0]->id_pessoa . '"，看起来字段名不匹配（cliente里的是id_pessoa，但pessoa_fisica的主键是id_pessoa_fisica），这会导致查询不到正确的个人信息。

修复后的代码示例 #

MODEL层代码修改

////Veficar matrícula cadastro.
function verificaNome($nome) {
    // 使用参数绑定防止SQL注入，同时添加表关联条件（假设cadastro.id_pessoa_fisica关联pessoa_fisica.id_pessoa_fisica）
    $sql = $this->db->query(
        "SELECT c.id_cadastro, pf.nome, c.id_pessoa_fisica AS id_pessoa, c.id_endereco, pf.id_pessoa_fisica, c.email, c.telefone 
         FROM cadastro c 
         JOIN pessoa_fisica pf ON c.id_pessoa_fisica = pf.id_pessoa_fisica 
         WHERE pf.nome = ?",
        array($nome)
    );

    if ($sql->num_rows() >= 1) {
        $cliente = $sql->row(); // 直接取单条匹配记录（如果姓名唯一），如果姓名可能重复，需要处理多条的情况

        // 查询个人信息，使用正确的关联字段
        $sqlPessoa = $this->db->query(
            'SELECT * FROM pessoa_fisica WHERE id_pessoa_fisica = ?',
            array($cliente->id_pessoa_fisica)
        );
        $pessoa = $sqlPessoa->row();

        // 查询地址信息
        $sqlEndereco = $this->db->query(
            'SELECT * FROM endereco WHERE id_endereco = ?',
            array($cliente->id_endereco)
        );
        $endereco = $sqlEndereco->row();

        // 组装返回数据
        $dados = [
            'cliente' => $cliente ?? [],
            'pessoa' => $pessoa ?? [],
            'endereco' => $endereco ?? []
        ];
        return $dados;
    } else {
        return []; // 没有匹配记录时返回空数组，而不是之前的$sql2结果（$sql2和$sql完全重复，没有意义）
    }
}

CONTROLLER层代码优化

//Verificar nome.
function verificaNomeExists() {
    $nome = $this->input->post('nome');
    if (empty($nome)) {
        echo json_encode(['error' => 'Nome não informado'], true);
        return;
    }

    $res = $this->compras_m->verificaNome($nome);
    if (!empty($res['cliente'])) {
        echo json_encode($res, true);
    } else {
        echo "0";
    }
}

关键修改说明 #

• 添加表关联：用JOIN语句明确关联cadastro和pessoa_fisica的外键（这里假设是c.id_pessoa_fisica = pf.id_pessoa_fisica，你需要根据实际数据库结构调整这个关联条件），这样只会返回姓名匹配且关联正确的客户记录。
• 参数绑定：使用?作为占位符，把变量通过数组传入query方法，避免SQL注入，同时让数据库正确解析字符串。
• 使用row()替代result()[0]：row()直接返回单条记录，比取结果集第一个元素更简洁安全，如果你允许姓名重复，需要改成result()并处理多条记录的情况。
• 修复字段匹配错误：把查询pessoa_fisica的条件改成id_pessoa_fisica = ?，对应cliente里的id_pessoa_fisica字段，确保能拿到正确的个人信息。
• 移除冗余代码：删除了重复的$sql2查询，因为它和$sql完全一样，没有实际作用。

额外建议 #

• 如果姓名不是唯一字段，建议在前端添加自动补全时显示更多信息（比如ID、邮箱），让用户选择正确的记录，避免返回多条结果时出现混淆。
• 数据库层面可以给pessoa_fisica.nome添加索引，提升查询性能。

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