在工作中遇到需要对某些参数进行业务逻辑过滤处理,在原方法中过滤成功。将业务过滤代码抽成公共方法后,过滤失败。当时情况如下:
原代码:
public static void main(String[] args) {
Boolean filter = true;
List<Integer> list = new ArrayList<>();
list.add(1);
list.add(2);
list.add(3);
list.add(4);
if (filter) {
List<Integer> collect = list.stream().filter(value -> value > 2).collect(Collectors.toList());
list = collect;
}else {
List<Integer> collect = list.stream().filter(value -> value < 2).collect(Collectors.toList());
list = collect;
}
System.out.println(list);
}
抽出业务代码为公共方法后:
public static void main(String[] args) {
Boolean filter = true;
List<Integer> test = new ArrayList<>();
doFilter(filter, test);
System.out.println(test);
}
private static void doFilter(Boolean filter, List<Integer> list) {
list.add(1);
list.add(2);
list.add(3);
list.add(4);
if (filter) {
List<Integer> collect = list.stream().filter(value -> value > 2).collect(Collectors.toList());
list = collect;
}else {
List<Integer> collect = list.stream().filter(value -> value < 2).collect(Collectors.toList());
list = collect;
}
}
如上所示,将业务抽成方法后过滤逻辑失效了,这个问题原因涉及到指针和java的引用传递。
mian 函数中的test 指向内存地址addr1,调用doFilter方法时test作为参数传进去,list参数也指向了addr1。在if 判断中,list 过滤出一个新的集合collect ,这时collect 的地址为addr2 ,执行list = collect 后,参数list 的值变成了我们理想中的值,但list 指向的地址为addr2,也就是说我们实际想要的值地址是addr2。而main 函数中test 的地址为addr1 。
要避免这种情况,就需要将list 参数作为函数返回值返回,将main 函数中的test 地址指向addr2。
public static void main(String[] args) {
Boolean filter = true;
List<Integer> test = new ArrayList<>();
test = doFilter(filter, test);
System.out.println(test);
}
private static List<Integer> doFilter(Boolean filter, List<Integer> list) {
list.add(1);
list.add(2);
list.add(3);
list.add(4);
if (filter) {
List<Integer> collect = list.stream().filter(value -> value > 2).collect(Collectors.toList());
list = collect;
}else {
List<Integer> collect = list.stream().filter(value -> value < 2).collect(Collectors.toList());
list = collect;
}
return list;
}