¿Advertencia de Eclipse sobre el acceso sintético para clases anidadas estáticas privadas en Java?

Question

Mi compañero de trabajo sugirió que se hicieran más rigurosos los ajustes de formato de código y advertencia de Eclipse. La mayoría de estos cambios tienen sentido, pero recibo esta rara advertencia en Java. Aquí hay algunos códigos de prueba para reproducir el "problema":

package com.example.bugs; 

public class WeirdInnerClassJavaWarning { 
    private static class InnerClass 
    { 
     public void doSomething() {} 
    } 

    final private InnerClass anInstance; 

    { 
     this.anInstance = new InnerClass(); // !!! 
     this.anInstance.doSomething(); 
    } 
} 
// using "this.anInstance" instead of "anInstance" prevents another warning, 
// Unqualified access to the field WeirdInnerClassJavaWarning.anInstance  

La línea con el !!! me da esta advertencia en Eclipse con mis nuevas opciones de advertencia:

acceso a encerrar constructor WeirdInnerClassJavaWarning.InnerClass() se emula mediante un método de síntesis de acceso . Al aumentar su visibilidad, mejorará su rendimiento.

¿Qué significa esto? La advertencia desaparece cuando cambio "clase estática privada" a "clase estática protegida", lo cual no tiene sentido para mí.

---

edición: que finalmente descubrió la solución "correcta". El verdadero problema aquí parece ser que a esta clase estática privada anidada le falta un constructor público. Que un pellizco retira la advertencia:

package com.example.bugs; 

public class WeirdInnerClassJavaWarning { 
    private static class InnerClass 
    { 
     public void doSomething() {} 
     public InnerClass() {} 
    } 

    final private InnerClass anInstance; 

    { 
     this.anInstance = new InnerClass(); 
     this.anInstance.doSomething(); 
    } 
} 

Quiero la clase que sea una clase anidada privada (por lo que ninguna otra clase puede tener acceso a ella, incluyendo las subclases de la clase envolvente) y yo quiero que sea un estático clase.

Todavía no entiendo por qué hacer que la clase anidada sea protegida en lugar de privada es otro método para solucionar el "problema", pero tal vez sea una peculiaridad o error de Eclipse.

(disculpas, debería haber llamado NestedClass en lugar de InnerClass a ser más clara.)

Answer 1

Usted puede deshacerse de la advertencia de la siguiente manera:

package com.example.bugs; 

public class WeirdInnerClassJavaWarning { 
    private static class InnerClass { 
     protected InnerClass() {} // This constructor makes the warning go away 
     public void doSomething() {} 
    } 

    final private InnerClass anInstance; 
    { 
     this.anInstance = new InnerClass(); 
     this.anInstance.doSomething(); 
    } 
} 

Como han dicho otros, Eclipse se quejaba de que una clase privada con ningún constructor explícito no se pueden crear instancias desde el exterior, excepto a través del método de síntesis que el El compilador de Java crea.Si se toma el código, compilarlo, y luego descompilarlo con jad (*), se obtienen los siguientes (reordenada):

public class Test { 
    private static class InnerClass { 
    public void doSomething() {} 
    // DEFAULT CONSTRUCTOR GENERATED BY COMPILER: 
    private InnerClass() {} 

    // SYNTHETIC METHOD GENERATED BY THE JAVA COMPILER:  
    InnerClass(InnerClass innerclass) { 
     this(); 
    } 
    } 

    public Test() { 
    anInstance.doSomething(); 
    } 

    // Your instance initialization as modified by the compiler: 
    private final InnerClass anInstance = new InnerClass(null); 
} 

Si añade un constructor protegido, el código sintético es innecesaria. El código sintético es teóricamente, supongo, más lento en una cantidad minúscula que el código no sintético que usa un constructor público o protegido.

(*) Para jad, he vinculado a una página de Wikipedia ... el dominio que alojó este programa ha expirado, pero la Wikipedia enlaza con otro que no he probado yo mismo. Sé que hay otros descompiladores (posiblemente más recientes), pero este es el que comencé a usar. Nota: Es se queja al descompilar archivos recientes de la clase Java, pero aún así hace un buen trabajo.

Answer 2

No se puede crear una instancia de InnerClass WeirdInnerClassJavaWarning. Es privado, JVM no te lo permitió, pero el lenguaje Java (por alguna razón) sí lo haría.

Por lo tanto, javac crearía un método adicional en InnerClass que simplemente devolvería una nueva InnerClass(), lo que le permitiría crear instancias de InnerClass desde WeirdInnerClassJavaWarning.

No creo que realmente deba deshacerse de él porque la caída del rendimiento sería muy pequeña. Sin embargo, puedes si realmente quieres.

Answer 3

Usted debe ser capaz de deshacerse de él utilizando el ámbito predeterminado en lugar de privada o protegida, es decir

static class InnerClass ... 

Es también digno de mención que mi colocando el cursor en la línea de código con la advertencia y Al presionar ctrl-1, Eclipse puede solucionarlo automáticamente.

Answer 4

Todavía no entiendo por qué hacer la clase anidada protegida en lugar de privada es otro método para fijar el "problema", pero tal vez eso es una peculiaridad/fallo de Eclipse

Eso no es una peculiaridad/error de Eclipse, solo una característica de Java. El Java Language Specification, 8.8.9 dice:

... si la clase se declara protegida, entonces el constructor por defecto se da implícitamente el modificador de acceso protegido ...

Answer 5

Por cierto, el ajuste para activar la alerta apagado está en la página de Java Errores/Advertencias bajo "estilo código" y se llama:

acceso a un miembro no accesible de un tipo envolvente

Answer 6

Para ayudar a la gente a cabo, esto es lo que se obtiene si se utiliza el código de la clase original en la pregunta con

javac -XD-printflat WeirdInnerClassJavaWarning.java -d tmp 

salida de crudo, compilador añade los comentarios. Tenga en cuenta la adición de la clase privada del paquete sintético y el constructor.

public class WeirdInnerClassJavaWarning { 
    { 
    } 

    public WeirdInnerClassJavaWarning() { 
     super(); 
    } 
    { 
    } 
    private final WeirdInnerClassJavaWarning$InnerClass anInstance; 
    { 
     this.anInstance = new WeirdInnerClassJavaWarning$InnerClass(null); 
     this.anInstance.doSomething(); 
    } 
} 

class WeirdInnerClassJavaWarning$InnerClass { 

    /*synthetic*/ WeirdInnerClassJavaWarning$InnerClass(WeirdInnerClassJavaWarning$1 x0) { 
     this(); 
    } 

    private WeirdInnerClassJavaWarning$InnerClass() { 
     super(); 
    } 

    public void doSomething() { 
    } 
} 

/*synthetic*/ class WeirdInnerClassJavaWarning$1 { 
}