1

Java GUIを使用して、さまざまな場所のさまざまな時間を示す6つの時計のセットを作成する必要があるプロジェクトを行っています。スレッドを使用して 1 つの時計を表示するコードを作成できました。しかし、6 つのクロックを処理するために 6 つの異なるスレッドを作成する方法は? これは、単一の時計を作成する次のクラスです。

public class analogClockGUI extends JPanel implements Runnable{

    int secx,secy,lsecx,lsecy;               
    int minx,miny,lminx,lminy;          
    int hrx,hry,lhrx,lhry;
    int secRadius,minRadius,hrRadius,center;
    int pointx1,pointy1,pointx2,pointy2;

    double dhs,dhm,dhh;        
    Random rndm;               
    Thread t;
    Font clockFont;
    Date incre;
    int s,h,m;

    boolean off=false,drawFirst=true;
    Dimension dimension;
    Color wallColor,backColor=Color.black;

    int previousMin,previousSec;
    int change;    

    public analogClockGUI()
    {

        addMouseListener(new MouseAdapter()
        {

            public void mouseEntered(MouseEvent me)
            {
                dimension=getSize();


                if(change!=dimension.height)
                {
                    center=dimension.height/2;
                    setSize(new Dimension(dimension.height,dimension.height));
                }
                /*else
                {
                    center=dimension.width/2;
                    setSize(new Dimension(dimension.width,dimension.width));
                }*/
                drawFirst=true;
                change=dimension.height;     
                secRadius=(int)(0.7*center);
                minRadius=(int)(0.75*center);
                hrRadius=(int)(0.5*center);

            }
        });
        init();
    }



