[
https://issues.apache.org/jira/browse/GROOVY-11792?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18034734#comment-18034734
]
Eric Milles commented on GROOVY-11792:
--------------------------------------
A closure-shared variable -- "n" in this case -- is a Reference. So when your
suppliers are executed the value of the reference is 3. The local capture
variable is new for each loop, so the value is not changed for each supplier
that uses it. I doubt this is a trivial thing to change and is likely breaking
for someone.
> Incorrect captured variable in closure
> --------------------------------------
>
> Key: GROOVY-11792
> URL: https://issues.apache.org/jira/browse/GROOVY-11792
> Project: Groovy
> Issue Type: Bug
> Affects Versions: 2.5.23, 3.0.25, 4.0.29, 5.0.2
> Reporter: Daniel Sun
> Priority: Major
>
> Expect 14(1+4+9), but get 27(9+9+9)
> {code:java}
> import java.util.function.Supplier
> def numbers = [1, 2, 3]
> List suppliers = []
> for (n in numbers) {
> println "v: ${n}"
> Supplier s = {
> def r = n * n
> println "$n: $r"
> return r
> }
> suppliers << s
> }
> println "Result: ${suppliers.sum({e -> e.get()})}"
> {code}
> yields:
> {code:java}
> v: 1
> v: 2
> v: 3
> 3: 9
> 3: 9
> 3: 9
> Result: 27
> {code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
