关闭。这个问题是基于意见的。它目前不接受答案。
**想改进这个问题吗？**更新这个问题，这样就可以通过编辑这篇文章用事实和引文来回答。

四年前关门了。
改进这个问题
我有枚举：

public enum Colour {
   RED, BLUE, BLACK;
}

我还有汽车课：

public class Car {
   private int id;
   private Colour colour;
   //other fields, getters and setters
}

我有很多车在里面 Map<Integer, Car> MAP_OF_CARS . 例如，我必须显示所有红色的汽车。所以我的问题是，什么是更好的解决方案。遍历Map中的所有汽车或创建Map，其中汽车列表将由颜色定义？
第一种解决方案：

public List<Car> getCarsByColor(Colour colour) {
   List<Car> sameColour = new ArrayList<>();
   for (Car car : MAP_OF_CARS.values()) {
      if(car.getColour().equals(colour) {
         sameColour.add(car);
      }
   }
   return sameColour;
}

第二种解决方案：在创建新车对象时，我会将其添加到Map中，结果是：

Map<Colour, List<Car>> CARS_BY_COLOURS;
CARS_BY_COLOURS.put(Colour.RED, car1);

而不是for循环，我将这样做：

public List<Car> getCarsByColor(Colour colour) {
   return CARS_BY_COLOURS.get(colour);
}

哪种解决方案更专业？