JButton Konflikt mit Graphics

[NicoHatProbleme2]

Ich habe einen JButton erstellt, doch er zerstört irgendwie meine restlichen Felder, die mit Graphics erstellt worden sind... auch, wenn er zwar visible, aber unsichtbar ist.
Wie kann ich das ändern?

 

[NicoHatProbleme2]

Da die Position durch einen Thread bestimmt wird, ist es komplett zufällig

 

[KonradN]

Kannst Du bitte genau beschreiben, was Du machst und was das genaue Problem ist. Deine kurze Aussage ist auch etwas widersprüchlich: Visible ist sichtbar, also ein Button ist entweder sichtbar oder unsichtbar. Also visible und unsichtbar kann er nicht sein ...

Und wenn Positionen geändert werden: Evtl. einmal mit dem Volatile beschäftigen - dazu hatte ich im anderen Thread ja auch extra einen Link zu gebracht. Da ist das Verständnis wichtig.

Je nachdem, was Du da machst, kann es viele Möglichkeiten geben: Evtl. ein Container, der nur ein Element beachtet oder wenn Du selbst etwas malst und Controls verwendest, dann hast Du evtl. bei den paint* Methoden etwas nicht richtig gemacht (also z.B. die Methode, die Du überschrieben hast, nicht aufgerufen oder so). ... Aber da bringt eine Spekulation relativ wenig weshalb ich das an dieser Stelle einfach einstelle.

 

[Neumi5694]

So viele Infos zum verwendeten Layout ...
Was du machen kannst, ist einen zweiten Layer zu verwenden. Füge dem Container zwei Panels hinzu:
Das erste mit Opaque = false und absoluteLayout, das zweite ist das, auf dem deine Felder gezeichnet werden.

Ansonsten gibt's da noch das GlassPane, das eigentlich dafür da wäre, aber dazu müsste man wissen, was du eigentlich machst.

 

[NicoHatProbleme2]

package Tools.Pixel2D.POFLibary;

import java.awt.Graphics;
import java.awt.event.*;

import javax.swing.*;

import Tools.Pixel2D.*;
import Tools.Menu.*;

public class clickable extends PixelObjectFunction implements ActionListener {
    JButton button;
    PixelImportable imp;
    PixelObject obj;
    public clickable(PixelObject object, String name, PixelImportable imp) {
        super(object, name, true);
        this.imp = imp;
        obj = object;
        button = new InvisButton();
        object.frame.add(button);
        button.setBounds(object.x*object.frame.getMinimumPixelSize(), object.y*object.frame.getMinimumPixelSize(), object.pixels[0].length*object.frame.getMinimumPixelSize(), object.pixels.length*object.frame.getMinimumPixelSize());
        button.addActionListener(this);
        
    }
    public void run() {
        button.setBounds(object.x*object.frame.getMinimumPixelSize(), object.y*object.frame.getMinimumPixelSize(), object.pixels[0].length*object.frame.getMinimumPixelSize(), object.pixels.length*object.frame.getMinimumPixelSize());
        //System.out.println(button.getBounds().toString());
    }
    @Override
    public void actionPerformed(ActionEvent e) {
        System.out.println("Ja");
        imp.action(this);
    }
    class InvisButton extends JButton {
        @Override
        public void paint(Graphics g){
              // Do nothing here
        }
    }
}

Hier erstmal der code... habe auch versucht, dem JButton einfach das hier zu geben:

button.setOpaque(false);
button.setContentAreaFilled(false);
button.setBorderPainted(false);

Hat aber auch nichts gebracht

 

[NicoHatProbleme2]

Ich meinte nur, dass der zwar visible:true ist, aber trotzdem nicht sichtbar ist (siehe oben)

 

[NicoHatProbleme2]

Ich benutze ein JPanel

 

[NicoHatProbleme2]

