循环中的逻辑错误导致不需要的输出

Question

我不确定如何解决这个问题。

 List<Temporary> getTemporaryList = getTemporary();
      Map<Integer, Set<Attribute>> attributeOrderMap = attributeOrderMap(getTemporaryList);
      Map<Integer, Set<OrderSample>> sampleOrderMap = sampleOrderMap(getTemporaryList);

      Set<OrderSample> temporaryOrderSampleSet = Sets.newHashSet();
      java.util.ListIterator<Order> it = orders.listIterator();
      List<Order> finalOrder = Lists.newArrayList();

for (Order order : orders) {
         temporaryOrderSampleSet.clear();
         if (sampleOrderMap.containsKey((order.getId()))) {
            Set<OrderSample> samples = sampleOrderMap.get(order.getId());

            for (OrderSample orderSample : samples) {
               Set<Attribute> attributes = attributeOrderMap.get(order.getId());
               orderSample.setAttributes(attributes);
               temporaryOrderSampleSet.add(orderSample);
               order.setSample(temporaryOrderSampleSet);
               finalOrder.add(order);
            }
         }
      }

我有一份订单清单。 1000 个订单。我有一组样本和一组属性。对于第一个属性，我想把它放在第一个样本中，然后把它放在第一个顺序中。然后对第二个属性也一样，把它放在第二个样本中，然后把那个样本放在第二个顺序中。

所以我的列表如下所示

Attribute1 --> sample1 --> Order1

Attribute2 --> sample2 --> Order2

Attribute3 --> sample3 --> Order3

Attribute4 --> sample4 --> Order4

我当前的代码一次又一次地添加样本。所以我的第一个列表开始时很小，然后我的第二个更大，因为它包含了第一个列表和第二个列表中的所有内容，然后第三个更大等等。我被卡住了，不知道该怎么办。

Answer 1

您需要在for 循环内创建一个新的temporaryOrderSampleSet。

for (Order order : orders) {
         Set<OrderSample> temporaryOrderSampleSet = Sets.newHashSet();
         if (sampleOrderMap.containsKey((order.getId()))) {

否则，您会一遍又一遍地放置 same 列表，然后将其清除，以便只有最后一次迭代设置它的值。

另一个问题：

order.setSample(temporaryOrderSampleSet);

可以设置在内部 for 循环之外。在您的代码中，您只需一次又一次地覆盖该值。