No es una clase envolvente Java

Estoy tratando de hacer un juego de Tetris y obtengo el error del compilador

Shape is not an enclosing class

cuando trato de crear un objeto

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

Estoy usando clases internas para cada forma. Aquí hay parte de mi código

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

Qué estoy haciendo mal ?

ZShape no es estático, por lo que requiere una instancia de la clase externa.

La solución más simple es hacer ZShape y cualquier clase anidada staticsi puede.

También haría cualquier campo finalo static finalque tú también puedas.

Suponga que RetailerProfileModel es su clase Main y RetailerPaymentModel es una clase interna dentro de ella. Puede crear un objeto de la clase interna fuera de la clase de la siguiente manera:

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

Lo que sugeriría es no convertir la clase no estática en una clase estática porque en ese caso, su clase interna no puede acceder a los miembros no estáticos de la clase externa.

Ejemplo:

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

Entonces, en tal caso, puede hacer algo como:

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

Como se indica en los documentos :

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

A veces, necesitamos crear una nueva instancia de una clase interna que no pueda ser estática porque depende de algunas variables globales de la clase primaria. En esa situación, si intenta crear la instancia de una clase interna que no sea estática, se genera un not an enclosing classerror.

Tomando el ejemplo de la pregunta, ¿qué ZShapepasa si no puede ser estático porque necesita una variable global de Shapeclase?

¿Cómo puedes crear una nueva instancia de ZShape? Así es como:

Agregue un captador en la clase padre:

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

Acceda así:

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

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

He encontrado el mismo problema. Lo resolví creando una instancia para cada clase pública interna. En cuanto a su situación, le sugiero que use una herencia distinta de las clases internas.

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
}

entonces puedes nueva Forma (); y visite ZShape a través de shape.zShape;

No es necesario que la clase anidada sea estática, pero debe ser pública

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

Una cosa que no me di cuenta al principio al leer la respuesta aceptada fue que hacer que una clase interna sea estática es básicamente lo mismo que moverla a su propia clase separada.

Por lo tanto, al obtener el error

xxx no es una clase de cierre

Puede resolverlo de cualquiera de las siguientes maneras:

• Agregue la staticpalabra clave a la clase interna, o
• Muévelo a su propia clase separada.

En caso de que la clase Parent sea singleton, use la siguiente manera:

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

donde getInstance()devolverá el objeto singleton de la clase padre.

Para lograr el requisito de la pregunta, podemos poner clases en la interfaz:

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

y luego usar como autor probado antes:

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

Si buscamos la solución "lógica" adecuada, se debe utilizar el fabricpatrón de diseño