Ok... jetzt habe ich einen 2. Panel hinzugefügt... gute Sache: er macht nichts kaputt... schlechte Sache: er ist nichtmehr clickable und ist nicht mehr sichtbar, auch wenn ich ihm nichts gebe, was ihn unsichtbar machen könnte

 

[Neumi5694]

Ok... jetzt habe ich einen 2. Panel hinzugefügt... gute Sache: er macht nichts kaputt... schlechte Sache: er ist nichtmehr clickable und ist nicht mehr sichtbar, auch wenn ich ihm nichts gebe, was ihn unsichtbar machen könnte

Die Reihenfolge der Panels ist wichtig. Das obere Panel muss zuerst hinzugefügt werden.
Falls es damit nicht klappt, lies dich mal in LayeredPanes ein.

ps: Wenn dein Button nicht sichtbar sein soll, ist es sehr sehr sinnlos, einen zu verwenden.
Besser wäre ein Mauslistener auf dem Panel, der anhand der Koordinaten auswertet, was passieren soll.

 

[NicoHatProbleme2]

Die Reihenfolge der Panels ist wichtig. Das obere Panel muss zuerst hinzugefügt werden.
Falls es damit nicht klappt, lies dich mal in LayeredPanes ein.

ps: Wenn dein Button nicht sichtbar sein soll, ist es sehr sehr sinnlos, einen zu verwenden.
Besser wäre ein Mauslistener auf dem Panel, der anhand der Koordinaten auswertet, was passieren soll.

Mouselistener? Wie das?

 

[Neumi5694]

Das wäre ein Ansatz. Es existiert eine Liste mit anklickbaren Flächen. Für jede der Flächen ist eine Action hinterlegt, die ausgeführt wird, falls innerhalb des Shapes (z.B. ein Rechteck) geklickt wurde.
Du kannst natürlich auch andere Klassen verwenden, z.B. einen Container, der mehr Eigenschaften hat als nur ein Panel. Alternativ kannst du auch einfach dem record mehr Eigenschaften hinzufügen, der Phantasie sind wenig Grenzen gesetzt.

[CODE lang="java" title="Ansatz"] //Das Panel, in dem die Felder angezeigt werden
Component panel = ...;
//Hier werden die klickbaren Flächen hinterlegt und was gemacht werden soll, wenn eine angeklickt wurde
List<ShapeAndAction> areasAndActions = ...;

void init() {
panel.addMouseListener(new MouseAdapter() {
@Override
public void mouseClicked(MouseEvent e) {
//Hier abfragen, ob shift oder alt gedrückt wurde oder um welche der Maustasten es sich handelt. Kann beliebig erweitert werden
//Falls die Abfrage komplex ist, in eine Methode auslagern.
//e.getModifiersEx() liefert alle Zustände, die mit dem Klick verbunden sind. Kann mit einer Maske verglichen werden, das geht schneller und ist übersichtlicher, als alles einzeln abzufragen.
if (!e.isMetaDown() && !e.isShiftDown()) {
var o = determineAction(e.getX(), e.getY());
if (o.isPresent()) {
o.get().accept(panel); //Annahme. Keine der Actions ist null
}
}
}

//Alernativ in in MouseReleased, Click ist etwas empfindlich, wenn man während des Klickens die Maus leicht bewegt.
@Override
public void mouseReleased(MouseEvent e) {
//...
}

});
}
/**
* Liefert die Action, die ausgeführt werden soll, falls an dieser Stelle geklickt wurde
* @return Optional.empty(), falls keine Fläche gefunden wurde.
*/
private Optional<Action> determineAction(int mousePosX, int mousePosY) {
for (var entry : areasAndActions) {
if (entry.shape().contains(mousePosX, mousePosY)) {
return Optional.of(entry.action());
}
}
return Optional.empty();
}

/**
* Klasse zum Speichern von Shape und Action Tupeln.
*/
record ShapeAndAction(Shape shape, Action action) {}
[/CODE]

 

[NicoHatProbleme2]

