I'm trying to compile a Java class which javac
rejects with an illegal forward reference error, where the offending reference is lexically after the referenced field. The following class is stripped down as much as possible while showing the same behavior:
java.util.concurrent.Callable
and the many uses of Object
are just used as placeholders to remove irrelevant pieces of code.
public class Test {
static final Object foo = method(new java.util.concurrent.Callable<Object>() {
@Override
public Object call() throws Exception {
return bar;
}
});
static final Object bar = foo;
static Object method(Object binder) {
return null;
}
}
When compiled using javac Test.java
, javac prints the following error message:
Test.java:9: illegal forward reference
static final Object bar = foo;
^
So the compiler complains about bar
's declaration referencing foo
while foo
should be in the scope of bar
's declaration. But as soon as the reference of bar
in foo
's declaration is removed, e.g. by changing line 5 from return bar;
to return null;
, the class is accepted by the compiler.
How can this be explained? Is my understanding of forward as meaning lexically after wrong or is this some special case I'm not aware of?
Your understanding of forward reference is correct. The reference to foo
on line 9 isn't a forward reference at all since it doesn't appear textually before its declaration (see the definition of what constitutes a forward reference in section 8.3.2.3 of The Java Language Specification).
The behavior you observe is a symptom of a javac bug. See this bug report. The problem appears to be fixed in newer versions of the compiler, e.g. OpenJDK 7.
It only affects forward references used as initializers to final fields. The issue appears to affect static and non-static fields equally.
Note that the reference to bar
in call()
is a legal forward reference since it occurs inside a different class (see examples in section 8.3.2.3 of The Java Language Specification).
Also, note that each of the following alterations make the error disappear:
Making bar
non-final:
static Object bar = foo;
Initializing bar
in static or instance initializer block:
static final Object bar;
static {
bar = foo;
}
Moving the initialization of foo
to an initializer block also helps.
Initializing bar
from a non-final temporary reference to foo
:
static Object tmp = foo;
static final Object bar = tmp;
Initializing bar
with Test.foo
(found by Tom Anderson) or with this.foo
in non-static case:
static final Object bar = Test.foo;
Removing bar
and referring to the object using foo
inside call()
:
static final Object foo = method(new java.util.concurrent.Callable<Object>() {
@Override
public Object call() throws Exception {
return foo;
}
});