    public void init()
    {

        rndm=new Random();
        t=new Thread(this);
        dimension=getSize();
        center=dimension.height/2;

        secRadius=(int)(0.7*center);   
        minRadius=(int)(0.75*center);
        hrRadius=(int)(0.5*center);
        wallColor=new Color(rndm.nextInt(255),rndm.nextInt(255),rndm.nextInt(255));
        t.start();
    }
    public void run() throws NullPointerException
    {
        while(true)
        {
            s=new Date().getSeconds();//+incre.getSeconds(); 

            if(s!=previousSec)
            {

                dhs=Math.toRadians(s*6-90);
                secx=(int)(secRadius*Math.cos(dhs))+center;
                secy=(int)(secRadius*Math.sin(dhs))+center;

                pointx1=(int)((secRadius+center*0.15)*Math.cos(dhs))+center;
                pointy1=(int)((secRadius+center*0.15)*Math.sin(dhs))+center;


                m=new Date().getMinutes();
                dhm=Math.toRadians(m*6-90);
                minx=(int)(minRadius*Math.cos(dhm))+center;
                miny=(int)(minRadius*Math.sin(dhm))+center;

                pointx2=(int)((secRadius+center*0.15)*Math.cos(dhm))+center;
                pointy2=(int)((secRadius+center*0.15)*Math.sin(dhm))+center;


                h =new Date().getHours();
                dhh=Math.toRadians(h*30-90+((m-1)/2));
                hrx=(int)(hrRadius*Math.cos(dhh))+center;
                hry=(int)(hrRadius*Math.sin(dhh))+center;

                repaint();
                previousSec=s;
            }
            try
            {
                Thread.sleep(100);
                if(off)
                    break;
            }catch(InterruptedException e){}          
        }
    }
    public void upgate(Graphics g) throws NullPointerException
    {

        if(drawFirst==true)
        {

            clockFont=new Font("Serif",Font.PLAIN,(int)(0.2*center));
            drawFirst=false;
        }
        if(m!=previousMin){
            if(s==0)
            {

                wallColor=new Color(rndm.nextInt(255),rndm.nextInt(255),rndm.nextInt(255));
                g.setColor(wallColor);
                g.fillOval(0,0,2*center,2*center);
                g.setColor(backColor);
                g.fillOval((int)(0.1*center),(int)(0.1*center),(int)(center*1.8),(int)(center*1.8));

                clear(g);
                previousMin=m;
            }}
        drawPoints(g);
        g.setColor(Color.white);
        g.setFont(clockFont);

        g.drawString("12",(int)(0.9*center),(int)(0.25*center));
        g.drawString("6",(int)(0.95*center),(int)(1.885*center));
        g.drawString("9",(int)(0.11*center),(int)(1.06*center));
        g.drawString("3",(int)(1.805*center),(int)(1.07*center));
        g.drawString("1",(int)(1.369*center),(int)(0.36*center));
        g.drawString("2",(int)(1.674*center),(int)(0.645*center));
        g.drawString("4",(int)(1.665*center),(int)(1.485*center));
        g.drawString("5",(int)(1.372*center),(int)(1.778*center));
        g.drawString("7",(int)(0.545*center),(int)(1.777*center));
        g.drawString("8",(int)(0.227*center),(int)(1.485*center));
        g.drawString("10",(int)(0.212*center),(int)(0.675*center));
        g.drawString("11",(int)(0.522*center),(int)(0.365*center));


        if(lsecx!=0)
        {
            g.setColor(backColor);
            g.drawLine(center,center,lsecx,lsecy);

        }


        g.setColor(Color.green);
        g.drawLine(center,center,hrx,hry);
        g.drawLine(center+1,center,hrx,hry);
        g.drawLine(center-1,center,hrx,hry);
        g.drawLine(center,center+1,hrx,hry);
        g.drawLine(center,center-1,hrx,hry);
        g.drawLine(center+2,center,hrx,hry);
        g.drawLine(center-2,center,hrx,hry);
        g.drawLine(center,center+2,hrx,hry);
        g.drawLine(center,center-2,hrx,hry);

        //show the minute hand with bigger size
        g.setColor(Color.blue);
        g.drawLine(center,center,minx,miny);
        g.drawLine(center+1,center,minx,miny);
        g.drawLine(center-1,center,minx,miny);
        g.drawLine(center,center+1,minx,miny);
        g.drawLine(center,center-1,minx,miny);

        g.setColor(Color.red);
        g.drawLine(center,center,secx,secy);
        g.fillOval(center-3,center-3,6,6);       //center point

        if(m%5!=0)
        {
            g.setColor(Color.cyan);
            g.fillOval(pointx2-2,pointy2-2,4,4);

        }

        if(s%5!=0)
        {
            g.setColor(Color.red);
            g.fillOval(pointx1-2,pointy1-2,4,4);

        }

        lminx=minx;
        lminy=miny;
        lhrx=hrx;
        lhry=hry;
        lsecx=secx;
        lsecy=secy;



    }
    public void paint(Graphics g)  throws NullPointerException
    {
        g.setColor(wallColor);
        g.fillOval(0,0,2*center,2*center);
        g.setColor(backColor);
        g.fillOval((int)(0.1*center),(int)(0.1*center),(int)(center*1.8),(int)(center*1.8));
        upgate(g);
    }
    public void clear(Graphics g)  throws NullPointerException
    {
        if(lsecx!=0)
        {
            g.setColor(backColor);
            g.drawLine(center,center,lhrx,lhry);
            g.drawLine(center+1,center,lhrx,lhry);
            g.drawLine(center-1,center,lhrx,lhry);
            g.drawLine(center,center+1,lhrx,lhry);
            g.drawLine(center,center-1,lhrx,lhry);
            g.drawLine(center+2,center,lhrx,lhry);
            g.drawLine(center-2,center,lhrx,lhry);
            g.drawLine(center,center+2,lhrx,lhry);
            g.drawLine(center,center-2,lhrx,lhry);

            g.drawLine(center,center,lminx,lminy);
            g.drawLine(center+1,center,lminx,lminy);
            g.drawLine(center-1,center,lminx,lminy);
            g.drawLine(center,center+1,lminx,lminy);
            g.drawLine(center,center-1,lminx,lminy);


        }


    }
    public void drawPoints(Graphics g) throws NullPointerException
    {
        int x1,y1;
        double p;
        for(int i=1;i<=60;i++)
        {
            if(i%5!=0)
            {
                p=i*0.10472;
                x1=(int)((secRadius+center*0.15)*Math.cos(p))+center;
                y1=(int)((secRadius+center*0.15)*Math.sin(p))+center;
                g.setColor(Color.gray);
                g.fillOval(x1-2,y1-2,4,4);

            }
        }


    }
    public void stop() throws NullPointerException /*close the thread when applet stopped*/
    {
        off=true;
        t=null;
    }

}
4

