Ostrzeżenie Eclipse na temat syntetycznego akcesorium dla prywatnych statycznych klas zagnieżdżonych w Javie?

Question

Mój współpracownik zasugerował, aby kilka z ustawień formatowania i ostrzeżeń Eclipse było bardziej rygorystycznych. Większość tych zmian ma sens, ale dostaję to jedno dziwne ostrzeżenie w Javie. Oto kod testowy do odtworzenia "problemu":

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  

Linia z !!! Daje mi to ostrzeżenie w Eclipse z moimi nowymi ustawieniami Ostrzeżenie:

Dostęp do załączając konstruktora WeirdInnerClassJavaWarning.InnerClass() jest emulowane przez syntetyczną accessor metody. Zwiększenie jego widoczności zwiększy Twoją wydajność.

Co to oznacza? Ostrzeżenie zniknie, gdy zmienię "prywatną klasę statyczną" na "chronioną klasę statyczną", co nie ma dla mnie żadnego sensu.

---

edit: w końcu zorientowali się, "prawidłowy" poprawkę. Prawdziwym problemem wydaje się być to, że tej zagnieżdżonej prywatnej klasie statycznej brakuje publicznego konstruktora. Że jeden uszczypnąć usuwane ostrzeżenie:

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(); 
    } 
} 

chcę klasa być prywatny klasy zagnieżdżone (więc żadna inna klasa może mieć do niego dostęp, w tym podklas klasy okalającego) i chcę go mieć statyczny klasa.

Nadal nie rozumiem, dlaczego tworzenie zagnieżdżonej klasy chronionej, a nie prywatnej, jest inną metodą naprawy "problemu", ale być może jest to dziwactwo/błąd Eclipse.

(przepraszam, powinienem nazwali to NestedClass zamiast InnerClass być bardziej jasne.)

Answer 1

można pozbyć się ostrzeżenia, co następuje:

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(); 
    } 
} 

Jak mówili inni, Eclipse narzeka ponieważ prywatny klasa bez wyraźnej konstruktor nie może być instancja z zewnątrz, z wyjątkiem metodą syntetyczną, że Kompilator Java tworzy.Jeśli wziąć swój kod, skompilować go, a następnie dekompilować go jad (*), pojawi się następujący (sformatowany):

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); 
} 

jeśli dodać zabezpieczony konstruktora, kod syntetyczny jest niepotrzebna. Syntetyczny kod teoretycznie jest, moim zdaniem, wolniejszy o wiele mniejszy od niesyntetycznego kodu przy użyciu publicznego lub chronionego konstruktora.

(*) Dla Jad, połączyłem się ze stroną w Wikipedii ... Domena, która hostowała ten program wygasła, ale Wikipedia łączy się z inną, której sam nie przetestowałem. Wiem, że są inne (prawdopodobnie nowsze) dekompilatory, ale to jest ten, którego zacząłem używać. Uwaga: To narzeka podczas dekompilacji ostatnich plików klasy Java, ale nadal robi dobrą robotę.

Answer 2

Nie można utworzyć wystąpienia InnerClass z WeirdInnerClassJavaWarning. Jest prywatny, JVM nie pozwalał ci, ale język Java (z jakiegoś powodu) by.

Dlatego też javac wykona dodatkową metodę w InnerClass, która właśnie zwróci nową InnerClass(), umożliwiając tworzenie instancji InnerClass z WeirdInnerClassJavaWarning.

Nie sądzę, że naprawdę trzeba się go pozbyć, ponieważ spadek wydajności byłby niewspółmiernie mały. Możesz jednak, jeśli naprawdę chcesz.

Answer 3

powinien być w stanie pozbyć się go za pomocą domyślnego zakresu zamiast prywatne lub chronione, tj

static class InnerClass ... 

Warto również zauważyć, że mój umieszczając kursor na linii kodu z ostrzeżeniem i naciśnięcie ctrl-1, Eclipse może być w stanie naprawić to automatycznie dla ciebie.

Answer 4

ja nadal nie rozumiem, dlaczego co zagnieżdżony klasy chronione niż prywatny jest inna metoda ustalania „problem”, ale być może jest to dziwactwo/bug Eclipse

To nie jest dziwactwo/błąd Eclipse, tylko cecha Javy. Java Language Specification, 8.8.9 mówi:

... jeśli klasa jest zadeklarowana chronione, wówczas konstruktor domyślny jest niejawnie otrzymują modyfikator dostępu zabezpieczonego ...

Answer 5

Nawiasem mówiąc, ustawienie, aby włączyć ostrzeganie off jest na stronie Błędy/ostrzeżenia Java pod „styl Code” i nazywa się:

dostęp do niedostępnej członek typu otaczającej

Answer 6

Aby pomóc ludzie się tutaj jest to, co masz, jeśli używasz oryginalnego kodu klasy w pytaniu z

javac -XD-printflat WeirdInnerClassJavaWarning.java -d tmp 

wyjście Raw, kompilator dodaje komentarze. Zwróć uwagę na dodanie klasy prywatnej i konstruktora pakietu syntetycznego.

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 { 
}