JPanel im JPanel bewegen

[Morphius]

Hi,
was ich erreichen möchte ist ein großes JPanel mit einer Gitterstruktur auf der ich kleine JPanels am Gitter ausrichten kann.

Bevor ich jetzt die ganze Zeit vom großen und kleinen JPanel rede, mach ich das mal am Beispiel von Scrabble fest. Also die Buchstaben, die auf dem Spielbrett verschoben werden können und sich am Gitter ausrichten.

Also, ich habe an das Spielbrett einen Listener gehängt, der per getComponentAt sagt, welcher Buchstabe ausgewählt wurde. Wenn man dann die Maus bewegt, dann wird die Position des Buchstabens immer aktualisiert und die repaint-Methode aufgerufen. Zur Positionsspeicherung habe ich mir eine extra Variable angelegt. Diese wird in der paintComponent-Methode des Buchstabens im Zusammenhang mit der setLocation-Methode benutzt. Das funktioniert auch ganz gut soweit. Wenn ich allerdings den Buchstaben etwas schneller bewege, dann wird er für kurze Zeit am oberen Rand des Spielbrettes dargestellt und das ist unschön. Zur Info, wenn mehrere Buchstaben auf dem Feld sind, und ich einen schnell hin und her bewege, dann werden alle nebeneinander am oberen Rand kurz dargstellt und sind dann wieder auf ihrer Position.

Also es fühlt sich so an, als wenn das Spielbrett neu erschaffen wird, die Buchstaben im "FlowLayout" geaddet werden und dann erst positioniert werden.
Aber wie gesagt, das passiert nur, wenn ich den Buchstaben schnell bewege. Wenn ich ihn ganz langsam von Zelle zu Zelle (wegen Gitterstruktur) bewege, dann passiert dieser Effekt nicht.

Weiß jemand, woran das liegen kann bzw. wie man diesen Effekt verhinden kann.

Grüße Morph

 

[Marco13]

Je nachdem, wie das ganze im Detail gelöst ist: Kann es sein, dass es damit zusammenhängt, dass die Maus den Buchstaben bei schneller Bewegung kurz verläßt?

 

[Morphius]

Hi,
danke schon mal für die Antwort, ich habe es gerade ausprobiert. Also der Effekt tritt auch auf, wenn ich die Maus zwar schnell bewege, aber so dass sie immer übe rdem Buchstaben ist.

Ich poste mal meinen Quell-Code:

Das ist das große JPanel (das mit dem Gitter):

public class TablePanel extends JPanel {
	
	DragAndDropObserver dadob;
	
	public TablePanel(DragAndDropObserver dado){
		this.dadob = dado;
		this.addMouseListener(new MouseAdapter() {
			@Override
			public void mousePressed(MouseEvent e) {
				Component component = e.getComponent().getComponentAt(e.getPoint());
				if (component instanceof ActivityPanel){
					ActivityPanel activityPanel = (ActivityPanel)component;
					dadob.activityPanelPressed(activityPanel, e.getLocationOnScreen());
				}				
			}
			@Override
			public void mouseReleased(MouseEvent e) {
				dadob.activityPanelReleased();
			}
			@Override
			public void mouseEntered(MouseEvent e) {
				dadob.enterTable((TablePanel)e.getSource());
			}
			@Override
			public void mouseExited(MouseEvent e) {
				dadob.exitTable((TablePanel)e.getSource());
			}
		});
		this.addMouseMotionListener(new MouseAdapter() {
			@Override
			public void mouseDragged(MouseEvent e) {
				dadob.tableDragged(e.getLocationOnScreen());
			}
		});
		this.setSize(Settings.TABLEWIDTH, Settings.TABLEHEIGHT);
		this.setPreferredSize(this.getSize());
		
		this.add(new ActivityPanel(400, 300, Color.blue, 4, "Operation 1"));
		this.add(new ActivityPanel(200, 500, Color.red, 8, "Operation 2"));
		this.add(new ActivityPanel(600, 100, Color.yellow, 2, "Operation 3"));
		this.add(new ActivityPanel(200, 50, Color.cyan, 5, "Operation 4"));
	}

...
}

Der DragAndDropObserver (eigentlich Manager, das wird noch umbenannt) verwaltet die Aktionen und wird vom Listener des TablePanels aufgerufen.

