Question

我目前正在制作一个地形生成器，一切都可以在一个课程中正常工作，但我将扩展我的应用程序。

目前我有一个JFrame类，可以保存所有内容，生成地形，绘制地形，查找位置等。

我想添加另一个将生成地形的类，但是当我创建这个类时，我需要访问主JFrame类中的字段，当我这样做时，我得到一个堆栈溢出错误 - 这是我的代码。

public class Simulator extends Applet
{
//fields

public Simulator()
{
    grid = new int[100][100];
    inhabGrid = new boolean[grid.length][grid.length];
    gridSize = grid.length - 1;
    dist = grid.length;
            TerrainGenerator gen = new TerrainGenerator();
    setSize(dist,dist);
    seedGrid();
    findInhabLocation();
    printGridToConsole();
}

public void paint(Graphics g)   
{
    //panting the grid
}

public void seedGrid()
{
    //seeding

}

public boolean generateTerrain(int x1,int y1, int x2, int y2) 
{

    //terrain generator
}

public boolean mouseUp(Event evt, int x, int y)
{
    seedGrid(); //Create a new map
    findInhabLocation();
    repaint();
    printGridToConsole();
    return true;
}

public boolean keyEvents(Event evt, int x, int y)
{
    seedGrid(); //Create a new map
    findInhabLocation();
    repaint();
    printGridToConsole();
    return true;
}

public void findInhabLocation()
{
    //find best inhabitant location
}

public int locateWater(int x, int y)
{

    //finding closest water
}

public int locateJungle(int x, int y)
{
    //finding closest jungle
}


}
}

在自己的类中工作正常，但是当我创建一个类时，例如：

public class TerrainGenerator 
{
Simulator sim = new Simulator();
}

我知道这与构造函数有关，而且我正在做的事情很愚蠢，将这个应用程序分成类的最佳方法是什么，例如地形生成器，居民等等。

例如，我希望能够从'TerrainGenerator'类调用一个方法并调用，即terrainGenerator.generateTerrain

Answer 1

您的TerrainGenerator创建了一个Simulator对象，反之亦然，因此您最终会得到无限多个对象（但在某些时候堆栈已满并且会引发堆栈溢出异常。。）

不应在Simulator中创建新的TerrainGenerator，而应该传递对当前Simulator的引用（实际上，这也不是一个好的设计，但是我'我不会把你与循环引用的问题混淆起来。）

Answer 2

赫斯特回答是正确的，此外，我认为您可以查看MVC来帮助您组织课程。

Answer 3

根据哪个应该是父级，您可以将实例化的类传递给另一个，即;

private final TerrainGenerator gen;  //if you need to save this.
public Simulator(TerrainGenerator terrainGenerator)
{

  this.gen = terrainGenerator;
 ....etc
}


public class TerrainGenerator 
{
Simulator sim = new Simulator(this);
}

或

private final TerrainGenerator gen;  //if you need to save this.
public Simulator()
{

  this.gen = new TerrainGenerator(this);
 ....etc
}

private final Simulator sim; //If you need to save it.
public class TerrainGenerator 
{
  public TerrainGenerator(Simulator simulator) {
    this.sim = simulator;
  }
}

从另一个类的JFrame调用方法

3 个答案: