Swing JPanel flüssig verschieben

[Marten87]

Hallo bin neu hier und brauche etwas Hilfe,
versuche ein JPanel flüssig der Drag&Drop zu verschieben. Ich weiss, irgendwo in diesem riesigen Forum wird mein Problem gelöst, aber darum geht es nicht in erster Linie. Ich möchte einfach erst mal verstehen, warum mein Code nicht flüssig funktioniert. Das Panel flakker z.B. beim Draggen und ich kann den Mauszeiger auch aus dem Panel rausziehen. Also warum kommt das Panel nicht schnell genug hinterher?
Hie mein Code:

package basic3;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class StartFrame extends JFrame {

    private Zeichenflaeche zeichenflaeche;

    MyPanel b1;


    int bauteilX = 300;
    int bauteilY = 300;
    int offsetX, offsetY;

    JLabel testfeld;

    Graphics g;

    MausBeweger b = new MausBeweger();
   
    public StartFrame() {
        super();
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 600);
        setVisible(true);
        setResizable(false);

        zeichenflaeche = new Zeichenflaeche();
        zeichenflaeche.setLayout(null);
        zeichenflaeche.setBounds(0, 0, 800, 600);
        add(zeichenflaeche);


        zeichenflaeche.addMouseMotionListener(b);
       
        b1 = new MyPanel();
        b1.setBounds(100, 100, b1.breite, b1.laenge);
        b1.addMouseMotionListener(b);
        b1.addMouseListener(b);

        zeichenflaeche.add(b1);
       
       

        //g = zeichenflaeche.getGraphics();

    }

    private class Zeichenflaeche extends JPanel {

        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            b1.setLocation(bauteilX, bauteilY);
        }

    }

    private class MausBeweger extends MouseAdapter {

        @Override
        public void mouseDragged(MouseEvent e) {
            bauteilX = e.getX() - offsetX;
            bauteilY = e.getY() - offsetY;
            repaint();

        }

        @Override
        public void mousePressed(MouseEvent e) {
            int x = e.getX();
            int y = e.getY();
            offsetX = x - bauteilX;
            offsetY = y - bauteilY;

        }

       
    }
    private class MyPanel extends JPanel {
        String name;
        int breite = 60;
        int laenge = 60;


        MyPanel() {
            super();
        }

        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            setBackground(Color.YELLOW);
            g.setColor(Color.RED);
            g.fillRect(20, 20, 20, 20);
        }

    }
   
    }

Ich starte von der Main

package basic3;

public class MainBasic3 {

    public static void main(String[] args){
        StartFrame f = new StartFrame();
    }

}

Hilfe bzw. ein paar Tipps/Erklärungen wären super!

 

[Thallius]

Nur mal grob geraten da es mir noch zu früh zum ausprobieren ist.

Du machst ein repaint deiner ganzen Zeichenfläche. Mach doch einfach mal nur ein repaint des Panels das zu verschiebst.

Gruß

Claus

 

[Meniskusschaden]

Ich glaube, das Problem ist, dass du die aktuelle Position als relative Position innerhalb des Panels benutzt. Das Panel bewegt sich aber laufend, so dass sich aus der relativen Position ein geringerer absoluter Abstand zur Ursprungsposition ergibt, als eigentlich gewünscht ist. Versuche doch mal, getX()/getY() durch getXOnScreen()/getYOnScreen() zu ersetzen.

 

[Marten87]

Vielen Dank für Eure schnellen Antworten!
@claus: Ich habe es mit b1.repaint() versucht, aber das bewegte sich das Panel überhaupt nicht mehr. Der Hinweis wird mir aber trotzdem nützlich sein, da ich dieser Unterscheidung zwischem dem "repainten" einer Komponente und dem "repainten" das ganzen Bildschirm so gar noch nicht auf dem Schirm hatte.
@Meniskusschaden: Das war in der Tat die Lösung des Problems - super. Für meine Zwecke ist das Problem damit gelöst und ich werde es so lassen. Allerdings zieht das Panel jetzt immer noch einen winzigen "Schweif" hinter sich her. Liegt das an der Hardware oder am Code?
Für andere Nutzer hier noch meinen

