通常需要对查询的结果进行转换,如:
select category, count(*)
from table
group by category到一个映射,其中键是类别,值是属于同一类别的记录的计数。
许多持久性框架返回像List<Object[]>这样的查询的结果,其中对象数组包含两个元素(类别和每个返回的结果集行的计数)。
我试图找到最易读的方法来将这个列表转换成相应的地图。
当然,传统的方法包括创建地图和手动放置条目:
Map<String, Integer> map = new HashMap<>();
list.stream().forEach(e -> map.put((String) e[0], (Integer) e[1]));我想到的第一个问题是利用开箱即用的Collectors.toMap收集器:
Map<String, Integer> map = list.stream().collect(toMap(e -> (String) e[0], e -> (Integer) e[1]));但是,我发现这种e -> (T) e[i]语法的可读性比传统方法要差一些。为了克服这一问题,我可以创建一个util方法,在所有这种情况下都可以重用它:
public static <K, V> Collector<Object[], ?, Map<K, V>> toMap() {
return Collectors.toMap(e -> (K) e[0], e -> (V) e[1]);
}那我就有了一条完美的衬垫:
Map<String, Integer> map = list.stream().collect(Utils.toMap());由于类型推断,甚至不需要转换键和值。但是,对于代码的其他读者(在util方法签名中的Collector<Object[], ?, Map<K, V>>,等等)来说,这有点难理解。
我想知道,在java 8工具箱中还有什么可以帮助以更易读/更优雅的方式实现这一点吗?
发布于 2016-02-28 22:55:41
我觉得你现在的“一条线”还行。但是,如果您不特别喜欢命令中内置的神奇索引,则可以将其封装在枚举中:
enum Column {
CATEGORY(0),
COUNT(1);
private final int index;
Column(int index) {
this.index = index;
}
public int getIntValue(Object[] row) {
return (int)row[index]);
}
public String getStringValue(Object[] row) {
return (String)row[index];
}
}那么您的提取代码就会变得更清晰一些:
list.stream().collect(Collectors.toMap(CATEGORY::getStringValue, COUNT::getIntValue));理想情况下,您应该向列中添加一个类型字段,并检查调用了正确的方法。
在问题范围之外,理想情况下,您应该创建一个类,表示封装查询的行。如下所示(为了清晰起见,跳过getter):
class CategoryCount {
private static final String QUERY = "
select category, count(*)
from table
group by category";
private final String category;
private final int count;
public static Stream<CategoryCount> getAllCategoryCounts() {
list<Object[]> results = runQuery(QUERY);
return Arrays.stream(results).map(CategoryCount::new);
}
private CategoryCount(Object[] row) {
category = (String)row[0];
count = (int)row[1];
}
}这将查询和行解码之间的依赖关系放在同一个类中,并向用户隐藏所有不必要的详细信息。
然后创建您的地图将变成:
Map<String,Integer> categoryCountMap = CategoryCount.getAllCategoryCounts()
.collect(Collectors.toMap(CategoryCount::getCategory, CategoryCount::getCount));发布于 2016-03-12 21:41:26
我不需要隐藏类强制转换,而是创建几个函数来帮助提高可读性:
Map<String, Integer> map = results.stream()
.collect(toMap(
columnToObject(0, String.class),
columnToObject(1, Integer.class)
));完整的例子:
package com.bluecatcode.learning.so;
import com.google.common.collect.ImmutableList;
import java.util.List;
import java.util.Map;
import java.util.function.Function;
import static java.lang.String.format;
import static java.util.stream.Collectors.toMap;
public class Q35689206 {
public static void main(String[] args) {
List<Object[]> results = ImmutableList.of(
new Object[]{"test", 1}
);
Map<String, Integer> map = results.stream()
.collect(toMap(
columnToObject(0, String.class),
columnToObject(1, Integer.class)
));
System.out.println("map = " + map);
}
private static <T> Function<Object[], T> columnToObject(int index, Class<T> type) {
return e -> asInstanceOf(type, e[index]);
}
private static <T> T asInstanceOf(Class<T> type, Object object) throws ClassCastException {
if (type.isAssignableFrom(type)) {
return type.cast(object);
}
throw new ClassCastException(format("Cannot cast object of type '%s' to '%s'",
object.getClass().getCanonicalName(), type.getCanonicalName()));
}
}https://stackoverflow.com/questions/35689206
复制相似问题