Symfony CollectionType集合表单提交后数据未持久化问题排查

• 未正确获取请求对象：你在控制器中手动new Request()创建了空白请求实例，没有拿到用户提交的表单数据，导致表单无法正确处理提交内容。
• 产品实体实例化位置错误：你在循环外仅创建了1个Product实例，循环时会反复覆盖同一个实例的属性，最终只会保留最后一条产品数据。
• 冗余的字段赋值操作：ProductType已经配置了data_class关联Product实体，表单子项可以直接返回映射完成的Product对象，无需手动逐个取值赋值。
• 不合理的flush调用：将flush放在循环内会导致每次循环都执行一次数据库写入，性能极差，统一在循环结束后执行一次即可。

//[...]
use Symfony\Component\HttpFoundation\Request; // 别忘了导入Request类

class ProductController extends AbstractController
{
    /**
     * @Route("/product", name="product")
     */
    public function index(Request $request): Response // 从参数注入请求对象
    {
        $formProduct = $this->createForm(ProductsType::class); // 用类名::class语法更稳妥
        $entityManager = $this->getDoctrine()->getManager();
        
        $formProduct->handleRequest($request);
        if ($formProduct->isSubmitted() && $formProduct->isValid()) {
            // 直接获取集合字段的所有数据，已经自动映射为Product对象数组
            foreach ($formProduct->get('products')->getData() as $product)
            {
                $entityManager->persist($product);
            }
            // 统一执行一次写入
            $entityManager->flush();

            return $this->redirectToRoute('task_success');
        }

        return $this->render('product/index.html.twig', [
            'formProduct' => $formProduct->createView(),
        ]);
    }
}

如果需要保留原有手动取值的写法，只要把$product = new Product();移动到循环内部、替换请求注入逻辑也可以正常运行，但直接用表单自动映射的写法更简洁易维护。

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