Non è una classe acclusa Java

Sto cercando di creare un gioco Tetris e sto ricevendo l'errore del compilatore

Shape is not an enclosing class

quando provo a creare un oggetto

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

Sto usando le classi interne per ogni forma. Ecco parte del mio codice

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

Che cosa sto facendo di sbagliato ?

ZShape non è statico, quindi richiede un'istanza della classe esterna.

La soluzione più semplice è creare ZShape e qualsiasi classe nidificata, staticse possibile.

Vorrei anche fare qualsiasi campo finalo static finalche tu possa pure.

Supponiamo che RetailerProfileModel sia la tua classe principale e RetailerPaymentModel sia una classe interna al suo interno. È possibile creare un oggetto della classe Inner all'esterno della classe come segue:

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

Quello che suggerirei non è convertire la classe non statica in una classe statica perché in tal caso, la tua classe interna non può accedere ai membri non statici della classe esterna.

Esempio :

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

Quindi, in tal caso, puoi fare qualcosa del tipo:

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

Come indicato nei documenti :

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

A volte, dobbiamo creare una nuova istanza di una classe interna che non può essere statica perché dipende da alcune variabili globali della classe genitore. In tale situazione, se si tenta di creare l'istanza di una classe interna non statica, not an enclosing classviene generato un errore.

Prendendo l'esempio della domanda, cosa succede se ZShapenon può essere statico perché necessita di una variabile di Shapeclasse globale ?

Come puoi creare una nuova istanza di ZShape? Questo è come:

Aggiungi un getter nella classe genitore:

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

Accedilo in questo modo:

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

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

Ho riscontrato lo stesso problema. Ho risolto creando un'istanza per ogni classe pubblica interna. per quanto riguarda la tua situazione, ti suggerisco di usare l'eredità diversa dalle classi interne.

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
}

allora puoi new Shape (); e visita ZShape attraverso shape.zShape;

Non è necessario rendere la classe nidificata come statica ma deve essere pubblica

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

Una cosa che all'inizio non ho capito quando ho letto la risposta accettata è che rendere statica una classe interna è praticamente la stessa cosa che spostarla in una classe separata.

Pertanto, quando si ottiene l'errore

xxx non è una classe chiusa

Puoi risolverlo in uno dei seguenti modi:

• Aggiungi la staticparola chiave alla classe interna o
• Spostalo nella sua classe separata.

Nel caso in cui la classe Parent sia singleton, utilizzare la seguente modalità:

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

dove getInstance()restituirà l'oggetto singleton della classe genitore.

Per raggiungere il requisito dalla domanda, possiamo mettere le classi in interfaccia:

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

e quindi utilizzare come autore provato prima:

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

Se stiamo cercando la corretta soluzione "logica", dovrebbe essere utilizzato il fabricmodello di progettazione