Symfony中按SUB类型QuestionCategory动态调整表单placeholder的方法

• 仓储方法findModule()返回值是未执行的QueryBuilder对象，任何非空对象在PHP条件判断中恒为true，导致分支判断完全失效，placeholder永远走同一套配置。
• DQL条件where('t.module = SUB')存在语法错误：DQL中字符串值必须加引号，直接写SUB会被解析为未知常量，即使执行查询也无法得到正确结果。
• 业务逻辑耦合不合理：将数据库查询逻辑直接硬编码在表单类内部，既降低表单复用性，也不符合Symfony表单的分层设计规范。

1. 修正仓储层查询，返回可直接判断的布尔结果 #

将原返回QueryBuilder的方法修改为直接执行查询、返回实际判断结果的方法，同时修正DQL语法，使用参数绑定规避语法问题和注入风险：

// src/Repository/QuestionCategoryRepository.php
public function hasSubModule(): bool
{
    $count = $this->createQueryBuilder('t')
        ->select('count(t.id)')
        ->where('t.module = :module')
        ->setParameter('module', 'SUB')
        ->getQuery()
        ->getSingleScalarResult();

    return $count > 0;
}

2. 调整表单类的参数传递机制 #

通过Symfony表单原生的选项系统传递动态判断值，不要在表单内部硬执行数据库查询：

class AbstractContactType extends AbstractType
{
    public function buildForm(FormBuilderInterface $builder, array $options): void
    {
        // 从表单配置项中读取提前计算好的判断值
        $isSubModule = $options['is_sub_module'];

        // 合并公共配置，按判断结果切换placeholder
        $contentConfig = [
            'required' => true,
            'attr' => [
                'rows' => '6',
                'placeholder' => $isSubModule ? 'Test ...' : 'Votre demande ...'
            ]
        ];

        // 对齐原有逻辑：SUB类型下不显示label，非SUB类型显示label
        if (!$isSubModule) {
            $contentConfig['label'] = 'Votre demande';
        }

        $builder->add('content', TextareaType::class, $contentConfig);
    }

    public function configureOptions(OptionsResolver $resolver): void
    {
        // 定义表单的自定义必填选项，限制传入值类型为布尔值
        $resolver->setRequired('is_sub_module')
            ->setAllowedTypes('is_sub_module', 'bool');
    }
}

代码优化点：原逻辑重复调用两次add()方法添加同名字段，Symfony中同名字段重复添加会触发配置覆盖，合并为一次配置后可大幅减少冗余代码。

3. 在上层完成查询并传入表单 #

将数据库查询动作放到控制器或对应业务服务层，拿到布尔结果后在创建表单时传入配置项：

// src/Controller/ContactController.php
public function index(QuestionCategoryRepository $categoryRepo, Request $request): Response
{
    // 执行业务查询拿到判断结果
    $isSubModule = $categoryRepo->hasSubModule();

    // 创建表单时传入动态配置
    $form = $this->createForm(ContactType::class, null, [
        'is_sub_module' => $isSubModule
    ]);

    $form->handleRequest($request);
    // ... 后续表单处理逻辑
}

如果需要判断的是当前绑定到表单的QuestionCategory实体本身的module属性是否为SUB，无需额外写仓储查询，可直接从表单绑定的数据中读取属性判断：

public function buildForm(FormBuilderInterface $builder, array $options): void
{
    /** @var QuestionCategory|null $category */
    $category = $builder->getData();
    $isSubModule = $category?->getModule() === 'SUB';

    // 后续placeholder配置逻辑同上
}

• 遵循Symfony的分层设计规范：表单类仅负责字段配置、渲染逻辑，数据库查询、业务判断等逻辑应放在上层服务/控制器中实现，提升代码可测试性。
• 编写DQL条件时始终使用参数绑定的方式传值，不要直接拼接字符串到查询语句中，既避免引号语法错误，也能防范SQL注入风险。

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