Laravel控制器传UUID至服务转内部ID的逻辑正确性咨询

你的核心逻辑方向是正确的：对外接收UUID参数，转换为内部业务ID后查询数据，完全符合「对外暴露UUID、内部使用自增ID」的设计原则。但代码存在几个可优化的点，能提升健壮性和规范性：

• 处理UUID未匹配到数据的场景
  原代码直接访问->id，如果服务层未找到对应记录（返回null），会触发Trying to get property 'id' of non-object的致命错误，而非友好的业务异常。建议先校验返回值，或确保服务层在未找到数据时主动抛出ModelNotFoundException：

  $business = $this->businessService->getBusinessByUUID($businessUUID);
  if (!$business) {
      throw new ModelNotFoundException(__('Business not found'));
  }
  $filters['business_id'] = $business->id;
  
  $businessType = $this->businessTypeService->getBusinessTypeByUUID($businessTypeUUID);
  if (!$businessType) {
      throw new ModelNotFoundException(__('Business type not found'));
  }
  $filters['business_type_id'] = $businessType->id;
  

• 避免过滤参数被意外覆盖
  如果传入的$filters中已存在business_id或business_type_id，原代码会直接覆盖。若业务禁止外部传入内部ID，可先主动移除这些键，避免恶意参数干扰：

  unset($filters['business_id'], $filters['business_type_id']);
  // 再执行UUID转ID的赋值逻辑
  

• 异常场景的合理性调整
  集合为空时抛出ModelNotFoundException需注意：该异常在Laravel中默认映射404响应。若业务上「无报价」属于正常场景，可考虑返回空集合而非抛出异常；若属于异常场景则没问题，但建议异常信息更精准，比如改为__('No active quotes found')。

• 代码可读性优化
  可将UUID转ID的逻辑提取为私有方法，简化控制器代码的同时方便复用：

  private function resolveBusinessIdFromUUID(string $uuid): int
  {
      $business = $this->businessService->getBusinessByUUID($uuid);
      if (!$business) {
          throw new ModelNotFoundException(__('Business not found'));
      }
      return $business->id;
  }
  
  // 控制器方法内调用
  $filters['business_id'] = $this->resolveBusinessIdFromUUID($businessUUID);
  $filters['business_type_id'] = $this->resolveBusinessTypeIdFromUUID($businessTypeUUID);
  

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