public class DragAndDropObserver {
	
	private Application application;
	private ActivityPanel draggedPanel;
	private int dy;
	
	public DragAndDropObserver(Application application) {
		this.application = application;
		draggedPanel = null;
	}
		
	public void tableDragged(Point position){
		if (draggedPanel != null){
			int relativX = position.x - application.getTablePanel().getLocationOnScreen().x;
			int relativY = position.y - application.getTablePanel().getLocationOnScreen().y -dy;
			Point cell = application.getTablePanel().getCellForLocation(relativX, relativY);	
			Point cellPos = application.getTablePanel().getPositionOfCell(cell);
			if (!draggedPanel.getLocation().equals(cellPos)){
				draggedPanel.setPosition(cellPos.x, cellPos.y);
				draggedPanel.repaint();
			}
		}
	}

...
}

Das ist das JPanel, was durch die Gegend gezogen wird.

public class ActivityPanel extends JPanel {
	
	Point position;
	JLabel label;
	
	public ActivityPanel(int x, int y, Color color, int duration, String name) {
		this.label = new JLabel();
		this.add(label);
		setPosition(x, y);
		Dimension dim = new Dimension(Settings.COLUMNWIDTH-1, (Settings.ROWHEIGHT*duration)-1);
		this.setBackground(color);
		this.setSize(dim);
		this.setPreferredSize(dim);
	}
	
	public void setPosition(int x, int y) {
		this.position = new Point(x,y);
	}
	
	@Override
	public void paintComponent(Graphics g) {
		this.setLocation(position);
		super.paintComponent(g);
	}
}

 

[Marco13]

Das

    public void setPosition(int x, int y) {
        this.position = new Point(x,y);
    }
    
    @Override
    public void paintComponent(Graphics g) {
        this.setLocation(position);
        super.paintComponent(g);
    }

sieht verdächtig aus: In der paint-Methode sollte NUR gezeichnet werden, und insbesondere keine Veränderungen an der Component vorgenommen werden - sonst legt sich Swing alles schön zurecht, damit es zeichnen kann, und wenn es dann zeichnen will, zieht man ihm die Information, die es dafür braucht, unterm Hintern weg. Probier mal

    public void setPosition(int x, int y) {
        this.position = new Point(x,y);
        this.setLocation(position); // Gleich setzen
    }
    /* Fällt dann wohl weg
    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
    }*/

(die position als Instanzvariable zu speichern ist dann evtl. überflüssig)

 

[Morphius]

Hi, danke nochmal für die Antwort.
Hab heute nochmal etwas rumprobiert und hab rausgefunden woran es lag, dass es flackert. Und zwar ist auf den Buchstaben ein JLabel, was bei jeder Bewegung seinen Text ändert. Wenn ich das nicht verändere, dann flackert es nicht mehr.
Wäre zwar cool, wenn es das macht, aber das muss es nicht unbedingt, somit hab ich es jetzt rausgenommen.

Allerdings gibt es ein neues Problem. Und zwar habe ich das setLocation jetzt aus der paintComponent-Methode rausgenommen (so wie du vorgeschlagen hattest und so hatte ich es auch zuerst). Das funktioniert zwar solange ich die Buchstaben auf dem Spielbrett hin- und herziehe, aber auf meiner Oberfläche existiert auch noch eine JList und wenn die sich aktualisiert, dann werden wieder alle Buchstaben an den oberen Rand gezeichnet. Ich weiß allerdings nicht wieso ???:L

Grüße Morph

 

[Morphius]

Nachtrag:
Das gleiche passiert auch beim Verändern der Fenstergröße. Da wird dann wahrscheinlich ein repaint() aufgerufen, ist nur die Frage, wieso die Buchstaben dann ihre Position verlieren ???:L

Grüße Morph

 

[Marco13]

Moment, auf die Schnelle hab ich im Code dazu jetzt nichts gesehen: Das Panel wo die unter-Panel drin liegen, welches Layout hat das? Du solltest diesem Ober-Panel mit
oberpanel.setLayout(null);
ein null-Layout geben...

 

[Morphius]

:toll:
Super, ich danke dir. Auf die Idee wäre ich im Leben nicht gekommen. Alle Panels sind und bleiben nun an Ihrem Fleck.