所以我想让它在这两个形状相互接触的地方关闭窗口。这是第一部分

public class Mayflower {

JFrame f = new JFrame();

public static void main(String[] args) {

    Mayflower bob = new Mayflower();
    bob.Start();

}

private void Start(int clothes, int food, int repair, int money) {
    int complete = 0;
    Mayflower bob = new Mayflower();
    //JOptionPane.showMessageDialog(null, "Your equipment:\nClothes - " + clothes + "\nFood - " + food + "\nrepair equipment - " + repair + "\nMoney left - $" + money);
    bob.epic(complete);
}

public void epic(int complete) {


    if (complete == 0){
    Iceberg Tim = new Iceberg();

    f.add(Tim);
    f.setVisible(true);
    f.setSize(600, 600);
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.setTitle("SAILIN BABEEEEY");
    f.setLocation(600, 200);

    }

    if(complete == 1){
        System.out.println("odeyladoeijoo");
        f.dispose();
     }


}

}


然后它调用迷你游戏所在的构造函数冰山,我删除了所有与动作无关的动作输入:

package mayflower;


public class Iceberg extends JPanel implements ActionListener, KeyListener {

Timer time = new Timer(5, this);
int x = 260;
int y = 500;
int velx = 0;
int vely = 0;

int hitscany = -4000;
int hitscanvely = -1;


public Iceberg() {

    time.start();
    addKeyListener(this);
    setFocusable(true);
    setFocusTraversalKeysEnabled(false);

}

@Override
public void paintComponent(Graphics g) {

    super.paintComponent(g);
    Graphics2D g2 = (Graphics2D) g;

g.setColor(MyColor1);
    g.fillRect(x, y, 40, 60);

    g.setColor(Color.GRAY);
    g.fillRect(0, hitscany, 650, 0);

    if (y == hitscany) {
        int complete = 1;
        Mayflower bob = new Mayflower();
        bob.epic(complete);

    }

    time.start();

}


因此,我到达了“ hitscan”对象在屏幕上向下移动的位置,当它触摸该对象时,该窗口应该关闭。当我的if语句(因为两个对象的y坐标是否相等)调用公共空空史诗时,它应该“激活” if complete的if语句== 1并处理框架,但是由于某种原因,它不会“ Ť

最佳答案

因此,我认为,此(以前已被删除)的代码在您的Iceberg密钥处理程序代码中可以找到位置...

if ((((x - icex)) >= -40 && ((x - icex) - 180) <= -130) && (((y - icey)) >= -60 && ((y - icey) - 180) <= -130)) {
    int complete = 1;
    Mayflower bob = new Mayflower();
    bob.epic(complete);
}


这突出了许多问题。首先,您要创建Mayflower的另一个实例,这是JFrame的另一个实例,这是要处理的内容,而不是原始帧。

Iceberg确实不需要与Mayflower进行交互,这超出了责任范围。取而代之的是,“ Iceberg”应该向有关方面生成有关其状态更改的事件通知。

为此,我们需要一个观察者模式!

让我们从一个简单的interface开始,它描述Iceberg愿意发出的所有通知...

public interface GameListener {
    public void completed(Iceberg berg);
}


接下来,我们需要某种方法来管理Iceberg中的这些侦听器...

public class Iceberg extends JPanel implements ActionListener, KeyListener {

    private List<GameListener> listeners = new ArrayList<>(25);


    public void addGameListener(GameListener listener) {
        listeners.add(listener);
    }

    public void removeGameListener(GameListener listener) {
        listeners.remove(listener);
    }


最后,是一种生成通知的方法...

public class Iceberg extends JPanel implements ActionListener, KeyListener {
    //...
    protected void fireCompleted() {
        for (GameListener listener : listeners) {
            listener.completed(this);
        }
    }


现在,当您处于“完成”状态时,您可以通知感兴趣的各方...

if ((((x - icex)) >= -40 && ((x - icex) - 180) <= -130) && (((y - icey)) >= -60 && ((y - icey) - 180) <= -130)) {
    fireCompleted();
}


现在,在您的start方法中,您只需要创建一个Iceberg实例,注册一个GameListener并将其全部启动...

private void Start(int clothes, int food, int repair, int money) {
    Iceberg Tim = new Iceberg();
    Tim.addGameListener(new GameListener() {
        @Override
        public void completed(Iceberg berg) {
            f.dispose();
        }
    });

    f.add(Tim);
    f.setVisible(true);
    f.setSize(600, 600);
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.setTitle("SAILIN BABEEEEY");
    f.setLocation(600, 200);
}


观察...

好的,您的代码示例有很多值得担心的地方,但让我们从...开始

@Override
public void paintComponent(Graphics g) {

    super.paintComponent(g);
    Graphics2D g2 = (Graphics2D) g;

    g.setColor(Color.RED);
    g.fillRect(x, y, 40, 60);

    g.setColor(Color.GRAY);
    g.fillRect(0, hitscany, 650, 0);

    if (y == hitscany) {
        int complete = 1;
        Mayflower bob = new Mayflower();
        bob.epic(complete);

    }

    time.start();

}



paintComponent绝对不能为public,没有人可以直接调用它。
您声明但不使用g2


这个...

if (y == hitscany) {
    int complete = 1;
    Mayflower bob = new Mayflower();
    bob.epic(complete);

}


在许多层面上都是个坏主意。绘制应该绘制组件的当前状态,别无其他,它不应该对组件的状态做出决定。这属于你的主循环

和...

time.start();


我无法开始告诉你这有多可怕。 paintComponent将经常被调用(如果您正在执行动画),这意味着您将不断重置TimerTimer的状态永远不能在paintComponent内部修改。相反,它应该通过其他方式进行控制,例如构造函数或启动/停止方法

如今,KeyListener是一个糟糕的选择。它遭受了许多众所周知且有据可查的短期缺陷。更好的全面解决方案是使用Key Bindings API,它旨在以可靠且可靠的方式帮助解决这些问题。

08-25 02:24