package basic3;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class StartFrame extends JFrame {

    private Zeichenflaeche zeichenflaeche;

    MyPanel b1;


    int bauteilX = 300;
    int bauteilY = 300;
    int offsetX, offsetY;

    JLabel testfeld;

    Graphics g;

    MausBeweger b = new MausBeweger();
   
    public StartFrame() {
        super();
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 600);
        setVisible(true);
        setResizable(false);

        zeichenflaeche = new Zeichenflaeche();
        zeichenflaeche.setLayout(null);
        zeichenflaeche.setBounds(0, 0, 800, 600);
        add(zeichenflaeche);


        zeichenflaeche.addMouseMotionListener(b);
       
        b1 = new MyPanel();
        b1.setBounds(100, 100, b1.breite, b1.laenge);
        b1.addMouseMotionListener(b);
        b1.addMouseListener(b);

        zeichenflaeche.add(b1);
       
       

        //g = zeichenflaeche.getGraphics();

    }

    private class Zeichenflaeche extends JPanel {

        @Override
        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            b1.setLocation(bauteilX, bauteilY);
        }

    }

    private class MausBeweger extends MouseAdapter {

        @Override
        public void mouseDragged(MouseEvent e) {
            bauteilX = e.getXOnScreen() - offsetX;
            bauteilY = e.getYOnScreen() - offsetY;
            repaint();

        }

        @Override
        public void mousePressed(MouseEvent e) {
            int x = e.getXOnScreen();
            int y = e.getYOnScreen();
            offsetX = x - bauteilX;
            offsetY = y - bauteilY;

        }

       
    }
    private class MyPanel extends JPanel {
        String name;
        int breite = 60;
        int laenge = 60;


        MyPanel() {
            super();
        }

        protected void paintComponent(Graphics g) {
            super.paintComponent(g);
            setBackground(Color.YELLOW);
            g.setColor(Color.RED);
            g.fillRect(20, 20, 20, 20);
        }

    }
   
    }

berichtigten Code:

 

[Marten87]

Hallo,
habe gerade Zeit gehabt mal wieder ein bischen herumzuspielen und meine Vorlage etwas erweitert. Man kann jetzt (beliebig) viele Panels verschieben. Zudem ist eine Randbegrenzung eingebaut, d.h. die Panels sollten eigentlich nicht auf der ContentPane des Frames verschoeben werden können. Bin erstmal zufrieden, weil es so einigermaßen funktioniert. Mich würde aber interessieren, wie man es noch besser machen kann:

1. Die MouseSteuerung ist im Ganzen etwas "hackelig". Bitte mal ausprobieren. Ich kann es hier schlecht beschreiben. Manchmal hängen sich die Panels doch am Rand fest. Dann bekommt man zwar meist weg, wenn man die Richtung ändert; aber eben nicht immer.
   
2. Die Randprüfung erfolgt im Panel mit Hilfe von MagicNumbers, was sicherlich keine schöne Lösung ist! D.h. ich habe ausprobiert, wie weit ich schieben darf und dann die entsprechende Pixelanzahl per IF - Prüfung ausgeschlossen. Diese Prüfung habe ich im Panel implementiert. Ist es dort falsch?
3. Wenn man zu schnell "draggt" wandert der Cursor aus dem Panel und dann ist es besonders fehleranfällig.

Ich freue mich über jede Antwort; egal wie ausführlich oder fundiert sie ist. Ich möchte mich einfach verbessern. EIne gute und flüssige GUI für mein Vorhaben hinzubekommen kann doch nicht so schwer sein!
Vielleicht sollte man solche Probleme gänzlich anders z.B. mit einer GlassPane etc. angehen. Davon habe ich aber noch gar keine Ahnung. Geht es damit besser, d.h. kann man auf diese Weise Probleme vermeiden, die mir hier das Leben hier schwer machen? Haben Profis die gleichen Probleme oder gibt es Pattern dafür - Fragen über Fragen.
Ich würde z.B. gern auch noch eine Kollisionerkennung bauen. Ich habe auch schon eine Idee dafür. Man müsste die Rändergrenzen der anderen Panels, die in einer Liste gespeicher durchlaufen und Prüfen, ob eine Überschreitung vorliegt. Aber da prüft sich das Maschinchen doch beim Draggen einen Wolf ....

