Quel est le problème avec une classe interne n'utilisant pas une classe externe en Java?

Question

J'utilise un analyseur statique dans Eclipse pour examiner mon code. Une classe, foo, a une classe interne, un bar. J'obtiens l'erreur suivante:

JAVA0043 Inner class 'bar' does not use outer class 'foo' 

Pourquoi est-ce une erreur? Tant que la classe externe utilise la classe interne n'est pas suffisante pour rendre cette information cachée utile et correcte?

La classe interne n'est pas statique.

Answer 1

Si la classe interne ne peut jamais être utilisé par la classe externe, mais la classe interne ne nécessite aucune référence à la classe externe, vous pouvez le faire private static.

Si la classe interne est connue de quelqu'un d'autre que la classe externe, alors elle pourrait tout aussi bien être une classe de premier niveau.

Answer 2

S'il ne fait aucune référence à la classe externe, il peut tout aussi bien s'agir d'une classe régulière complète. Comme il ne dépend pas de la classe externe, il peut se suffire à lui-même. Je soupçonne que c'est la raison de "l'erreur".

Answer 3

Le point entier de la classe interne est qu'elle a accès à la classe externe. Si vous n'êtes pas réellement en utilisant la classe externe, faites-en une classe régulière et complète.

Answer 4

On dirait un Enerjy Error:

// Incorrect 
class Log { 
    // Position never uses the enclosing Log instance, 
    // so it should be static 
    class Position { 
    private int line; 
    private int column; 
    Position(int line, int column) { 
     this.line = line; 
     this.column = column; 
    } 
    } 
} 

Une classe imbriquée qui n'utilise pas toutes les variables d'instance ou des méthodes de l'une des ses classes extérieures peut être déclarée statique.
Cela réduit la dépendance entre les deux classes, ce qui améliore la lisibilité et la maintenance.

// Correct 
class Log { 
    static class Position { 
    private int line; 
    private int column; 
    Position(int line, int column) { 
     this.line = line; 
     this.column = column; 
    } 
    } 
} 

Answer 5

Une classe interne non statique possède une référence implicite à une instance de sa classe externe. Cette référence masquée peut retarder (voire empêcher) la récupération de place sur la classe externe et créer des problèmes de sérialisation. Vous ne devez donc utiliser que des classes internes non statiques lorsque vous en avez besoin. Il est facile d'oublier de déclarer la classe statique, donc l'analyse de code vous avertit quand elle n'est pas nécessaire.