为什么这个简单的循环导致我的JFrame的问题行为

时间:2016-11-10 00:53:51

标签: java swing graphics jframe awt

我正在从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();
    }

}

2 个答案:

答案 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()

改善行为

请参阅Java Event-Dispatching Thread explanation

答案 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平滑动画”有很多例子,如何做到这一点,双缓冲和所有。