kotlin - Kotlin空安全容器”是一种可变属性，该属性可能已在此时更改

Question

class ExchangeRatesServiceImpl : ExchangeRatesService {

    private var container: ExchangeRatesContainer? = null

    /**
     * {@inheritDoc}
     */
    override val currentRates: Map<Currency, BigDecimal>
        get() {
            if (container == null || container.date != LocalDate.now()) {
                container = client.getRates(Currency.getBase())
                        log.info("exchange rates has been updated: {}", container)
            }
            return ImmutableMap.of<Currency, BigDecimal>(
                    Currency.EUR, container.rates[Currency.EUR.name],
                    Currency.RUB, container.rates[Currency.RUB.name],
                    Currency.USD, BigDecimal.ONE
            )
        }
}

如果(容器== null || container.date!= LocalDate.now())行
错误【因为“容器”是可变属性，而这一次可能已经更改了)

我该如何编译？

Answer 1

container在函数范围之外声明，因此从理论上讲，在检查null然后使用它之间，另一个线程可以修改它。为了防止这种情况，您应该创建一个初始化为container的局部变量，并像这样使用它:

class ExchangeRatesServiceImpl : ExchangeRatesService {

private var container: ExchangeRatesContainer? = null

/**
 * {@inheritDoc}
 */
override val currentRates: Map<Currency, BigDecimal>
    get() {
        val tempContainer = container
        if (tempContainer == null || tempContainer.date != LocalDate.now()) {
            tempContainer = client.getRates(Currency.getBase())
                    log.info("exchange rates has been updated: {}", tempContainer)
        }
        return ImmutableMap.of<Currency, BigDecimal>(
                Currency.EUR, container.rates[Currency.EUR.name],
                Currency.RUB, container.rates[Currency.RUB.name],
                Currency.USD, BigDecimal.ONE
        )
    }
}