Symfony5如何将控制器逻辑拆分至Repository与Factory，表单传参是否正确

架构老林

2026-9-30

实现方案评估

首先你的传入Form对象的实现是完全正确的，且优于传入Request的方案，原因如下：

数据安全性更高：Form组件已经完成了参数校验、类型转换、数据过滤，你从$form->get()->getData()拿到的是符合TestType中定义规则的合法数据，避免了直接从Request取原始参数可能带来的脏数据注入问题。
复用性更强：工厂依赖Form对象而非Request对象，不会和HTTP层强绑定。后续如果需要在命令行、内部服务调用等非HTTP场景调用工厂创建对象，只要构造符合规则的Form实例即可，不需要再模拟Request对象。
职责更清晰：控制器负责处理请求和表单逻辑，工厂只负责根据校验后的数据创建业务对象，符合单一职责原则。

可优化的建议

补充表单有效性判断
你当前的控制器代码只判断了$form->isSubmitted()，建议增加$form->isValid()判断，避免非法数据进入后续业务流程：
```
if ($form->isSubmitted() && $form->isValid()) {
    $test = $factory->create($form);
    $repository->save($test);
}
```
简化数据获取逻辑
如果你在TestType中已经配置了data_class => Test::class，直接调用$form->getData()就可以返回已经填充好所有字段的Test对象，不需要自己写工厂逐个字段赋值。只有当你需要做自定义字段转换、关联其他业务数据生成对象时，才需要单独封装Factory。

进一步解耦工厂（可选）
如果希望工厂完全不依赖Symfony的Form组件，提升跨场景复用性，可以把工厂参数改为数组，控制器传入表单提交的结构化数据即可：

// 优化后的TestFactory
class TestFactory
{   
    public function create(array $data)
    {
        $test = new Test();
        $test->setTest1($data['test1']);
        $test->setTest2($data['test2']);
        // 自定义业务逻辑可在这里补充
        return $test;
    }
}

// 控制器调用代码
$test = $factory->create($form->getData());