我正在从2007年编写的Java Exposure教科书中完成一项任务。本书包含了一些我经常更新的代码,以便使用一些最近的功能(只是基本的东西)。然而,在这一个我遇到了问题。我尝试做的只是将show
替换为setVisible(true)
并将Frame
更改为JFrame
并添加gfx.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
。但是,我注意到这实际上不会导致窗口关闭。如果我多次点击,可能是1/30尝试它会关闭。如果我将延迟从10减少到1,它通常会在2次尝试中关闭。这当然让我相信delay
方法导致了这种不稳定的行为。我试过了Thread.sleep
,但当然没有用。 是否有任何简单的方法来获取此代码,以便在我按下关闭按钮时框架将关闭?如果没有,那么做什么不那么简单的方法呢?
以下是代码:
// Lab30st.java
// The Screen Saver Program
// Student Version
import java.awt.*;
import java.awt.event.*;
import java.applet.*;
import javax.swing.JOptionPane;
public class Lab30st
{
public static void main(String args[])
{
GfxApp gfx = new GfxApp();
gfx.setSize(800,600);
gfx.addWindowListener(new WindowAdapter() {public void
windowClosing(WindowEvent e) {System.exit(0);}});
gfx.show();
}
}
class GfxApp extends Frame
{
private int circleCount, circleSize;
public GfxApp()
{
circleCount = 50;
circleSize = 30;
}
class Coord
{
private int xPos;
private int yPos;
public Coord(int x, int y)
{
xPos = x;
yPos = y;
}
}
public void paint(Graphics g)
{
int incX = 5;
int incY = 5;
int diameter = 30;
int timeDelay = 10;
Circle c = new Circle(g,diameter,incX,incY,timeDelay);
for (int k = 1; k <= 2000; k++)
{
c.drawCircle(g);
c.hitEdge();
}
}
}
class Circle
{
private int tlX; // top-left X coordinate
private int tlY; // top-left Y coordinate
private int incX; // increment movement of X coordinate
private int incY; // increment movement of Y coordinate
private boolean addX; // flag to determine add/subtract of increment for X
private boolean addY; // flag to determine add/subtract of increment for Y
private int size; // diameter of the circle
private int timeDelay; // time delay until next circle is drawn
public Circle(Graphics g, int s, int x, int y, int td)
{
incX = x;
incY = y;
size = s;
addX = true;
addY = false;
tlX = 400;
tlY = 300;
timeDelay = td;
}
public void delay(int n)
{
long startDelay = System.currentTimeMillis();
long endDelay = 0;
while (endDelay - startDelay < n)
endDelay = System.currentTimeMillis();
}
public void drawCircle(Graphics g)
{
g.setColor(Color.blue);
g.drawOval(tlX,tlY,size,size);
delay(timeDelay);
if (addX)
tlX+=incX;
else
tlX-=incX;
if (addY)
tlY+=incY;
else
tlY-=incY;
}
public void newData()
{
incX = (int) Math.round(Math.random() * 7 + 5);
incY = (int) Math.round(Math.random() * 7 + 5);
}
public void hitEdge()
{
boolean flag = false;
if (tlX < incX)
{
addX = true;
flag = true;
}
if (tlX > 800 - (30 + incX))
{
addX = false;
flag = true;
}
if (tlY < incY + 30) // The +30 is due to the fact that the title bar covers the top 30 pixels of the window
{
addY = true;
flag = true;
}
if (tlY > 600 - (30 + incY))
{
addY = false;
flag = true;
}
if (flag)
newData();
}
}
答案 0 :(得分:2)
您正在使用
“冻结”事件派遣线程public void delay(int n)
{
long startDelay = System.currentTimeMillis();
long endDelay = 0;
while (endDelay - startDelay < n)
endDelay = System.currentTimeMillis();
}
这意味着所有其他尝试发生的事情(如关闭窗口)必须等到线程退出“休眠”。 基本上你不应该在EDT中做延迟,它应该在不同的线程上,然后要求EDT线程更新。
您的“忙碌等待”延迟也可能导致其他问题。您可以使用Thread.sleep()
答案 1 :(得分:1)
那太可怕了 您需要重新构建整个代码。
让我们从非常糟糕的开始:
{\rtf1\ansi\ansicpg1252\deff0\deflang1033{\fonttbl
{\f0\fswiss\fcharset0 Courier New;}}\r\n{\colortbl;\red65\green0\blue0;\red0\green0\blue0;}
\r\n\viewkind4\uc1\pard\cf1\f0\fs24 se\\cf5\cf2 cf1lect left\par\r\n}\r\n
(几乎)是一个忙碌的等待,因为BASIC是现代的,所以我没有看到忙碌的等待。它基本上保留了线程的CPU主机,它不仅没有做任何事情,也没有其他线程(几乎)可以使用时间片。我说的原因几乎是调用系统时间函数会导致上下文切换,这可能允许其他线程运行,但它仍然很糟糕。
仍然非常糟糕:
替换为Rtf
。更好的是,没有忙碌的等待,但你仍然持有唯一的UI线程。这意味着在关闭主窗口之前不会发生其他UI工作。
需要做什么:
获取外部计时器(例如RichTextBox
)以触发绘制事件并执行动画的下一部分。
搜索“Java平滑动画”有很多例子,如何做到这一点,双缓冲和所有。