Question

 MyObject structure is: 

public class MyObject { 
 String gender; 
 String state; 
  int quantity; 
  int Salary;}

我有以下功能。它允许我在性别和对象的状态维度中聚合数据：

   private static Collection<MyObject> aggregate(List<MyObject> objects) {

      int nombre=0;
      Map<String, MyObject> map = new HashMap<String, MyObject>();

      for (MyObject current : objects) {
           String key = String.format("%s:%s", current.gender, current.state);
           MyObject aggregated = map.get(key);
           if (aggregated == null) {
              aggregated = new MyObject();
              aggregated.setGender(current.getGender());
              aggregated.setState(current.state);
              map.put(key, aggregated);

           }
           if(current.getSalary()!=0)
              nombre+=1;

           aggregated.setQuantity(aggregated.getQuantity() + current.getQuantity());
           aggregated.setSalary((aggregated.getSalary() + current.getSalary())/nombre);

        }
        return map.values();
  }

我希望获得汇总数据的平均工资。＆＃34; nombre＆＃34;出了点问题。属性。

实际上它给了我以下结果：

 element : 1  M  40  1200
 element : 1  F  35  2000
 element : 1  M  20  200
 element : 1  M  80  300
 element : 1  F  35  2000

聚会后的内容：

`element : 1  M  140  333
 element : 1  F  70  2000`

但我期待：

聚合后的内容： element : 1 M 140 566 element : 1 F 70 2000

Answer 1

我在这里至少看到四个问题：

如果当前工资为0，则无论如何都会计算它，如果它是第一个你删除0 - 哎哟。
nombre是int，假设getSalary()也会返回int，因为当所有操作数都是整数时，因为1/2 = 0，所以会出现截断/舍入问题。
您可能希望在总结所有工资后计算平均，而不是每次迭代。

您似乎根据性别和状态汇总，但您只有一个计数器（nombre）。您希望每个聚合都有一个计数器。

Answer 2

  private static Collection<MyObject> aggregate(List<MyObject> objects) {
        Map<String, MyObject> map = new HashMap<String, MyObject>();

        for (MyObject current : objects) {
         String key = String.format("%s:%s", current.gender, current.state);
           MyObject aggregated = map.get(key);
          if (aggregated == null) {
                aggregated = new MyObject();
               aggregated.setGender(current.getGender());
               aggregated.setState(current.state);
               map.put(key, aggregated);
          }
          aggregated.nombre+=1;
           aggregated.setQuantity(aggregated.getQuantity() + current.getQuantity());
            aggregated.setSalary((aggregated.getSalary() + current.getSalary()));

         }
            for(MyObject c : map.values()){
            c.setSalary(c.getSalary()/c.nombre);

        }

        return map.values();
    }

我在MyObject中添加了一个属性nombre。

计算平均值，同时汇总数据

2 个答案: