非主线程的图片不会移动

时间:2015-04-30 08:27:47

标签: java eclipse multithreading swing

我是java新手,现在我正在开发Threads.But现在我遇到了一个大问题: 当我运行以下代码时,我除了两个图像在屏幕上移动,但其中一个现在移动,甚至另一个没有显示。我想知道问题是什么: 阶级敌人:

import java.awt.Point;
import java.awt.Rectangle;
import java.awt.image.BufferedImage;
import java.io.File;
import javax.imageio.ImageIO;
import javax.swing.*;

public class Enemy extends JLabel implements Runnable{
    private BufferedImage i;
    private int x,y;
    Enemy()
    {
        x=y=0;
        try{
            i=ImageIO.read(new File("m2.jpg"));
        }catch (Exception e){}
        this.setIcon(new ImageIcon(i));
        this.setLocation(x,y);
        this.setBounds(new Rectangle(new Point(x,y),this.getPreferredSize()));
    }
    Enemy(int X,int Y)
    {
        x=X;
        y=Y;
        try{
            i=ImageIO.read(new File("m2.jpg"));
        }catch (Exception e){}
        this.setIcon(new ImageIcon(i));
        this.setLocation(x,y);
        this.setBounds(new Rectangle(new Point(x,y),this.getPreferredSize()));
    }

    public void move()
    {
        while(true)
        {
            int p=(int)Math.random()*4+1;
            switch (p)
            {
            case 1:x++;break;
            case 2:y++;break;
            case 3:x--;break;
            case 4:y--;break;
            }
            this.setLocation(x,y);
            this.setBounds(new Rectangle(new Point(x,y),this.getPreferredSize()));
        }
    }

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

班级球员:

import java.awt.Point;
import java.awt.Rectangle;
import java.awt.image.BufferedImage;
import java.io.File;

import javax.imageio.ImageIO;
import javax.swing.ImageIcon;
import javax.swing.JLabel;


public class Player extends JLabel implements Runnable{
    private BufferedImage i;
    private int x,y;
    Player()
    {
        x=y=0;
        try{
            i=ImageIO.read(new File("m1.jpg"));
        }catch (Exception e){}
        this.setIcon(new ImageIcon(i));
        this.setLocation(x,y);
        this.setBounds(new Rectangle(new Point(x,y),this.getPreferredSize()));
    }

    public void move()
    {
        int p=(int)Math.random()*4+1;
        switch (p)
        {
        case 1:x++;break;
        case 2:y++;break;
        case 3:x--;break;
        case 4:y--;break;
        }
        this.setLocation(x,y);
        this.setBounds(new Rectangle(new Point(x,y),this.getPreferredSize()));
    }

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

主要课程:

import javax.swing.JFrame;


public class Test {
    public static void main(String[] args) {
        Enemy e=new Enemy(150,150);
        Player p=new Player();
        JFrame f=new JFrame("Test");
        f.setSize(300,300);
        f.setLayout(null);
        f.setDefaultCloseOperation(f.EXIT_ON_CLOSE);
        f.add(e);
        f.add(p);
        Thread pThread=new Thread(p);
        Thread eThread=new Thread(e);
        pThread.start();
        eThread.start();
        while(true)
        {
            f.setVisible(true);
            try
            {
                Thread.sleep(50);
            }catch (Exception q){}
            f.remove(p);
            f.remove(e);
            f.add(p);
            f.add(e);
        }


    }

}

1 个答案:

答案 0 :(得分:1)

Math.random()Enemy类中对Player的调用始终四舍五入为零。你可能想要:

int p = (int) (Math.random() * 4) + 1;

x方法中限制ymove的可能值也是一个好主意(maxX =帧宽 - 敌人/玩家宽度; maxY =帧高 - 敌人/玩家身高):

switch (p) {
    case 1:
        if (x < maxX)
            x++;
        break;
    case 2:
        if (y < maxY)
            y++;
        break;
    case 3:
        if (x > 0)
            x--;
        break;
    case 4:
        if (y > 0)
            y--;
        break;
}

最后,您可以通过为EnemyPlayer类创建公共类来节省一些工作。在这里你可以放置所有共享代码,此时此代码可以是所有代码;两个对象都可以是同一个类的实例:

GameObject e = new GameObject(150, 150, size - 150, size - 150, "m2.jpg");
GameObject p = new GameObject(100, 100, size - 100, size - 100, "m1.jpg");

GameObject类可能如下所示:

import java.awt.Point;
import java.awt.Rectangle;
import java.awt.image.BufferedImage;
import java.io.File;

import javax.imageio.ImageIO;
import javax.swing.ImageIcon;
import javax.swing.JLabel;

public class GameObject extends JLabel implements Runnable {
    private BufferedImage i;
    private int x, y;
    private int maxX;
    private int maxY;

    public GameObject(int X, int Y, int maxX, int maxY, String imagePath) {
        x = X;
        y = Y;
        this.maxX = maxX;
        this.maxY = maxY;
        try {
            this.i = ImageIO.read(new File(imagePath));
        } catch (Exception e) {
            e.printStackTrace();
        }
        this.setIcon(new ImageIcon(this.i));
        this.setLocation(x, y);
        this.setBounds(new Rectangle(new Point(x, y), this.getPreferredSize()));
    }

    public void move() {
        while (i != null) {
            int p = (int) (Math.random() * 4) + 1;
            switch (p) {
                case 1:
                    if (x < maxX)
                        x++;
                    break;
                case 2:
                    if (y < maxY)
                        y++;
                    break;
                case 3:
                    if (x > 0)
                        x--;
                    break;
                case 4:
                    if (y > 0)
                        y--;
                    break;
            }
            this.setLocation(x, y);
            this.setBounds(new Rectangle(new Point(x, y), this.getPreferredSize()));
        }
    }

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