使用Symfony框架执行DQL查询FOSUser表时出现SQL错误

嘿，我注意到你在尝试用DQL从FOSUserBundle生成的用户表中获取用户名，你的代码里有几个需要调整的地方，我来帮你修正并给出最佳实践的实现方式：

首先指出现有代码的问题 #

• 逻辑不匹配：方法名是findEmailDQL，参数是$email，但查询里却用u.id = $email——如果是通过邮箱找用户名，应该匹配u.email；如果是通过ID找，参数名和方法名应该更准确，避免混淆。
• SQL注入风险：直接把变量拼接进DQL字符串是非常危险的操作，很容易被利用进行SQL注入攻击，必须用Doctrine的参数绑定机制。
• 返回值冗余：getResult()会返回一个包含数组的数组（比如[['username' => 'xxx']]），如果只需要单个用户名，用更高效的单值查询方法更合适。

修正后的UserRepository代码 #

根据你的需求，我写了两个常用的方法：一个通过ID找用户名，一个通过邮箱找用户名，你可以按需使用：

namespace UserBundle\Repository;

use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\NonUniqueResultException;

class UserRepository extends EntityRepository {
    /**
     * 通过用户ID获取用户名
     */
    public function findUsernameById(int $userId): ?string {
        try {
            return $this->getEntityManager()->createQuery(
                "SELECT u.username FROM UserBundle:User u WHERE u.id = :userId"
            )
            ->setParameter('userId', $userId)
            ->getSingleScalarResult();
        } catch (NonUniqueResultException $e) {
            // ID是唯一的，这里仅做异常兜底，返回null表示未找到
            return null;
        }
    }

    /**
     * 通过邮箱获取用户名（更符合你原方法名的逻辑）
     */
    public function findUsernameByEmail(string $email): ?string {
        try {
            return $this->getEntityManager()->createQuery(
                "SELECT u.username FROM UserBundle:User u WHERE u.email = :email"
            )
            ->setParameter('email', $email)
            ->getSingleScalarResult();
        } catch (NonUniqueResultException $e) {
            return null;
        }
    }
}

控制器中的调用示例 #

假设你的控制器方法是通过邮箱查找用户名，调用方式如下：

use Symfony\Component\HttpFoundation\Response;
use UserBundle\Entity\User;

// 控制器方法示例
public function findByEmailAction(string $email) {
    $userRepo = $this->getDoctrine()->getRepository(User::class);
    $username = $userRepo->findUsernameByEmail($email);

    if ($username) {
        return new Response("找到的用户名：{$username}");
    }

    return new Response("未找到该邮箱对应的用户");
}

补充说明 #

• 参数绑定：setParameter方法会自动处理变量的转义，彻底避免SQL注入，这是Doctrine查询的标准做法。
• 单值查询：getSingleScalarResult()专门用于只查询单个字段的场景，直接返回该字段的字符串值（或null），比getResult()更简洁高效。
• 异常处理：捕获NonUniqueResultException是为了应对极端情况（比如数据库约束失效导致邮箱重复），保证代码的健壮性。

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