JAVA8 lambdas表达式 改变外面变量的值

Question

如图，我在外部定义了一个etotalPrice，然后再两个for循环里想要改变这个值，结果给我报错了，该怎么解决？
(NumberUtil.add和mutiplyu是基础的+*保留2位小数的方法)

Answer 1

在Java的经典著作《Effective Java》、《Java Concurrency in Practice》里，大神们都提到：匿名函数里的变量引用，也叫做变量引用泄露，会导致线程安全问题，因此在Java8之前，如果在匿名类内部引用函数局部变量，必须将其声明为final，即不可变对象。

Java8这里加了一个语法糖：在lambda表达式以及匿名类内部，如果引用某局部变量，则直接将其视为final。

建议你重构一下这段代码：使用lambda返回一个值，赋给外面的变量。

Answer 2

【更新】忘记回答原始的问题了。
如果要在Lambdas里更改外部变量，一般有两种方法：

1. 定义一个长度为1的数组

final double[] totalPrice = new double[];
// lambdas 块：
{
    totalPrice[0] += // TODO
}

1. 定义Holder类，或着用MutableDouble

public class Holder<T> {
    private T value;
    
    Holder(T value) {
        this.value = value;
    }
    
    public static <T> Holder<T> of(T value) {
        return new Holder<>(value);
    }
    
    public T getValue() {
        return value;
    }
    
    public void setValue(T value) {
        this.value = value;
    }
    
    // ... TODO
}

如果我没有搞错的话，你问题代码用lambda更标准的写法是：

puchasePrices.filter(skuVo -> skuVo.getCode() == code).mapToDouble(skuVo -> count * skuVo.getPurchasePrice()).sum();

完整的写法

Map<String, Integer> codeMap = ...;
codeMap.entrySet().stream().mapToDouble(e -> puchasePrices.filter(skuVo -> skuVo.getCode() == e.getKey()).mapToDouble(skuVo -> e.getValue() * skuVo.getPurchasePrice()).sum()).sum();

一种可能更好的写法（再一次，因为没有看到你全部的代码，我只能猜测）

puchasePrices.filter(skuVo -> codeMap.containsKey(skuVo.getCode)).mapToDouble(skuVo -> codeMap.get(skuVo.getCode()) * skuVo.getPurchasePrice()).sum();

或者：

puchasePrices.mapToDouble(skuVo -> codeMap.getOrDefault(skuVo.getCode(), 0) * skuVo.getPurchasePrice()).sum();

建议你在原问题里粘贴代码，图片别人没法拷贝。
顺便再提一个建议，根据你贴出来的代码，puchasePrices在我看来好像该是orderItems什么之类，变量名是很重要，如果是我弄错了，请原谅。

Answer 3

意思是totalPrice在lambada表达式中，应该是final类型，final类型初始化后就不可以改变了，所以再次给totalPrice赋值会错误。所以应该重新定义一个变量保存新的值，而不是再次将值复制给totalPrice,如果改变变量不可以，就不要使用lambada表达式。

Answer 4

@泊浮目 已经将原因说得很清楚了，给你提供一个变相的解决方式。使用数组或者集合（不推荐，仅供参考）

public static void main(String[] args) {

        int[] sum = new int[1];

        List<Integer> nums = new ArrayList<Integer>(){{
            add(1);
            add(2);
        }};

        nums.stream().forEach(n -> sum[0]+=n);
    }

Answer 5

你这段代码是用函数式的接口写命令式的程序，那为什么不用for循环做呢？
如果采用函数式思路的话，大致写法如下

Double totalPrice = 
codeMap.map( (code, count) -> {
    return NumberUtil.multiply(count,getPrice(code),2);
}).reduce(0.0, (sum, eachPrice) -> { 
    return NumberUtil.add(sum, eachPrice, 2) 
});

Answer 6

你可以将你想要改变的值放到一个类的实例中做为属性，在 lambda 中修改实例中的属性:

PriceObj price=new PriceObj();

codeMap.forEach( (code, count) -> {
    price.totalPrice=xxxxx;
})

类似这样

Answer 7

final 的，当然不可变啦。如果必须要变就不要用lambda，用lambda就别修改值