所以我想让它在这两个形状相互接触的地方关闭窗口。这是第一部分
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
将经常被调用(如果您正在执行动画),这意味着您将不断重置Timer
。 Timer
的状态永远不能在paintComponent
内部修改。相反,它应该通过其他方式进行控制,例如构造函数或启动/停止方法如今,
KeyListener
是一个糟糕的选择。它遭受了许多众所周知且有据可查的短期缺陷。更好的全面解决方案是使用Key Bindings API,它旨在以可靠且可靠的方式帮助解决这些问题。