检查列表中两个对象的字段

Question

我有对象列表：

{
  "rn": "14",
  "time": "2020-06-16 16:47:55",
  "id": 42332392988,
  "termname": "S1160AFB",
  "type": "100",
  "trancode": "110",
  "pan": "4790729949581839",
  "amount": "360.96",
  "approvalcode": "44444"
},
{
  "rn": "15",
  "time": "2020-06-16 16:48:55",
  "id": 42330308379,
  "termname": "S1160AFB",
  "type": "100",
  "trancode": "110",
  "pan": "4149510064617121",
  "amount": "360.96",
  "approvalcode": "55555"
},
{
  "rn": "13",
  "time": "2020-06-16 16:49:11",
  "id": 42332530886,
  "termname": "S1160AFB",
  "type": "420",
  "trancode": "110",
  "pan": "4790729949581839",
  "amount": "360.96",
  "approvalcode": "44444"
}

如果出现以下情况，我需要从列表对象中删除：

1. 它们具有相同的字段批准代码。 （批准代码 - 唯一。并且不能超过两个相同）
2. 在具有相同批准代码的两个对象中，您需要检查日期较大的对象，以及该对象类型是否为 420。从列表中删除这两个对象。

我是这样决定的：

1. 我按批准代码和日期对列表进行排序。

    List<Transaction> filteredTrans = transResponse.getTransaction().stream().sorted(Comparator.comparing(Transaction::approvalcode).thenComparing(Transaction::getTime))
    .collect(Collectors.toList());
   

排序后，我比较两个最接近的对象。

  for (int i=0; i<filteredTrans.size()-1; i++) {
  if (filteredTrans.get(i).getApprovalcode().equals(filteredTrans.get(j).getApprovalcode()) && filteredTrans.get(j).getType().equals("420")) {
      filteredTrans.remove(i);
      filteredTrans.remove(j);
      j++;
  }
}

但我不喜欢代码的第二部分。能否请教一下如何改进？

Answer 1

您应该使用地图而不是排序。以下是您可以继续进行此操作的方法

import java.time.LocalDateTime;
import java.util.ArrayList;
import java.util.List;
import java.util.function.Function;
import java.util.stream.Collectors;

public class Test {
  static class Transaction {
    String approvalCode;
    LocalDateTime time;
    int type;

    public Transaction(String approvalCode, LocalDateTime time, int type) {
      this.approvalCode = approvalCode;
      this.time = time;
      this.type = type;
    }

    public int getType() {
      return type;
    }

    public LocalDateTime getTime() {
      return time;
    }

    public String getApprovalCode() {
      return approvalCode;
    }

      @Override
      public String toString() {
          return "Transaction{" +
                  "approvalCode='" + approvalCode + '\'' +
                  ", time=" + time +
                  ", type=" + type +
                  '}';
      }
  }

  public static void main(String[] args) {
    List<Transaction> transactions = new ArrayList<>();
    transactions.add(new Transaction("1", LocalDateTime.now(), 420));
    transactions.add(new Transaction("1", LocalDateTime.now(), 11));
    transactions.add(new Transaction("1", LocalDateTime.now(), 12));
    transactions.add(new Transaction("2", LocalDateTime.now(), 13));
    transactions.add(new Transaction("2", LocalDateTime.now(), 14));
    transactions.add(new Transaction("2", LocalDateTime.now(), 420));
    System.out.println(
        transactions.stream()
            .collect(
                Collectors.toMap(
                    Transaction::getApprovalCode,
                    Function.identity(),
                    (u, v) -> u.getTime().compareTo(v.getTime()) > 0 ? u : v))
            .values()
            .stream()
            .filter(v -> v.getType() != 420)
            .collect(Collectors.toList()));
  }
}

当遍历一个列表时，不要从同一个列表中删除。它可能导致并发修改异常。相反，将所有必需的元素复制到新列表/使用流过滤器

Answer 2

谢谢大家的帮助。 这段代码对我有帮助：

List<Transaction> filteredTrans = transResponse.getTransaction().stream()
    .collect(Collectors.groupingBy(Transaction::getApprovalcode))
    .values()
    .stream()
    .filter(l -> {
        if (l.size() == 1) return true;
        Transaction maxTimeTr = l.stream().max(Comparator.comparing(Transaction::getTime)).orElseThrow();
        return !maxTimeTr.getType().equals("420");
    })
    .flatMap(Collection::stream)
    .collect(Collectors.toList());