Symfony 3.4：含复杂查询的Product实体代码放置建议咨询

代码应该放在 ProductRepository，而非 Controller #

• Controller 的核心职责是处理请求、调度资源、返回响应，不适合承载可复用的业务逻辑，尤其是这种需要在多个页面展示的计算逻辑。
• Repository 是专门负责实体相关数据库查询与业务逻辑的组件，符合单一职责原则，也方便后续多场景复用。

Repository 中完全可以使用实体对象 #

你可以在 Repository 的方法里接收 Product 实体作为参数，直接调用它的 getter 方法获取属性，再结合 Query Builder 完成复杂查询，两者完全兼容。

重构示例代码 #

1. 迁移逻辑到 ProductRepository

// src/Repository/ProductRepository.php
public function calculateSetupCompletion(Product $product): array
{
    $setup = [
        'active' => $product->isActive(),
        'ref'    => !empty($product->getRef()),
        'tags'   => !$product->getTags()->isEmpty(),
    ];

    // 构建复杂查询（示例）
    $qb = $this->createQueryBuilder('p');
    $qb->select('COUNT(p.id)')
        ->where('p.id = :productId')
        // 这里添加你的复杂查询条件
        ->setParameter('productId', $product->getId());
    
    $recordCount = $qb->getQuery()->getSingleScalarResult();
    $setup['records'] = $recordCount > 0; // 根据实际需求调整判断逻辑

    // 计算完成度百分比
    $score = count(array_filter($setup));
    $setup['completion'] = round(($score / count($setup)) * 100);

    return $setup;
}

2. 在 Controller 中调用该方法

// src/Controller/ProductController.php
public function detailAction(Product $product, ProductRepository $productRepo): Response
{
    $setupData = $productRepo->calculateSetupCompletion($product);
    
    return $this->render('product/detail.html.twig', [
        'product' => $product,
        'setup' => $setupData,
    ]);
}

额外建议 #

如果后续逻辑需要扩展更多非数据库相关的业务判断，可以考虑封装成独立的服务类（比如 ProductSetupService），让 Repository 专注于数据库操作，服务类负责业务逻辑的组合。不过针对当前需求，放在 Repository 已经足够简洁合理。

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