ไม่ใช่คลาส Java ที่ล้อมรอบ

ฉันกำลังพยายามสร้างเกม Tetris และฉันได้รับข้อผิดพลาดของคอมไพเลอร์

Shape is not an enclosing class

เมื่อฉันพยายามสร้างวัตถุ

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

ฉันใช้คลาสภายในสำหรับแต่ละรูปร่าง นี่คือส่วนหนึ่งของรหัสของฉัน

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

ผมทำอะไรผิดหรือเปล่า ?

ZShape ไม่คงที่ดังนั้นจึงต้องการอินสแตนซ์ของคลาสภายนอก

ทางออกที่ง่ายที่สุดคือการสร้าง ZShape และคลาสที่ซ้อนกันstaticหากคุณทำได้

ฉันจะทำฟิลด์ใดก็ได้finalหรือstatic finalคุณสามารถทำได้เช่นกัน

สมมติว่า RetailerProfileModel เป็นคลาสหลักของคุณและ RetailerPaymentModel เป็นคลาสภายในที่อยู่ภายใน คุณสามารถสร้างวัตถุของชั้นภายในภายนอกชั้นเรียนดังต่อไปนี้:

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

สิ่งที่ฉันอยากจะแนะนำไม่ได้แปลงคลาสที่ไม่คงที่ให้เป็นคลาสแบบคงที่เพราะในกรณีนี้คลาสภายในของคุณไม่สามารถเข้าถึงสมาชิกที่ไม่คงที่ของคลาสภายนอก

ตัวอย่าง:

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

ดังนั้นในกรณีเช่นนี้คุณสามารถทำสิ่งที่ชอบ:

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

ตามที่ระบุไว้ในเอกสาร :

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

บางครั้งเราจำเป็นต้องสร้างอินสแตนซ์ใหม่ของคลาสภายในที่ไม่สามารถคงที่ได้เพราะมันขึ้นอยู่กับตัวแปรส่วนกลางบางอย่างของคลาสพาเรนต์ ในสถานการณ์นั้นถ้าคุณพยายามสร้างอินสแตนซ์ของคลาสภายในที่ไม่คงที่not an enclosing classข้อผิดพลาดขึ้น

ยกตัวอย่างของคำถามจะทำอย่างไรถ้าZShapeไม่สามารถคงที่ได้เพราะมันต้องการตัวแปรShapeระดับโลก

คุณจะสร้างตัวอย่างใหม่ได้ZShapeอย่างไร นี่คือวิธี:

เพิ่ม getter ในคลาสพาเรนต์:

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

เข้าถึงแบบนี้:

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

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

ฉันพบปัญหาเดียวกัน ฉันแก้ไขโดยการสร้างอินสแตนซ์สำหรับทุกคลาสสาธารณะ สำหรับสถานการณ์ของคุณฉันขอแนะนำให้คุณใช้การสืบทอดอื่นที่ไม่ใช่คลาสภายใน

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
}

จากนั้นคุณสามารถรูปร่างใหม่ (); และเยี่ยมชม ZShape ผ่าน shape.zShape;

ไม่จำเป็นต้องทำให้คลาสที่ซ้อนกันเป็นแบบสแตติก แต่ต้องเป็นแบบสาธารณะ

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

สิ่งหนึ่งที่ฉันไม่ได้ตระหนักในตอนแรกเมื่ออ่านคำตอบที่ยอมรับได้คือการทำให้ชั้นในคงที่นั้นเป็นสิ่งเดียวกันกับการย้ายไปยังชั้นเรียนที่แยกต่างหาก

ดังนั้นเมื่อได้รับข้อผิดพลาด

xxx ไม่ใช่คลาสที่ล้อมรอบ

คุณสามารถแก้ไขได้ด้วยวิธีใดวิธีหนึ่งต่อไปนี้:

• เพิ่มstaticคำหลักให้กับคลาสภายในหรือ
• ย้ายออกไปยังคลาสแยกต่างหาก

ในกรณีที่คลาส Parent เป็น singleton ให้ใช้วิธีดังต่อไปนี้:

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

โดยที่getInstance()จะส่งคืนวัตถุระดับชั้นเดียวของวัตถุ

เพื่อให้บรรลุความต้องการจากคำถามเราสามารถนำคลาสเข้าสู่อินเตอร์เฟส:

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

จากนั้นใช้เป็นผู้เขียนทดลองก่อนหน้านี้:

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

หากเรากำลังมองหาวิธีการแก้ปัญหา "ตรรกะ" ที่เหมาะสมควรใช้fabricรูปแบบการออกแบบ