2013-06-05 36 views
4

我目前正在为我创建的游戏制作地图生成程序,所以我试图能够使用我的鼠标修改地图上的各个图块。我已经成功实现了点击图块并更改其值(从4路瓦片到2路瓦片)的功能,但是我无法使mouseDragged工作。根据Java文档,我已经将MouseDragged接口正确实现到了MouseEventHandler中,但是当我拖动鼠标并将鼠标移动到这些事件的任何一处时,并且抛出println文本,使它们向右移动现在。如果有人可以说出为什么这不适合我,我会喜欢这个。谢谢!MouseDragged&MouseMoved不能在Java Applet中工作

一个方面的问题:如果任何人有更好的方法来尝试生成这样的地图,或者任何其他方法(大概比我做的更好),其他方法检测单个mousePressed在瓷砖上,我很乐意听到。这只是我第一次与这类事情合作,所以我很缺乏经验,并且会喜欢一些建议。

至于代码本身:

文件1:OneQuestMapgen.java

package OneQuestMapgen; 

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

import tiles.Tile; 
import tiles.TileSet; 

public class OneQuestMapgen extends JApplet{ 

    /** 
    * 
    */ 

    public static OneQuestMapgen instance; 

    ArrayList<ArrayList<Tile>> map = new ArrayList<ArrayList<Tile>>(); 
    TileSet tileSet = new TileSet(); 

    public void init(){ 
     this.setSize(950,600); 
    } 

    public void start(){ 
     this.addMouseListener(new MouseEventListener(this)); 

     int tileSize = tileSet.get("grasslands")[1].getHeight(); 

     for (int i = 0; i < getHeight(); i += tileSize) { 
      ArrayList<Tile> temp = new ArrayList<Tile>(); 
      for (int j = 0; j < getWidth(); j += tileSize) { 
       temp.add(new Tile(j, i, tileSize, tileSet.get("grasslands"))); 
      } 
      map.add(temp); 
     } 

    } 

    public void paint(Graphics g){ 

     for (int i = 0; i < map.size(); i++){ 
      for(int j = 0; j < map.get(i).size(); j++){ 
       map.get(i).get(j).render(g); 
      }    
     } 
    } 

    public void stop(){ 
    } 

    public void destroy(){ 
    } 

    public ArrayList<ArrayList<Tile>> getMap(){ 
     return map; 
    } 
} 

文件2:MouseEventListener.java

package OneQuestMapgen; 

import java.awt.event.MouseEvent; 
import java.awt.event.MouseListener; 
import java.awt.event.MouseMotionListener; 
import java.util.ArrayList; 

import tiles.Tile; 

public class MouseEventListener implements MouseListener, MouseMotionListener { 

    OneQuestMapgen instance; 
    ArrayList<ArrayList<Tile>> map; 

    public MouseEventListener(OneQuestMapgen instance) { 
     this.instance = instance; 
    } 

    @Override 
    public void mouseClicked(MouseEvent e) { 
     System.out.println(e.getX() + " , " + e.getY()); 
     map = instance.getMap(); 
     for (int i = 0; i < map.size(); i++) { 
      for (int j = 0; j < map.get(i).size(); j++) 
       if (map.get(i).get(j).bounds(e)) { 
        map.get(i).get(j).onClicked(instance.getGraphics()); 
       } 
     } 
    } 

