N'est pas une classe englobante Java

J'essaie de créer un jeu Tetris et j'obtiens l'erreur du compilateur

Shape is not an enclosing class

quand j'essaye de créer un objet

public class Test {
    public static void main(String[] args) {
        Shape s = new Shapes.ZShape();
    }
}

J'utilise des classes internes pour chaque forme. Voici une partie de mon code

public class Shapes {
    class AShape {
    }
    class ZShape {
    }
}

Qu'est-ce que je fais mal ?

ZShape n'est pas statique, il nécessite donc une instance de la classe externe.

La solution la plus simple consiste à créer ZShape et toute classe imbriquée staticsi vous le pouvez.

Je ferais aussi des champs finalou static finalque vous pouvez aussi.

Supposons que RetailerProfileModel est votre classe principale et RetailerPaymentModel est une classe interne à l'intérieur. Vous pouvez créer un objet de la classe Inner en dehors de la classe comme suit:

RetailerProfileModel.RetailerPaymentModel paymentModel
        = new RetailerProfileModel().new RetailerPaymentModel();

Ce que je suggérerais n'est pas de convertir la classe non statique en une classe statique car dans ce cas, votre classe interne ne peut pas accéder aux membres non statiques de la classe externe.

Exemple :

class Outer
{
    class Inner
    {
        //...
    }
}

Donc, dans ce cas, vous pouvez faire quelque chose comme:

Outer o = new Outer();
Outer.Inner obj = o.new Inner();

Comme indiqué dans les documents :

OuterClass.InnerClass innerObject = outerObject.new InnerClass();

Parfois, nous devons créer une nouvelle instance d'une classe interne qui ne peut pas être statique car elle dépend de certaines variables globales de la classe parente. Dans cette situation, si vous essayez de créer l'instance d'une classe interne qui n'est pas statique, une not an enclosing classerreur est levée.

En prenant l'exemple de la question, que faire si elle ZShapene peut pas être statique car elle a besoin d'une variable globale de Shapeclasse?

Comment pouvez-vous créer une nouvelle instance de ZShape? C'est ainsi:

Ajoutez un getter dans la classe parent:

public ZShape getNewZShape() {
    return new ZShape();
}

Accédez-y comme ceci:

Shape ss = new Shape();
ZShape s = ss.getNewZShape();

Shape shape = new Shape();
Shape.ZShape zshape = shape.new ZShape();

J'ai rencontré le même problème. J'ai résolu en créant une instance pour chaque classe publique interne. Quant à votre situation, je vous suggère d'utiliser un héritage autre que les classes internes.

public class Shape {

    private String shape;

    public ZShape zShpae;
    public SShape sShape;

    public Shape(){
      int[][] coords =  noShapeCoords;
      shape = "NoShape";
      zShape = new ZShape();
      sShape = new SShape();
    }

    class ZShape{
      int[][] coords =  zShapeCoords;
      String shape = "ZShape";
    }

    class SShape{
      int[][] coords = sShapeCoords;
      String shape = "SShape";
    }

 //etc
}

alors vous pouvez nouveau Shape (); et visitez ZShape à travers shape.zShape;

Pas besoin de rendre la classe imbriquée statique mais elle doit être publique

public class Test {
    public static void main(String[] args) {
        Shape shape = new Shape();
        Shape s = shape.new Shape.ZShape();
    }
}

Une chose que je n'avais pas réalisé au début en lisant la réponse acceptée était que rendre une classe interne statique est fondamentalement la même chose que la déplacer vers sa propre classe séparée.

Ainsi, lors de l'obtention de l'erreur

xxx n'est pas une classe englobante

Vous pouvez le résoudre de l'une des manières suivantes:

• Ajoutez le staticmot clé à la classe interne, ou
• Déplacez-le dans sa propre classe distincte.

Dans le cas où la classe Parent est singleton, utilisez la méthode suivante:

Parent.Child childObject = (Parent.getInstance()).new Child();

où getInstance()renverra l'objet singleton de la classe parent.

Pour atteindre l'exigence de la question, nous pouvons mettre les classes en interface:

public interface Shapes {
    class AShape{
    }
    class ZShape{
    }
}

puis utiliser comme auteur essayé auparavant:

public class Test {
    public static void main(String[] args) {
        Shape s = new Shapes.ZShape();
    }
}

Si nous recherchons la solution "logique" appropriée, il faut utiliser le fabricmodèle de conception