4

問題は、正方形 JPanel の背景色を square.setBackground(colors[j]) として設定すると、正方形は他の 3 を表示せずに色のリストの最初の色のみを描画することです。これは私のコードです:

import java.awt.*;
import java.util.*;
import javax.swing.*;
import java.awt.*;

@SuppressWarnings({ "unused", "serial" })

public class RegionPartition extends JFrame
{
    JLayeredPane layeredPane;
    JPanel regionBoard;
    JLabel regionPiece;

    private static int DELAY = 200;

    private Color[] colors = new Color[]{Color.PINK, Color.GREEN, Color.BLACK, Color.RED};

    public RegionPartition()
    {
        Dimension boardSize = new Dimension(500, 500);

        //  Use a Layered Pane for this this application
        layeredPane = new JLayeredPane();
        getContentPane().add(layeredPane);
        layeredPane.setPreferredSize(boardSize);

        regionBoard = new JPanel();

        layeredPane.add(regionBoard, JLayeredPane.DEFAULT_LAYER);

        regionBoard.setLayout( new GridLayout(4, 4) );

        regionBoard.setPreferredSize( boardSize );
        regionBoard.setBounds(0, 0, boardSize.width, boardSize.height);

        Random random = new Random();


        for (int i = 0; i < 16; i++) {          
            JPanel square = new JPanel(new BorderLayout());
            square.setBorder(BorderFactory.createLineBorder(Color.black));
            regionBoard.add( square );  

            square.setBackground(Color.green);

            int j=0;

            square.setBackground(colors[j]);

            j++;
        }
    }

    {
        JPanel panel = new JPanel()  
        {  
            Clients[] c = new Clients[128];

            Random random = new Random();

            private final int SIZE = 450;
            private int DELAY = 9999999;
            public void paintComponent (Graphics g)
            {
                super.paintComponent(g);

                for (int i=0; i<c.length; i++)
                {
                    int x = ( int ) ( random.nextFloat() * SIZE ) + 10;
                    int y = ( int ) ( random.nextFloat() * SIZE ) + 10;

                    g.drawOval( x, y, 10, 10 );
                    g.fillOval(x, y, 10, 10);
                }

                for (int j=0; j<DELAY; j++)
                {
                    repaint();
                }
            }
        };  

        panel.setOpaque(false);  

        //Set the glass pane in the JFrame  
        setGlassPane(panel);  

        //Display the panel  

        panel.setVisible(true);  
    }

    public static void main(String[] args)
    {
        JFrame frame = new RegionPartition();

        frame.setDefaultCloseOperation(DISPOSE_ON_CLOSE );
        frame.pack();
        frame.setResizable(true);
        frame.setLocationRelativeTo( null );
        frame.setVisible(true);

    }

    protected void paintComponent(Graphics g)
    {
        // TODO Auto-generated method stub
    }
}
4

3 に答える 3

5

これは、反復ごとに常にjを0に設定しているためです。

    int j=0;

    square.setBackground(colors[j]);

    j++;

ここで実際に何をしたいかに応じて、jをiに変更するか、ネストされたループを実行することができます。

16個の正方形すべてに4色すべてをグリッドのように配置したい場合は、ループを次のように変更することをお勧めします。

     for (int i = 0; i < 16; i++) {          
        JPanel square = new JPanel(new GridLayout(2,2));
        square.setBorder(BorderFactory.createLineBorder(Color.black));
        regionBoard.add( square );  



        for(int j=0; j<4; ++j){
          JPanel insideSquare = new JPanel();
          insideSquare.setBackground(colors[j]);
          square.add(insideSquare);
        }
    }
于 2012-08-29T13:01:13.727 に答える
4

配列には 4 色しかありませんcolorが、ループ インデックスがこれを超えているため、次を使用できます。

square.setBackground(colors[ i % colors.length]);

あなたの正方形の色を交互にします。

于 2012-08-29T13:10:31.947 に答える
3

forループのスコープ内でインスタンス化int jしているため、その値は複数の反復にわたって保持されません。コード内のある時点で宣言して、forループ全体をスコープできるようにする必要があります。

int j = 0;
<for loop>
    square.setBackground(colors[j]);
    j++;
<end for>

ただし、この状況では、配列インデックスとして十分でjあるという目的を果たしています。完全に削除して、代わりに次の手順を実行する方が正しいでしょう。iij

    square.setBackground(colors[i]);
于 2012-08-29T13:01:30.577 に答える