    @Override 
    public void mouseEntered(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 

    @Override 
    public void mouseExited(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 

    @Override 
    public void mousePressed(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 

    @Override 
    public void mouseReleased(MouseEvent e) { 
     // TODO Auto-generated method stub 

    } 

    @Override 
    public void mouseDragged(MouseEvent arg0) { 

     System.out.println("Mouse Dragged: " + arg0.getX() + " , " + arg0.getY());   
    } 

    @Override 
    public void mouseMoved(MouseEvent arg0) { 
     // TODO Auto-generated method stub 
     System.out.println("Mouse Moved: " + arg0.getX() + " , " + arg0.getY()); 

    } 

} 

文件3:Tile.java

package tiles; 

import java.awt.Graphics; 
import java.awt.event.MouseEvent; 
import java.awt.image.BufferedImage; 

@SuppressWarnings("serial") 
public class Tile extends Square { 

    BufferedImage[] tiles; 
    int tileValue = 0; 

    public Tile (int x, int y, int size, BufferedImage[] tileSet){ 
     super(x,y,size); 
     tiles = tileSet; 
    } 

    public void render(Graphics g) { 

     g.drawImage(tiles[tileValue], getX(), getY(), getDimension(), getDimension(), null);   
    } 

    public void onClicked(Graphics g) { 
     tileValue++; 
     System.out.println(tileValue); 
     g.drawImage(tiles[tileValue], getX(), getY(), getDimension(), getDimension(), null); 
    } 

    public boolean bounds(MouseEvent e){ 
     if (e.getX() <= getX() + getDimension() && e.getX() >= getX() && e.getY() <= getY() + getDimension() && e.getY() >= getY()) 
      return true; 
     return false; 
    } 
} 

文件4: TileSet.java

package tiles; 

import java.awt.image.BufferedImage; 
import java.net.URL; 
import java.util.concurrent.ConcurrentHashMap; 

import javax.imageio.ImageIO; 

public final class TileSet { 

    ConcurrentHashMap<String, BufferedImage[]> container = new ConcurrentHashMap<String, BufferedImage[]>(11); 
    BufferedImage[] grasslands = new BufferedImage[11]; 

    public TileSet(){ 

     boolean toAdd = true; 

     try { 
      grasslands[0] = ImageIO.read(new URL("http://i675.photobucket.com/albums/vv118/va023/OneQuest/GrassBlank_zpse99de845.png")); 
      grasslands[1] = ImageIO.read(new URL("http://i675.photobucket.com/albums/vv118/va023/OneQuest/4WayTile_zps49ebbeea.png")); 
      grasslands[2] = ImageIO.read(new URL("http://i675.photobucket.com/albums/vv118/va023/OneQuest/LineHorizontal_zpsc7bd45d5.png")); 
      grasslands[3] = ImageIO.read(new URL("http://i675.photobucket.com/albums/vv118/va023/OneQuest/LineVertical_zps9719b63f.png")); 
      grasslands[4] = ImageIO.read(new URL("http://i675.photobucket.com/albums/vv118/va023/OneQuest/TShapeTop_zpsa4b2aaa1.png"))   
     } 

     catch (Exception e) { 
      System.out.println("Unable to load resources for Grasslands"); 
     } 

     for (int i = 0; i < grasslands.length; i++) 
      if (grasslands[i].getWidth() != grasslands[i].getHeight()) { 
       System.out.println("Error, tiles are not square. Grasslands has not been added."); 
       toAdd = false; 
      } 

     if (toAdd) { 
      container.put("grasslands".toLowerCase(), grasslands); 
     } 
    } 

    public BufferedImage[] get(String s) { 
     return container.get(s.toLowerCase()); 
    } 
} 

文件5:Square.java

package tiles; 

import javax.swing.JLabel; 

public abstract class Square extends JLabel{ 

    /** 
    * 
    */ 
    private static final long serialVersionUID = 1L; 

    private int x; 
    private int y; 
    private int dimension; 

    public Square(int x, int y, int dimension){ 

     this.x = x; 
     this.y = y; 
     this.dimension = dimension; 
    } 

    public int getX(){ 
     return x; 
    } 

    public int getY(){ 
     return y; 
    } 

    public int getDimension(){ 
     return dimension; 
    } 
} 

注意:如果你宁愿能够把它全部关闭github上的,我添加了一个链接到我的github回购以下这个项目:

网址:https://github.com/vagrawal1/OneQuestMapgen

HTTPS:https://github.com/vagrawal1/OneQuestMapgen.git

SSH:[email protected]:vagrawal1/OneQuestMapgen.git

的Git只读:git的:再次//github.com/vagrawal1/OneQuestMapgen.git

感谢您的帮助,谢谢您花时间阅读本文。

+3

圣洁,这是很多代码!你能简化它吗?也许创建一个[sscce](http://sscce.org)?请记住,我们是志愿者。你的努力使我们的工作更容易帮助,总是非常感激。谢谢! –

+0

sscce?好的,我会尝试通过代码并简化它。我认为大部分情况与这个问题无关,所以我会删除它。 – Vasu

+1

顺便说一句,你当然知道,你的'的getX()'和'的getY()'Square类的方法重载的JLabel的同样的方法,而如果你不小心,这些和正确地使用他们,他们可以完全搞乱你的组件在GUI中的位置,对吧?我自己,我非常努力地避免压倒这些非常方法。如果压倒性不是你的意图,考虑改变他们的名字。 –

回答

8

您需要添加一个MouseMotionListener才能进行工作。你只是添加一个MouseListener。

public void start() { 
    MouseEventListener myMouseEventListener = new MouseEventListener(this); 
    addMouseListener(myMouseEventListener); 
    addMouseMotionListener(myMouseEventListener); 

    //this.addMouseListener(new MouseEventListener(this)); 

而且按照我的意见,Square类的你的getX()和的getY()方法重载的JLabel的同样的方法,而如果你不小心,这些和正确使用它们可以完全搞乱你的组件在GUI中的位置。我自己,我非常努力地避免压倒这些非常方法。如果压倒性不是你的意图,考虑改变他们的名字。

此外,我倾向于倾向mousePressed(...)而不是mouseClicked(...)方法,因为前者即使按下后移动鼠标也会对印刷机作出响应。


编辑:另一个bug!通过调用组件上的getGraphics(),您几乎不会得到您的Graphics实例,因为返回的Graphics对象是eva逝的,可能很快就会变为null。相反,可以使用JComponent或JPanel的paintComponent(...)中显示的BufferedImage或仅绘制paintComponent(...)本身。


编辑2:为什么重写paint(...)?这对我来说似乎有点危险,特别是因为你没有调用超级方法,特别是因为不需要做这种事情。

+0

这就是我忘了。非常感谢您提供快速,完美的答案!对此,我真的非常感激。 – Vasu

+0

@ Holyhades666:请参阅修改。 –

+0

气垫船,你说的涂装方法是指我压倒一切?我没有在'OneQuestMapgen'类的'paint(...)'之上有@Override语句,除非你的意思是在那个类的paint方法中我调用了我自己创建的渲染方法因为我不太确定如何以及何时调用paintComponent()。 – Vasu

相关问题