Spring+React商品应用指定品牌过滤失败问题求助

核心问题分析 #

1. 参数名不匹配：前端请求的参数是brand，但后端控制器接收参数的@RequestParam指定的value是brands，导致后端无法获取过滤条件，直接返回全部商品。
2. 空指针异常：数据库中存在brand字段为null的商品，调用product.getBrand().toLowerCase()时触发NullPointerException。
3. 查询效率低下：当前服务层先查询全部商品再做内存过滤，数据量大时性能很差，应该直接让MongoDB完成过滤逻辑。

分步修复 #

1. 修复参数名匹配问题 #

方案一：修改后端控制器参数名

将ProductController中的@RequestParam的value改为brand，匹配前端传入的参数：

@GetMapping
public List<Product> getProducts(@RequestParam(value = "brand", required = false) List<String> brandProducts) {
    if (brandProducts == null || brandProducts.isEmpty()) {
        return productService.allProducts();
    } else {
        return productService.findByBrand(brandProducts);
    }
}

方案二：修改前端请求参数名

保持后端参数名brands不变，修改前端请求的参数key：

useEffect(() => {
  api.get("/api/v1/products", {
    params: {
      brands: ['clinique', 'smashbox'] // 改为brands，和后端@RequestParam的value一致
    }
  })
  .then((response) => {
    setProducts(response.data);
  })
  .catch(error => console.error("加载商品失败：", error));
}, []);

2. 解决空指针异常 #

修改ProductService的findByBrand方法，先判断brand是否为null，再执行大小写转换和匹配：

public List<Product> findByBrand(List<String> brands) {
    // 统一将传入的品牌转为小写，避免大小写不匹配
    List<String> lowerCaseBrands = brands.stream()
        .map(String::toLowerCase)
        .collect(Collectors.toList());

    return productRepository.findAll().stream()
        .filter(product -> product.getBrand() != null 
            && lowerCaseBrands.contains(product.getBrand().toLowerCase()))
        .collect(Collectors.toList());
}

3. 优化查询效率（推荐） #

直接通过MongoDB查询过滤，避免内存遍历。在ProductRepository中添加查询方法：

import org.springframework.data.mongodb.repository.MongoRepository;
import org.springframework.data.mongodb.repository.Query;
import java.util.List;

public interface ProductRepository extends MongoRepository<Product, Integer> {
    // 忽略大小写的品牌查询（适配数据库中品牌大小写不统一的场景）
    @Query("{ 'brand': { $in: ?0, $options: 'i' } }")
    List<Product> findByBrandInIgnoreCase(List<String> brands);
    
    // 如果数据库中品牌都是小写存储，直接用这个更简洁
    // List<Product> findByBrandIn(List<String> brands);
}

然后修改ProductService的findByBrand方法：

public List<Product> findByBrand(List<String> brands) {
    // 使用忽略大小写的查询，无需手动转小写
    return productRepository.findByBrandInIgnoreCase(brands);
    
    // 如果是精确小写匹配：
    // List<String> lowerCaseBrands = brands.stream().map(String::toLowerCase).collect(Collectors.toList());
    // return productRepository.findByBrandIn(lowerCaseBrands);
}

4. 前端调试优化 #

添加错误捕获逻辑，方便排查请求失败的原因：

useEffect(() => {
  api.get("/api/v1/products", {
    params: {
      brands: ['clinique', 'smashbox']
    }
  })
  .then((response) => {
    setProducts(response.data);
  })
  .catch(error => {
    console.error("请求商品接口失败：", error.response?.data || error.message);
  });
}, []);

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