2 に答える 2

1

あなたの最大の問題はThread#runコードにあります...クロックのすべてのインスタンスが同じDate値を使用しています。

public void run() throws NullPointerException {
    while (true) {
        // !! This is wrong !!
        // Each clock should have its own concept of it's start time, then each "second" you
        // should advance the time by a single second...
        s = new Date().getSeconds();//+incre.getSeconds(); 

もう 1 つの問題は、スレッドのコンテンツ内で UI が依存する値を変更してはならないということです...

        // This is a bad idea...
        if (s != previousSec) {

            dhs = Math.toRadians(s * 6 - 90);
            secx = (int) (secRadius * Math.cos(dhs)) + center;
            secy = (int) (secRadius * Math.sin(dhs)) + center;

            pointx1 = (int) ((secRadius + center * 0.15) * Math.cos(dhs)) + center;
            pointy1 = (int) ((secRadius + center * 0.15) * Math.sin(dhs)) + center;


            m = new Date().getMinutes();
            dhm = Math.toRadians(m * 6 - 90);
            minx = (int) (minRadius * Math.cos(dhm)) + center;
            miny = (int) (minRadius * Math.sin(dhm)) + center;

            pointx2 = (int) ((secRadius + center * 0.15) * Math.cos(dhm)) + center;
            pointy2 = (int) ((secRadius + center * 0.15) * Math.sin(dhm)) + center;


            h = new Date().getHours();
            dhh = Math.toRadians(h * 30 - 90 + ((m - 1) / 2));
            hrx = (int) (hrRadius * Math.cos(dhh)) + center;
            hry = (int) (hrRadius * Math.sin(dhh)) + center;

            repaint();
            previousSec = s;
        }

問題は、すべての値が更新される前に UI が更新される可能性があり、奇妙で予測不可能なペイント技術が発生することです。

そして、本当に 100 ミリ秒の更新が必要ですか?? スリープを 500 ミリ秒にスケールアウトして、2 番目の変更より少し早くまたは少し遅くしたほうがよいでしょう。

あなたの絵も私をびっくりさせます。非常に注意してください。JPanelすでにupdate(Graphics)から継承されたメソッドContainerがあります。名前の競合が発生してプログラムがクラッシュする可能性があります (update再描画エンジンで使用されているように)。

paint直接オーバーライドする必要はほとんどないため、 を使用することをお勧めしますpaintComponent。これにより、各ペイント サイクルでコンポーネントが更新され、適切に準備されます。プロセスの一部としてpaintComponent、グラフィック コンテキストもクリアされるため、このメソッドは必要ありませんclear(ペイント チェーンを尊重して呼び出している限り)。super.paintComponent

アイデアと解決策

私がすることは、登録されたクラスに「ティック」が発生したことを通知することだけを担当する「ティッカー」クラスを設定することです。このクラスでは、javax.swing.Timer. これにより、「ティック」イベントがイベント ディスパッチ スレッドのコンテキスト内から強制的に呼び出され、ペイント メソッドが依存する変数への変更が同じスレッドのコンテキスト内で更新されることが保証されます。250 ミリ秒程度の遅延を使用することを検討して、通知サイクルでの余分な混乱を考慮してください。

Thread使用するか、または使用するかに関係なくTimer、「ティック」を更新すると、ボトルネックが発生します。

BufferedImageコンポーネントのサイズが変わらない限り変更される可能性は低いため、時計の文字盤を にペイントすることも検討します。これで塗装が早くなります。

于 2013-02-03T08:31:24.333 に答える
1

これらのクロックは固定時間オフセットだけが異なるため、おそらく個々のスレッドや個々のタイマー タスクさえも必要としません。paintComponent同じ基本時間値を使用している異なる時間オフセットを持つ複数のクロックを描画する、より大きく複雑なメソッドを記述します。

于 2013-02-03T11:10:20.697 に答える