代码优化 List<String> allRows = this.getHBaseRows(); if (CollectionUtils.isEmpty(allRows)) { return success("暂无数据"); } // 数据转换 List<PositionInfo> allInfos = allRows.stream() .map(ar -> JSONObject.parseObject(ar, PositionInfo.class)) .filter(ar-> ar.getTimestamp() > DateUtils.addHours(new Date(),-1).getTime()) .collect(Collectors.toList()); // 获取每个用户最近一次的记录 List<PositionInfo> result = new ArrayList<>(allInfos.stream() .collect(Collectors.toMap(PositionInfo::getUserId, Function.identity(), BinaryOperator.maxBy(Comparator.comparing(PositionInfo::getTimestamp)))) .values());

这段代码可以进行以下优化：

1. 使用Optional避免空指针异常

可以使用Optional避免空指针异常，如下：

List<String> allRows = this.getHBaseRows();
return Optional.ofNullable(allRows)
        .filter(CollectionUtils::isNotEmpty)
        .map(rows -> {
            // 数据转换
            List<PositionInfo> allInfos = rows.stream()
                    .map(ar -> JSONObject.parseObject(ar, PositionInfo.class))
                    .filter(ar -> ar.getTimestamp() > DateUtils.addHours(new Date(), -1).getTime())
                    .collect(Collectors.toList());
            // 获取每个用户最近一次的记录
            List<PositionInfo> result = new ArrayList<>(allInfos.stream()
                    .collect(Collectors.toMap(PositionInfo::getUserId, Function.identity(),
                            BinaryOperator.maxBy(Comparator.comparing(PositionInfo::getTimestamp))))
                    .values());
            return success(result);
        })
        .orElse(success("暂无数据"));

2. 使用Lambda表达式简化代码

可以使用Lambda表达式简化代码，如下：

List<String> allRows = this.getHBaseRows();
return Optional.ofNullable(allRows)
        .filter(CollectionUtils::isNotEmpty)
        .map(rows -> rows.stream()
                .map(ar -> JSONObject.parseObject(ar, PositionInfo.class))
                .filter(ar -> ar.getTimestamp() > DateUtils.addHours(new Date(), -1).getTime())
                .collect(Collectors.groupingBy(PositionInfo::getUserId,
                        Collectors.collectingAndThen(Collectors.maxBy(Comparator.comparing(PositionInfo::getTimestamp)),
                                Optional::get)))
                .values())
        .map(ArrayList::new)
        .map(this::success)
        .orElse(success("暂无数据"));

这段代码使用了groupingBy方法将数据按照userId分组，然后使用collectingAndThen方法将每组数据取最大的timestamp，最后使用values方法获取所有分组的结果。