Wie gesagt: vielen Dank für großen und kleine Tipps!

package blah_mit_Randerkennung2;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.awt.geom.Point2D;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;

public class PanelTest extends JFrame {

    private JPanel contentPane;
    private Shape ausgewShape;
   
    private MouseHandler mh = new MouseHandler();

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    PanelTest frame = new PanelTest();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    /**
     * Create the frame.
     */
    public PanelTest() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(0, 0, 500, 500);
        setResizable(false);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        setContentPane(contentPane);
        contentPane.setLayout(null);

        Shape panel = new Shape(new Point2D.Float(60, 100), new Dimension(60, 60), Color.PINK);

        panel.addMouseListener(mh);
        panel.addMouseMotionListener(mh);

        Shape panel1 = new Shape(new Point2D.Float(120, 150), new Dimension(60, 60), Color.RED);

        panel1.addMouseListener(mh);
        panel1.addMouseMotionListener(mh);

        Shape panel2 = new Shape(new Point2D.Float(200, 250), new Dimension(60, 60), Color.blue);

        panel2.addMouseListener(mh);
        panel2.addMouseMotionListener(mh);

       
        contentPane.addMouseListener(mh);
        contentPane.addMouseMotionListener(mh);

        contentPane.add(panel);
        contentPane.add(panel1);
        contentPane.add(panel2);

    }// PanelTest2

    private class MouseHandler extends MouseAdapter {
        int dx, dy;
        @Override
        public void mouseDragged(MouseEvent arg0) {
            if (ausgewShape.beweglich) {
                ausgewShape.setXpos(arg0.getXOnScreen() - dx);
                ausgewShape.setYpos(arg0.getYOnScreen() - dy);
                ausgewShape.bewegeDich();
            } // if
        }// Dragged

        @Override
        public void mouseMoved(MouseEvent arg0) {
        }// Moved

        @Override
        public void mousePressed(MouseEvent arg0) {
            ausgewShape.beweglich = true;
            dx = arg0.getXOnScreen() - ausgewShape.getXpos();
            dy = arg0.getYOnScreen() - ausgewShape.getYpos();
        }// Pressed

        @Override
        public void mouseReleased(MouseEvent arg0) {
            ausgewShape.beweglich = false;
            dx=0;
            dy=0;
        }// Released

        @Override
        public void mouseEntered(MouseEvent arg0) {
            if (arg0.getSource().getClass().getName().equals("blah_mit_Randerkennung2.Shape")) {
                ausgewShape = (Shape) arg0.getSource();
            }

        }// Entered

    }// MouseHandler
}

... und hier das Panel

package blah_mit_Randerkennung2;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.geom.Point2D;

import javax.swing.JPanel;

class Shape extends JPanel {
    boolean beweglich;
    private int xpos, ypos;

    public int getXpos() {
        return xpos;
    }

    public void setXpos(int xpos) {
        this.xpos = xpos;
    }

    public int getYpos() {
        return ypos;
    }

    public void setYpos(int ypos) {
        this.ypos = ypos;
    }

    Shape(Point2D position, Dimension groesse, Color c) {
        super();
        setBackground(c);
        xpos = (int) position.getX();
        ypos = (int) position.getY();
        setLocation(xpos, ypos);
        setSize(groesse.width, groesse.height);
        beweglich = false;

    }// Konstruktor

    public void bewegeDich() {
        if (xpos < 430 && xpos > 0 && ypos < 420 && ypos > 0) {
            setLocation(xpos, ypos);
        }
    }
   
   

}// class Shape

 

[Marten87]

nicht beachten