Das wäre ein Ansatz. Es existiert eine Liste mit anklickbaren Flächen. Für jede der Flächen ist eine Action hinterlegt, die ausgeführt wird, falls innerhalb des Shapes (z.B. ein Rechteck) geklickt wurde.
Du kannst natürlich auch andere Klassen verwenden, z.B. einen Container, der mehr Eigenschaften hat als nur ein Panel. Alternativ kannst du auch einfach dem record mehr Eigenschaften hinzufügen, der Phantasie sind wenig Grenzen gesetzt.

[CODE lang="java" title="Ansatz"] //Das Panel, in dem die Felder angezeigt werden
Component panel = ...;
//Hier werden die klickbaren Flächen hinterlegt und was gemacht werden soll, wenn eine angeklickt wurde
List<ShapeAndAction> areasAndActions = ...;

void init() {
panel.addMouseListener(new MouseAdapter() {
@Override
public void mouseClicked(MouseEvent e) {
//Hier abfragen, ob shift oder alt gedrückt wurde oder um welche der Maustasten es sich handelt. Kann beliebig erweitert werden
//Falls die Abfrage komplex ist, in eine Methode auslagern.
//e.getModifiersEx() liefert alle Zustände, die mit dem Klick verbunden sind. Kann mit einer Maske verglichen werden, das geht schneller und ist übersichtlicher, als alles einzeln abzufragen.
if (!e.isMetaDown() && !e.isShiftDown()) {
var o = determineAction(e.getX(), e.getY());
if (o.isPresent()) {
o.get().accept(panel); //Annahme. Keine der Actions ist null
}
}
}

//Alernativ in in MouseReleased, Click ist etwas empfindlich, wenn man während des Klickens die Maus leicht bewegt.
@Override
public void mouseReleased(MouseEvent e) {
//...
}

});
}
/**
* Liefert die Action, die ausgeführt werden soll, falls an dieser Stelle geklickt wurde
* @return Optional.empty(), falls keine Fläche gefunden wurde.
*/
private Optional<Action> determineAction(int mousePosX, int mousePosY) {
for (var entry : areasAndActions) {
if (entry.shape().contains(mousePosX, mousePosY)) {
return Optional.of(entry.action());
}
}
return Optional.empty();
}

/**
* Klasse zum Speichern von Shape und Action Tupeln.
*/
record ShapeAndAction(Shape shape, Action action) {}
[/CODE]

benutzt du extend?

 

[Neumi5694]

benutzt du extend?

Falls du damit das Schlüsselwort "extends" meinst ... sicher
Ansonsten weiß ich nicht, was du meinst. Das ist ganz normaler Java17 Code.
Falls du eine ältere Version verwendest, musst du halt etwas mehr Tiparbeit leisten (anstatt des Records selbst eine Klasse anlegen z.B.)

 

[NicoHatProbleme2]

hat jetzt geklappt...

        r = new Rectangle();
        r.setBounds(object.x*object.frame.getMinimumPixelSize(), object.y*object.frame.getMinimumPixelSize(), object.pixels[0].length*object.frame.getMinimumPixelSize(), object.pixels.length*object.frame.getMinimumPixelSize());
        obj.frame.panel.addMouseListener(new MouseAdapter() {
            @Override
             public void mouseClicked(MouseEvent e) {
                if(r.contains(e.getPoint())) {
                    if(object.visible | useIfInvis)
                    action();
                }
            }
        }
        );

 

[Neumi5694]

Als erster Schritt nicht schlecht, ich würde den Listener aber nicht zusammen mit dem object erstellen, da kriegst du noch Probleme. Der Listener sollte einmalig definiert werden, außerhalb des Threads. Der Thread befüllt dann nur noch die Daten des Listeners.
Ich mein ... so lange du nur ein object hast, das sich nicht ändert, klappt es auch so, aber bei allem, was darüber hinausgeht, wirst du Probleme kriegen. Weiterer Denkanstoß: Was passiert, wenn du ein object löschen willst?