0

そこで、画面をクリックすると、落下して下の大きなブロックと衝突してくっつくブロックを作成する簡単なプログラムを作成しようとしています。単純な衝突プログラムのようなものです。問題は、1 つのブロックを作成すると、以前にブロックが削除されることです。配列を作成しましたが、それでもこれを行います。私が間違っていることを知っている人はいますか？簡単な修正だと確信しています。

    public class Screen extends JPanel implements Runnable {

public static JLabel statusbar; //displays a status bar showing what mouse movements are taking place
private Image cat;  //image of the cat
public int xCoord ; //get the coordinates of the mouse pressed
public int yCoord ;
public int xCreate;
public int yCreate;
public Rectangle Ground;
public Rectangle Block;
public boolean isClicked = false;
public int clickCount = 0;
Rectangle blocks[] = new Rectangle[10];

int blocknum = 0;

    public Screen(Frame frame) {
            loadPic(); //calls the loadPic method above 
                Handlerclass handler = new Handlerclass(); //creates a new class to use the mouse motion listener
                    System.out.println("mouse works!");
         addMouseListener(handler);
         addMouseMotionListener(handler);
         statusbar = new JLabel("default");
            add(statusbar);
    }
    public void run(){ //this is the game run loop
        System.out.println("this is running");   
        try{
        } catch(Exception e) {} //exception handling

    }
    public void loadPic(){ //loads the picture from the other project but its the same pic
        cat = new ImageIcon("C:\\Users\\Camtronius\\Documents\\NetBeansProjects\\Moving Block Proj\\src\\MovingBlock\\catIcon1.png").getImage(); //gets the image
        System.out.println("Image Loaded!");
    }

     @Override public void paintComponent(Graphics g){
        super.paintComponent(g); //paints the component, the picture, on top
            Graphics2D g2d = (Graphics2D) g.create();

             g2d.drawImage(cat, xCoord, yCoord, null);      

             g2d.setColor(Color.BLUE);
             Ground = new Rectangle(0,450,550,50);
             g2d.fillRect(0,450, 550, 50);

                 for(Rectangle blocknum : blocks){
                  if (blocks != null) {    
                      g2d.setColor(Color.RED);
                       g2d.fillRect(xCreate,yCreate,50,50); 
                       System.out.println(blocknum);
                  }

                 }
                 //move();           
    }

   public void move(){   
    if(yCreate<400){
        yCreate+=1;
    }else{
        }
    if(Ground.intersects(blocks[blocknum])){ 
            yCreate=400;
            System.out.println("contains!");
        }
    }    
     private class Handlerclass implements MouseListener, MouseMotionListener{
        public void mouseClicked(MouseEvent event){

        }
        public void mousePressed(MouseEvent event){

        }
        public void mouseReleased(MouseEvent event){

         if(blocknum<blocks.length){
            xCreate=event.getX();
            yCreate=event.getY(); 
            blocks[blocknum] = new Rectangle(50,50, xCreate, yCreate);
            repaint();
         }
        blocknum=blocknum+1;
        }
        public void mouseEntered(MouseEvent event){

        }
        public void mouseExited(MouseEvent event){

        } 
        public void mouseDragged(MouseEvent event){

        }
        public void mouseMoved(MouseEvent event){
           statusbar.setText(String.format("Coordinates are: %d, %d", event.getX(),event.getY()));
            xCoord=event.getX();
            yCoord=event.getY(); 
        }
    }
}
4

1 に答える 1

2

塗装は破壊的なプロセスです。つまり、新しい描画サイクルが実行されると、Graphicsコンテキストの以前の内容がクリアされます...

したがって、あなたpaintComponentの方法では、最後のブロックのみをペイントしています...

if(isClicked = true){
    blocks[blocknum] = new Rectangle(50,50, xCreate, yCreate);
    g2d.setColor(Color.RED);
    g2d.fillRect(xCreate,yCreate,50,50);
    System.out.println(blocknum);
    repaint(); // THIS IS A BAD IDEA
}

呼び出される可能性のあるメソッドを呼び出さないでrepaintください。これにより、CPU を消費する潜在的な死のサイクルに陥ります。

blocks代わりに、配列をループして、それぞれをペイントする必要があります...

for (Rectangle block : blocks) {
    if (block != null) {
        g2d.setColor(Color.RED);
        g2d.fill(block);
    }
}

そして、あなたmouseReleasedの方法では、新しい長方形を追加する必要があります...

public void mouseReleased(MouseEvent event){
    blocknum=blocknum+1;
    if (blocknum < blocks.length) {
        xCreate=event.getX();
        yCreate=event.getY(); 
        blocks[blocknum] = new Rectangle(xCreate, yCreate, 50, 50);
    }
}

詳細については、 Custom Painting、Paint in AWT 、および SwingとConcurrency in Swingをご覧になることをお勧めします

于 2013-05-29T04:59:36.627 に答える