Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Illegal forward reference error for static final fields

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?

like image 518
Feuermurmel Avatar asked Nov 22 '11 22:11

Feuermurmel


2 Answers

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;
    }   
});
like image 129
Adam Zalcman Avatar answered Sep 19 '22 11:09

Adam Zalcman


The Java Language Specifications specifically mentions restrictions on object fields during the initializations phase, specifically (C is an interface or class):

A compile-time error for forward references occurs when these conditions are fulfilled:

  • The usage occurs in an instance (respectively static) variable initializer of C or in an instance (respectively static) initializer of C.
  • The usage is not on the left hand side of an assignment.
  • The usage is via a simple name.
  • C is the innermost class or interface enclosing the usage.

The article What are the forward reference rules? contains an excellent explanation of the rules and the restrictions when it comes to initializing members and forward referencing.

like image 2
Mechkov Avatar answered Sep 19 '22 11:09

Mechkov