0

問題が発生しています。四角形 (ハエ) を描画し、動きを示すために再描画したいと考えています。このコードでは、ボタンが押されたときにハエが「移動」しますが、古い正方形は削除されません。environmentPanel.repaint() updateui() と removeall() を試しましたが、動作しません。それらのいずれかを使用すると、形状が表示されず、空白の画面が表示されます。

import java.util.Random;


public class Fly implements Runnable{
private int xPosition;
private int yPosition;
private boolean eaten;

public Fly(){
    Random randomGenerator = new Random();
    xPosition = randomGenerator.nextInt(690) + 10;
    yPosition = randomGenerator.nextInt(690) + 10;
    eaten = false;
}

public int getxPosition() {
    return xPosition;
}

public void setxPosition(int xPosition) {
    this.xPosition = xPosition;
}

public int getyPosition() {
    return yPosition;
}

public void setyPosition(int yPosition) {
    this.yPosition = yPosition;
}

public boolean isEaten() {
    return eaten;
}

public void setEaten(boolean eaten) {
    this.eaten = eaten;
}

public void move(){
      Random randomGenerator = new Random();

        int xChange = -10 + randomGenerator.nextInt(20);
        int yChange = -10 + randomGenerator.nextInt(20);
        xPosition = xPosition + xChange;
        yPosition = yPosition + yChange;
        try {
            Thread.sleep(1000);
        } catch (InterruptedException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
        }
        move();

}

@Override
public String toString() {
    return "Fly [xPosition=" + xPosition + ", yPosition=" + yPosition
            + ", eaten=" + eaten + "]";
}

@Override
public void run() {
    move();
}


}

import java.awt.Color;
import java.awt.FlowLayout;
import java.awt.Graphics;
import java.awt.Panel;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.util.ArrayList;
import javax.swing.*;
import java.awt.*;
import javax.imageio.ImageIO;

public class Enviroment2 implements Runnable,ActionListener{ 
private JFrame frame;
private JPanel enviromentPanel,totalGUI,enviromentButtonPanel;
private JButton newFrogButton, resetButton, hungryButton;
private JTextField enterName;
private JLabel hungryLabel;
private ArrayList<Frog> frogs;
private ArrayList<Fly> flys;



public Enviroment2(){
totalGUI = new JPanel();
flys = new ArrayList<Fly>();
frogs = new ArrayList<Frog>();
enviromentPanel = new JPanel();
enviromentButtonPanel = new JPanel();
newFrogButton = new JButton("New Frog");
enterName = new JTextField("Enter name");

hungryButton = new JButton("Hungry!");

resetButton = new JButton("Reset");
frame = new JFrame("[=] Hungry Cyber Pet [=]");
JFrame.setDefaultLookAndFeelDecorated(true);

frame.setContentPane(runEnviroment());

frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setSize(740, 800);
frame.setVisible(true);



}



public JPanel runEnviroment(){

totalGUI.setLayout(null);

enviromentPanel.setLayout(null);
enviromentPanel.setLocation(10, 10);
enviromentPanel.setSize(700, 700);
enviromentPanel.setBackground(Color.WHITE);
totalGUI.add(enviromentPanel);


FlowLayout experimentLayout = new FlowLayout();
enviromentButtonPanel.setLayout(experimentLayout);
enviromentButtonPanel.setLocation(10, 710);
enviromentButtonPanel.setSize(700, 50);
totalGUI.add(enviromentButtonPanel);

newFrogButton.setLocation(0, 0);
newFrogButton.setSize(120, 30);
newFrogButton.addActionListener(this);
enviromentButtonPanel.add(newFrogButton);

enterName.setLocation(140,0);
enterName.setSize(120,30);
enviromentButtonPanel.add(enterName);

hungryButton.setLocation(280, 0);
hungryButton.setSize(120, 30);
hungryButton.addActionListener(this);
enviromentButtonPanel.add(hungryButton);

resetButton.setLocation(420, 0);
resetButton.setSize(120, 30);
resetButton.addActionListener(this);
enviromentButtonPanel.add(resetButton);


totalGUI.setOpaque(true);

return totalGUI;
}


public void draw(){     

Graphics paper = enviromentPanel.getGraphics();

for (int i = 0; i <= flys.size()-1; i++){
    System.out.println("hi");
    paper.setColor(Color.BLACK);

    paper.fillRect(flys.get(i).getxPosition(), flys.get(i).getyPosition(), 10,      10);




}
try {
    Thread.sleep(1000);
} catch (InterruptedException e) {
    // TODO Auto-generated catch block
    e.printStackTrace();
}

draw();

}

public void actionPerformed(ActionEvent e) {
 if(e.getSource() == newFrogButton){
     Frog frog = new Frog(enterName.getText());
     frogs.add(frog);
     Fly fly = new Fly();
     Thread t = new Thread(fly);
     t.start();
     flys.add(fly);
     showFlys();

  }
  else if(e.getSource() == hungryButton){
  }
  else if(e.getSource() == resetButton){
      frogs.clear();
      flys.clear();
      System.out.println(frogs);
      System.out.println(flys);


  }
}


public void showFlys(){
for (int i = 0; i <= flys.size()-1; i++){
    System.out.println(flys.get(i));
}

}



@Override
public void run() {
draw(); 
}



}
4

1 に答える 1

5

これGraphics paper = enviromentPanel.getGraphics()が問題の始まりです。これは、カスタム ペインティングがどのように行われるかではありません。

getGraphics最後のペイント サイクルで使用されたグラフィックス コンテキストを返しますnull。

Graphics自分が作成していないコンテキストへの参照を維持するべきではありません。それらは変化する可能性があり、順番を変えてペイントすると、予期しない結果が生じる可能性があります。

代わりに、paintComponentメソッドを (おそらく でenvironmentPanel) オーバーライドし、すべてのカスタム ペイントをその中で行う必要があります。

2 番目の問題は、Swing のスレッド ルールに違反していることです。EDT 以外のスレッドで UI コンポーネントを作成または変更しないでください。

あなたは一読したいかもしれません

于 2013-02-15T10:32:22.460 に答える